linux-kernel - Re: [PATCH v2] selftests: x86: conform test to TAP format output

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <0b731ef9-3110-44d8-b768-ccbf7585a08d@collabora.com>
Date: Thu, 11 Jul 2024 11:52:50 +0500
From: Muhammad Usama Anjum <usama.anjum@...labora.com>
To: Shuah Khan <skhan@...uxfoundation.org>,
 "Chang S. Bae" <chang.seok.bae@...el.com>,
 Binbin Wu <binbin.wu@...ux.intel.com>, Ingo Molnar <mingo@...nel.org>,
 "Kirill A. Shutemov" <kirill.shutemov@...ux.intel.com>
Cc: Muhammad Usama Anjum <usama.anjum@...labora.com>, kernel@...labora.com,
 linux-kselftest@...r.kernel.org, linux-kernel@...r.kernel.org,
 Shuah Khan <shuah@...nel.org>
Subject: Re: [PATCH v2] selftests: x86: conform test to TAP format output

On 7/10/24 9:16 PM, Shuah Khan wrote:
> On 7/10/24 03:37, Muhammad Usama Anjum wrote:
>> This patch brings just readability implements by using kselftests wrappers
>> instead of manual pass/fail test cases counting. It has been on mailing
>> list from several months now. Please can someone ack or nack?
>>
> 
> Okay. I think I responded to your other patches that are adding TAP
> to individual tests when kselftest wrapped does it for you based on
> return values.
The current test doesn't return any exit value (hence implicitly always 0
is returned). The return value in addition to some other changes is getting
fixed in this patch.

> 
> The reason I don't want to take this patch is if you run the test
> using the recommended method:
> 
> make -C tools/testing/selftests/vDSO/ run_tests you will get the
> TAP output because lib.mk runtests framework takes care of this.
This patch is required to correctly return the value so that runtests
fraework can correctly mark the result of the test.

> 
> or
> 
> make kselftest TARGETS=vDSO will do the same.
> 
> Please don't send TAP conversions for individual runs. You will
> start seeing duplicate TAP output which will make it unreadable.
> 
> Run the test using make -C or make kselftest TARGETS before
> investing time to concert to TAP. I am not going to take TAP
> conversions patches if make -C or make kselftest TARGETS
> shows TAP.
> 
> thanks,
> -- Shuah
> 

-- 
BR,
Muhammad Usama Anjum