From: Gao Xiang <hsiangkao@linux.alibaba.com>
To: Giuseppe Scrivano <gscrivan@redhat.com>
Cc: linux-erofs@lists.ozlabs.org, cyphar@cyphar.com,
linux-fsdevel@vger.kernel.org
Subject: Re: [PATCH v3] erofs: accept source file descriptor via fsconfig
Date: Fri, 17 Jul 2026 16:37:27 +0800 [thread overview]
Message-ID: <5f5e668f-c371-4b8a-9ca3-91aa8fa10fbf@linux.alibaba.com> (raw)
In-Reply-To: <87cxwm2f6z.fsf@redhat.com>
On 2026/7/17 16:28, Giuseppe Scrivano wrote:
> Gao Xiang <hsiangkao@linux.alibaba.com> writes:
>
>> Hi Giuseppe,
>>
>> On 2026/7/14 23:48, Giuseppe Scrivano wrote:
>>> Allow userspace to pass an already-opened file descriptor as the mount
>>> source instead of a path string. This is useful for tools that already
>>> hold an fd to the image, such as composefs reusing an existing erofs
>>> backing file.
>>> Signed-off-by: Giuseppe Scrivano <gscrivan@redhat.com>
>>
>> Use the current erofs-utils testsuite, the following tests seems skipped incorrectly.
>>
>> [Error 22] Invalid argument
>> SKIP: erofs/008
>> [Error 22] Invalid argument
>> SKIP: erofs/009
>> [Error 22] Invalid argument
>> SKIP: erofs/010
>> [Error 22] Invalid argument
>> SKIP: erofs/011
>>
>> [Error 22] Invalid argument
>> SKIP: erofs/017
>> [Error 22] Invalid argument
>> SKIP: erofs/018
>>
>> And the kernel message shows:
>> [ 116.477816] erofs: source is unsupported
>> ...
>>
>> git://git.kernel.org/pub/scm/linux/kernel/git/xiang/erofs-utils.git -b experimental-tests
>> $ sudo make check
>>
>> Thanks,
>> Gao Xiang
>
> it seems to be caused by the different error returned by erofs_fc_get_tree.
>
> Would you be fine if I amend this on top of v3?
I think you could just send out the diff from the baseline
since it might be harder to follow an incremental diff here.
But anyway, new versions are always welcome, I will play
with them when I am availble.
Thanks,
Gao Xiang
>
> Thanks,
> Giuseppe
prev parent reply other threads:[~2026-07-17 8:37 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2026-07-14 15:48 [PATCH v3] erofs: accept source file descriptor via fsconfig Giuseppe Scrivano
2026-07-17 3:42 ` Gao Xiang
2026-07-17 8:28 ` Giuseppe Scrivano
2026-07-17 8:37 ` 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=5f5e668f-c371-4b8a-9ca3-91aa8fa10fbf@linux.alibaba.com \
--to=hsiangkao@linux.alibaba.com \
--cc=cyphar@cyphar.com \
--cc=gscrivan@redhat.com \
--cc=linux-erofs@lists.ozlabs.org \
--cc=linux-fsdevel@vger.kernel.org \
/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.