[PATCH RESEND] ecryptfs: Log function name only once in decode_and_decrypt_filename

Thorsten Blum posted 1 patch 3 weeks, 2 days ago
fs/ecryptfs/crypto.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
[PATCH RESEND] ecryptfs: Log function name only once in decode_and_decrypt_filename
Posted by Thorsten Blum 3 weeks, 2 days ago
ecryptfs_printk() already prints the function name using %s and
__func__. Drop the redundant function name from the debug log message.

Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
---
 fs/ecryptfs/crypto.c | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/fs/ecryptfs/crypto.c b/fs/ecryptfs/crypto.c
index 3b59346d68c5..d4e714912429 100644
--- a/fs/ecryptfs/crypto.c
+++ b/fs/ecryptfs/crypto.c
@@ -1925,8 +1925,7 @@ int ecryptfs_decode_and_decrypt_filename(char **plaintext_name,
 						  decoded_name_size);
 		if (rc) {
 			ecryptfs_printk(KERN_DEBUG,
-					"%s: Could not parse tag 70 packet from filename\n",
-					__func__);
+					"Could not parse tag 70 packet from filename\n");
 			goto out_free;
 		}
 	} else {
Re: [PATCH RESEND] ecryptfs: Log function name only once in decode_and_decrypt_filename
Posted by Tyler Hicks 1 week, 1 day ago
On Tue, 10 Mar 2026 11:26:53 +0100, Thorsten Blum wrote:
> ecryptfs_printk() already prints the function name using %s and
> __func__. Drop the redundant function name from the debug log message.

Thank you! This has been applied to the next branch of the tyhicks/ecryptfs.git tree.

You can find a direct link below but please be aware that the commit hash is
unstable and, therefore, the URL may not be valid in the future.

[1/1] ecryptfs: Log function name only once in decode_and_decrypt_filename
      https://git.kernel.org/tyhicks/ecryptfs/c/1601fe9e0423d813b1158a52e051bd3059f74197

Tyler