diff mbox series

[kvm-unit-tests,v2,2/2] powerpc: Make h_cede_tm test run by default

Message ID 20190515002801.20517-2-sjitindarsingh@gmail.com (mailing list archive)
State New, archived
Headers show
Series [kvm-unit-tests,v2,1/2] powerpc: Allow for a custom decr value to be specified to load on decr excp | expand

Commit Message

Suraj Jitindar Singh May 15, 2019, 12:28 a.m. UTC
This test was initially designed to test for a known bug where
performing a sequence of H_CEDE hcalls while suspended would cause a
vcpu to lockup in the host. The fix has been available for some time
now, so to increase coverage of this test remove the no-default flag.

Signed-off-by: Suraj Jitindar Singh <sjitindarsingh@gmail.com>
---
 powerpc/unittests.cfg | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Laurent Vivier May 15, 2019, 4:25 p.m. UTC | #1
On 15/05/2019 02:28, Suraj Jitindar Singh wrote:
> This test was initially designed to test for a known bug where
> performing a sequence of H_CEDE hcalls while suspended would cause a
> vcpu to lockup in the host. The fix has been available for some time
> now, so to increase coverage of this test remove the no-default flag.
> 
> Signed-off-by: Suraj Jitindar Singh <sjitindarsingh@gmail.com>
> ---
>  powerpc/unittests.cfg | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/powerpc/unittests.cfg b/powerpc/unittests.cfg
> index af535b7..1e74948 100644
> --- a/powerpc/unittests.cfg
> +++ b/powerpc/unittests.cfg
> @@ -64,7 +64,7 @@ file = emulator.elf
>  file = tm.elf
>  smp = 2,threads=2
>  extra_params = -machine cap-htm=on -append "h_cede_tm"
> -groups = nodefault,h_cede_tm
> +groups = h_cede_tm
>  
>  [sprs]
>  file = sprs.elf
> 

Reviewed-by: Laurent Vivier <lvivier@redhat.com>
Thomas Huth May 17, 2019, 10:13 a.m. UTC | #2
On 15/05/2019 02.28, Suraj Jitindar Singh wrote:
> This test was initially designed to test for a known bug where
> performing a sequence of H_CEDE hcalls while suspended would cause a
> vcpu to lockup in the host. The fix has been available for some time
> now, so to increase coverage of this test remove the no-default flag.
> 
> Signed-off-by: Suraj Jitindar Singh <sjitindarsingh@gmail.com>
> ---
>  powerpc/unittests.cfg | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/powerpc/unittests.cfg b/powerpc/unittests.cfg
> index af535b7..1e74948 100644
> --- a/powerpc/unittests.cfg
> +++ b/powerpc/unittests.cfg
> @@ -64,7 +64,7 @@ file = emulator.elf
>  file = tm.elf
>  smp = 2,threads=2
>  extra_params = -machine cap-htm=on -append "h_cede_tm"
> -groups = nodefault,h_cede_tm
> +groups = h_cede_tm
>  
>  [sprs]
>  file = sprs.elf

Acked-by: Thomas Huth <thuth@redhat.com>
diff mbox series

Patch

diff --git a/powerpc/unittests.cfg b/powerpc/unittests.cfg
index af535b7..1e74948 100644
--- a/powerpc/unittests.cfg
+++ b/powerpc/unittests.cfg
@@ -64,7 +64,7 @@  file = emulator.elf
 file = tm.elf
 smp = 2,threads=2
 extra_params = -machine cap-htm=on -append "h_cede_tm"
-groups = nodefault,h_cede_tm
+groups = h_cede_tm
 
 [sprs]
 file = sprs.elf