On Tue, Jun 17, 2025 at 09:31:52AM +0200, Thomas Weißschuh wrote:
Please use "selftests: harness:" as subject prefix for the patches. Also mention the specific typo to make the subject more unique.
Thanks.
Will adjust it.
On Mon, Jun 16, 2025 at 12:23:36PM +0000, Wei Yang wrote:
The name is __constructor_order_forward.
Just correct it.
Signed-off-by: Wei Yang richard.weiyang@gmail.com
With the above comments addressed: Reviewed-by: Thomas Weißschuh thomas.weissschuh@linutronix.de
tools/testing/selftests/kselftest_harness.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/tools/testing/selftests/kselftest_harness.h b/tools/testing/selftests/kselftest_harness.h index 2925e47db995..674a6112e6e1 100644 --- a/tools/testing/selftests/kselftest_harness.h +++ b/tools/testing/selftests/kselftest_harness.h @@ -936,7 +936,7 @@ static inline bool __test_passed(struct __test_metadata *metadata)
- list so tests are run in source declaration order.
- https://gcc.gnu.org/onlinedocs/gccint/Initialization.html
- However, it seems not all toolchains do this correctly, so use
- __constructor_order_foward to detect which direction is called first
*/
- __constructor_order_forward to detect which direction is called first
- and adjust list building logic to get things running in the right
- direction.
-- 2.34.1