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

[Xen-devel] RE: xl features missing



On Fri, 16 Jul 2010, Dan Magenheimer wrote:
> > From: Dan Magenheimer
> > Subject: xl features missing
> > 
> > FYI, I finally got a chance to play with xl a bit in
> > xen-unstable and quickly discovered the following:
> 
> > - xl create doesn't handle the possibility that tmem
> >   has memory which is freeable (xm create does)
> 
> This appears to be an indicator of a bigger issue unless
> there is some code somewhere I am missing.
> 
> Can "xl create" deal with ballooning memory from dom0, as is
> done in the free() method in tools/python/xen/xend/balloon.py?
> If not, doesn't that mean that xl is useless on systems
> where dom0_mem= isn't specified as a boot option?
>
> Since tmem leverages this method to recover freeable
> memory when creating a new guest, tmem also needs this
> else xl is also useless on a tmem-enabled system.
> 
> And in either case, it appears that the error handling
> for "out of memory" spews meaningless error messages.

xl doesn't have an equivalent of the free() method in xend, so it would
be useless without dom0_mem= at the moment.
However reimplementing that method shouldn't be hard, the only problem
is that it would probably require some sort of global locking among all
the various xl processes running concurrently.
Maybe we need a lockfile?


> 
> > - xl tmem-freeable doesn't exist (xm tmem-freeable does)
> 
> Patch submitted.

thanks


> 
> > - xl vncviewer doesn't exist (xm vncviewer does)
> 
> Doesn't look too hard, but haven't tried it yet.  This
> is something I use all the time though so would be nice
> to have.
 
it should be something similar to the 'xl console' command that we
already have.

_______________________________________________
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®.