Re: [PATCH] virtio: include asm/barrier explicitly

2013-07-07 Thread Stephen Rothwell
Hi Rusty, On Mon, 08 Jul 2013 11:31:45 +0930 Rusty Russell wrote: > > PS. I'll squeeze these into this merge window, since they're trivial. s/'re trivial/fix bugs/ ;-) -- Cheers, Stephen Rothwells...@canb.auug.org.au pgpyfEvEm5Jnm.pgp Description: PGP signature

Re: [PATCH] virtio: include asm/barrier explicitly

2013-07-07 Thread Rusty Russell
"Michael S. Tsirkin" writes: > virtio_ring.h uses mb() and friends, make > it pull in asm/barrier.h itself, not rely > on other headers to do it. > > Signed-off-by: Michael S. Tsirkin Applied. Thanks, Rusty. PS. I'll squeeze these into this merge window, since they're trivial. -- To

Re: [PATCH] virtio: include asm/barrier explicitly

2013-07-07 Thread Michael S. Tsirkin
On Sun, Jul 07, 2013 at 05:20:19PM +0300, Michael S. Tsirkin wrote: > virtio_ring.h uses mb() and friends, make > it pull in asm/barrier.h itself, not rely > on other headers to do it. > > Signed-off-by: Michael S. Tsirkin In particular this fixes warnings during test build: [linux]$ make -C

[PATCH] virtio: include asm/barrier explicitly

2013-07-07 Thread Michael S. Tsirkin
virtio_ring.h uses mb() and friends, make it pull in asm/barrier.h itself, not rely on other headers to do it. Signed-off-by: Michael S. Tsirkin --- include/linux/virtio_ring.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/linux/virtio_ring.h b/include/linux/virtio_ring.h index

[PATCH] virtio: include asm/barrier explicitly

2013-07-07 Thread Michael S. Tsirkin
virtio_ring.h uses mb() and friends, make it pull in asm/barrier.h itself, not rely on other headers to do it. Signed-off-by: Michael S. Tsirkin m...@redhat.com --- include/linux/virtio_ring.h | 1 + 1 file changed, 1 insertion(+) diff --git a/include/linux/virtio_ring.h

Re: [PATCH] virtio: include asm/barrier explicitly

2013-07-07 Thread Michael S. Tsirkin
On Sun, Jul 07, 2013 at 05:20:19PM +0300, Michael S. Tsirkin wrote: virtio_ring.h uses mb() and friends, make it pull in asm/barrier.h itself, not rely on other headers to do it. Signed-off-by: Michael S. Tsirkin m...@redhat.com In particular this fixes warnings during test build: [linux]$

Re: [PATCH] virtio: include asm/barrier explicitly

2013-07-07 Thread Rusty Russell
Michael S. Tsirkin m...@redhat.com writes: virtio_ring.h uses mb() and friends, make it pull in asm/barrier.h itself, not rely on other headers to do it. Signed-off-by: Michael S. Tsirkin m...@redhat.com Applied. Thanks, Rusty. PS. I'll squeeze these into this merge window, since they're

Re: [PATCH] virtio: include asm/barrier explicitly

2013-07-07 Thread Stephen Rothwell
Hi Rusty, On Mon, 08 Jul 2013 11:31:45 +0930 Rusty Russell ru...@rustcorp.com.au wrote: PS. I'll squeeze these into this merge window, since they're trivial. s/'re trivial/fix bugs/ ;-) -- Cheers, Stephen Rothwells...@canb.auug.org.au pgpyfEvEm5Jnm.pgp Description: