Re: svn commit: r240427 - head/sys/dev/virtio

2012-09-14 Thread Konstantin Belousov
...@freebsd.org Sent: Thursday, September 13, 2012 1:56:03 PM Subject: Re: svn commit: r240427 - head/sys/dev/virtio On Thursday, September 13, 2012 12:40:42 pm Bryan Venteicher wrote: Would it be possible to use atomic_load/store() instead of direct memory barriers? For example

Re: svn commit: r240427 - head/sys/dev/virtio

2012-09-14 Thread John Baldwin
On Friday, September 14, 2012 3:55:20 am Konstantin Belousov wrote: On Fri, Sep 14, 2012 at 12:47:52AM -0500, Bryan Venteicher wrote: I also found myself wanting an atomic_load_rel_*() type function. That would be odd I think. _rel barriers only affect stores, so there would be no

Re: svn commit: r240427 - head/sys/dev/virtio

2012-09-14 Thread Bryan Venteicher
, September 14, 2012 7:17:54 AM Subject: Re: svn commit: r240427 - head/sys/dev/virtio On Friday, September 14, 2012 3:55:20 am Konstantin Belousov wrote: On Fri, Sep 14, 2012 at 12:47:52AM -0500, Bryan Venteicher wrote: I also found myself wanting an atomic_load_rel_*() type function

Re: svn commit: r240427 - head/sys/dev/virtio

2012-09-13 Thread John Baldwin
On Wednesday, September 12, 2012 8:36:47 pm Peter Grehan wrote: Author: grehan Date: Thu Sep 13 00:36:46 2012 New Revision: 240427 URL: http://svn.freebsd.org/changeset/base/240427 Log: Relax requirement of certain mb()s Submitted by: Bryan Venteicher bryanv at

Re: svn commit: r240427 - head/sys/dev/virtio

2012-09-13 Thread Bryan Venteicher
Hi, - Original Message - From: John Baldwin j...@freebsd.org To: Peter Grehan gre...@freebsd.org Cc: svn-src-head@freebsd.org, svn-src-...@freebsd.org, src-committ...@freebsd.org Sent: Thursday, September 13, 2012 7:59:38 AM Subject: Re: svn commit: r240427 - head/sys/dev/virtio

Re: svn commit: r240427 - head/sys/dev/virtio

2012-09-13 Thread John Baldwin
, 2012 7:59:38 AM Subject: Re: svn commit: r240427 - head/sys/dev/virtio On Wednesday, September 12, 2012 8:36:47 pm Peter Grehan wrote: Author: grehan Date: Thu Sep 13 00:36:46 2012 New Revision: 240427 URL: http://svn.freebsd.org/changeset/base/240427 Log: Relax

Re: svn commit: r240427 - head/sys/dev/virtio

2012-09-13 Thread John Baldwin
On Thursday, September 13, 2012 12:40:42 pm Bryan Venteicher wrote: Would it be possible to use atomic_load/store() instead of direct memory barriers? For example: I've been sitting on a (lightly tested) patch [1] for awhile that does just that, but am not very happy with it. A lot of

Re: svn commit: r240427 - head/sys/dev/virtio

2012-09-13 Thread Bryan Venteicher
: Re: svn commit: r240427 - head/sys/dev/virtio On Thursday, September 13, 2012 12:40:42 pm Bryan Venteicher wrote: Would it be possible to use atomic_load/store() instead of direct memory barriers? For example: I've been sitting on a (lightly tested) patch [1] for awhile that does

svn commit: r240427 - head/sys/dev/virtio

2012-09-12 Thread Peter Grehan
Author: grehan Date: Thu Sep 13 00:36:46 2012 New Revision: 240427 URL: http://svn.freebsd.org/changeset/base/240427 Log: Relax requirement of certain mb()s Submitted by: Bryan Venteicher bryanv at daemoninthecloset org Modified: head/sys/dev/virtio/virtqueue.c Modified: