From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mailman by lists.gnu.org with tmda-scanned (Exim 4.43) id 1O4Z9O-0002LD-5g for qemu-devel@nongnu.org; Wed, 21 Apr 2010 08:35:30 -0400 Received: from [140.186.70.92] (port=54793 helo=eggs.gnu.org) by lists.gnu.org with esmtp (Exim 4.43) id 1O4Z9D-0002CZ-ML for qemu-devel@nongnu.org; Wed, 21 Apr 2010 08:35:29 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.69) (envelope-from ) id 1O4Z9B-0000A6-Jk for qemu-devel@nongnu.org; Wed, 21 Apr 2010 08:35:19 -0400 Received: from bhuna.collabora.co.uk ([93.93.128.226]:53599) by eggs.gnu.org with esmtp (Exim 4.69) (envelope-from ) id 1O4Z9B-00009i-FO for qemu-devel@nongnu.org; Wed, 21 Apr 2010 08:35:17 -0400 Message-ID: <4BCEF0B9.2050704@collabora.co.uk> Date: Wed, 21 Apr 2010 13:34:01 +0100 From: Ian Molton MIME-Version: 1.0 Subject: Re: [Qemu-devel] [PATCH 2/2] VirtIO RNG References: <4BB2053C.6000701@collabora.co.uk> <201004031606.26893.paul@codesourcery.com> <4BC482A6.4040504@collabora.co.uk> <201004131632.25820.paul@codesourcery.com> <4BCDC51F.2030205@collabora.co.uk> <20100420161302.GA11723@shareable.org> <4BCE061B.2030506@collabora.co.uk> <20100420205654.GI11723@shareable.org> <4BCE1D3B.7000306@collabora.co.uk> <4BCEAC99.8000206@redhat.com> <20100421094007.GC13114@shareable.org> In-Reply-To: <20100421094007.GC13114@shareable.org> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit List-Id: qemu-devel.nongnu.org List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Jamie Lokier Cc: qemu-devel@nongnu.org, Gerd Hoffmann , Paul Brook Jamie Lokier wrote: > But enough of that: It's history now; the guest virtio-rng has existed > for more than a year. It is also amazingly short and simple. Yay for Rusty! Ok, so... If we can now take steps to integrate the code... I'd like to get some feedback (or merging!) of the other parts of the patch, namely:- * the SIZE property patch: Msg-Id: <4BB206B9.80501@collabora.co.uk> * the socket reconnect patch: Msg-Id: <4B18055B.1030606@collabora.co.uk> If we can get these patches sorted out, the only outstanding issues are where the EGD protocol support and rate limiting should go. IMO, both are so small and simple that I'd leave them in the driver, however I'm open to (realistic) suggestions on how else to go about this. AIUI qemus chardev layer doesnt 'do' line disciplines, which complicates matters. For reference, the EGD protocol is implemented in 5 lines of code, and rate limiting in about 10. I don't mind changing qemu_gettimeofday for some other counter (does qemu provide a monotonic counter (from the hosts perspective) of some sort?) - if anyone is /really/ bothered by the fact that one buffer fill could happen at 2x the set rate should the timeofday be glitched. :-D -Ian