fs: use fput_close() in path_openat()

mjguzik · brauner · commit 606623de503f · 2025-03-05T18:31:23.000+01:00
This bumps failing open rate by 1.7% on Sapphire Rapids by avoiding one atomic. Signed-off-by: Mateusz Guzik <mjguzik@gmail.com> Link: https://lore.kernel.org/r/20250305123644.554845-5-mjguzik@gmail.com Signed-off-by: Christian Brauner <brauner@kernel.org>
diff --git a/fs/namei.c b/fs/namei.c
@@ -3995,7 +3995,7 @@ static struct file *path_openat(struct nameidata *nd,
 		WARN_ON(1);
 		error = -EINVAL;
 	}
-	fput(file);
+	fput_close(file);
 	if (error == -EOPENSTALE) {
 		if (flags & LOOKUP_RCU)
 			error = -ECHILD;

Original file line number	Diff line number	Diff line change
`@@ -3995,7 +3995,7 @@ static struct file path_openat(struct nameidata nd,`
`3995`	`3995`	`WARN_ON(1);`
`3996`	`3996`	`error = -EINVAL;`
`3997`	`3997`	`}`
`3998`		`- fput(file);`
	`3998`	`+ fput_close(file);`
`3999`	`3999`	`if (error == -EOPENSTALE) {`
`4000`	`4000`	`if (flags & LOOKUP_RCU)`
`4001`	`4001`	`error = -ECHILD;`