
11 Feb
2009
11 Feb
'09
9:01 a.m.
On Tue, Feb 10, 2009 at 09:32:49PM +0100, Stefan de Konink wrote:
Guido Günther wrote:
On Mon, Feb 09, 2009 at 11:30:30AM -0500, Cole Robinson wrote: [..snip..]
No fault of this patch, but this makes me ask the question: what exactly are attachDevice and detachDevice supposed to do?
In the qemu driver, it is entirely hotplug/hotunplug, and deliberately fails on an inactive domain. This patch seems to add offline support We don't even persist the devices being hot{added,removed} into the domain xml so acting on inactive domains has no meaning or am I missing something?
When the domain is enabled, and you store it... you basically get that behavior. Exactly my point: when the domain is *enabled*. -- Guido