From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:49102) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WOAck-0001HY-Sc for qemu-devel@nongnu.org; Thu, 13 Mar 2014 14:45:03 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1WOAcc-00022P-MM for qemu-devel@nongnu.org; Thu, 13 Mar 2014 14:44:58 -0400 Received: from cantor2.suse.de ([195.135.220.15]:37161 helo=mx2.suse.de) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1WOAcc-00022D-BT for qemu-devel@nongnu.org; Thu, 13 Mar 2014 14:44:50 -0400 Message-ID: <5321FC9F.2010601@suse.de> Date: Thu, 13 Mar 2014 19:44:47 +0100 From: =?UTF-8?B?QW5kcmVhcyBGw6RyYmVy?= MIME-Version: 1.0 References: <1394722501-32326-1-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] [PULL for-2.0-rc0 00/58] QOM CPUState patch queue 2014-03-13 List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: Igor Mammedov , Eduardo Habkost , QEMU Developers , Anthony Liguori , Christian Borntraeger Am 13.03.2014 18:10, schrieb Peter Maydell: > On 13 March 2014 14:54, Andreas F=C3=A4rber wrote: >> Hello Peter, >> >> This is my current QOM CPU patch queue. Please pull. >> >> Since this touches on bsd-user code, request to apply the build fix fi= rst. >> >> NB: I was only able to test x86/ppc/arm KVM this time; s390x KVM was r= e-reviewed only. >> >> Regards, >> Andreas >> >> Cc: Peter Maydell >> Cc: Anthony Liguori >> >> Cc: Eduardo Habkost >> Cc: Igor Mammedov >> Cc: Christian Borntraeger >> >> The following changes since commit be86c53c058d75fc3938b1b54f363259f28= 2b3d5: >> >> Merge remote-tracking branch 'remotes/afaerber/tags/ppc-for-2.0' int= o staging (2014-03-13 13:19:46 +0000) >> >> are available in the git repository at: >> >> >> git://github.com/afaerber/qemu-cpu.git tags/qom-cpu-for-2.0 >> >> for you to fetch changes up to 98f11363e7add1e750f76e3d81750348f0470c2= 9: >> >> user-exec: Change exception_action() argument to CPUState (2014-03-1= 3 15:35:04 +0100) >=20 > Fails to compile on systems without CONFIG_INT128 (eg 32 bit host): >=20 > /root/qemu/target-s390x/int_helper.c: In function =E2=80=98helper_divu6= 4=E2=80=99: > /root/qemu/target-s390x/int_helper.c:111:9: error: passing argument 1 > of =E2=80=98cpu_abort=E2=80=99 from incompatible pointer type [-Werror] > In file included from /root/qemu/include/exec/cpu-all.h:26:0, > from /root/qemu/target-s390x/cpu.h:41, > from /root/qemu/target-s390x/int_helper.c:21: > /root/qemu/include/qom/cpu.h:620:37: note: expected =E2=80=98struct CPU= State > *=E2=80=99 but argument is of type =E2=80=98struct CPUS390XState *=E2=80= =99 > cc1: all warnings being treated as errors Thanks for catching that, the following seems to fix: diff --git a/target-s390x/int_helper.c b/target-s390x/int_helper.c index 85e49aa..5ffc5a8 100644 --- a/target-s390x/int_helper.c +++ b/target-s390x/int_helper.c @@ -106,9 +106,10 @@ uint64_t HELPER(divu64)(CPUS390XState *env, uint64_t ah, uint64_t al, runtime_exception(env, PGM_FIXPT_DIVIDE, GETPC()); } #else + S390CPU *cpu =3D s390_env_get_cpu(env); /* 32-bit hosts would need special wrapper functionality - just abort if we encounter such a case; it's very unlikely anyways. */ - cpu_abort(env, "128 -> 64/64 division not implemented\n"); + cpu_abort(CPU(cpu), "128 -> 64/64 division not implemented\n"); #endif } return ret; git-grep showed no further missed occurrences of cpu_abort(). Will submit v2 once my testing completes. Regards, 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