diff mbox series

btrfs: remove colon from messages with state

Message ID 20240409141954.16446-1-dsterba@suse.com (mailing list archive)
State New, archived
Headers show
Series btrfs: remove colon from messages with state | expand

Commit Message

David Sterba April 9, 2024, 2:19 p.m. UTC
The message format in syslog is usually made of two parts:

  prefix ":" message

Various tools parse the prefix up to the first ":". When there's
an additional status of a btrfs filesystem like

  [5.199782] BTRFS info (device nvme1n1p1: state M): use zstd compression, level 9

where 'M' is for remount, there's one more ":" that does not conform to
the format. Remove it.

Signed-off-by: David Sterba <dsterba@suse.com>
---
 fs/btrfs/messages.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Anand Jain April 11, 2024, 8:29 a.m. UTC | #1
On 4/9/24 22:19, David Sterba wrote:
> The message format in syslog is usually made of two parts:
> 
>    prefix ":" message
> 
> Various tools parse the prefix up to the first ":". When there's
> an additional status of a btrfs filesystem like
> 
>    [5.199782] BTRFS info (device nvme1n1p1: state M): use zstd compression, level 9
> 
> where 'M' is for remount, there's one more ":" that does not conform to
> the format. Remove it.
> 

Reviewed-by: Anand Jain <anand.jain@oracle.com>
diff mbox series

Patch

diff --git a/fs/btrfs/messages.c b/fs/btrfs/messages.c
index c96dd66fd0f7..210d9c82e2ae 100644
--- a/fs/btrfs/messages.c
+++ b/fs/btrfs/messages.c
@@ -7,7 +7,7 @@ 
 
 #ifdef CONFIG_PRINTK
 
-#define STATE_STRING_PREFACE	": state "
+#define STATE_STRING_PREFACE	" state "
 #define STATE_STRING_BUF_LEN	(sizeof(STATE_STRING_PREFACE) + BTRFS_FS_STATE_COUNT + 1)
 
 /*