[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <200804080834.56943.rusty@rustcorp.com.au>
Date: Tue, 8 Apr 2008 08:34:55 +1000
From: Rusty Russell <rusty@...tcorp.com.au>
To: Jonathan Corbet <corbet@....net>
Cc: netdev@...r.kernel.org, virtualization@...ts.linux-foundation.org,
Max Krasnyansky <maxk@...lcomm.com>,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH RFC 1/5] vringfd syscall
On Tuesday 08 April 2008 03:54:34 Jonathan Corbet wrote:
> Hey, Rusty,
>
> > For virtualization, we've developed virtio_ring for efficient
> > communication. This would also work well for userspace-kernel
> > communication, particularly for things like the tun device. By using the
> > same ABI, we can join guests to the host kernel trivially.
>
> I'm *sure* you meant to document that somewhat non-trivial proposed new
> kernel API as soon as you got a moment.
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@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists