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