linux-erofs.lists.ozlabs.org archive mirror
 help / color / mirror / Atom feed
From: Huang Jianan <jnhuang95@gmail.com>
To: Tom Rini <trini@konsulko.com>
Cc: u-boot@lists.denx.de, xiang@kernel.org, linux-erofs@lists.ozlabs.org
Subject: Re: [PATCH v2 1/3] fs/erofs: add erofs filesystem support
Date: Mon, 30 Aug 2021 23:31:28 +0800	[thread overview]
Message-ID: <35ce7ab3-a21c-0401-c677-eb2140ea908d@gmail.com> (raw)
In-Reply-To: <177141f0-ebbd-017e-ab63-9445b3f53ac1@gmail.com>

在 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@oppo.com>
>>>
>>> This patch mainly deals with uncompressed files.
>>>
>>> Signed-off-by: Huang Jianan <huangjianan@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.


  reply	other threads:[~2021-08-30 15:31 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-08-22 15:48 [PATCH 1/3] fs/erofs: new filesystem Huang Jianan
2021-08-22 15:48 ` [PATCH 2/3] fs/erofs: add lz4 1.8.3 decompressor Huang Jianan
2021-08-22 15:48 ` [PATCH 3/3] fs/erofs: add lz4 decompression support Huang Jianan
2021-08-23  4:24 ` [PATCH 1/3] fs/erofs: new filesystem Bin Meng
2021-08-23 11:46   ` Huang Jianan
2021-08-23 12:36 ` [PATCH v2 0/3] " Huang Jianan
2021-08-23 12:36   ` [PATCH v2 1/3] fs/erofs: add erofs filesystem support Huang Jianan
2021-08-25 22:39     ` Tom Rini
2021-08-30 13:27       ` Huang Jianan
2021-08-30 15:31         ` Huang Jianan [this message]
2021-08-30 16:06           ` Tom Rini
2021-08-30 16:23             ` Gao Xiang
2021-08-23 12:36   ` [PATCH v2 2/3] fs/erofs: add lz4 1.8.3 decompressor Huang Jianan
2021-08-25 22:39     ` Tom Rini
2021-08-30 13:30       ` Huang Jianan
2021-08-23 12:36   ` [PATCH v2 3/3] fs/erofs: add lz4 decompression support Huang Jianan
2021-08-25  1:23   ` [PATCH v2 0/3] fs/erofs: new filesystem Gao Xiang
2021-08-25 22:40   ` Tom Rini

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=35ce7ab3-a21c-0401-c677-eb2140ea908d@gmail.com \
    --to=jnhuang95@gmail.com \
    --cc=linux-erofs@lists.ozlabs.org \
    --cc=trini@konsulko.com \
    --cc=u-boot@lists.denx.de \
    --cc=xiang@kernel.org \
    --subject='Re: [PATCH v2 1/3] fs/erofs: add erofs filesystem support' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).