Whamcloud - gitweb
LU-10973 lutf: Fix order of linking for python modules 78/46478/15
authorJames Simmons <jsimmons@infradead.org>
Sun, 31 Jul 2022 14:06:08 +0000 (10:06 -0400)
committerOleg Drokin <green@whamcloud.com>
Wed, 13 Dec 2023 12:18:46 +0000 (12:18 +0000)
commitdc128be9dd4e0a4bb59b2243612f298c6c6d3e70
treee2dd50c94a0fb3a57eb6934c5209a0110c096a67
parentbeaa2e03765655656f5a3befdb4b8d8cccfa60e8
LU-10973 lutf: Fix order of linking for python modules

LUTF normally works but in some test cases at startup we got:

ImportError: lustre/test/lutrf/src/_lnetconfig.so:
undefined symbol: lustre_lnet_del_ni

If you check the symbol is there. The issue is the linking order.
We need to put the generated module name before all its
dependencies.

Also, remove cfs_expr_list_match from string.h, move the definition
to nidstrings.c, and make it static.

Test-Parameters: @lnet
Change-Id: Ia57fbd9d5795d845ea14bc1416f968383afcba2b
Signed-off-by: James Simmons <jsimmons@infradead.org>
Signed-off-by: Timothy Day <timday@amazon.com>
Reviewed-on: https://review.whamcloud.com/c/fs/lustre-release/+/46478
Tested-by: jenkins <devops@whamcloud.com>
Tested-by: Maloo <maloo@whamcloud.com>
Reviewed-by: Serguei Smirnov <ssmirnov@whamcloud.com>
Reviewed-by: Frank Sehr <fsehr@whamcloud.com>
Reviewed-by: Cyril Bordage <cbordage@whamcloud.com>
Reviewed-by: Oleg Drokin <green@whamcloud.com>
libcfs/include/libcfs/util/string.h
libcfs/libcfs/util/nidstrings.c
libcfs/libcfs/util/string.c
lustre/tests/lutf/src/Makefile.am