mbox series

[kvm-unit-tests,v4,0/1] s390x: add migration test for storage keys

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

Message

Nico Boehr June 8, 2022, 1:13 p.m. UTC 
v3->v4:
----
* remove useless goto (Thanks Thomas)

v2->v3:
----
* remove some useless variables, style suggestions, improve commit description
  (thanks Janis)
* reverse christmas tree (thanks Claudio)

v1->v2:
----
* As per discussion with Janis and Claudio, remove the actual access check from
  the test. This also allows us to remove the check_pgm_int_code_xfail() patch.
* Typos/Style suggestions (thanks Janis)

Upon migration, we expect storage keys set by the guest to be preserved,
so add a test for it.

We keep 128 pages and set predictable storage keys. Then, we migrate and check
they can be read back.

Nico Boehr (1):
  s390x: add migration test for storage keys

 s390x/Makefile         |  1 +
 s390x/migration-skey.c | 73 ++++++++++++++++++++++++++++++++++++++++++
 s390x/unittests.cfg    |  4 +++
 3 files changed, 78 insertions(+)
 create mode 100644 s390x/migration-skey.c

Comments

Claudio Imbrenda June 8, 2022, 1:39 p.m. UTC | #1 
On Wed,  8 Jun 2022 15:13:27 +0200
Nico Boehr <nrb@linux.ibm.com> wrote:

thanks, queued

> v3->v4:
> ----
> * remove useless goto (Thanks Thomas)
> 
> v2->v3:
> ----
> * remove some useless variables, style suggestions, improve commit description
>   (thanks Janis)
> * reverse christmas tree (thanks Claudio)
> 
> v1->v2:
> ----
> * As per discussion with Janis and Claudio, remove the actual access check from
>   the test. This also allows us to remove the check_pgm_int_code_xfail() patch.
> * Typos/Style suggestions (thanks Janis)
> 
> Upon migration, we expect storage keys set by the guest to be preserved,
> so add a test for it.
> 
> We keep 128 pages and set predictable storage keys. Then, we migrate and check
> they can be read back.
> 
> Nico Boehr (1):
>   s390x: add migration test for storage keys
> 
>  s390x/Makefile         |  1 +
>  s390x/migration-skey.c | 73 ++++++++++++++++++++++++++++++++++++++++++
>  s390x/unittests.cfg    |  4 +++
>  3 files changed, 78 insertions(+)
>  create mode 100644 s390x/migration-skey.c
>