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

Re: [Xen-devel] [PATCH] tools: remove unused wrappers for python



On Tue, Oct 06, 2015 at 12:18:50PM +0100, Wei Liu wrote:
> On Tue, Oct 06, 2015 at 12:46:08PM +0200, Juergen Gross wrote:
> > Remove lots of functions in tools/python/xen/lowlevel/xc/xc.c as they
> > are not used anywhere in the tree. In fact only one function is still
> > being used from pygrub, namely "xeninfo". All other users seem to have
> > gone with nuking xm/xend.
> > 
> > Signed-off-by: Juergen Gross <jgross@xxxxxxxx>
> 
> Andrew said the python module is useful for debugging purpose. There are
> out-of-tree users as well.
> 
> I'm not too fussed about either keeping these functions or removing
> them. But I would like to leave some time for other people to object.

In QubesOS we use xc bindings. In the most recent version only for
memory managing (domain_set_target_mem, domain_setmaxmem,
domain_getinfo, physinfo).
https://github.com/QubesOS/qubes-core-admin/blob/master/qmemman/qmemman.py

This is used for real-time memory adjustment based on domain needs
(think: give domain more memory when needed, before it start swapping),
so directly manipulating with xc/xs is preferred over much slower
alternatives like libvirt. Libxl would be probably enough, but libxl
python bindings effectively doesn't exists.

-- 
Best Regards,
Marek Marczykowski-GÃrecki
Invisible Things Lab
A: Because it messes up the order in which people normally read text.
Q: Why is top-posting such a bad thing?

Attachment: pgpqmhaJ5uzGw.pgp
Description: PGP signature

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel

 


Rackspace

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