I think the approach here is a good compromise. Should fix recent platforms and not break older ones. Maybe commit message could say use of VGA cards on AGP and (parallel) PCI slots may have issues due to aliasing, and reference the error messages we add here.

Patch set 2:Code-Review +1

1 comment:

File src/device/device.c:
- Patch Set #2, Line 799:
  Can we add this:
```
   bool vga_io_10bit_decode(void)
   {
      return (vga_pri && vga_pri->bus->no_vga16);    
   }
```
  I might try to fix the 'aliasing avoidance' or just throw errors after resource assignments.

To view, visit change 35516. To unsubscribe, or for help writing mail filters, visit settings.