On Mon, 2011-10-10 at 09:20 +0100, David Laight wrote: > > For the above I'd actually suggest making 'doorbell_qpn' have the > correct endianness in order to avoid the (potential) swap every > time it is set.
Well, the problem is that either you'll end up swapping on x86 or you'll end up swapping on ppc, there is no "native" MMIO accessor that allow you to do a no-swap access whatever the arch you are on. Or rather, there is the __raw_ one but you shouldn't use it for most things :-) (Because it also doesn't have the right memory barriers). So I'd rather they do it right using the simpler method, the cost of swap is going to be negligible, probably not even measurable, and if and only if they think they can improve on that in a second step, then consider doing otherwise with appropriate measurements showing a significant difference. > You also need to treble-check the required endianness for the > 'vlan_tag' in the tx descriptor. What would be needed is the > MAC PCI slave were on an x86 (LE) system. Cheers, Ben. _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@lists.ozlabs.org https://lists.ozlabs.org/listinfo/linuxppc-dev