[PATCH] ucc_geth: Rework the TX logic.

Li Yang leoli at freescale.com
Tue Mar 31 19:16:58 EST 2009


On Tue, Mar 31, 2009 at 1:22 AM, Scott Wood <scottwood at freescale.com> wrote:
> Joakim Tjernlund wrote:
>>
>> gianfar does not seem to use in_/out_ functions for the BDs. Works just
>> fine that too it seems.
>
> It does now that it has explicit barriers in a few places.  Before they were
> added, it would sometimes fail under load.  That was due to a compiler
> reordering, but CPU reordering was possible as well.

I noticed that in gianfar these memory access is not protected by
"volatile".  Can this be the reason why the compiler did some unwanted
optimization?

- Leo



More information about the Linuxppc-dev mailing list