[U-Boot] [PATCH] arm: Kirkwood: Add memory barriers to kwgbe_send/recv
Prafulla Wadaskar
prafulla at marvell.com
Thu Jul 2 10:12:08 CEST 2009
> -----Original Message-----
> From: Simon Kagstrom [mailto:simon.kagstrom at netinsight.net]
> Sent: Thursday, July 02, 2009 12:17 PM
> To: Prafulla Wadaskar
> Cc: U-Boot ML
> Subject: Re: [U-Boot] [PATCH] arm: Kirkwood: Add memory
> barriers to kwgbe_send/recv
>
> On Wed, 1 Jul 2009 09:57:10 -0700
> Prafulla Wadaskar <prafulla at marvell.com> wrote:
> > > kwgbe_send/recv both have loops waiting for the hardware
> to set a
> > > bit.
> > > GCC 4.3.3 cleverly optimizes this to ... a while(1); loop.
> > Struct used is defined as volatile here.
> > Is this not sufficient?
>
> Right, but that was only for recv - send isn't volatile. I'll
> send a new patch with the required changes. The question is which :-):
>
> 1. Make structure volatile in send as well
Hi Simon
I suggest to implement this change
Regards..
Prafulla . .
More information about the U-Boot
mailing list