From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:39222) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WLnOo-00012p-OQ for qemu-devel@nongnu.org; Fri, 07 Mar 2014 00:32:52 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WLnOi-00016O-OJ for qemu-devel@nongnu.org; Fri, 07 Mar 2014 00:32:46 -0500 Received: from mx1.redhat.com ([209.132.183.28]:64268) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WLnOi-00016H-Ge for qemu-devel@nongnu.org; Fri, 07 Mar 2014 00:32:40 -0500 Message-ID: <1394170376.2663.24.camel@localhost.localdomain> From: Marcel Apfelbaum Date: Fri, 07 Mar 2014 07:32:56 +0200 In-Reply-To: <5319087A.3090403@suse.de> References: <1394040647-20083-1-git-send-email-marcel.a@redhat.com> <1394040647-20083-4-git-send-email-marcel.a@redhat.com> <5319087A.3090403@suse.de> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH v3 3/3] hw/boards: converted current_machine to be an instance of MachineCLass List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Andreas =?ISO-8859-1?Q?F=E4rber?= Cc: qemu-devel@nongnu.org, ehabkost@redhat.com, mst@redhat.com, Alexey Kardashevskiy , armbru@redhat.com, lcapitulino@redhat.com, blauwirbel@gmail.com, aliguori@amazon.com, pbonzini@redhat.com, imammedo@redhat.com On Fri, 2014-03-07 at 00:44 +0100, Andreas F=C3=A4rber wrote: > Am 05.03.2014 18:30, schrieb Marcel Apfelbaum: > > In order to allow attaching machine options to a machine instance, > > current_machine is converted into MachineState. > > As a first step of deprecating QEMUMachine, some of the functions > > were modified to return MachineCLass. > >=20 > > Signed-off-by: Marcel Apfelbaum >=20 > Looks mostly good, but same issue as Alexey's patch: We are risking > qdev_get_machine() creating a Container-typed /machine node. >=20 > What about the following on top? Hi Andreas, I checked with the debugger and qdev_get_machine is called long after we add the machine to the QOM tree. However, the race still exists as someone can call qdev_get_machine before the machine is added to the tree, not being aware of that. Your change solves the problem, thank you! Do you want me to add this diff and resend, or I will send yours separately? Thanks, Marcel >=20 > Alexey, if we reach agreement here, this means for you that we can just > use type_register_static() in place of qemu_machine_register() to > register your custom machine type with interface added. >=20 > Regards, > Andreas >=20 > diff --git a/hw/core/qdev.c b/hw/core/qdev.c > index b6deebd..749c83a 100644 > --- a/hw/core/qdev.c > +++ b/hw/core/qdev.c > @@ -861,7 +861,8 @@ Object *qdev_get_machine(void) > static Object *dev; >=20 > if (dev =3D=3D NULL) { > - dev =3D container_get(object_get_root(), "/machine"); > + dev =3D object_resolve_path("/machine", NULL); > + g_assert(dev); > } >=20 > return dev; >=20 >=20