From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:55092) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z8AT5-000605-F0 for qemu-devel@nongnu.org; Thu, 25 Jun 2015 12:57:40 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Z8AT2-0000Gv-4w for qemu-devel@nongnu.org; Thu, 25 Jun 2015 12:57:39 -0400 Received: from cantor2.suse.de ([195.135.220.15]:58506 helo=mx2.suse.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Z8AT1-0000GX-V9 for qemu-devel@nongnu.org; Thu, 25 Jun 2015 12:57:36 -0400 Message-ID: <558C32FE.60501@suse.de> Date: Thu, 25 Jun 2015 18:57:34 +0200 From: =?ISO-8859-15?Q?Andreas_F=E4rber?= MIME-Version: 1.0 References: <7d54c33fa2a928fd317f3a95af61f854f43ba23c.1435195913.git.zhugh.fnst@cn.fujitsu.com> In-Reply-To: <7d54c33fa2a928fd317f3a95af61f854f43ba23c.1435195913.git.zhugh.fnst@cn.fujitsu.com> Content-Type: text/plain; charset=iso-8859-15 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [RESEND PATCH v8 2/4] hw: add a wrapper for registering reset handler List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Zhu Guihua , qemu-devel@nongnu.org Cc: chen.fan.fnst@cn.fujitsu.com, imammedo@redhat.com, izumi.taku@jp.fujitsu.com, ehabkost@redhat.com, pbonzini@redhat.com Am 25.06.2015 um 04:17 schrieb Zhu Guihua: > Add a wrapper to specify reset order when registering reset handler, > instead of non-obvious initiazation code ordering. "initialization", and this sentence is not really telling to me. What issue is this solving or, more likely, working around? In the next patch the APIC is being moved from ICC bus (which, as it has a device parent, would seem to not register its own reset handler in qdev.c) to no bus and thus seemingly no implicit reset handler either. We've been trying to get away from reset handlers and move to reset callbacks that propagate through devices and buses for a long time. This patch feels like a step backwards. If there is indeed no simpler solution, this at least deserves a better justification. If more people start using values like 0x1, we'll have the same ordering issues unless we use some global enum to coordinate them. >=20 > Signed-off-by: Zhu Guihua > --- > include/hw/hw.h | 4 ++++ > vl.c | 18 +++++++++++++++++- > 2 files changed, 21 insertions(+), 1 deletion(-) Regards, Andreas --=20 SUSE Linux GmbH, Maxfeldstr. 5, 90409 N=FCrnberg, Germany GF: Felix Imend=F6rffer, Jane Smithard, Dilip Upmanyu, Graham Norton; HRB 21284 (AG N=FCrnberg)