From: Paul Durrant <Paul.Durrant@citrix.com>
To: 'Stefano Stabellini' <sstabellini@kernel.org>
Cc: Wei Liu <wei.liu2@citrix.com>,
Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>,
Andrew Cooper <Andrew.Cooper3@citrix.com>,
"Tim (Xen.org)" <tim@xen.org>,
George Dunlap <George.Dunlap@citrix.com>,
Julien Grall <julien.grall@arm.com>,
Jan Beulich <jbeulich@suse.com>,
Ian Jackson <Ian.Jackson@citrix.com>,
"xen-devel@lists.xenproject.org" <xen-devel@lists.xenproject.org>
Subject: Re: [PATCH] tools: specifically enable VirtFS in Linux QEMU builds
Date: Fri, 7 Sep 2018 17:00:11 +0000 [thread overview]
Message-ID: <f4556059fd9340a8b8f8edb105a87784@AMSPEX02CL03.citrite.net> (raw)
In-Reply-To: <alpine.DEB.2.10.1809070941480.14449@sstabellini-ThinkPad-X260>
> -----Original Message-----
> From: Stefano Stabellini [mailto:sstabellini@kernel.org]
> Sent: 07 September 2018 17:58
> To: Paul Durrant <Paul.Durrant@citrix.com>
> Cc: 'Stefano Stabellini' <sstabellini@kernel.org>; xen-
> devel@lists.xenproject.org; Andrew Cooper <Andrew.Cooper3@citrix.com>;
> George Dunlap <George.Dunlap@citrix.com>; Ian Jackson
> <Ian.Jackson@citrix.com>; Jan Beulich <jbeulich@suse.com>; Julien Grall
> <julien.grall@arm.com>; Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>;
> Tim (Xen.org) <tim@xen.org>; Wei Liu <wei.liu2@citrix.com>
> Subject: RE: [PATCH] tools: specifically enable VirtFS in Linux QEMU builds
>
> On Fri, 7 Sep 2018, Paul Durrant wrote:
> > > -----Original Message-----
> > > From: Stefano Stabellini [mailto:sstabellini@kernel.org]
> > > Sent: 06 September 2018 19:12
> > > To: Paul Durrant <Paul.Durrant@citrix.com>
> > > Cc: xen-devel@lists.xenproject.org; Andrew Cooper
> > > <Andrew.Cooper3@citrix.com>; George Dunlap
> > > <George.Dunlap@citrix.com>; Ian Jackson <Ian.Jackson@citrix.com>; Jan
> > > Beulich <jbeulich@suse.com>; Julien Grall <julien.grall@arm.com>;
> Konrad
> > > Rzeszutek Wilk <konrad.wilk@oracle.com>; Stefano Stabellini
> > > <sstabellini@kernel.org>; Tim (Xen.org) <tim@xen.org>; Wei Liu
> > > <wei.liu2@citrix.com>
> > > Subject: Re: [PATCH] tools: specifically enable VirtFS in Linux QEMU builds
> > >
> > > On Thu, 6 Sep 2018, Paul Durrant wrote:
> > > > 9pfs support has been a documented feature since Xen 4.9, but QEMU
> will
> > > > not be built with backend support unless libcap and libattr dev packages
> > > > are installed.
> > > >
> > > > This patch modifies the README to call out those packages as pre-
> > > requisites
> > > > for Linux builds and specifically enables VirtFS in the configure line
> > > > for QEMU so that an error message is displayed if they are missing.
> > > >
> > > > Signed-off-by: Paul Durrant <paul.durrant@citrix.com>
> > >
> > > Thank you, Paul!
> >
> > NP. I'm at the point of seriously considering a Windows frontend, which is
> why I've been playing.
> >
> > BTW, as a heads-up... Before I realised QEMU was not building the
> backend, I tried a modprobe of the Xen 9pfs transport module in a Linux
> guest and immediately hit a kernel BUG (NULL ptr IIRC) which was apparently
> down to the frontend expecting the backend to be present when its probe
> routine is run, rather than deferring things until the backend goes into
> InitWait (which is what a frontend generally should do). Also, it's non-obvious
> why I have to explicitly modprobe anyway... Shouldn't the module get
> demand-loaded when I run mount -t 9pfs blah?
>
> Interesting. I don't recall ever testing this is a scenario, so I am not
> surprised it is broken. I don't know why the module doesn't get loaded
> automatically. FYI another thing missing is suspend/resume support in
> the driver.
I see. I guess that probably means cycling through closing->closed and back to initwait has not been exercised in the backend. I'll bear that in mind when I get to that stage. Thanks :-)
Paul
_______________________________________________
Xen-devel mailing list
Xen-devel@lists.xenproject.org
https://lists.xenproject.org/mailman/listinfo/xen-devel
prev parent reply other threads:[~2018-09-07 17:00 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-09-06 12:06 [PATCH] tools: specifically enable VirtFS in Linux QEMU builds Paul Durrant
2018-09-06 12:51 ` George Dunlap
2018-09-06 13:08 ` Paul Durrant
2018-09-06 18:11 ` Stefano Stabellini
2018-09-07 8:35 ` Paul Durrant
2018-09-07 14:57 ` Roger Pau Monné
2018-09-07 15:25 ` George Dunlap
2018-09-07 15:27 ` Paul Durrant
2018-09-07 16:18 ` Ian Jackson
2018-09-07 16:20 ` Paul Durrant
2018-09-07 16:55 ` Stefano Stabellini
2018-09-07 16:57 ` Stefano Stabellini
2018-09-07 17:00 ` Paul Durrant [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=f4556059fd9340a8b8f8edb105a87784@AMSPEX02CL03.citrite.net \
--to=paul.durrant@citrix.com \
--cc=Andrew.Cooper3@citrix.com \
--cc=George.Dunlap@citrix.com \
--cc=Ian.Jackson@citrix.com \
--cc=jbeulich@suse.com \
--cc=julien.grall@arm.com \
--cc=konrad.wilk@oracle.com \
--cc=sstabellini@kernel.org \
--cc=tim@xen.org \
--cc=wei.liu2@citrix.com \
--cc=xen-devel@lists.xenproject.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 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).