From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:43887) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1eEFGk-0007Tu-IU for qemu-devel@nongnu.org; Mon, 13 Nov 2017 08:59:23 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1eEFGh-0002ux-Ga for qemu-devel@nongnu.org; Mon, 13 Nov 2017 08:59:22 -0500 Received: from mx1.redhat.com ([209.132.183.28]:56168) by eggs.gnu.org with esmtps (TLS1.0:DHE_RSA_AES_256_CBC_SHA1:32) (Exim 4.71) (envelope-from ) id 1eEFGh-0002uj-AQ for qemu-devel@nongnu.org; Mon, 13 Nov 2017 08:59:19 -0500 References: <1504286483-23327-1-git-send-email-eric.auger@redhat.com> <1504286483-23327-16-git-send-email-eric.auger@redhat.com> <40a0ece3-7498-0dda-9e13-234ba09bedf1@redhat.com> <0e23f2bb-a3be-2ba5-5d90-2b7c42b68d5b@redhat.com> From: Auger Eric Message-ID: Date: Mon, 13 Nov 2017 14:59:15 +0100 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH v7 15/20] hw/arm/sysbus-fdt: Pass the VirtMachineState to the node creation functions List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: QEMU Developers Hi, On 13/11/2017 14:44, Peter Maydell wrote: > On 13 November 2017 at 13:37, Auger Eric wrote: >> On 13/11/2017 14:08, Peter Maydell wrote: >>> Not being able to conveniently wire up a sysbus device on the >>> command line or pass it options are general problems. I don't >>> think the SMMU is a special case that should work around these >>> general issues by being created in a different way to everything >>> else. If the "hard to pass options to the device" problem needs >>> solving (which it does anyway if we want to drop '-net' for >>> configuring embedded ethernet devices) we should solve it, >>> not have some small set of sysbus devices be weirdly magic. >> do you have examples of other SysbusDevices whose instantiation is made >> conditional with "-device" option and which address the dt node/ACPI >> table creation in a more standard manner? Or do you want me to drop the >> "-device" requirement. > > I think that you just can't create sysbus devices with -device. > They're hardwired into the machine model. OK > >> I may manage reaching my goal with yet another machine init done >> notifier that would create the dt node from virt code at fixed base >> address. But this solution still may be be virt specific. Is it the >> direction you want me to follow at the moment? > > I still don't really understand why the SMMU has to be any > different from the UART, or the PCI controller, or any of > the other devices in the virt board model. None of those > try to be creatable from the command line. To me the difference is, if you don't want to use UART or PCI controller, you don't suffer performance downgrade if they are instantiated. On the opposite, if the SMMU is instantiated whereas you don't need his functionality you suffer performance downgrade. Thanks Eric > > thanks > -- PMM >