lkml.org 
[lkml]   [2013]   [Dec]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH] xen/pvhvm: If xen_platform_pci=0 is set don't blow up.
    On Wed, 4 Dec 2013, Ian Campbell wrote:
    > > +bool xen_has_pv_devices(void)
    > > +{
    > > + if (!xen_domain())
    > > + return false;
    > > +
    > > + if (xen_hvm_domain()) {
    > > + /* User requested no unplug, so no PV drivers. */
    > > + if (xen_emul_unplug & XEN_UNPLUG_NEVER)
    > > + return false;
    >
    > I think you need
    > if (xen_emul_unpug & XEN_UNPLUG_UNNECESSARY)
    > return true;
    > don't you?

    XEN_UNPLUG_UNNECESSARY was introduced to enable the platform PCI device
    even if it didn't respond properly to the unplug protocol.
    The corresponding parameter is called "unnecessary" because if you pass
    it to the kernel you mean that it is unnecessary to unplug the emulated
    devices but you can use the pv devices anyway.

    So no, we shouldn't check for XEN_UNPLUG_UNNECESSARY here.



    > > + /* And user has xen_platform_pci=0 set in guest config as
    > > + * driver did not modify the value. */
    > > + if (!xen_platform_pci_unplug)
    > > + return false;
    > > + }
    > > + return true;
    > > +}
    > > +EXPORT_SYMBOL_GPL(xen_has_pv_devices);
    > > +
    > > void xen_unplug_emulated_devices(void)
    > > {
    > > int r;
    >
    >


    \
     
     \ /
      Last update: 2013-12-04 12:01    [W:4.139 / U:0.316 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site