From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:47495)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <afaerber@suse.de>) id 1WQQjf-0000Fr-OY
	for qemu-devel@nongnu.org; Wed, 19 Mar 2014 20:21:33 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <afaerber@suse.de>) id 1WQQjZ-0002a9-Tx
	for qemu-devel@nongnu.org; Wed, 19 Mar 2014 20:21:27 -0400
Message-ID: <532A347F.5000100@suse.de>
Date: Thu, 20 Mar 2014 01:21:19 +0100
From: =?ISO-8859-15?Q?Andreas_F=E4rber?= <afaerber@suse.de>
MIME-Version: 1.0
References: <1395273809-12809-1-git-send-email-aik@ozlabs.ru>
In-Reply-To: <1395273809-12809-1-git-send-email-aik@ozlabs.ru>
Content-Type: text/plain; charset=ISO-8859-15
Content-Transfer-Encoding: quoted-printable
Subject: Re: [Qemu-devel] [PATCH v5 0/3] spapr: fix H_SET_MODE
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Alexey Kardashevskiy <aik@ozlabs.ru>, qemu-devel@nongnu.org
Cc: qemu-ppc@nongnu.org, Alexander Graf <agraf@suse.de>, Greg Kurz <gkurz@linux.vnet.ibm.com>

Am 20.03.2014 01:03, schrieb Alexey Kardashevskiy:
> This fixes H_SET_MODE.
> Changes:
> v5:
> * fixed code style
>=20
> v4:
> * rebased on top of current ppc-next.
> * changed PPC2_ISA207S flag value
>=20
>=20
> Alexey Kardashevskiy (3):
>   target-ppc: introduce powerisa-207-server flag
>   spapr_hcall: fix little-endian resource handling in H_SET_MODE

Thanks, applied these two to my ppc-next:
https://github.com/afaerber/qemu-cpu/commits/ppc-next

>   spapr_hcall: add address-translation-mode-on-interrupt resource in
>     H_SET_MODE

This doesn't strike me as a bug fix worthy of -rc1? Does anything
severely break without it that would justify putting it in -rc2?

Also some additional reviewer would assure me there's been no oversight.

Regards,
Andreas

>  hw/ppc/spapr_hcall.c        | 67 +++++++++++++++++++++++++++++++++++++=
++------
>  include/hw/ppc/spapr.h      |  9 ++++--
>  target-ppc/cpu.h            |  4 +++
>  target-ppc/translate_init.c |  3 +-
>  4 files changed, 72 insertions(+), 11 deletions(-)

--=20
SUSE LINUX Products GmbH, Maxfeldstr. 5, 90409 N=FCrnberg, Germany
GF: Jeff Hawn, Jennifer Guild, Felix Imend=F6rffer; HRB 16746 AG N=FCrnbe=
rg