Re: [PULL 47/47] tests/tcg/multiarch: add vma-pthread.c

[Richard Henderson]

On 12/29/22 16:02, Richard Henderson wrote:
> From: Ilya Leoshkevich <iii@linux.ibm.com>
>
> Add a test that locklessly changes and exercises page protection bits
> from various threads. This helps catch race conditions in the VMA
> handling.
>
> Acked-by: Alex Bennée <alex.bennee@linaro.org>
> Signed-off-by: Ilya Leoshkevich <iii@linux.ibm.com>
> Message-Id: <20221223120252.513319-1-iii@linux.ibm.com>
> Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
> ---
>   tests/tcg/multiarch/nop_func.h       |  25 ++++
>   tests/tcg/multiarch/munmap-pthread.c |  16 +--
>   tests/tcg/multiarch/vma-pthread.c    | 207 +++++++++++++++++++++++++++
>   tests/tcg/multiarch/Makefile.target  |   3 +
>   4 files changed, 236 insertions(+), 15 deletions(-)
>   create mode 100644 tests/tcg/multiarch/nop_func.h
>   create mode 100644 tests/tcg/multiarch/vma-pthread.c

Hi Peter,

From the failures I see on the gitlab merge job, I think I need to resubmit with this new 
test adjusted to loop less,

> +    for (i = 0; i < 50000; i++) {

here.

The failing jobs are --enable-debug, and take about 115 seconds to run manually on our 
aarch64 test host, exceeding the 90 second timeout.

I'll cut this down to 10000 loops and double-check times before resubmitting.


r~