Roland Dreier wrote:
 > So after all we do need a flag?

No, after thinking about it I don't think there's any reason to use
__GFP_HIGHMEM... it would use less kernel memory on highmem systems
but I don't think it really helps in the end.

I see that none of the drivers for NICs which set the NETIF_F_HIGHDMA bit use some _HIGHMEM GFP directive for their skb allocation, so maybe the two flags need not be correlated. So the NETIF_F_HIGHDMA just comes to say that if from some reason an skb which was not allocated by the driver points to buffer in high-memory, its fine.

Anyway, I am fine with the patch once the change-log comment is changed AND the ipath issue is resolved with agreement of the parties (Roland, Ralph), if no device capability flag nor implementing high-mem support in the ipath driver is needed, let it be.

Or.

_______________________________________________
general mailing list
[email protected]
http://lists.openfabrics.org/cgi-bin/mailman/listinfo/general

To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general

Reply via email to