Dear MdeModulePkg maintainer,

On September 14 SVN rev 16104 there was a change to SnpDxe to improve the 
detection of BARs to replace the hardcoded MemoryBarIndex=0 and IoBarIndex=1 
with some intelligent scanning using PCI_IO GetBarAttributes instead.

This change had the side effect of breaking our UNDI drivers (from two 
different manufacturers) because of a requirement that both IO and Memory BARs 
exist:

   if ((Snp->MemoryBarIndex == PCI_MAX_BAR) || (Snp->IoBarIndex == 
PCI_MAX_BAR)) {
     goto Error_DeleteSNP;
   }

Our NICs only implement memory BARs and have no IO bars so this check fails and 
SnpDxe will no longer bind to these adapters.

I'm not aware of a requirement that adapters must implement both Memory and IO 
BARs.  I'm thinking SnpDxe should handle all of these cases and not just the 
dual memory- and io-BAR case.

Eugene
 

------------------------------------------------------------------------------
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel

Reply via email to