From: Gao Xiang <xiang@kernel.org>
To: Yifan Zhao <yifan.yfzhao@foxmail.com>
Cc: Gao Xiang <xiang@kernel.org>, linux-erofs@lists.ozlabs.org
Subject: Re: [PATCH v2 1/2] erofs-utils: mkfs: defer compressed metadata generation
Date: Fri, 11 Sep 2026 18:33:30 +0800 [thread overview]
Message-ID: <aqPY-mqxmuZcugRb@XiangdeMacBook-Pro.local> (raw)
In-Reply-To: <tencent_56C5B5ED8833DCAB79CB06F495CF93AE5808@qq.com>
Hi Yifan,
On Fri, Sep 11, 2026 at 04:35:37PM +0800, Yifan Zhao wrote:
>
> On 11/9/26 11:59, Gao Xiang wrote:
> > Generally, unique addresses should be assigned after the size of
> > the primary device is decided.
> >
> > Defer compressed metadata generation until after
> > erofs_update_all_devices(). This is used to support multi-device
> > compressed filesystems (including fsmerge support for compressed
> > images.)
> >
> > Signed-off-by: Gao Xiang <xiang@kernel.org>
> > ---
...
> > + ++nexts;
> > }
> > - ++nexts;
> > + inode->z_extents = nexts;
> Hi Xiang,
>
> I think this line seems redundant as `z_extents` is not used with encoded
> extents disabled.
I've addressed most of your comments, except above.
Since it's no harm to leave a valid number here and it can be used
later if needed.
Thanks,
Gao Xiang
prev parent reply other threads:[~2026-09-11 10:33 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-09-11 3:59 [PATCH v2 1/2] erofs-utils: mkfs: defer compressed metadata generation Gao Xiang
2026-09-11 3:59 ` [PATCH v2 2/2] erofs-utils: mkfs: enable `--blobdev` for compressed inode layouts Gao Xiang
2026-09-11 8:35 ` [PATCH v2 1/2] erofs-utils: mkfs: defer compressed metadata generation Yifan Zhao
2026-09-11 10:33 ` Gao Xiang [this message]
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=aqPY-mqxmuZcugRb@XiangdeMacBook-Pro.local \
--to=xiang@kernel.org \
--cc=linux-erofs@lists.ozlabs.org \
--cc=yifan.yfzhao@foxmail.com \
/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.