[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH] Prevent xl save from segfaulting when control/shutdown key is removed



True. I was conflating the key rm issue with the fact that the XenServer 
Windows PV guest agent gets unhappy because it can't advertise its features 
(which I hadn't mentioned before).

I suppose that libxl will just have to continue to speculatively write 
control/shutdown and timeout if it's not NUL-ed or rm-ed as it does now.

  Paul

> -----Original Message-----
> From: Ian Campbell
> Sent: 23 November 2011 13:05
> To: Paul Durrant
> Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
> Subject: RE: [Xen-devel] [PATCH] Prevent xl save from segfaulting
> when control/shutdown key is removed
> 
> On Wed, 2011-11-23 at 12:59 +0000, Paul Durrant wrote:
> > > -----Original Message-----
> > > From: Ian Campbell
> > > Sent: 23 November 2011 11:24
> > > To: Paul Durrant
> > > Cc: xen-devel@xxxxxxxxxxxxxxxxxxx
> > > Subject: RE: [Xen-devel] [PATCH] Prevent xl save from
> segfaulting
> > > when control/shutdown key is removed
> > >
> > > On Wed, 2011-11-23 at 11:19 +0000, Paul Durrant wrote:
> > > > what is the reason for creating control ro to the guest?
> > >
> > > In general libxl prefers to whitelist paths which the guest can
> > > write too, just to prevent a complete free for all, keep things
> > > somewhat under control and to help avoid situations where tools
> > > might inadvertently rely on a guest-writeable key in an unsafe
> way..
> > >
> > > >  In XenServer we allow the guest to write the control key to
> > > advertise
> > > > feature-shutdown, feature-suspend etc. so that the tools know
> what
> > > > values of control/shutdown the guest will respond to.
> > >
> > > The libxl way would be to create these at build time (perhaps
> empty)
> > > with the appropriate permissions.
> > >
> > > It's not clear how that functionality can be added in a way
> which is
> > > compatible with existing guests though, e.g. no Linux guest
> writes
> > > those but many can be suspended etc.
> > >
> >
> > So the simple solution, for compatibility's sake, is to make
> control rw isn't it?
> 
> The problem I'm thinking of exists even if control is rw.
> 
> Given an empty control directory how do you know if a guest supports
> suspend or not, given that most existing guests which do support
> suspend do not write any key there?
> 
> Ian.

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel


 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.