From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:51650) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Tkwti-0001Lu-Jr for qemu-devel@nongnu.org; Tue, 18 Dec 2012 08:07:57 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Tkwta-0006wr-Ff for qemu-devel@nongnu.org; Tue, 18 Dec 2012 08:07:50 -0500 Received: from mx1.redhat.com ([209.132.183.28]:6884) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Tkwta-0006wM-6P for qemu-devel@nongnu.org; Tue, 18 Dec 2012 08:07:42 -0500 Date: Tue, 18 Dec 2012 15:10:43 +0200 From: "Michael S. Tsirkin" Message-ID: <20121218131043.GA26110@redhat.com> References: <1354887155-32281-1-git-send-email-fred.konrad@greensocs.com> <20121217154508.GA28712@redhat.com> <20121218110153.GC22586@redhat.com> <50D05898.9030506@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Subject: Re: [Qemu-devel] [RFC PATCH v6 0/6] Virtio refactoring. List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: aliguori@us.ibm.com, e.voevodin@samsung.com, mark.burton@greensocs.com, qemu-devel@nongnu.org, stefanha@redhat.com, cornelia.huck@de.ibm.com, Paolo Bonzini , afaerber@suse.de, fred.konrad@greensocs.com On Tue, Dec 18, 2012 at 12:06:39PM +0000, Peter Maydell wrote: > On 18 December 2012 11:50, Paolo Bonzini wrote: > > Il 18/12/2012 12:26, Peter Maydell ha scritto: > >> On 18 December 2012 11:01, Michael S. Tsirkin wrote: > >>> This is what I am saying: create your own bus and put > >>> your devices there. > >> > >> What bus? > > > > A virtio bus like the one in these patches. But mst is suggesting to > > leave virtio.c aside, and only use the virtio bus in the virtio-x-mmio > > devices, to connect to the virtio-mmio device provided by the board. > > That doesn't make any sense to me -- why would you want to make > mmio be randomly different from the other virtio transports? Not different at all. virtio-pci uses the pci bus. It's you who's saying mmio is different and can not work with existing bindings. > The code > as it stands is a mess which ought to be cleaned up anyway... My experience is all working code is somewhat messy. This is working code. I'm sure we shouldnot add more ways to create devices, we have two for each device already, that's too much. You can't seriously add yet another way, keep the two existing ones around as legacy and claim it's a cleanup. You are single-handedly making the testing matrix bigger by 1/3. And what makes virtio so special anyway? e1000 can be used without exposing users to internal buses and all kind of nastiness like this. People just want to install a driver and have a faster IO. > To the extent that it's painful for users to manipulate qdev devices on > the command line, that's a problem we ought to address at some point > anyhow. > > -- PMM It's pretty painful, yes, but adding yet another way to do it is not addressing the problem. -- MST