All of lore.kernel.org
 help / color / mirror / Atom feed
From: Tom Rini <trini@konsulko.com>
To: Huang Jianan <jnhuang95@gmail.com>
Cc: u-boot@lists.denx.de, linux-erofs@lists.ozlabs.org
Subject: Re: [PATCH v4 0/5] fs/erofs: new filesystem
Date: Thu, 3 Mar 2022 14:15:24 -0500	[thread overview]
Message-ID: <20220303191524.GF5020@bill-the-cat> (raw)
In-Reply-To: <a028ed71-b694-2c95-094b-c50551245770@gmail.com>

[-- Attachment #1: Type: text/plain, Size: 3623 bytes --]

On Thu, Mar 03, 2022 at 10:51:18PM +0800, Huang Jianan wrote:

> Hi Tom,
> 
> Would you mind taking some time to check if this version meets
> the requirements ?So we could have a chance to be merged
> into the next version ?
> 
> I have triggered a CI via Github PR based on this version :
> https://github.com/u-boot/u-boot/pull/133

It seems fine, yes.  The thing about that PR is that it doesn't use a
custom build of the Docker container that also has the tools, so the FS
tests aren't actually run.  I'll do that when I get to testing and
applying this, but if you can do that before hand to ensure the tests
really do run and pass, especially on Azure where things can be a tiny
bit trickier (since source directory is enforced read-only), that would
be great.

> 
> Thanks,
> Jianan
> 
> 在 2022/2/26 15:05, Huang Jianan 写道:
> > Changes since v3:
> >   - update tools/docker/Dockerfile;
> > 
> > Changes since v2:
> >   - sync up with erofs-utils 1.4;
> >   - update lib/lz4 to v1.8.3;
> >   - add test for filesystem functions;
> > 
> > Changes since v1:
> >   - fix the inconsistency between From and SoB;
> >   - add missing license header;
> > 
> > Huang Jianan (5):
> >    fs/erofs: add erofs filesystem support
> >    lib/lz4: update LZ4 decompressor module
> >    fs/erofs: add lz4 decompression support
> >    fs/erofs: add filesystem commands
> >    test/py: Add tests for the erofs
> > 
> >   MAINTAINERS                         |   9 +
> >   cmd/Kconfig                         |   6 +
> >   cmd/Makefile                        |   1 +
> >   cmd/erofs.c                         |  42 ++
> >   configs/sandbox_defconfig           |   1 +
> >   fs/Kconfig                          |   2 +
> >   fs/Makefile                         |   1 +
> >   fs/erofs/Kconfig                    |  21 +
> >   fs/erofs/Makefile                   |   9 +
> >   fs/erofs/data.c                     | 311 +++++++++++++
> >   fs/erofs/decompress.c               |  78 ++++
> >   fs/erofs/decompress.h               |  24 +
> >   fs/erofs/erofs_fs.h                 | 436 ++++++++++++++++++
> >   fs/erofs/fs.c                       | 267 +++++++++++
> >   fs/erofs/internal.h                 | 313 +++++++++++++
> >   fs/erofs/namei.c                    | 252 +++++++++++
> >   fs/erofs/super.c                    | 105 +++++
> >   fs/erofs/zmap.c                     | 601 ++++++++++++++++++++++++
> >   fs/fs.c                             |  22 +
> >   include/erofs.h                     |  19 +
> >   include/fs.h                        |   1 +
> >   include/u-boot/lz4.h                |  49 ++
> >   lib/lz4.c                           | 679 ++++++++++++++++++++--------
> >   lib/lz4_wrapper.c                   |  23 +-
> >   test/py/tests/test_fs/test_erofs.py | 211 +++++++++
> >   tools/docker/Dockerfile             |   1 +
> >   26 files changed, 3269 insertions(+), 215 deletions(-)
> >   create mode 100644 cmd/erofs.c
> >   create mode 100644 fs/erofs/Kconfig
> >   create mode 100644 fs/erofs/Makefile
> >   create mode 100644 fs/erofs/data.c
> >   create mode 100644 fs/erofs/decompress.c
> >   create mode 100644 fs/erofs/decompress.h
> >   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 fs/erofs/zmap.c
> >   create mode 100644 include/erofs.h
> >   create mode 100644 test/py/tests/test_fs/test_erofs.py
> > 
> 

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]

