* [PATCH] af_unix: add missing annotation at bpf_iter_unix_seq_stop()
@ 2021-10-01 23:49 Jules Irenge
2021-10-02 1:04 ` Jakub Kicinski
0 siblings, 1 reply; 2+ messages in thread
From: Jules Irenge @ 2021-10-01 23:49 UTC (permalink / raw)
To: linux-kernel
Cc: Jules Irenge, David S. Miller, Jakub Kicinski, Alexei Starovoitov,
Daniel Borkmann, Andrii Nakryiko, Martin KaFai Lau, Song Liu,
Yonghong Song, John Fastabend, KP Singh, Al Viro, Cong Wang,
Eric Dumazet, Rao Shoaib, Jiang Wang, Kuniyuki Iwashima
Sparse reports a warning at bpf_iter_unix_seq_stop()
The root cause is a missing annotation at bpf_iter_unix_seq_stop()
Add the missing __releases(unix_table_lock) annotation
Signed-off-by: Jules Irenge <jbi.octave@gmail.com>
---
net/unix/af_unix.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/net/unix/af_unix.c b/net/unix/af_unix.c
index efac5989edb5..9838d4d855e0 100644
--- a/net/unix/af_unix.c
+++ b/net/unix/af_unix.c
@@ -3291,6 +3291,7 @@ static int bpf_iter_unix_seq_show(struct seq_file *seq, void *v)
}
static void bpf_iter_unix_seq_stop(struct seq_file *seq, void *v)
+ __releases(unix_table_lock)
{
struct bpf_iter_meta meta;
struct bpf_prog *prog;
--
2.32.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] af_unix: add missing annotation at bpf_iter_unix_seq_stop()
2021-10-01 23:49 [PATCH] af_unix: add missing annotation at bpf_iter_unix_seq_stop() Jules Irenge
@ 2021-10-02 1:04 ` Jakub Kicinski
0 siblings, 0 replies; 2+ messages in thread
From: Jakub Kicinski @ 2021-10-02 1:04 UTC (permalink / raw)
To: Jules Irenge
Cc: linux-kernel, David S. Miller, Alexei Starovoitov,
Daniel Borkmann, Andrii Nakryiko, Martin KaFai Lau, Song Liu,
Yonghong Song, John Fastabend, KP Singh, Al Viro, Cong Wang,
Eric Dumazet, Rao Shoaib, Jiang Wang, Kuniyuki Iwashima
On Sat, 2 Oct 2021 00:49:37 +0100 Jules Irenge wrote:
> Sparse reports a warning at bpf_iter_unix_seq_stop()
> The root cause is a missing annotation at bpf_iter_unix_seq_stop()
>
> Add the missing __releases(unix_table_lock) annotation
>
> Signed-off-by: Jules Irenge <jbi.octave@gmail.com>
> ---
> net/unix/af_unix.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/net/unix/af_unix.c b/net/unix/af_unix.c
> index efac5989edb5..9838d4d855e0 100644
> --- a/net/unix/af_unix.c
> +++ b/net/unix/af_unix.c
> @@ -3291,6 +3291,7 @@ static int bpf_iter_unix_seq_show(struct seq_file *seq, void *v)
> }
>
> static void bpf_iter_unix_seq_stop(struct seq_file *seq, void *v)
> + __releases(unix_table_lock)
> {
> struct bpf_iter_meta meta;
> struct bpf_prog *prog;
You need to CC bpf@vger... and netdev@vger...
You can drop the CC for linux-kernel@, approximately nobody reads that.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-10-02 1:04 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-10-01 23:49 [PATCH] af_unix: add missing annotation at bpf_iter_unix_seq_stop() Jules Irenge
2021-10-02 1:04 ` Jakub Kicinski
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox