Re: [PATCH v2] ubifs: ubifs to export filesystem error counters

From: Richard Weinberger
Date: Tue Oct 26 2021 - 15:36:48 EST


Stefan,

----- Ursprüngliche Mail -----
> Von: "schaecsn" <schaecsn@xxxxxxx>
> An: "richard" <richard@xxxxxx>, "linux-mtd" <linux-mtd@xxxxxxxxxxxxxxxxxxx>, "linux-kernel"
> <linux-kernel@xxxxxxxxxxxxxxx>
> CC: "schaecsn" <schaecsn@xxxxxxx>, "Stefan Schaeckeler" <sschaeck@xxxxxxxxx>
> Gesendet: Sonntag, 10. Oktober 2021 06:22:39
> Betreff: [PATCH v2] ubifs: ubifs to export filesystem error counters

> Not all ubifs filesystem errors are propagated to userspace.
>
> Export bad magic, bad node and crc errors via sysfs. This allows userspace
> to notice filesystem errors:
>
> /sys/fs/ubifs/ubiX_Y/errors_magic
> /sys/fs/ubifs/ubiX_Y/errors_node
> /sys/fs/ubifs/ubiX_Y/errors_crc
>
> The counters are reset to 0 with a remount.
>
> Signed-off-by: Stefan Schaeckeler <sschaeck@xxxxxxxxx>
> ---
> Changes in v2:
> - merged sysfs.h header into ubifs.h.
> - io.c: wrapped incrementing error counters into new functions
> record_*_error().
> - sysfs.c: removed sysfs write callback for clearing stats.
> - sysfs.c: ubifs_sysfs_register(): free c->stats on error.
> - sysfs.c: ubifs_attr_show(): use sysfs_emit().
>
> fs/ubifs/Makefile | 2 +-
> fs/ubifs/io.c | 21 +++++++
> fs/ubifs/super.c | 16 ++++-
> fs/ubifs/sysfs.c | 153 ++++++++++++++++++++++++++++++++++++++++++++++
> fs/ubifs/ubifs.h | 35 +++++++++++
> 5 files changed, 225 insertions(+), 2 deletions(-)
> create mode 100644 fs/ubifs/sysfs.c

Queued for 5.16.

I just noticed forgot to request sysfs documentation in Documentation/ABI/.
Can you please send a followup patch?

Thanks,
//richard