Hi, this allows bridges to refuse forwarding VGA, and reports this upwards as an error, because we cannot set up valid decoding for the requested device in this case.
I think it should be fine to leave VGA forwarding enabled on lower bridges if a bridge closer to the root refused to enable forwarding, because no accesses can reach there anyway. Simon Simon Richter (5): vgaarb: pass vga_get errors to userspace vgaarb: pass errors from pci_set_vga_state up vgaarb: mark vga_get family as __must_check pci: check if VGA decoding was really activated pci: mark return value of pci_set_vga_state as __must_check drivers/pci/pci.c | 6 ++++++ drivers/pci/vgaarb.c | 22 +++++++++++++++++++--- include/linux/pci.h | 2 +- include/linux/vgaarb.h | 9 +++++---- 4 files changed, 31 insertions(+), 8 deletions(-) base-commit: 9702969978695d9a699a1f34771580cdbb153b33 -- 2.47.3
