From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:56491) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1V7GcF-0007YG-IR for qemu-devel@nongnu.org; Wed, 07 Aug 2013 23:10:25 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1V7Gc9-0002H8-Cp for qemu-devel@nongnu.org; Wed, 07 Aug 2013 23:10:19 -0400 Received: from mail-pd0-f176.google.com ([209.85.192.176]:51385) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1V7Gc9-0002Go-6f for qemu-devel@nongnu.org; Wed, 07 Aug 2013 23:10:13 -0400 Received: by mail-pd0-f176.google.com with SMTP id q10so2074125pdj.21 for ; Wed, 07 Aug 2013 20:10:11 -0700 (PDT) Message-ID: <52030C0B.8020705@ozlabs.ru> Date: Thu, 08 Aug 2013 13:10:03 +1000 From: Alexey Kardashevskiy MIME-Version: 1.0 References: <1375777673-20274-1-git-send-email-aik@ozlabs.ru> <1375777673-20274-6-git-send-email-aik@ozlabs.ru> <5200C789.5030802@suse.de> <5201E3F4.8070409@ozlabs.ru> <5201F146.5060400@suse.de> <5201F6BE.9090804@ozlabs.ru> <5202581C.3070700@suse.de> In-Reply-To: <5202581C.3070700@suse.de> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Subject: Re: [Qemu-devel] [PATCH 5/6] xics: split to xics and xics-common List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: =?UTF-8?B?QW5kcmVhcyBGw6RyYmVy?= Cc: Anthony Liguori , Alexander Graf , qemu-devel@nongnu.org, qemu-ppc@nongnu.org, Paul Mackerras , David Gibson On 08/08/2013 12:22 AM, Andreas Färber wrote: > Am 07.08.2013 09:26, schrieb Alexey Kardashevskiy: >> On 08/07/2013 05:03 PM, Andreas Färber wrote: >>> Am 07.08.2013 08:06, schrieb Alexey Kardashevskiy: >>>> [...] How do I test it? >>> >>> ./QMP/qom-list to find the path, if not fixed in code yet, and >> >> Something is wrong. XICS does not have an id but it should not be a >> problem, right (btw how to set it from the code?)? >> >> [aik@dyn232 ~]$ ./qemu-impreza/QMP/qom-list -s ./qmp-sock >> / > > That's expected for lack of argument. > > $ ./QMP/qom-list -s ./qmp-sock /machine > unattached/ > peripheral/ > peripheral-anon/ > type > > This confirms "path ... not fixed in code yet" (otherwise there would've > been an "spapr/" entry or anything else telling. > > So you need to look through /machine/unassigned for the device[n] with > qom-get /machine/unassigned/device[n].type matching your type, possibly > device[0] as in my case. From there on you see your icp[0]/ and ics/ > children without searching the unassigned list again. > > Or simply use my qom-tree script: > http://lists.nongnu.org/archive/html/qemu-devel/2013-07/txte1WIbWzu5Z.txt Yep. That works, thanks. > Setting a canonical path is done via object_property_add_child() after > instantiating and before realizing a device. > For x86 the northbridge is used as name, i.e. "i440fx" for pc and "q35" > for q35. Suggest to discuss with Anthony how to structure the > composition tree for sPAPR. I tried inserting this: object_property_add_child(qdev_get_machine(), busname, OBJECT(dev), NULL); in TYPE_SPAPR_PCI_HOST_BRIDGE's spapr_phb_init() (I thought this is what i440fx_init() does in the very beginning) but when I do that, spapr_phb already has a parent of a "container" type so assert happens. What is the aim of all of this? Is it not to have "unattached" in a path starting with /machine? btw I have added notes in the previous response against splitting ICS initialization into 2 functions and you skipped it. Does this mean you do not want to waste more of your time persuading me and this is the real stopped or you just gave up? :) Thanks. -- Alexey