On 06/15/2009 05:24 PM, Anthony Liguori wrote:
Dor Laor wrote:
Libvirt does not support r2d. I hope it won't start to support it.
It supports mips, sparc, and ppc machines now.  I don't see why it 
wouldn't support r2d.  For ppcemb, I expect this same problem to 
occur.  This sort of restriction is going to be common with embedded 
boards.
I expect these restrictions will have to be known by the management 
application.  Otherwise the users will try invalid configurations only 
to receive errors when they launch them.  GUIs exist to guide users, not 
as an inefficient means of trial-and-error.
We can have default values for these types of devices or something like pci_addr=auto.
Why wouldn't libvirt always use pci_addr=auto?  If the only argument 
for having libvirt do pci slot allocation is error messages, can't we 
find a nice way to allow libvirt to create friendly error messages 
when QEMU fails?
Error messages are not the only argument for pushing slot allocation to 
management.  See my previous messages on the topic.
If you let QEMU allocate which PCI slot a device goes in, we can hide this detail from libvirt. If you have libvirt do PCI slot allocation by default, it has to know about this restriction in the r2d board unless you have a clever way to express this sort of information.
Once QEMU has allocated a device to a slot, libvirt can do a good 
job maintaining that relationship.
The end user should have a mechanism to control device slot 
positioning. For example, if you have several pci devices, some
get high rate of interrupts and some not, if you want to optimize you 
guest you should isolate the high rate 'interesting' devices.
This is something the user will need to do. I agree that the default 
behavior might be 'auto'
I'm not at all arguing against pci_addr.  I'm arguing about how 
libvirt should use it with respect to the "genesis" use-case where 
libvirt has no specific reason to choose one PCI slot over another.  
In that case, I'm merely advocating that we want to let QEMU make the 
decision.

However this may end up, isn't it offtopic? Whatever we do we have to support both pci_addr= and default placement, so we can push this discussion to livirt-devel and bid them godspeed.
--
error compiling committee.c: too many arguments to function

--
To unsubscribe from this list: send the line "unsubscribe kvm" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to