This is an archived copy of the Xen.org mailing list, which we have preserved to ensure that existing links to archives are not broken. The live archive, which contains the latest emails, can be found at http://lists.xen.org/
Home Products Support Community News


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

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.


Xen-devel mailing list

<Prev in Thread] Current Thread [Next in Thread>