Re: Problem with xen config

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 8/10/20 10:55 AM, Laine Stump wrote:
On 8/10/20 4:38 AM, Christoph wrote:

xen_platform_pci=1

This setting is not supported in the libvirt libxl driver, but AFAICT
libxl sets the default to 'true' for HVM guests.

pci_msitranslate=1

This is also not supported in libvirt and unfortunately defaults to
'false'.

pci_permissive = 1

Patches have been submitted for this setting but so far there has not
been agreement on the schema change

https://www.redhat.com/archives/libvir-list/2020-April/msg01230.html

Hmm, so I see that discussion stalled after I asked a few questions, and then you (Jim) made a few suggestions about where the attribute should go and what it should be named. (note for newcomers - the original patch messages linked above were at the end of April, but the replies discussing them were in May, so they don't show up directly in the primitive mailman archive page for the original messages; to see them you need to go here:

https://www.redhat.com/archives/libvir-list/2020-May/thread.html

then search for "permissive" in the page - this will show you followups to all the original patch mails.


I'm not sure who should be pushing the discussion/decision - I've already said my piece, and would be just as happy with anything as long as it doesn't interfere with usage for qemu. Maybe someone else has a stronger opinion.

I just posted a V3 that supports a 'writeFiltering' attribute on the <source> element

https://www.redhat.com/archives/libvir-list/2020-August/msg00465.html

E.g.

    <hostdev mode='subsystem' type='pci'>
      ...
      <source writeFiltering='no'>
        <address .../>
      </source>
    </hostdev>

However, after reading your reply again, I realize I forgot to add the check you requested in the qemu driver's post-parse validation logic. I can do that in V4 but I'd like to get some feedback on the modeling approach first.

Regards,
Jim




[Index of Archives]     [Virt Tools]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]

  Powered by Linux