lkml.org 
[lkml]   [2008]   [Apr]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH 2/5] /dev/vring: simple userspace-kernel ringbuffer interface.
Date
On Saturday 19 April 2008 20:22:15 Evgeniy Polyakov wrote:
> Hi.
>
> On Fri, Apr 18, 2008 at 02:39:48PM +1000, Rusty Russell
(rusty@rustcorp.com.au) wrote:
> > +int vring_get_buffer(struct vring_info *vr,
> > + struct iovec *in_iov,
> > + unsigned int *num_in, unsigned long *in_len,
> > + struct iovec *out_iov,
> > + unsigned int *num_out, unsigned long *out_len)
> > +{
> > + unsigned int i, in = 0, out = 0;
> > + unsigned long dummy;
> > + u16 avail, last_avail, head;
> > + struct vring_desc d;
>
> Should this whole function and vring_used_buffer() be protected with
> vr->lock mutex?

No; it's up to the caller to make sure that they are serialized. In the case
of tun that happens naturally.

There are two reasons not to grab the lock. It turns out that if we tried to
lock here, we'd deadlock, since the callbacks are called under the lock.
Secondly, it's possible to implement an atomic vring_used_buffer variant,
which could fail: this would avoid using the thread most of the time.

Hope that helps,
Rusty.


\
 
 \ /
  Last update: 2008-04-19 18:09    [W:0.123 / U:0.004 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site