mbox series

[v2,0/2] KVM: selftests: enable the memslot tests in ARM64

Message ID 20210907180957.609966-1-ricarkol@google.com (mailing list archive)
Headers show
Series KVM: selftests: enable the memslot tests in ARM64 | expand

Message

Ricardo Koller Sept. 7, 2021, 6:09 p.m. UTC
Enable memslot_modification_stress_test and memslot_perf_test in ARM64
(second patch). memslot_modification_stress_test builds and runs in
aarch64 without any modification. memslot_perf_test needs some nits
regarding ucalls (first patch).

Can anybody try these two tests in s390, please?

Changes:
v2: Makefile tests in the right order (from Andrew).

Ricardo Koller (2):
  KVM: selftests: make memslot_perf_test arch independent
  KVM: selftests: build the memslot tests for arm64

 tools/testing/selftests/kvm/Makefile          |  2 +
 .../testing/selftests/kvm/memslot_perf_test.c | 56 +++++++++++--------
 2 files changed, 36 insertions(+), 22 deletions(-)

Comments

Andrew Jones Sept. 7, 2021, 6:17 p.m. UTC | #1
On Tue, Sep 07, 2021 at 11:09:55AM -0700, Ricardo Koller wrote:
> Enable memslot_modification_stress_test and memslot_perf_test in ARM64
> (second patch). memslot_modification_stress_test builds and runs in
> aarch64 without any modification. memslot_perf_test needs some nits
> regarding ucalls (first patch).
> 
> Can anybody try these two tests in s390, please?
> 
> Changes:
> v2: Makefile tests in the right order (from Andrew).

Hi Ricardo,

You could have collected all the r-b's too.

Thanks,
drew

> 
> Ricardo Koller (2):
>   KVM: selftests: make memslot_perf_test arch independent
>   KVM: selftests: build the memslot tests for arm64
> 
>  tools/testing/selftests/kvm/Makefile          |  2 +
>  .../testing/selftests/kvm/memslot_perf_test.c | 56 +++++++++++--------
>  2 files changed, 36 insertions(+), 22 deletions(-)
> 
> -- 
> 2.33.0.153.gba50c8fa24-goog
>
Ricardo Koller Sept. 7, 2021, 8:52 p.m. UTC | #2
On Tue, Sep 07, 2021 at 08:17:37PM +0200, Andrew Jones wrote:
> On Tue, Sep 07, 2021 at 11:09:55AM -0700, Ricardo Koller wrote:
> > Enable memslot_modification_stress_test and memslot_perf_test in ARM64
> > (second patch). memslot_modification_stress_test builds and runs in
> > aarch64 without any modification. memslot_perf_test needs some nits
> > regarding ucalls (first patch).
> > 
> > Can anybody try these two tests in s390, please?
> > 
> > Changes:
> > v2: Makefile tests in the right order (from Andrew).
> 
> Hi Ricardo,
> 
> You could have collected all the r-b's too.

Thanks Andrew. Will try to do it next time (if changes are small like
in this patch set.

Thanks,
Ricardo

> 
> Thanks,
> drew
> 
> > 
> > Ricardo Koller (2):
> >   KVM: selftests: make memslot_perf_test arch independent
> >   KVM: selftests: build the memslot tests for arm64
> > 
> >  tools/testing/selftests/kvm/Makefile          |  2 +
> >  .../testing/selftests/kvm/memslot_perf_test.c | 56 +++++++++++--------
> >  2 files changed, 36 insertions(+), 22 deletions(-)
> > 
> > -- 
> > 2.33.0.153.gba50c8fa24-goog
> > 
>
Ricardo Koller Oct. 19, 2021, 9:17 p.m. UTC | #3
On Tue, Sep 07, 2021 at 08:17:37PM +0200, Andrew Jones wrote:
> On Tue, Sep 07, 2021 at 11:09:55AM -0700, Ricardo Koller wrote:
> > Enable memslot_modification_stress_test and memslot_perf_test in ARM64
> > (second patch). memslot_modification_stress_test builds and runs in
> > aarch64 without any modification. memslot_perf_test needs some nits
> > regarding ucalls (first patch).
> > 
> > Can anybody try these two tests in s390, please?
> > 
> > Changes:
> > v2: Makefile tests in the right order (from Andrew).
> 
> Hi Ricardo,
> 
> You could have collected all the r-b's too.
> 
> Thanks,
> drew
> 

Friendly ping on this one, please.

> > 
> > Ricardo Koller (2):
> >   KVM: selftests: make memslot_perf_test arch independent
> >   KVM: selftests: build the memslot tests for arm64
> > 
> >  tools/testing/selftests/kvm/Makefile          |  2 +
> >  .../testing/selftests/kvm/memslot_perf_test.c | 56 +++++++++++--------
> >  2 files changed, 36 insertions(+), 22 deletions(-)
> > 
> > -- 
> > 2.33.0.153.gba50c8fa24-goog
> > 
>
Marc Zyngier Oct. 21, 2021, 12:47 p.m. UTC | #4
On Tue, 7 Sep 2021 11:09:55 -0700, Ricardo Koller wrote:
> Enable memslot_modification_stress_test and memslot_perf_test in ARM64
> (second patch). memslot_modification_stress_test builds and runs in
> aarch64 without any modification. memslot_perf_test needs some nits
> regarding ucalls (first patch).
> 
> Can anybody try these two tests in s390, please?
> 
> [...]

Applied to next, thanks!

[1/2] KVM: selftests: make memslot_perf_test arch independent
      commit: ffb4ce3c49366f02f1c064fbe2e66a96ab5f98b8
[2/2] KVM: selftests: build the memslot tests for arm64
      commit: 358928fd5264f069b9758f8b29297c7bff2a06de

Cheers,

	M.