xen/pciback: Don't allow MSI-X ops if PCI_COMMAND_MEMORY is not set.
authorKonrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Mon, 2 Nov 2015 23:13:27 +0000 (18:13 -0500)
committerSasha Levin <sasha.levin@oracle.com>
Mon, 7 Mar 2016 17:26:48 +0000 (12:26 -0500)
commitfda3e3e7b638f742149fb32aa9b691413bb91f41
treed60eaab93e8939a4a04db5f6c4bac43a552d82e3
parenta6c762381cc34ff6df5f75afa46e7f153116cf78
xen/pciback: Don't allow MSI-X ops if PCI_COMMAND_MEMORY is not set.

[ Upstream commit 408fb0e5aa7fda0059db282ff58c3b2a4278baa0 ]

commit f598282f51 ("PCI: Fix the NIU MSI-X problem in a better way")
teaches us that dealing with MSI-X can be troublesome.

Further checks in the MSI-X architecture shows that if the
PCI_COMMAND_MEMORY bit is turned of in the PCI_COMMAND we
may not be able to access the BAR (since they are memory regions).

Since the MSI-X tables are located in there.. that can lead
to us causing PCIe errors. Inhibit us performing any
operation on the MSI-X unless the MEMORY bit is set.

Note that Xen hypervisor with:
"x86/MSI-X: access MSI-X table only after having enabled MSI-X"
will return:
xen_pciback: 0000:0a:00.1: error -6 enabling MSI-X for guest 3!

When the generic MSI code tries to setup the PIRQ without
MEMORY bit set. Which means with later versions of Xen
(4.6) this patch is not neccessary.

This is part of XSA-157

CC: stable@vger.kernel.org
Reviewed-by: Jan Beulich <jbeulich@suse.com>
Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Signed-off-by: Sasha Levin <sasha.levin@oracle.com>
drivers/xen/xen-pciback/pciback_ops.c