WARNING: multiple messages have this Message-ID (diff)
From: Tom Rini <trini@konsulko.com>
To: Huang Jianan <jnhuang95@gmail.com>
Cc: u-boot@lists.denx.de, linux-erofs@lists.ozlabs.org, xiang@kernel.org
Subject: Re: [PATCH v4 0/5] fs/erofs: new filesystem
Date: Thu, 3 Mar 2022 14:15:24 -0500	[thread overview]
Message-ID: <20220303191524.GF5020@bill-the-cat> (raw)
In-Reply-To: <a028ed71-b694-2c95-094b-c50551245770@gmail.com>

[-- Attachment #1: Type: text/plain, Size: 3623 bytes --]

On Thu, Mar 03, 2022 at 10:51:18PM +0800, Huang Jianan wrote:

> Hi Tom,
> 
> Would you mind taking some time to check if this version meets
> the requirements ?So we could have a chance to be merged
> into the next version ?
> 
> I have triggered a CI via Github PR based on this version :
> https://github.com/u-boot/u-boot/pull/133

It seems fine, yes.  The thing about that PR is that it doesn't use a
custom build of the Docker container that also has the tools, so the FS
tests aren't actually run.  I'll do that when I get to testing and
applying this, but if you can do that before hand to ensure the tests
really do run and pass, especially on Azure where things can be a tiny
bit trickier (since source directory is enforced read-only), that would
be great.

> 
> Thanks,
> Jianan
> 
> 在 2022/2/26 15:05, Huang Jianan 写道:
> > Changes since v3:
> >   - update tools/docker/Dockerfile;
> > 
> > Changes since v2:
> >   - sync up with erofs-utils 1.4;
> >   - update lib/lz4 to v1.8.3;
> >   - add test for filesystem functions;
> > 
> > Changes since v1:
> >   - fix the inconsistency between From and SoB;
> >   - add missing license header;
> > 
> > Huang Jianan (5):
> >    fs/erofs: add erofs filesystem support
> >    lib/lz4: update LZ4 decompressor module
> >    fs/erofs: add lz4 decompression support
> >    fs/erofs: add filesystem commands
> >    test/py: Add tests for the erofs
> > 
> >   MAINTAINERS                         |   9 +
> >   cmd/Kconfig                         |   6 +
> >   cmd/Makefile                        |   1 +
> >   cmd/erofs.c                         |  42 ++
> >   configs/sandbox_defconfig           |   1 +
> >   fs/Kconfig                          |   2 +
> >   fs/Makefile                         |   1 +
> >   fs/erofs/Kconfig                    |  21 +
> >   fs/erofs/Makefile                   |   9 +
> >   fs/erofs/data.c                     | 311 +++++++++++++
> >   fs/erofs/decompress.c               |  78 ++++
> >   fs/erofs/decompress.h               |  24 +
> >   fs/erofs/erofs_fs.h                 | 436 ++++++++++++++++++
> >   fs/erofs/fs.c                       | 267 +++++++++++
> >   fs/erofs/internal.h                 | 313 +++++++++++++
> >   fs/erofs/namei.c                    | 252 +++++++++++
> >   fs/erofs/super.c                    | 105 +++++
> >   fs/erofs/zmap.c                     | 601 ++++++++++++++++++++++++
> >   fs/fs.c                             |  22 +
> >   include/erofs.h                     |  19 +
> >   include/fs.h                        |   1 +
> >   include/u-boot/lz4.h                |  49 ++
> >   lib/lz4.c                           | 679 ++++++++++++++++++++--------
> >   lib/lz4_wrapper.c                   |  23 +-
> >   test/py/tests/test_fs/test_erofs.py | 211 +++++++++
> >   tools/docker/Dockerfile             |   1 +
> >   26 files changed, 3269 insertions(+), 215 deletions(-)
> >   create mode 100644 cmd/erofs.c
> >   create mode 100644 fs/erofs/Kconfig
> >   create mode 100644 fs/erofs/Makefile
> >   create mode 100644 fs/erofs/data.c
> >   create mode 100644 fs/erofs/decompress.c
> >   create mode 100644 fs/erofs/decompress.h
> >   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 fs/erofs/zmap.c
> >   create mode 100644 include/erofs.h
> >   create mode 100644 test/py/tests/test_fs/test_erofs.py
> > 
> 

-- 
Tom

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 659 bytes --]

  reply	other threads:[~2022-03-03 19:15 UTC|newest]

