From mboxrd@z Thu Jan 1 00:00:00 1970 From: Laurent Vivier Date: Tue, 01 Mar 2016 08:54:23 +0000 Subject: Re: [kvm-unit-tests PATCH v3 2/2] powerpc: select endianness Message-Id: <56D558BF.4020002@redhat.com> List-Id: References: <1456768146-10136-1-git-send-email-lvivier@redhat.com> <1456768146-10136-3-git-send-email-lvivier@redhat.com> <56D5559B.8050808@redhat.com> In-Reply-To: <56D5559B.8050808@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Thomas Huth , kvm@vger.kernel.org, kvm-ppc@vger.kernel.org Cc: drjones@redhat.com, dgibson@redhat.com, agraf@suse.de, pbonzini@redhat.com On 01/03/2016 09:40, Thomas Huth wrote: > 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. Thanks Thomas, I will. Laurent