[PATCH v2 1/3] fs/erofs: add erofs filesystem support

Tom Rini trini at konsulko.com
Mon Aug 30 18:06:46 CEST 2021


On Mon, Aug 30, 2021 at 11:31:28PM +0800, Huang Jianan wrote:
> 在 2021/8/30 21:27, Huang Jianan 写道:
> > 
> > 
> > 在 2021/8/26 6:39, Tom Rini 写道:
> > > On Mon, Aug 23, 2021 at 08:36:44PM +0800, Huang Jianan wrote:
> > > 
> > > > From: Huang Jianan <huangjianan at oppo.com>
> > > > 
> > > > This patch mainly deals with uncompressed files.
> > > > 
> > > > Signed-off-by: Huang Jianan <huangjianan at oppo.com>
> > > > ---
> > > >   fs/Kconfig          |   1 +
> > > >   fs/Makefile         |   1 +
> > > >   fs/erofs/Kconfig    |  12 ++
> > > >   fs/erofs/Makefile   |   7 +
> > > >   fs/erofs/data.c     | 124 ++++++++++++++
> > > >   fs/erofs/erofs_fs.h | 384
> > > > ++++++++++++++++++++++++++++++++++++++++++++
> > > >   fs/erofs/fs.c       | 231 ++++++++++++++++++++++++++
> > > >   fs/erofs/internal.h | 203 +++++++++++++++++++++++
> > > >   fs/erofs/namei.c    | 238 +++++++++++++++++++++++++++
> > > >   fs/erofs/super.c    |  65 ++++++++
> > > >   fs/fs.c             |  22 +++
> > > >   include/erofs.h     |  19 +++
> > > >   include/fs.h        |   1 +
> > > >   13 files changed, 1308 insertions(+)
> > > >   create mode 100644 fs/erofs/Kconfig
> > > >   create mode 100644 fs/erofs/Makefile
> > > >   create mode 100644 fs/erofs/data.c
> > > >   create mode 100644 fs/erofs/erofs_fs.h
> > > >   create mode 100644 fs/erofs/fs.c
> > > >   create mode 100644 fs/erofs/internal.h
> > > >   create mode 100644 fs/erofs/namei.c
> > > >   create mode 100644 fs/erofs/super.c
> > > >   create mode 100644 include/erofs.h
> > > Do the style problems checkpatch.pl complains about here match what's in
> > > the linux kernel?  I expect at lease some of them come from using custom
> > > debug/etc macros rather than the standard logging functions. Thanks.
> > 
> > The code is mainly come from erofs-utils, thems it has the same problem,
> > i
> > will fix it ASAP.
> > 
> > Thanks,
> > Jianan
> > 
> I have checked checkpatch.pl complains, some need to be fixed, and some
> come frome using custom macros. It seems that there are still some warnings
> that are inconsistent with the linux kernel, such as :
> 
> WARNING: Use 'if (IS_ENABLED(CONFIG...))' instead of '#if or #ifdef' where
> possible
> #835: FILE: fs/erofs/fs.c:224:
> +#ifdef CONFIG_LIB_UUID
> 
> WARNING: Possible switch case/default not preceded by break or fallthrough
> comment
> #763: FILE: fs/erofs/zmap.c:489:
> +       case Z_EROFS_VLE_CLUSTER_TYPE_NONHEAD:
> 
> erofs-utils is written according to the linux kernel coding style, I expect
> this
> part can be consistent in order to reduce maintenance burden and keep
> with the latest feature.

Yes, please fix what can be easily fixed and still kept in sync with
other projects.

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20210830/9e5910c6/attachment.sig>


More information about the U-Boot mailing list