Re: Configuration vs. compat hints [was Re: [Qemu-devel] [PATCHv3 03/13] qemu: add routines to manage PCI capabilities]

Avi Kivity
2009-06-15T17:09:22Z

[-- Attachment #1.1: Type: text/plain, Size: 1669 bytes --]

On 06/15/2009 07:27 PM, Mark McLoughlin wrote:
>> 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.
>>
>
> Presumably you're not proposing that qemu-devel completely ignore the
> typical requirements of management apps?
>

We propose to allow both qemu-allocated slots and user-allocated slots,
so we're only ignoring the actual decision by the management tool
providers, not their requirements.

> You can push the discussion to libvirt-devel, and the conclusion would
> most likely be:
>
> "We can do slot allocation if you provide us with a way to query free
> slots, or we can use qemu's default allocation if you provide us a
> way to query the allocation.
>
> We'd prefer the default allocation problem, but we don't really
> care. Both require about the same amount of work for us."
>

Well, they'll find out if they try default allocation. It's traditional
to try all the complicated solutions before trying the simplest one, so
I guess we'll just have to let them.

> libvirt was only mentioned in this thread as a concrete example of how
> the suggested solutions would actually be used by management apps.
>

True, others will wind up doing things differently. In fact, I'm a
little surprised that libvirt is involved, since the place to do
inventory is in the management app itself (it's true that libvirt also
maintains its own database, so the line is blurred).

--
I have a truly marvellous patch that fixes the bug which this
signature is too narrow to contain.

[-- Attachment #1.2: Type: text/html, Size: 2316 bytes --]

[-- Attachment #2: Type: text/plain, Size: 184 bytes --]

_______________________________________________
Virtualization mailing list
Virtualization@lists.linux-foundation.org