Re: [PATCH] shared: linux: remove unused variables

[Sid Kumar <sidhartha.kumar@oracle.com>]

On 9/2/24 11:29 PM, Ba Jing wrote:
> These variables are never referenced in the code, just remove them.

Hello,

I think these variables are needed and they are referenced in main.c 
within the radix-tree directory. With these lines removed and running 
make within tools/testing/radix-tree I get the following errors:

/usr/bin/ld: 
/home/ubuntu/linux-sidhakum-public/tools/testing/radix-tree/main.c:74:(.text+0x57a): 
undefined reference to `test_verbose'
/usr/bin/ld: main.o: in function `check_copied_tags':
/home/ubuntu/linux-sidhakum-public/tools/testing/radix-tree/main.c:135:(.text+0xc3f): 
undefined reference to `test_verbose'


/home/ubuntu/linux-sidhakum-public/tools/testing/radix-tree/main.c:243:(.text+0x1d68): 
undefined reference to `preempt_count'
/usr/bin/ld: 
/home/ubuntu/linux-sidhakum-public/tools/testing/radix-tree/main.c:243:(.text+0x1d8d): 
undefined reference to `preempt_count'

adding back these variables allows make to run successfully again.

Thanks,

Sidhartha Kumar


>
> Signed-off-by: Ba Jing <bajing@cmss.chinamobile.com>
> ---
>   tools/testing/shared/linux.c | 2 --
>   1 file changed, 2 deletions(-)
>
> diff --git a/tools/testing/shared/linux.c b/tools/testing/shared/linux.c
> index 17263696b5d8..eb1b1878cf64 100644
> --- a/tools/testing/shared/linux.c
> +++ b/tools/testing/shared/linux.c
> @@ -13,8 +13,6 @@
>   #include <urcu/uatomic.h>
>   
>   int nr_allocated;
> -int preempt_count;
> -int test_verbose;
>   
>   struct kmem_cache {
>   	pthread_mutex_t lock;