xen-devel.lists.xenproject.org archive mirror
 help / color / mirror / Atom feed
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

      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).