[PATCH] 9p: v9fs_file_do_lock: replace WARN_ONCE with p9_debug

Dominique Martinet via B4 Relay posted 1 patch 1 week, 3 days ago
fs/9p/vfs_file.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
[PATCH] 9p: v9fs_file_do_lock: replace WARN_ONCE with p9_debug
Posted by Dominique Martinet via B4 Relay 1 week, 3 days ago
From: Dominique Martinet <asmadeus@codewreck.org>

This warning depends on server-provided data, we should not use
WARN here

Reported-by: Yifei Chu <yifeichu24@gmail.com>
Closes: https://lore.kernel.org/r/CAPJnbgJ7ZK7DCjCfG56hd_iKGePmAzudb4hOWd4=9r32nM+KcA@mail.gmail.com
Signed-off-by: Dominique Martinet <asmadeus@codewreck.org>
---
 fs/9p/vfs_file.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/fs/9p/vfs_file.c b/fs/9p/vfs_file.c
index c5e73c37baea55394294b939fdda7170db8503c0..cddfb4b7ce4e22aacad7b51f6e6cbd164b7c2f8f 100644
--- a/fs/9p/vfs_file.c
+++ b/fs/9p/vfs_file.c
@@ -198,7 +198,7 @@ static int v9fs_file_do_lock(struct file *filp, int cmd, struct file_lock *fl)
 		res = -EAGAIN;
 		break;
 	default:
-		WARN_ONCE(1, "unknown lock status code: %d\n", status);
+		p9_debug(P9_DEBUG_ERROR, "unknown lock status code: %d\n", status);
 		fallthrough;
 	case P9_LOCK_ERROR:
 	case P9_LOCK_GRACE:

---
base-commit: 3ec9e935015b11236ba778885d89e399f00d6b50
change-id: 20260529-lock-warn-f014cac9badd

Best regards,
-- 
Dominique Martinet <asmadeus@codewreck.org>