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

RE: [Xen-devel] [PATCH] Virtual machine queue NIC support incontrol panel



On Sun, 2008-02-03 at 14:36 +0800, Zhao, Yu wrote:
> Currently "accel" only supports static configuration, which means user
> has to set its values before the associated VIF is allocated. I'd like
> to turn "accel" into be a dynamic option, so user can enable an
> accelerator for a VIF even when this VIF is running.

That sounds like a good idea, and should be pretty straightforward.  It
would need a watch on the accel xenbus entry in
drivers/xen/netback/accel.c and then some care to ensure that any
previous accelerator was removed before the new one gets added when the
watch fires.  The frontend already watches the configured accelerator so
that side of things should just work already.

> And the content of "accel" also could be in a multi-column format,
> rather than NIC name. For example, if "accel" could recognize
> "eth_name:feature:parameters", then user can pass flags to
> acceleration plug-in. This gives more flexibility.

I'm not sure if there's a standard xenstore way of doing things like
that, but I'd suggest that having the features and parameters as
separate entries in xenstore would make the implementation simpler as
you wouldn't have to parse the option to work out what had changed when
the configuration is modified.

Kieran


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