Re: [kvm-unit-tests PATCH v3 0/7] s390x: basic test infrastructure

From: Paolo Bonzini <pbonzini@redhat.com>
To: David Hildenbrand <david@redhat.com>, kvm@vger.kernel.org
Cc: "Radim Krčmář" <rkrcmar@redhat.com>,
	"Thomas Huth" <thuth@redhat.com>,
	"Christian Borntraeger" <borntraeger@de.ibm.com>,
	"Cornelia Huck" <cornelia.huck@de.ibm.com>
Subject: Re: [kvm-unit-tests PATCH v3 0/7] s390x: basic test infrastructure
Date: Thu, 25 May 2017 18:37:30 +0200	[thread overview]
Message-ID: <8592cbf5-9206-aeb6-caa1-78870f234cc8@redhat.com> (raw)
In-Reply-To: <20170522085702.17856-1-david@redhat.com>

On 22/05/2017 10:56, David Hildenbrand wrote:
> This adds the very basic infrastructure for s390x.
> 
> For now, only a basic self test is provided. This one will also run under
> TCG.
> 
> Parts taken from a prototype by Thomas Huth.
> 
> v2 -> v3:
> - Fixed it up to allow for out-of-tree builds, added it to .travis.yml
> - Selftest: removed parameter intendation, dropped a ":" from inline asm
> 
> 
> David Hildenbrand (7):
>   s390x: initial infrastructure
>   s390x: basic self test
>   s390x: copy sclp.h and sclp-ascii.c from QEMU
>   s390x: compile sclp-ascii.c
>   s390x: wire up sclp console output
>   .travis.yml: add s390x
>   MAINTAINERS: add s390x maintainers
> 
>  .travis.yml              |   5 +++
>  MAINTAINERS              |   7 +++
>  lib/s390x/asm/barrier.h  |  16 +++++++
>  lib/s390x/asm/io.h       |  18 ++++++++
>  lib/s390x/asm/page.h     |  16 +++++++
>  lib/s390x/asm/spinlock.h |  16 +++++++
>  lib/s390x/asm/stack.h    |  21 +++++++++
>  lib/s390x/io.c           |  49 +++++++++++++++++++++
>  lib/s390x/sclp-ascii.c   |  68 +++++++++++++++++++++++++++++
>  lib/s390x/sclp.h         | 108 +++++++++++++++++++++++++++++++++++++++++++++++
>  lib/s390x/stack.c        |  28 ++++++++++++
>  s390x/Makefile           |  37 ++++++++++++++++
>  s390x/cstart64.S         |  44 +++++++++++++++++++
>  s390x/flat.lds           |  42 ++++++++++++++++++
>  s390x/run                |  56 ++++++++++++++++++++++++
>  s390x/selftest.c         |  41 ++++++++++++++++++
>  s390x/unittests.cfg      |  24 +++++++++++
>  17 files changed, 596 insertions(+)
>  create mode 100644 lib/s390x/asm/barrier.h
>  create mode 100644 lib/s390x/asm/io.h
>  create mode 100644 lib/s390x/asm/page.h
>  create mode 100644 lib/s390x/asm/spinlock.h
>  create mode 100644 lib/s390x/asm/stack.h
>  create mode 100644 lib/s390x/io.c
>  create mode 100644 lib/s390x/sclp-ascii.c
>  create mode 100644 lib/s390x/sclp.h
>  create mode 100644 lib/s390x/stack.c
>  create mode 100644 s390x/Makefile
>  create mode 100644 s390x/cstart64.S
>  create mode 100644 s390x/flat.lds
>  create mode 100755 s390x/run
>  create mode 100644 s390x/selftest.c
>  create mode 100644 s390x/unittests.cfg
> 

Thanks, didn't have time to test it today.  Will push tomorrow.

Paolo