From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:46178) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gP582-0007IR-Ej for qemu-devel@nongnu.org; Tue, 20 Nov 2018 07:27:43 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gP57z-0000yV-B2 for qemu-devel@nongnu.org; Tue, 20 Nov 2018 07:27:42 -0500 Received: from mx1.redhat.com ([209.132.183.28]:38310) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1gP57z-0000xv-2l for qemu-devel@nongnu.org; Tue, 20 Nov 2018 07:27:39 -0500 Message-ID: <9bc6ced6b0491643775081b8f8437065684e9959.camel@redhat.com> From: Andrea Bolognani Date: Tue, 20 Nov 2018 13:27:05 +0100 In-Reply-To: <20181119140813-mutt-send-email-mst@kernel.org> References: <20181114233831.10374-1-ehabkost@redhat.com> <20181116034551.GK3807@habkost.net> <20181119114105.4da89f2c.cohuck@redhat.com> <20181119125519-mutt-send-email-mst@kernel.org> <20181119193238.117c2f4e.cohuck@redhat.com> <20181119133434-mutt-send-email-mst@kernel.org> <20181119195638.49a9c21e.cohuck@redhat.com> <20181119140813-mutt-send-email-mst@kernel.org> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH for-4.0 v2] virtio: Provide version-specific variants of virtio PCI devices List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: "Michael S. Tsirkin" , Cornelia Huck Cc: Eduardo Habkost , qemu-devel@nongnu.org, Gonglei , Paolo Bonzini , Amit Shah , Cleber Rosa , Marcel Apfelbaum , Fam Zheng , Kevin Wolf , Max Reitz , Jason Wang , Wainer dos Santos Moschetta , Philippe =?ISO-8859-1?Q?Mathieu-Daud=E9?= , libvir-list@redhat.com, Markus Armbruster , Laine Stump , Stefan Hajnoczi , Gerd Hoffmann , "Daniel P. =?ISO-8859-1?Q?Berrang=E9?=" , Caio Carrara On Mon, 2018-11-19 at 14:14 -0500, Michael S. Tsirkin wrote: > On Mon, Nov 19, 2018 at 07:56:38PM +0100, Cornelia Huck wrote: > > On Mon, 19 Nov 2018 13:42:58 -0500 "Michael S. Tsirkin" wrote: > > > We have this assumption that if we force a choice then people will > > > choose the right thing but in practice they will do what we all do, play > > > with it until it kind of works and leave well alone afterwards. > > > That's at best - at worst give up and use an easier tool. > > > > That implies that we (the developers) need to care and make sure that > > "model=virtio" gets them the best possible transport (i.e. on s390x, > > that would be ccw unless the user explicitly requests pci; I'm not sure > > what the situation with mmio is -- probably "use pci whenever > > possible"?) I think that's what libvirt already gives us today (I hope.) The interface at the libvirt level is exactly "model=virtio", with that ultimately translating to virtio-*-pci or virtio-*-ccw or virtio-*-device or whatever else based on the architecture, machine type and other information about the guest. > > What makes it messy on the pci side is that the "best option" actually > > depends on what kind of guest the user wants to run (if the guest is > > too old, you're stuck with transitional; if you want to reap the > > benefits of PCIe, you need non-transitional...) > > Well it works now - connect it to a bus and it figures out whether it > should do transitional or not. You can force transitional in PCIe anyway > but then you are limited to about 15 devices - probably sufficient for > most people ... That's not how it works, though: current virtio-*-pci devices will be transitional (and thus support older guest OS) or not based on the kind of slot you plug them into. >>From the management point of view that's problematic, because libvirt (which takes care of the virtual hardware, including assigning PCI addresses to devices) has no knowledge of the guest OS running on said hardware, and management apps (which know about the guest OS and can figure out its capabilities using libosinfo) don't want to be in the business of assigning PCI addresses themselves. Having separate transitional and non-transitional variants solves the issue because now management apps can query libosinfo to figure out whether the guest OS supports non-transitional virtio devices, and based on that they can ask libvirt to use either the transitional or non-transitional variant; from that, libvirt will be able to choose the correct slot for the device. None of the above quite works if we have a single variant that morphs based on the slot, as we have today. -- Andrea Bolognani / Red Hat / Virtualization