[PATCH net-next] udp: Remove duplicate included header file trace/events/udp.h

Thorsten Blum posted 1 patch 1 year, 7 months ago
net/ipv6/udp.c | 1 -
1 file changed, 1 deletion(-)
[PATCH net-next] udp: Remove duplicate included header file trace/events/udp.h
Posted by Thorsten Blum 1 year, 7 months ago
Remove duplicate included header file trace/events/udp.h and the
following warning reported by make includecheck:

  trace/events/udp.h is included more than once

Compile-tested only.

Signed-off-by: Thorsten Blum <thorsten.blum@toblux.com>
---
 net/ipv6/udp.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/net/ipv6/udp.c b/net/ipv6/udp.c
index c81a07ac0463..bfd7fff1bc0c 100644
--- a/net/ipv6/udp.c
+++ b/net/ipv6/udp.c
@@ -46,7 +46,6 @@
 #include <net/tcp_states.h>
 #include <net/ip6_checksum.h>
 #include <net/ip6_tunnel.h>
-#include <trace/events/udp.h>
 #include <net/xfrm.h>
 #include <net/inet_hashtables.h>
 #include <net/inet6_hashtables.h>
-- 
2.45.2
Re: [PATCH net-next] udp: Remove duplicate included header file trace/events/udp.h
Posted by Simon Horman 1 year, 7 months ago
On Sat, Jul 06, 2024 at 09:11:33AM +0200, Thorsten Blum wrote:
> Remove duplicate included header file trace/events/udp.h and the
> following warning reported by make includecheck:
> 
>   trace/events/udp.h is included more than once
> 
> Compile-tested only.
> 
> Signed-off-by: Thorsten Blum <thorsten.blum@toblux.com>

Thanks, I see that trace/events/udp.h is also included on line 36.

Reviewed-by: Simon Horman <horms@kernel.org>

> ---
>  net/ipv6/udp.c | 1 -
>  1 file changed, 1 deletion(-)
> 
> diff --git a/net/ipv6/udp.c b/net/ipv6/udp.c
> index c81a07ac0463..bfd7fff1bc0c 100644
> --- a/net/ipv6/udp.c
> +++ b/net/ipv6/udp.c
> @@ -46,7 +46,6 @@
>  #include <net/tcp_states.h>
>  #include <net/ip6_checksum.h>
>  #include <net/ip6_tunnel.h>
> -#include <trace/events/udp.h>
>  #include <net/xfrm.h>
>  #include <net/inet_hashtables.h>
>  #include <net/inet6_hashtables.h>
> -- 
> 2.45.2
> 
>