| |
|
 |
|
 |
|
|
|
| |
|
|
xen-devel
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.
Kieran
_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel
|
|
|
| |
|
Home