Re: [Xen-devel] [PATCH 13/45] xen: arm64: dcache flush

[Tim Deegan <tim@xxxxxxx>]

At 15:56 +0000 on 23 Jan (1358956579), Ian Campbell wrote:
> This drops the
>     "m" (*_p)
> asm contraint. I can't figure out what this was for.
> 

It's to stop the compiler hoisting writes to the wrong side of the
flush.  It's the lighter-weight equivalent to the "memory" clobber in
the range version.  By moving from inline "dsb;" to dsb() you've
reintroduced the full memory clobber here anyway. :(


_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxx
http://lists.xen.org/xen-devel