mbox series

[kvm-unit-tests,v3,0/2] s390x: add migration test for CMM

Message ID 20220512134233.1416490-1-nrb@linux.ibm.com (mailing list archive)
Headers show
Series s390x: add migration test for CMM | expand

Message

Nico Boehr May 12, 2022, 1:42 p.m. UTC 
v2->v3:
---
* remove unneeded include asm-offsets.h (Thanks Claudio)
* change prefix of test to match filename (migration-cmm instead of
  cmm-migration)

v1->v2:
---
* Rename cmm-migration.c to migration-cmm.c (Thanks Janosch)
* Replace switch-case with unrolled loop (Thanks Claudio)
* Migrate even when ESSA is not available so we don't hang forever

Upon migration, we expect the CMM page states to be preserved. Add a test which
checks for that.

The new test gets a new file so the existing cmm test can still run when the
prerequisites for running migration tests aren't given (netcat). Therefore, move
some definitions to a common header to be able to re-use them.

Nico Boehr (2):
  lib: s390x: add header for CMM related defines
  s390x: add cmm migration test

 lib/s390x/asm/cmm.h   | 50 ++++++++++++++++++++++++++++
 s390x/Makefile        |  1 +
 s390x/cmm.c           | 25 ++------------
 s390x/migration-cmm.c | 77 +++++++++++++++++++++++++++++++++++++++++++
 s390x/unittests.cfg   |  4 +++
 5 files changed, 135 insertions(+), 22 deletions(-)
 create mode 100644 lib/s390x/asm/cmm.h
 create mode 100644 s390x/migration-cmm.c

Comments

Claudio Imbrenda May 12, 2022, 3:44 p.m. UTC | #1 
On Thu, 12 May 2022 15:42:31 +0200
Nico Boehr <nrb@linux.ibm.com> wrote:

> v2->v3:
> ---
> * remove unneeded include asm-offsets.h (Thanks Claudio)
> * change prefix of test to match filename (migration-cmm instead of
>   cmm-migration)
> 
> v1->v2:
> ---
> * Rename cmm-migration.c to migration-cmm.c (Thanks Janosch)
> * Replace switch-case with unrolled loop (Thanks Claudio)
> * Migrate even when ESSA is not available so we don't hang forever
> 
> Upon migration, we expect the CMM page states to be preserved. Add a test which
> checks for that.
> 
> The new test gets a new file so the existing cmm test can still run when the
> prerequisites for running migration tests aren't given (netcat). Therefore, move
> some definitions to a common header to be able to re-use them.
> 

thanks, queued

> Nico Boehr (2):
>   lib: s390x: add header for CMM related defines
>   s390x: add cmm migration test
> 
>  lib/s390x/asm/cmm.h   | 50 ++++++++++++++++++++++++++++
>  s390x/Makefile        |  1 +
>  s390x/cmm.c           | 25 ++------------
>  s390x/migration-cmm.c | 77 +++++++++++++++++++++++++++++++++++++++++++
>  s390x/unittests.cfg   |  4 +++
>  5 files changed, 135 insertions(+), 22 deletions(-)
>  create mode 100644 lib/s390x/asm/cmm.h
>  create mode 100644 s390x/migration-cmm.c
>