From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:37740) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SXCl4-0001Sz-Bp for qemu-devel@nongnu.org; Wed, 23 May 2012 10:41:56 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1SXCkt-0006HA-Am for qemu-devel@nongnu.org; Wed, 23 May 2012 10:41:49 -0400 Received: from cantor2.suse.de ([195.135.220.15]:48074 helo=mx2.suse.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SXCkt-0006Gn-4R for qemu-devel@nongnu.org; Wed, 23 May 2012 10:41:39 -0400 Message-ID: <4FBCF71D.7050705@suse.de> Date: Wed, 23 May 2012 16:41:33 +0200 From: =?UTF-8?B?QW5kcmVhcyBGw6RyYmVy?= MIME-Version: 1.0 References: <1337504561-20297-1-git-send-email-gleb@redhat.com> In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH 1/2] Make pointer to fw_cfg device global. List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: qemu-devel@nongnu.org, Gleb Natapov Am 23.05.2012 14:44, schrieb Peter Maydell: > On 20 May 2012 10:02, Gleb Natapov wrote: >> There can be only one fw_cfg device, so saving global reference to it >> removes the need to pass its pointer around. >=20 > This seems like a backwards step to me: one of the things that prevents > us supporting "two separate machines in one emulation" is that we have > various things that assume there's only one instance of themselves and > use globals. I don't think we should be adding any more... While I concur with the design sentiment of avoiding globals where possible, I can understand having multiple CPU types in one machine, but multiple machines in one emulation? Do you have a specific example that cannot easily be modeled as multiple socket-connected executables? Andreas --=20 SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N=C3=BCrnberg, Germany GF: Jeff Hawn, Jennifer Guild, Felix Imend=C3=B6rffer; HRB 16746 AG N=C3=BC= rnberg