Messages in this thread | | | Date | Fri, 12 Nov 2021 18:57:58 +0800 | From | Gao Xiang <> | Subject | Re: [PATCH v5 1/2] erofs: add sysfs interface |
| |
On Fri, Nov 12, 2021 at 06:10:37PM +0800, Huang Jianan wrote: > 在 2021/11/12 13:36, Gao Xiang 写道: > > Hi Jianan, > > > > On Fri, Nov 12, 2021 at 10:30:02AM +0800, Huang Jianan wrote: > > > Add sysfs interface to configure erofs related parameters later. > > > > > > Signed-off-by: Huang Jianan <huangjianan@oppo.com> > > > Reviewed-by: Chao Yu <chao@kernel.org> > > > --- > > > since v4: > > > - Resend in a clean chain. > > > > > > since v3: > > > - Add description of sysfs in erofs documentation. > > > > > > since v2: > > > - Check whether t in erofs_attr_store is illegal. > > > - Print raw value for bool entry. > > > > > > since v1: > > > - Add sysfs API documentation. > > > - Use sysfs_emit over snprintf. > > > > > > Documentation/ABI/testing/sysfs-fs-erofs | 7 + > > > Documentation/filesystems/erofs.rst | 8 + > > > fs/erofs/Makefile | 2 +- > > > fs/erofs/internal.h | 10 + > > > fs/erofs/super.c | 12 ++ > > > fs/erofs/sysfs.c | 240 +++++++++++++++++++++++ > > > 6 files changed, 278 insertions(+), 1 deletion(-) > > > create mode 100644 Documentation/ABI/testing/sysfs-fs-erofs > > > create mode 100644 fs/erofs/sysfs.c > > > > > > diff --git a/Documentation/ABI/testing/sysfs-fs-erofs b/Documentation/ABI/testing/sysfs-fs-erofs > > > new file mode 100644 > > > index 000000000000..86d0d0234473 > > > --- /dev/null > > > +++ b/Documentation/ABI/testing/sysfs-fs-erofs > > > @@ -0,0 +1,7 @@ > > > +What: /sys/fs/erofs/features/ > > > +Date: November 2021 > > > +Contact: "Huang Jianan" <huangjianan@oppo.com> > > > +Description: Shows all enabled kernel features. > > > + Supported features: > > > + lz4_0padding, compr_cfgs, big_pcluster, device_table, > > > + sb_chksum. > > Please help submit a patch renaming lz4_0padding to 0padding globally > > since LZMA and later algorithms also need that... > > > > Also, lack of chunked_file and compr_head2 as well? > > It seems that these features are also missing in internal.h, I will send a > new > patchset containing these fixes.
Yeah, these were not used before, but as you're introducing the sysfs directories, they need to be filled up.
Thanks, Gao Xiang
> > Thanks, > Jianan > > > Thanks, > > Gao Xiang
| |