> -----Original Message----- > From: Dexuan Cui > Sent: Tuesday, March 6, 2018 10:22 AM > To: bhelg...@google.com; linux-...@vger.kernel.org; KY Srinivasan > <k...@microsoft.com>; > Stephen Hemminger <sthem...@microsoft.com>; o...@aepfle.de; > a...@canonical.com; > jasow...@redhat.com > Cc: linux-ker...@vger.kernel.org; driverdev-devel@linuxdriverproject.org; > Haiyang Zhang > <haiya...@microsoft.com>; vkuzn...@redhat.com; marcelo.ce...@canonical.com; > Michael > Kelley (EOSG) <michael.h.kel...@microsoft.com>; Dexuan Cui > <de...@microsoft.com>; > sta...@vger.kernel.org; Jack Morgenstein <ja...@mellanox.com> > Subject: [PATCH v3 6/6] PCI: hv: fix 2 hang issues in hv_compose_msi_msg() > > 1. With the patch "x86/vector/msi: Switch to global reservation mode" > (4900be8360), the recent v4.15 and newer kernels always hang for 1-vCPU > Hyper-V VM with SR-IOV. This is because when we reach hv_compose_msi_msg() > by request_irq() -> request_threaded_irq() -> __setup_irq()->irq_startup() > -> __irq_startup() -> irq_domain_activate_irq() -> ... -> > msi_domain_activate() -> ... -> hv_compose_msi_msg(), local irq is > disabled in __setup_irq(). > > Fix this by polling the channel. > > 2. If the host is ejecting the VF device before we reach > hv_compose_msi_msg(), in a UP VM, we can hang in hv_compose_msi_msg() > forever, because at this time the host doesn't respond to the > CREATE_INTERRUPT request. This issue also happens to old kernels like > v4.14, v4.13, etc. > > Fix this by polling the channel for the PCI_EJECT message and > hpdev->state, and by checking the PCI vendor ID. > > Note: actually the above issues also happen to a SMP VM, if > "hbus->hdev->channel->target_cpu == smp_processor_id()" is true. > > Signed-off-by: Dexuan Cui <de...@microsoft.com> > Tested-by: Adrian Suhov <v-ads...@microsoft.com> > Tested-by: Chris Valean <v-chv...@microsoft.com> > Cc: sta...@vger.kernel.org > Cc: Stephen Hemminger <sthem...@microsoft.com> > Cc: K. Y. Srinivasan <k...@microsoft.com> > Cc: Vitaly Kuznetsov <vkuzn...@redhat.com> > Cc: Jack Morgenstein <ja...@mellanox.com> > --- > drivers/pci/host/pci-hyperv.c | 58 > ++++++++++++++++++++++++++++++++++++++++++- > 1 file changed, 57 insertions(+), 1 deletion(-) >
Reviewed-by: Michael Kelley <mikel...@microsoft.com> _______________________________________________ devel mailing list de...@linuxdriverproject.org http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
