login
Header Space

 
 

Re: [PATCH RFC 1/5] vringfd syscall

Score:
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]
To: Jonathan Corbet <corbet@...>
Cc: <netdev@...>, <virtualization@...>, Max Krasnyansky <maxk@...>, <linux-kernel@...>
Date: Monday, April 7, 2008 - 6:34 pm

On Tuesday 08 April 2008 03:54:34 Jonathan Corbet wrote:

Actually, yes.  But I wanted to get it out there before I start the treck 
across to the virtualization summit.

A few points:
'The page alignment for the used array is important - that array might be 
mapped separately into kernel space.'
   Well, the used array is written by one side only, so it's possible to split 
the ring here and make each part r/o to the other side.  More importantly, a 
page boundary is almost certainly a cacheline boundary, and we already have a 
userspace interface for it.

'Note that the flags fields in the vring_avail and vring_used structures 
appear to be unused.'
   virtio uses these for wakeup/interrupt suppression.  It's a cheap way to 
avoid hypercalls, and we can use them the same way to avoid system calls (you 
set the suppression bit while you're actually looking at the ring).

The need for the kmap (and hence the atomic horror) has now been alleviated: I 
changed the shinfo destructor code to allow the destructor to hold onto the 
skb data so it can queue it and free it later.

BTW, the only place currently where both output and input buffers are used is 
the virtio_blk driver doing a read, where the header describes the operation, 
and the other buffers are overwritten with the data.

Thanks!
Rusty.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Previous message: [thread] [date] [author]
Next message: [thread] [date] [author]

Messages in current thread:
[PATCH RFC 1/5] vringfd syscall, Rusty Russell, (Sat Apr 5, 8:02 am)
Re: [PATCH RFC 1/5] vringfd syscall, Marcelo Tosatti, (Sat Apr 12, 1:18 pm)
Re: [PATCH RFC 1/5] vringfd syscall, Rusty Russell, (Sat Apr 12, 2:19 pm)
Re: [PATCH RFC 1/5] vringfd syscall, Marcelo Tosatti, (Sat Apr 12, 1:39 pm)
Re: [PATCH RFC 1/5] vringfd syscall, Jeremy Fitzhardinge, (Wed Apr 9, 3:28 pm)
Re: [PATCH RFC 1/5] vringfd syscall, Arnd Bergmann, (Mon Apr 7, 10:35 pm)
Re: [PATCH RFC 1/5] vringfd syscall , Jonathan Corbet, (Mon Apr 7, 1:54 pm)
Re: [PATCH RFC 1/5] vringfd syscall, Rusty Russell, (Mon Apr 7, 6:34 pm)
[PATCH RFC 2/5] vringfd base/offset, Rusty Russell, (Sat Apr 5, 8:04 am)
Re: [PATCH RFC 2/5] vringfd base/offset, Arnd Bergmann, (Tue Apr 8, 1:14 am)
Re: [PATCH RFC 2/5] vringfd base/offset, Avi Kivity, (Sat Apr 5, 8:44 am)
Re: [PATCH RFC 2/5] vringfd base/offset, Rusty Russell, (Sat Apr 5, 10:54 pm)
[PATCH RFC 3/5] tun: vringfd receive support., Rusty Russell, (Sat Apr 5, 8:05 am)
Re: [PATCH RFC 3/5] tun: vringfd receive support., Max Krasnyansky, (Tue Apr 8, 3:49 pm)
Re: [PATCH RFC 3/5] tun: vringfd receive support., Rusty Russell, (Thu Apr 10, 1:44 am)
Re: [PATCH RFC 3/5] tun: vringfd receive support., Max Krasnyanskiy, (Thu Apr 10, 1:18 pm)
Re: [PATCH RFC 3/5] tun: vringfd receive support., Max Krasnyanskiy, (Thu Apr 10, 1:02 pm)
[PATCH RFC 4/5] tun: vringfd xmit support., Rusty Russell, (Sat Apr 5, 8:06 am)
Re: [PATCH RFC 4/5] tun: vringfd xmit support., Herbert Xu, (Mon Apr 7, 1:13 am)
Re: [PATCH RFC 4/5] tun: vringfd xmit support., Rusty Russell, (Mon Apr 7, 3:24 am)
Re: [PATCH RFC 4/5] tun: vringfd xmit support., David Miller, (Mon Apr 7, 3:35 am)
Re: [PATCH RFC 4/5] tun: vringfd xmit support., Rusty Russell, (Mon Apr 7, 9:51 pm)
[PATCH RFC 5/5] lguest support, Rusty Russell, (Sat Apr 5, 8:09 am)
speck-geostationary