On Tue, 25 Mar 2008 07:59:15 +1100
Benjamin Herrenschmidt <[EMAIL PROTECTED]> wrote:
>
> On Mon, 2008-03-24 at 08:39 -0500, Josh Boyer wrote:
> > > +
> > > + local_irq_save(flags);
> > > + asm volatile ("sync" ::: "memory");
> >
> > Perhaps just call iosync() for these instead of the open coded asm
> > volatile stuff?
>
> Not sure about that. iosync() will do a sync but it's not meant at being
> used for things unrelated to IOs (and who knows... we might make it do
> something else one day ?).
>
> In low level arch code like that, an open coded sync might be proper
> especially if it matches some documented sequence.
Either way, it doesn't matter to me.
josh
_______________________________________________
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev
