From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Huth Date: Tue, 01 Mar 2016 08:40:59 +0000 Subject: Re: [kvm-unit-tests PATCH v3 2/2] powerpc: select endianness Message-Id: <56D5559B.8050808@redhat.com> List-Id: References: <1456768146-10136-1-git-send-email-lvivier@redhat.com> <1456768146-10136-3-git-send-email-lvivier@redhat.com> In-Reply-To: <1456768146-10136-3-git-send-email-lvivier@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Laurent Vivier , kvm@vger.kernel.org, kvm-ppc@vger.kernel.org Cc: drjones@redhat.com, dgibson@redhat.com, agraf@suse.de, pbonzini@redhat.com On 29.02.2016 18:49, Laurent Vivier wrote: > This patch allows to build tests for ppc64 little endian target > (ppc64le) on big and little endian hosts. > > We add a new parameter to configure to select the endianness of the > tests (--endian=little or --endian=big). > > I have built and tested big and little endian tests on a little > endian host, a big endian host, with kvm_hv and kvm_pr, and on > x86_64 with ppc64 as a TCG target. > > Signed-off-by: Laurent Vivier > --- > configure | 11 +++++++++++ > lib/powerpc/asm/ppc_asm.h | 19 +++++++++++++++++++ > lib/ppc64/asm/io.h | 8 ++++++++ > powerpc/Makefile.common | 3 ++- > powerpc/Makefile.ppc64 | 9 +++++++-- > powerpc/cstart64.S | 20 ++++++++++++++++---- > 6 files changed, 63 insertions(+), 7 deletions(-) Patch looks fine to me, so when you fixed the two little things that Andrew mentioned, feel free to add my Reviewed-by. Thomas From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Huth Subject: Re: [kvm-unit-tests PATCH v3 2/2] powerpc: select endianness Date: Tue, 1 Mar 2016 09:40:59 +0100 Message-ID: <56D5559B.8050808@redhat.com> References: <1456768146-10136-1-git-send-email-lvivier@redhat.com> <1456768146-10136-3-git-send-email-lvivier@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: drjones@redhat.com, dgibson@redhat.com, agraf@suse.de, pbonzini@redhat.com To: Laurent Vivier , kvm@vger.kernel.org, kvm-ppc@vger.kernel.org Return-path: Received: from mx1.redhat.com ([209.132.183.28]:34318 "EHLO mx1.redhat.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751965AbcCAIlE (ORCPT ); Tue, 1 Mar 2016 03:41:04 -0500 In-Reply-To: <1456768146-10136-3-git-send-email-lvivier@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: On 29.02.2016 18:49, Laurent Vivier wrote: > This patch allows to build tests for ppc64 little endian target > (ppc64le) on big and little endian hosts. > > We add a new parameter to configure to select the endianness of the > tests (--endian=little or --endian=big). > > I have built and tested big and little endian tests on a little > endian host, a big endian host, with kvm_hv and kvm_pr, and on > x86_64 with ppc64 as a TCG target. > > Signed-off-by: Laurent Vivier > --- > configure | 11 +++++++++++ > lib/powerpc/asm/ppc_asm.h | 19 +++++++++++++++++++ > lib/ppc64/asm/io.h | 8 ++++++++ > powerpc/Makefile.common | 3 ++- > powerpc/Makefile.ppc64 | 9 +++++++-- > powerpc/cstart64.S | 20 ++++++++++++++++---- > 6 files changed, 63 insertions(+), 7 deletions(-) Patch looks fine to me, so when you fixed the two little things that Andrew mentioned, feel free to add my Reviewed-by. Thomas