On 05/20/10 16:45, James Carlson wrote:
> Rishi Srivatsavai wrote:
>> -       if (mip->mi_bridge_link != NULL)
>> -               return (cap == MAC_CAPAB_NO_ZCOPY);
>> +       if (mip->mi_bridge_link != NULL && cap == MAC_CAPAB_NO_ZCOPY)
>> +               return (B_FALSE);
> 
> I think you want to return B_TRUE in this case.  MAC_CAPAB_NO_ZCOPY has
> inverted sense; "true" means "don't do this."

Ah yes my bad.

> Otherwise, I think this is ok.

Thank you for the quick review.

Rishi
_______________________________________________
rbridges-dev mailing list
[email protected]
http://mail.opensolaris.org/mailman/listinfo/rbridges-dev

Reply via email to