* [PATCH net-next] linux/netfilter.h: fix kernel-doc warnings
@ 2023-06-23 6:11 Randy Dunlap
2023-06-23 7:18 ` Simon Horman
2023-06-26 15:17 ` Pablo Neira Ayuso
0 siblings, 2 replies; 3+ messages in thread
From: Randy Dunlap @ 2023-06-23 6:11 UTC (permalink / raw)
To: netdev
Cc: Randy Dunlap, Florian Westphal, Pablo Neira Ayuso,
Jozsef Kadlecsik, netfilter-devel, coreteam, David S. Miller,
Eric Dumazet, Jakub Kicinski, Paolo Abeni
kernel-doc does not support DECLARE_PER_CPU(), so don't mark it with
kernel-doc notation.
One comment block is not kernel-doc notation, so just use
"/*" to begin the comment.
Quietens these warnings:
netfilter.h:493: warning: Function parameter or member 'bool' not described in 'DECLARE_PER_CPU'
netfilter.h:493: warning: Function parameter or member 'nf_skb_duplicated' not described in 'DECLARE_PER_CPU'
netfilter.h:493: warning: expecting prototype for nf_skb_duplicated(). Prototype was for DECLARE_PER_CPU() instead
netfilter.h:496: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
* Contains bitmask of ctnetlink event subscribers, if any.
Fixes: e7c8899f3e6f ("netfilter: move tee_active to core")
Fixes: fdf6491193e4 ("netfilter: ctnetlink: make event listener tracking global")
Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
Cc: Florian Westphal <fw@strlen.de>
Cc: Pablo Neira Ayuso <pablo@netfilter.org>
Cc: Jozsef Kadlecsik <kadlec@netfilter.org>
Cc: netfilter-devel@vger.kernel.org
Cc: coreteam@netfilter.org
Cc: "David S. Miller" <davem@davemloft.net>
Cc: Eric Dumazet <edumazet@google.com>
Cc: Jakub Kicinski <kuba@kernel.org>
Cc: Paolo Abeni <pabeni@redhat.com>
---
include/linux/netfilter.h | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff -- a/include/linux/netfilter.h b/include/linux/netfilter.h
--- a/include/linux/netfilter.h
+++ b/include/linux/netfilter.h
@@ -481,7 +481,7 @@ struct nfnl_ct_hook {
};
extern const struct nfnl_ct_hook __rcu *nfnl_ct_hook;
-/**
+/*
* nf_skb_duplicated - TEE target has sent a packet
*
* When a xtables target sends a packet, the OUTPUT and POSTROUTING
@@ -492,7 +492,7 @@ extern const struct nfnl_ct_hook __rcu *
*/
DECLARE_PER_CPU(bool, nf_skb_duplicated);
-/**
+/*
* Contains bitmask of ctnetlink event subscribers, if any.
* Can't be pernet due to NETLINK_LISTEN_ALL_NSID setsockopt flag.
*/
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH net-next] linux/netfilter.h: fix kernel-doc warnings
2023-06-23 6:11 [PATCH net-next] linux/netfilter.h: fix kernel-doc warnings Randy Dunlap
@ 2023-06-23 7:18 ` Simon Horman
2023-06-26 15:17 ` Pablo Neira Ayuso
1 sibling, 0 replies; 3+ messages in thread
From: Simon Horman @ 2023-06-23 7:18 UTC (permalink / raw)
To: Randy Dunlap
Cc: netdev, Florian Westphal, Pablo Neira Ayuso, Jozsef Kadlecsik,
netfilter-devel, coreteam, David S. Miller, Eric Dumazet,
Jakub Kicinski, Paolo Abeni
On Thu, Jun 22, 2023 at 11:11:01PM -0700, Randy Dunlap wrote:
> kernel-doc does not support DECLARE_PER_CPU(), so don't mark it with
> kernel-doc notation.
>
> One comment block is not kernel-doc notation, so just use
> "/*" to begin the comment.
>
> Quietens these warnings:
>
> netfilter.h:493: warning: Function parameter or member 'bool' not described in 'DECLARE_PER_CPU'
> netfilter.h:493: warning: Function parameter or member 'nf_skb_duplicated' not described in 'DECLARE_PER_CPU'
> netfilter.h:493: warning: expecting prototype for nf_skb_duplicated(). Prototype was for DECLARE_PER_CPU() instead
> netfilter.h:496: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
> * Contains bitmask of ctnetlink event subscribers, if any.
>
> Fixes: e7c8899f3e6f ("netfilter: move tee_active to core")
> Fixes: fdf6491193e4 ("netfilter: ctnetlink: make event listener tracking global")
> Signed-off-by: Randy Dunlap <rdunlap@infradead.org>
> Cc: Florian Westphal <fw@strlen.de>
> Cc: Pablo Neira Ayuso <pablo@netfilter.org>
> Cc: Jozsef Kadlecsik <kadlec@netfilter.org>
> Cc: netfilter-devel@vger.kernel.org
> Cc: coreteam@netfilter.org
> Cc: "David S. Miller" <davem@davemloft.net>
> Cc: Eric Dumazet <edumazet@google.com>
> Cc: Jakub Kicinski <kuba@kernel.org>
> Cc: Paolo Abeni <pabeni@redhat.com>
Reviewed-by: Simon Horman <simon.horman@corigine.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH net-next] linux/netfilter.h: fix kernel-doc warnings
2023-06-23 6:11 [PATCH net-next] linux/netfilter.h: fix kernel-doc warnings Randy Dunlap
2023-06-23 7:18 ` Simon Horman
@ 2023-06-26 15:17 ` Pablo Neira Ayuso
1 sibling, 0 replies; 3+ messages in thread
From: Pablo Neira Ayuso @ 2023-06-26 15:17 UTC (permalink / raw)
To: Randy Dunlap
Cc: netdev, Florian Westphal, Jozsef Kadlecsik, netfilter-devel,
coreteam, David S. Miller, Eric Dumazet, Jakub Kicinski,
Paolo Abeni
On Thu, Jun 22, 2023 at 11:11:01PM -0700, Randy Dunlap wrote:
> kernel-doc does not support DECLARE_PER_CPU(), so don't mark it with
> kernel-doc notation.
>
> One comment block is not kernel-doc notation, so just use
> "/*" to begin the comment.
>
> Quietens these warnings:
>
> netfilter.h:493: warning: Function parameter or member 'bool' not described in 'DECLARE_PER_CPU'
> netfilter.h:493: warning: Function parameter or member 'nf_skb_duplicated' not described in 'DECLARE_PER_CPU'
> netfilter.h:493: warning: expecting prototype for nf_skb_duplicated(). Prototype was for DECLARE_PER_CPU() instead
> netfilter.h:496: warning: This comment starts with '/**', but isn't a kernel-doc comment. Refer Documentation/doc-guide/kernel-doc.rst
> * Contains bitmask of ctnetlink event subscribers, if any.
Applied to nf.git, thanks
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2023-06-26 15:17 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2023-06-23 6:11 [PATCH net-next] linux/netfilter.h: fix kernel-doc warnings Randy Dunlap
2023-06-23 7:18 ` Simon Horman
2023-06-26 15:17 ` Pablo Neira Ayuso
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).