Re: [PATCH] s390x: intercept: fence one test when using TCG

[Thomas Huth]

On 21/07/2022 12.56, Claudio Imbrenda wrote:
> Qemu commit f8333de2793 ("target/s390x/tcg: SPX: check validity of new prefix")
> fixes a TCG bug discovered with a new testcase in the intercept test.
>
> The gitlab pipeline for the KVM unit tests uses TCG and it will keep
> failing every time as long as the pipeline uses a version of Qemu
> without the aforementioned patch.
>
> Fence the specific testcase for now. Once the pipeline is fixed, this
> patch can safely be reverted.
>
> Signed-off-by: Claudio Imbrenda <imbrenda@linux.ibm.com>
> ---
>   s390x/intercept.c | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/s390x/intercept.c b/s390x/intercept.c
> index 54bed5a4..c48818c2 100644
> --- a/s390x/intercept.c
> +++ b/s390x/intercept.c
> @@ -14,6 +14,7 @@
>   #include <asm/page.h>
>   #include <asm/facility.h>
>   #include <asm/time.h>
> +#include <hardware.h>
>   
>   static uint8_t pagebuf[PAGE_SIZE * 2] __attribute__((aligned(PAGE_SIZE * 2)));
>   
> @@ -76,7 +77,7 @@ static void test_spx(void)
>         check_pgm_int_code(PGM_INT_CODE_ADDRESSING);
>   
>   	new_prefix = get_ram_size() & 0x7fffe000;
> -       if (get_ram_size() - new_prefix < 2 * PAGE_SIZE) {
> +       if (!host_is_tcg() && (get_ram_size() - new_prefix < 2 * PAGE_SIZE)) {

Could you please add a comment à la

 /* TODO: Remove host_is_tcg() once CIs are using QEMU >= 7.1 */

?

 Thomas