On Fri, May 23, 2008 at 02:53:21PM +1000, Paul Mackerras wrote: > Nick Piggin writes: > > > There don't seem to actually be read*_relaxed calls that also use rmb > > in the same file (although there is no reason why they might not appear). > > But I must be thinking of are the raw_read accessors. They aren't ordered > > on powerpc, and a few drivers appear to hope rmb() will order them. > > Which drivers?
net/macb.c, net/sfc/falcon_io.c... hmm, I thought there was a couple more but maybe I was thinking of write side. Anyway, even if there were zero, then the point is still that you implement that API, so you should either strongly order your __raw_ and _relaxed then you can weaken your rmb, or you have to strengthen your rmb to match your weak read ops. Saying it doesn't matter because there are no drivers is likely to cause more headaches in future... _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev
