mbox

[kvm-unit-tests,GIT,PULL,00/11] s390x patches

Message ID 20200731094607.15204-1-frankja@linux.ibm.com (mailing list archive)
State New, archived
Headers show

Pull-request

https://github.com/frankjaa/kvm-unit-tests.git tags/s390x-2020-31-07

Message

Janosch Frank July 31, 2020, 9:45 a.m. UTC 
Hi Paolo,

The following changes since commit 82147b77199bbfec2c61c97685cfb34f3f97c889:

  fw_cfg: avoid index out of bounds (2020-07-30 17:57:55 -0400)

are available in the Git repository at:

  https://github.com/frankjaa/kvm-unit-tests.git tags/s390x-2020-31-07

for you to fetch changes up to c2f4799a861993b82950b9e5a3a44fc65ba05ec6:

  s390x: fix inline asm on gcc10 (2020-07-31 04:35:33 -0400)

----------------------------------------------------------------
* IO tests from Pierre
* GCC 10 compile fix from Claudio
* CPU model test fix from Thomas
----------------------------------------------------------------

Claudio Imbrenda (1):
  s390x: fix inline asm on gcc10

Pierre Morel (9):
  s390x: saving regs for interrupts
  s390x: I/O interrupt registration
  s390x: export the clock get_clock_ms() utility
  s390x: clock and delays calculations
  s390x: define function to wait for interrupt
  s390x: Library resources for CSS tests
  s390x: css: stsch, enumeration test
  s390x: css: msch, enable test
  s390x: css: ssch/tsch with sense and interrupt

Thomas Huth (1):
  s390x/cpumodel: The missing DFP facility on TCG is expected

 lib/s390x/asm/arch_def.h |  14 ++
 lib/s390x/asm/cpacf.h    |   5 +-
 lib/s390x/asm/time.h     |  50 ++++++
 lib/s390x/css.h          | 294 +++++++++++++++++++++++++++++++++++
 lib/s390x/css_dump.c     | 152 +++++++++++++++++++
 lib/s390x/css_lib.c      | 320 +++++++++++++++++++++++++++++++++++++++
 lib/s390x/interrupt.c    |  23 ++-
 lib/s390x/interrupt.h    |   8 +
 lib/s390x/vm.c           |  46 ++++++
 lib/s390x/vm.h           |  14 ++
 s390x/Makefile           |   4 +
 s390x/cpumodel.c         |  19 ++-
 s390x/css.c              | 150 ++++++++++++++++++
 s390x/cstart64.S         |  41 ++++-
 s390x/emulator.c         |  25 +--
 s390x/intercept.c        |  11 +-
 s390x/unittests.cfg      |   4 +
 17 files changed, 1147 insertions(+), 33 deletions(-)
 create mode 100644 lib/s390x/asm/time.h
 create mode 100644 lib/s390x/css.h
 create mode 100644 lib/s390x/css_dump.c
 create mode 100644 lib/s390x/css_lib.c
 create mode 100644 lib/s390x/interrupt.h
 create mode 100644 lib/s390x/vm.c
 create mode 100644 lib/s390x/vm.h
 create mode 100644 s390x/css.c

Comments

Paolo Bonzini July 31, 2020, 10:31 a.m. UTC | #1 
On 31/07/20 11:45, Janosch Frank wrote:
>   https://github.com/frankjaa/kvm-unit-tests.git tags/s390x-2020-31-07

Pulled, thanks.  FWIW you may want to gitlab in order to get the CI.

Paolo
Janosch Frank July 31, 2020, 11:31 a.m. UTC | #2 
On 7/31/20 12:31 PM, Paolo Bonzini wrote:
> On 31/07/20 11:45, Janosch Frank wrote:
>>   https://github.com/frankjaa/kvm-unit-tests.git tags/s390x-2020-31-07
> 
> Pulled, thanks.  FWIW you may want to gitlab in order to get the CI.
> 
> Paolo
> 

Hey Paolo, that repository is hooked up to travis already:
https://travis-ci.com/github/frankjaa/kvm-unit-tests/builds/177931162

I'll consider it if it has any benefit.
@Thomas: Are there differences in the CI?
Thomas Huth July 31, 2020, 12:09 p.m. UTC | #3 
On 31/07/2020 13.31, Janosch Frank wrote:
> On 7/31/20 12:31 PM, Paolo Bonzini wrote:
>> On 31/07/20 11:45, Janosch Frank wrote:
>>>   https://github.com/frankjaa/kvm-unit-tests.git tags/s390x-2020-31-07
>>
>> Pulled, thanks.  FWIW you may want to gitlab in order to get the CI.
>>
>> Paolo
>>
> 
> Hey Paolo, that repository is hooked up to travis already:
> https://travis-ci.com/github/frankjaa/kvm-unit-tests/builds/177931162
> 
> I'll consider it if it has any benefit.
> @Thomas: Are there differences in the CI?

Not that much, you get a good build test coverage with both. Travis uses
real (nested) KVM tests, but the compiler and QEMU versions are a little
bit backlevel (still using Ubuntu bionic). Gitlab-CI uses newer versions
(thanks to Fedora 32), but there is no KVM support here, so the tests
run with TCG only (I'm thinking of adding the cirrus-run script to the
Gitlab-CI, maybe we could get some KVM-coverage that way there, too, but
that will certainly take some time to figure it out).

 Thomas
Janosch Frank July 31, 2020, 12:31 p.m. UTC | #4 
On 7/31/20 2:09 PM, Thomas Huth wrote:
> On 31/07/2020 13.31, Janosch Frank wrote:
>> On 7/31/20 12:31 PM, Paolo Bonzini wrote:
>>> On 31/07/20 11:45, Janosch Frank wrote:
>>>>   https://github.com/frankjaa/kvm-unit-tests.git tags/s390x-2020-31-07
>>>
>>> Pulled, thanks.  FWIW you may want to gitlab in order to get the CI.
>>>
>>> Paolo
>>>
>>
>> Hey Paolo, that repository is hooked up to travis already:
>> https://travis-ci.com/github/frankjaa/kvm-unit-tests/builds/177931162
>>
>> I'll consider it if it has any benefit.
>> @Thomas: Are there differences in the CI?
> 
> Not that much, you get a good build test coverage with both. Travis uses
> real (nested) KVM tests, but the compiler and QEMU versions are a little
> bit backlevel (still using Ubuntu bionic). Gitlab-CI uses newer versions
> (thanks to Fedora 32), but there is no KVM support here, so the tests
> run with TCG only (I'm thinking of adding the cirrus-run script to the
> Gitlab-CI, maybe we could get some KVM-coverage that way there, too, but
> that will certainly take some time to figure it out).
> 
>  Thomas
> 

Thanks for the answer!

For now I'll stay with github/travis until I have a real reason to move
over. Maybe if the qemu CI run doesn't take ~2h on gitlab I'll start
considering moving.