U-Boot Archive on lore.kernel.org
 help / color / mirror / Atom feed
From: Tom Rini <trini@konsulko.com>
To: Kuan-Wei Chiu <visitorckw@gmail.com>
Cc: tuomas.tynkkynen@iki.fi, bmeng.cn@gmail.com, sjg@chromium.org,
	jerome.forissier@arm.com, jserv@ccns.ncku.edu.tw,
	eleanor15x@gmail.com, marscheng@google.com,
	u-boot@lists.u-boot-project.org
Subject: Re: [PATCH v3 0/6] fs: Add 9P filesystem support over virtio
Date: Wed, 7 Oct 2026 14:31:30 -0600	[thread overview]
Message-ID: <20261007203130.GF3079290@bill-the-cat> (raw)
In-Reply-To: <asar1Es-MIxmnyM_@google.com>

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

On Thu, Oct 08, 2026 at 04:30:12AM +0800, Kuan-Wei Chiu wrote:
> Hi Tom,
> 
> On Thu, Oct 08, 2026 at 04:13:33AM +0800, Kuan-Wei Chiu wrote:
> > Hi Tom,
> > 
> > On Wed, Oct 07, 2026 at 01:59:51PM -0600, Tom Rini wrote:
> > > On Wed, Oct 07, 2026 at 12:42:03PM -0600, Tom Rini wrote:
> > > > On Wed, 23 Sep 2026 21:29:56 +0000, Kuan-Wei Chiu wrote:
> > > > 
> > > > > Add 9P network filesystem support to U-Boot.
> > > > > 
> > > > > It allows U-Boot to directly mount host directories in virtualization
> > > > > environments via virtio. This is useful for loading kernel and
> > > > > initramfs images directly from the host into the VM during development
> > > > > and testing without having to rebuild disk images or set up a TFTP
> > > > > server.
> > > > > 
> > > > > [...]
> > > > 
> > > > Applied to u-boot/main, thanks!
> > > > 
> > > > [1/6] net: 9p: Add 9P2000.L protocol support
> > > >       commit: 48bb83195682d189273bdba813fa67f7f2760d59
> > > > [2/6] fs: 9p: Add 9P filesystem support
> > > >       commit: 9e14f904f8f12cd9df07e99a679453c8782e523b
> > > > [3/6] virtio: 9p: Add 9P transport driver
> > > >       commit: 09e668e32b4c14ce5c929d6cebfac75d2aefb90d
> > > > [4/6] doc: 9p: Add 9P filesystem documentation
> > > >       commit: 7d963df1bc38f5ae070ea8b96e03afc1ef7e23ed
> > > > [5/6] test: 9p: Add test for 9P filesystem
> > > >       commit: d446ef40b87d53ef7ac59ac9b1d79a16f0a065b0
> > > > [6/6] MAINTAINERS: Add entry for 9PFS
> > > >       commit: f6002d5e8018714ca80dcf1835c230c99c5090b3
> > > 
> > > Are there special permissions required for 9p to run? This works in
> > > Gitlab, but in Azure:
> > > test/py/tests/test_9p.py:46: in test_fs_9p
> > >     assert test_file_name in output
> > > E   assert '9p_test_small.txt' in "9P Error: No matching 9P transport device found for '-'"
> > > 
> > That's weird.  In theory, no special permissions should be required.
> > That error messag indicates that u-boot didn't detect any 9p transport
> > device. I'll take a look at it.
> > 
> Azure currently points to the github mirror of u-boot-test-hooks,
> which does not include my previous u-boot-test-hooks patch that adds
> the 9p device to arm64 qemu.
> 
> The following change should fix it, although I haven't tested it yet. I
> can submit it separately as a formal patch.
> 
> Regards,
> Kuan-Wei
> 
> diff --git a/.azure-pipelines.yml b/.azure-pipelines.yml
> index 015fc15d5c2..93bdbf4a4cf 100644
> --- a/.azure-pipelines.yml
> +++ b/.azure-pipelines.yml
> @@ -251,7 +251,7 @@ stages:
>            # the below corresponds to .gitlab-ci.yml "before_script"
>            cd \${WORK_DIR}
>            git config --global --add safe.directory \${WORK_DIR}
> -          git clone --depth=1 https://github.com/u-boot/u-boot-test-hooks /tmp/uboot-test-hooks
> +          git clone --depth=1 https://git.u-boot-project.org/u-boot/u-boot-test-hooks /tmp/uboot-test-hooks
>            # qemu_arm64_lwip_defconfig is the same as qemu_arm64 but with NET_LWIP enabled.
>            # The test config and the boardenv file from qemu_arm64 can be re-used so create symlinks
>            ln -s conf.qemu_arm64 /tmp/uboot-test-hooks/bin/travis-ci/conf.qemu_arm64_lwip_na

Ah! Thanks! Yes, please post a proper patch and I'll grab and push it
out.

-- 
Tom

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

      reply	other threads:[~2026-10-07 22:16 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2026-09-23 21:29 [PATCH v3 0/6] fs: Add 9P filesystem support over virtio Kuan-Wei Chiu
2026-09-23 21:29 ` [PATCH v3 1/6] net: 9p: Add 9P2000.L protocol support Kuan-Wei Chiu
2026-09-23 21:29 ` [PATCH v3 2/6] fs: 9p: Add 9P filesystem support Kuan-Wei Chiu
2026-09-23 21:29 ` [PATCH v3 3/6] virtio: 9p: Add 9P transport driver Kuan-Wei Chiu
2026-09-27 15:40   ` Simon Glass
2026-09-23 21:30 ` [PATCH v3 4/6] doc: 9p: Add 9P filesystem documentation Kuan-Wei Chiu
2026-09-27 15:40   ` Simon Glass
2026-09-23 21:30 ` [PATCH v3 5/6] test: 9p: Add test for 9P filesystem Kuan-Wei Chiu
2026-09-27 15:40   ` Simon Glass
2026-09-23 21:30 ` [PATCH v3 6/6] MAINTAINERS: Add entry for 9PFS Kuan-Wei Chiu
2026-10-07 18:42 ` [PATCH v3 0/6] fs: Add 9P filesystem support over virtio Tom Rini
2026-10-07 19:59   ` Tom Rini
2026-10-07 20:13     ` Kuan-Wei Chiu
2026-10-07 20:30       ` Kuan-Wei Chiu
2026-10-07 20:31         ` Tom Rini [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=20261007203130.GF3079290@bill-the-cat \
    --to=trini@konsulko.com \
    --cc=bmeng.cn@gmail.com \
    --cc=eleanor15x@gmail.com \
    --cc=jerome.forissier@arm.com \
    --cc=jserv@ccns.ncku.edu.tw \
    --cc=marscheng@google.com \
    --cc=sjg@chromium.org \
    --cc=tuomas.tynkkynen@iki.fi \
    --cc=u-boot@lists.u-boot-project.org \
    --cc=visitorckw@gmail.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox