On 10/03/16 15:04, Ladi Prosek wrote:
On Thu, Mar 10, 2016 at 3:12 PM, Michael S. Tsirkin <m...@redhat.com> wrote:
On Wed, Mar 09, 2016 at 07:20:03PM +0100, Ladi Prosek wrote:
+ /* Enable PCI bus master and reset NIC */
+ adjust_pci_device ( pci );
At the moment, this enables memory and io unconditionally. But I think
it should check BAR is assigned before enabling and using it, since
e.g. pci express devices might not have their BARs assigned.
I'm also not sure 64 bit BAR values will work under ipxe -
if not it's preferably to fall back on io or cfg access.
Hmm.. 64-bit BARs are indeed not supported. It would be better to
always use the cfg cap then and remove all that BAR mapping business.
Less code and fewer assumptions for the win.
64-bit BARs work as of about two weeks ago. You can now build iPXE to
run as a 64-bit binary under BIOS (e.g. "make bin-x86_64-pcbios/ipxe.usb").
Michael
_______________________________________________
ipxe-devel mailing list
ipxe-devel@lists.ipxe.org
https://lists.ipxe.org/mailman/listinfo.cgi/ipxe-devel
