[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <ZgZvaUbZIr0qpxK5@gmail.com>
Date: Fri, 29 Mar 2024 08:36:09 +0100
From: Ingo Molnar <mingo@...nel.org>
To: Shuah Khan <skhan@...uxfoundation.org>
Cc: Muhammad Usama Anjum <usama.anjum@...labora.com>,
Shuah Khan <shuah@...nel.org>,
"Chang S. Bae" <chang.seok.bae@...el.com>,
"Kirill A. Shutemov" <kirill.shutemov@...ux.intel.com>,
Binbin Wu <binbin.wu@...ux.intel.com>,
angquan yu <angquan21@...il.com>, kernel@...labora.com,
linux-kselftest@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3] selftests: x86: skip the tests if prerequisites
aren't fulfilled
* Shuah Khan <skhan@...uxfoundation.org> wrote:
> On 3/27/24 05:17, Muhammad Usama Anjum wrote:
> > Skip instead of failing when prerequisite conditions aren't fulfilled,
> > such as invalid xstate values etc. This patch would make the tests show
> > as skip when run by:
> > make -C tools/testing/selftests/ TARGETS=x86 run_tests
> >
> > ...
> > # timeout set to 45
> > # selftests: x86: amx_64
> > # # xstate cpuid: invalid tile data size/offset: 0/0
> > ok 42 selftests: x86: amx_64 # SKIP
> > # timeout set to 45
> > # selftests: x86: lam_64
> > # # Unsupported LAM feature!
> > ok 43 selftests: x86: lam_64 # SKIP
> > ...
> >
> > In amx test, Move away from check_cpuid_xsave() and start using
> > arch_prctl() to find out if amx support is present or not. In the
> > kernels where amx isn't present, arch_prctl returns -EINVAL. Hence it is
> > backward compatible.
> >
> > Reviewed-by: Chang S. Bae <chang.seok.bae@...el.com>
> > Acked-by: Kirill A. Shutemov <kirill.shutemov@...ux.intel.com>
> > Reviewed-by: Binbin Wu <binbin.wu@...ux.intel.com>
> > Signed-off-by: Muhammad Usama Anjum <usama.anjum@...labora.com>
> > ---
> > Changes since v2:
> > - Update the changelog
> >
>
> Thank you - applied to linux-kselftest next for 6.10-rc1
Please don't, I've applied the patch to tip:x86/cpu with a tidied up
changelog.
Thanks,
Ingo
Powered by blists - more mailing lists