linux-kernel - Re: [PATCH -next] trace: wwnr: Make local symbol 'rv

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: <378d61ab-8866-8026-d6ad-b8f6718ff345@kernel.org>
Date:   Tue, 23 Aug 2022 16:05:17 +0200
From:   Daniel Bristot de Oliveira <bristot@...nel.org>
To:     Zeng Heng <zengheng4@...wei.com>, rostedt@...dmis.org,
        mingo@...hat.com
Cc:     liwei391@...wei.com, linux-kernel@...r.kernel.org
Subject: Re: [PATCH -next] trace: wwnr: Make local symbol 'rv_wwnr' static

On 8/22/22 08:30, Zeng Heng wrote:
> The sparse tool complains as follows:
> 
> kernel/trace/rv/monitors/wwnr/wwnr.c:18:19:
> warning: symbol 'rv_wwnr' was not declared. Should it be static?
> 
> The `rv_wwnr` symbol is not dereferenced by other exter files,
> so add static qualifier for it.

Would you mind re-sending this patch with some changes?

- Use "rv/monitors:" as the subsystem
- Do the same change for wip monitor
- Add the fixes tags:
  Fixes:	ccc319dcb450 ("rv/monitor: Add the wwnr monitor")
  Fixes:	8812d21219b9 ("rv/monitor: Add the wip monitor skeleton created by dot2k")

Also, to fix this problem for new monitors, do another patch changing the dot2k
templates files, adding the same 'static' attribute. The files are:

- tools/verification/dot2/dot2k_templates/main_global.c
- tools/verification/dot2/dot2k_templates/main_per_cpu.c
- tools/verification/dot2/dot2k_templates/main_per_task.c

In this second patch, add the subsystem as "rv/dot2k" and the following tags:

  Fixes:	24bce201d798 ("tools/rv: Add dot2k")
  Suggested-by: Daniel Bristot de Oliveira <bristot@...nel.org>

Thanks!
-- Daniel


> Signed-off-by: Zeng Heng <zengheng4@...wei.com>
> ---
>  kernel/trace/rv/monitors/wwnr/wwnr.c | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/kernel/trace/rv/monitors/wwnr/wwnr.c b/kernel/trace/rv/monitors/wwnr/wwnr.c
> index 599225d9cf38..a063b93c6a1d 100644
> --- a/kernel/trace/rv/monitors/wwnr/wwnr.c
> +++ b/kernel/trace/rv/monitors/wwnr/wwnr.c
> @@ -15,7 +15,7 @@
>  
>  #include "wwnr.h"
>  
> -struct rv_monitor rv_wwnr;
> +static struct rv_monitor rv_wwnr;
>  DECLARE_DA_MON_PER_TASK(wwnr, unsigned char);
>  
>  static void handle_switch(void *data, bool preempt, struct task_struct *p,
> @@ -59,7 +59,7 @@ static void disable_wwnr(void)
>  	da_monitor_destroy_wwnr();
>  }
>  
> -struct rv_monitor rv_wwnr = {
> +static struct rv_monitor rv_wwnr = {
>  	.name = "wwnr",
>  	.description = "wakeup while not running per-task testing model.",
>  	.enable = enable_wwnr,