From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:41008) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UljXs-0003wW-TI for qemu-devel@nongnu.org; Sun, 09 Jun 2013 13:36:49 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1UljXs-0002Cj-0r for qemu-devel@nongnu.org; Sun, 09 Jun 2013 13:36:48 -0400 Received: from cantor2.suse.de ([195.135.220.15]:42151 helo=mx2.suse.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1UljXr-0002Ce-Qb for qemu-devel@nongnu.org; Sun, 09 Jun 2013 13:36:47 -0400 Message-ID: <51B4BD2A.1040703@suse.de> Date: Sun, 09 Jun 2013 19:36:42 +0200 From: =?UTF-8?B?QW5kcmVhcyBGw6RyYmVy?= MIME-Version: 1.0 References: <1370794247-28267-1-git-send-email-afaerber@suse.de> <1370794247-28267-19-git-send-email-afaerber@suse.de> In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Subject: Re: [Qemu-devel] [PATCH qom-cpu v4 18/18] memory_mapping: Build only once List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: qiaonuohan@cn.fujitsu.com, qemu-devel@nongnu.org, lcapitulino@redhat.com Am 09.06.2013 19:29, schrieb Peter Maydell: > On 9 June 2013 17:10, Andreas F=C3=A4rber wrote: >> Move RAMBlock, RAMList and ram_list to exec/cpu-common.h since they do >> not depend on CPUArchState. We can then use it together with qom/cpu.h >> to replace cpu.h and exec/cpu-all.h includes, making it fully >> target-independent. >=20 > Messing directly with RAMBlocks is (or should be) limited to > pretty small amounts of the codebase, so maybe it's worth putting > these structs in their own header in include/exec which only gets > pulled in by the code that really needs it? I thought about that, but this way seemed easier to get v4 out finally and the qom-cpu-10 series built on it. The patches I really care about in this series are 7-11 because they determine how arm and s390x implementations are to be added. I'll happily respin or drop part of the remainder of the series after I get my qom-cpu queue with the dump refactoring flushed out. ;) Thanks, 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