Hi,
We can't hard-wire the PCI_COMMAND_IO to 0 since virtio are(may be) 
transitional devices that support both IO and MEM. Since the device supports 
both modes is up to firmware/OS to choose a working mode. Once the mode is 
chosen it is expected to update the command register accordingly. This is why 
PCI spec has separate bits for IO/MEM . If we dig into virtio implementation we 
may find that we don't care if both bits are on and we may also be able to fix 
any potential problems.

If every byte is important for ipxe you may leave the patches out, consciously 
creating an "unfriendly"  driver (QEMU will create an io-region for that and 
will have some 'if' statements with politically correct comments: "Some drivers 
enable IO even if .."  :)  )

Bottom line, I would add the patches but definitely there are not a must.
I hope I helped,
Marcel

---
You are receiving this because you commented.
Reply to this email directly or view it on GitHub:
https://github.com/ipxe/ipxe/pull/55#issuecomment-227378966
_______________________________________________
ipxe-devel mailing list
ipxe-devel@lists.ipxe.org
https://lists.ipxe.org/mailman/listinfo.cgi/ipxe-devel

Reply via email to