From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:39703) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UCN23-0002p4-Nx for qemu-devel@nongnu.org; Sun, 03 Mar 2013 23:29:51 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UCN1z-0005ck-Cx for qemu-devel@nongnu.org; Sun, 03 Mar 2013 23:29:47 -0500 Received: from mx1.redhat.com ([209.132.183.28]:41972) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UCN1z-0005cb-4x for qemu-devel@nongnu.org; Sun, 03 Mar 2013 23:29:43 -0500 Date: Mon, 4 Mar 2013 09:59:35 +0530 From: Amit Shah Message-ID: <20130304042935.GA8219@amit.redhat.com> References: <512FF819.7050505@redhat.com> <51307A25.9030703@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <51307A25.9030703@redhat.com> Subject: Re: [Qemu-devel] virtio-rng and fd passing List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Paolo Bonzini Cc: "qemu-devel@nongnu.org" , Stefan Berger On (Fri) 01 Mar 2013 [10:51:33], Paolo Bonzini wrote: > Il 01/03/2013 01:36, Eric Blake ha scritto: > > For fd passing to work, we have to use qemu_open() instead of raw > > open(). Is there any way to enforce that all files being opened by qemu > > go through the appropriate qemu_open() wrapper? > > > > Meanwhile, we have a quandary on the libvirt side of things: qemu 1.4 > > supports fd passing in general, but does not support it for rng. I > > guess the same is true for -blockdev - we don't (yet) have a way to do > > fd passing for backing files. Do we need some sort of QMP command that > > will let libvirt query for a particular device whether that device is > > known to support fd passing, so that libvirt can use fd passing for all > > supported devices, while falling back to older direct open()s, and to > > know which instance of qemu can safely have open() blocked at the > > SELinux or syscall blacklist level? > > Let's change open to qemu_open for 1.4.1, and declare rng only supported > in 1.4.1... In addition, we can tweak checkpatch.pl to flag such usage as a warning. Amit