From mboxrd@z Thu Jan 1 00:00:00 1970 From: =?UTF-8?B?QW5kcmVhcyBGw6RyYmVy?= Subject: Re: [PATCH qom-next 00/59] QOM CPUState, part 4: CPU_COMMON Date: Wed, 23 May 2012 17:36:32 +0200 Message-ID: <4FBD0400.90101@suse.de> References: <1337742502-28565-1-git-send-email-afaerber@suse.de> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Return-path: In-Reply-To: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: qemu-devel-bounces+gceq-qemu-devel=gmane.org@nongnu.org Sender: qemu-devel-bounces+gceq-qemu-devel=gmane.org@nongnu.org To: Stefano Stabellini Cc: xen-devel , "qemu-devel@nongnu.org" List-Id: xen-devel@lists.xenproject.org Am 23.05.2012 13:27, schrieb Stefano Stabellini: > On Wed, 23 May 2012, Andreas F=C3=A4rber wrote: >> This series, based on qom-next and the two pending ARM cleanup patches= , starts >> moving fields from CPUArchState (CPU_COMMON) to QOM CPUState. It stops= short >> of moving all easily possible fields (i.e., those not depending on tar= get_ulong >> or target_phys_addr_t) since the series got too long already and is ex= pected to >> spark some controversies due to collisions with several other series. >> >> The series is structured as preparatory refactorings interwoven with t= he actual >> touch-all movement of one field ("cpu: Move ... to CPUState"), optiona= lly >> followed by type signature cleanups, culminating in the movement of tw= o fields >> that are tied together by VMState. >> Thus, unlike part 3, this series cannot randomly be cherry-picked to >> -next trees, only select parts thereof (e.g., use of cpu_s390x_i= nit()). >> >> Please review and test. [...] >> I have checked this to compile on ... >> * openSUSE 12.1 x86_64 w/KVM, >> * openSUSE Factory ppc w/KVM, >> * SLES 11 SP2 s390x w/KVM, >> * mingw32/64 cross-builds, >> * OpenBSD 5.1 amd64 (not for final version though, master doesn't buil= d). >> Untested: Xen. >=20 > I tested it on Xen: it works correctly. >=20 > Tested-by: Stefano Stabellini Thanks for the quick response! I've cherry-picked the preparatory patch to qom-next: http://repo.or.cz/w/qemu/afaerber.git/shortlog/refs/heads/qom-next /-F --=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