Thread overview: 42+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-02-26  7:05 [PATCH v4 0/5] fs/erofs: new filesystem Huang Jianan
2022-02-26  7:05 ` Huang Jianan
2022-02-26  7:05 ` [PATCH v4 1/5] fs/erofs: add erofs filesystem support Huang Jianan
2022-02-26  7:05   ` Huang Jianan
2022-03-16 12:10   ` Tom Rini
2022-03-16 12:10     ` Tom Rini
2022-02-26  7:05 ` [PATCH v4 2/5] lib/lz4: update LZ4 decompressor module Huang Jianan
2022-02-26  7:05   ` Huang Jianan
2022-03-16 12:10   ` Tom Rini
2022-03-16 12:10     ` Tom Rini
2024-05-24 14:26   ` Jonathan Liu
2024-05-24 14:26     ` Jonathan Liu
2024-05-24 16:52     ` Gao Xiang
2024-05-24 16:52       ` Gao Xiang
2024-05-26  8:06       ` Jonathan Liu
2024-05-26  8:06         ` Jonathan Liu
2024-05-26 12:18         ` Jianan Huang
2024-05-26 12:18           ` Jianan Huang
2024-05-28 13:28           ` Jonathan Liu
2024-05-28 13:28             ` Jonathan Liu
2024-05-28 14:03             ` Gao Xiang
2024-05-28 14:03               ` Gao Xiang
2022-02-26  7:05 ` [PATCH v4 3/5] fs/erofs: add lz4 decompression support Huang Jianan
2022-02-26  7:05   ` Huang Jianan
2022-03-16 12:10   ` Tom Rini
2022-03-16 12:10     ` Tom Rini
2022-02-26  7:05 ` [PATCH v4 4/5] fs/erofs: add filesystem commands Huang Jianan
2022-02-26  7:05   ` Huang Jianan
2022-03-16 12:10   ` Tom Rini
2022-03-16 12:10     ` Tom Rini
2022-02-26  7:05 ` [PATCH v4 5/5] test/py: Add tests for the erofs Huang Jianan
2022-02-26  7:05   ` Huang Jianan
2022-03-16 12:10   ` Tom Rini
2022-03-16 12:10     ` Tom Rini
2022-03-03 14:51 ` [PATCH v4 0/5] fs/erofs: new filesystem Huang Jianan
2022-03-03 14:51   ` Huang Jianan
2022-03-03 19:15   ` Tom Rini [this message]
2022-03-03 19:15     ` Tom Rini
2022-03-05 13:06     ` Huang Jianan
2022-03-05 13:06       ` Huang Jianan
2022-03-05 14:57       ` Tom Rini
2022-03-05 14:57         ` 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=20220303191524.GF5020@bill-the-cat \
    --to=trini@konsulko.com \
    --cc=jnhuang95@gmail.com \
    --cc=linux-erofs@lists.ozlabs.org \
    --cc=u-boot@lists.denx.de \
    /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
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.