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)
committerBen Hutchings <ben@decadent.org.uk>
Fri, 22 Jan 2016 21:40:07 +0000 (21:40 +0000)
commit16f592aba4a0e7741823a37b0e5064f08c5f6dc1
tree52cab6335eef0eb10a3cbded5dc5c58cc61d8dff
parent5a08e0db712f72ee8645ec8f38c6f0a937f26304
xen/pciback: Don't allow MSI-X ops if PCI_COMMAND_MEMORY is not set.

commit 408fb0e5aa7fda0059db282ff58c3b2a4278baa0 upstream.

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

Reviewed-by: Jan Beulich <jbeulich@suse.com>
Signed-off-by: Konrad Rzeszutek Wilk <konrad.wilk@oracle.com>
Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
drivers/xen/xen-pciback/pciback_ops.c