LKML Archive mirror
 help / color / mirror / Atom feed
* [PATCH net-next 3/3] tools: ynl: Remove duplicated include in devlink-user.c
@ 2023-06-09  8:52 Yang Li
  2023-06-09  8:52 ` [PATCH net-next 2/3] tools: ynl: Remove duplicated include in handshake-user.c Yang Li
                   ` (3 more replies)
  0 siblings, 4 replies; 6+ messages in thread
From: Yang Li @ 2023-06-09  8:52 UTC (permalink / raw)
  To: simon.horman
  Cc: davem, edumazet, kuba, pabeni, netdev, linux-hyperv, linux-kernel,
	Yang Li, Abaci Robot

./tools/net/ynl/generated/devlink-user.c: stdlib.h is included more than once.

Reported-by: Abaci Robot <abaci@linux.alibaba.com>
Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=5464
Signed-off-by: Yang Li <yang.lee@linux.alibaba.com>
---
 tools/net/ynl/generated/devlink-user.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/tools/net/ynl/generated/devlink-user.c b/tools/net/ynl/generated/devlink-user.c
index c3204e20b971..18157afd7c73 100644
--- a/tools/net/ynl/generated/devlink-user.c
+++ b/tools/net/ynl/generated/devlink-user.c
@@ -8,7 +8,6 @@
 #include "ynl.h"
 #include <linux/devlink.h>
 
-#include <stdlib.h>
 #include <stdio.h>
 #include <string.h>
 #include <libmnl/libmnl.h>
-- 
2.20.1.7.g153144c


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH net-next 2/3] tools: ynl: Remove duplicated include in handshake-user.c
  2023-06-09  8:52 [PATCH net-next 3/3] tools: ynl: Remove duplicated include in devlink-user.c Yang Li
@ 2023-06-09  8:52 ` Yang Li
  2023-06-09  8:52 ` [PATCH net-next 1/3] net: hv_netvsc: Remove duplicated include in rndis_filter.c Yang Li
                   ` (2 subsequent siblings)
  3 siblings, 0 replies; 6+ messages in thread
From: Yang Li @ 2023-06-09  8:52 UTC (permalink / raw)
  To: simon.horman
  Cc: davem, edumazet, kuba, pabeni, netdev, linux-hyperv, linux-kernel,
	Yang Li, Abaci Robot

./tools/net/ynl/generated/handshake-user.c: stdlib.h is included more than once.

Reported-by: Abaci Robot <abaci@linux.alibaba.com>
Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=5464
Signed-off-by: Yang Li <yang.lee@linux.alibaba.com>
---
 tools/net/ynl/generated/handshake-user.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/tools/net/ynl/generated/handshake-user.c b/tools/net/ynl/generated/handshake-user.c
index fe99c4ef7373..7a1f0364b88f 100644
--- a/tools/net/ynl/generated/handshake-user.c
+++ b/tools/net/ynl/generated/handshake-user.c
@@ -8,7 +8,6 @@
 #include "ynl.h"
 #include <linux/handshake.h>
 
-#include <stdlib.h>
 #include <stdio.h>
 #include <string.h>
 #include <libmnl/libmnl.h>
-- 
2.20.1.7.g153144c


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* [PATCH net-next 1/3] net: hv_netvsc: Remove duplicated include in rndis_filter.c
  2023-06-09  8:52 [PATCH net-next 3/3] tools: ynl: Remove duplicated include in devlink-user.c Yang Li
  2023-06-09  8:52 ` [PATCH net-next 2/3] tools: ynl: Remove duplicated include in handshake-user.c Yang Li
@ 2023-06-09  8:52 ` Yang Li
  2023-06-09 10:40 ` [PATCH net-next 3/3] tools: ynl: Remove duplicated include in devlink-user.c patchwork-bot+netdevbpf
  2023-06-09 11:38 ` Jiri Pirko
  3 siblings, 0 replies; 6+ messages in thread
From: Yang Li @ 2023-06-09  8:52 UTC (permalink / raw)
  To: simon.horman
  Cc: davem, edumazet, kuba, pabeni, netdev, linux-hyperv, linux-kernel,
	Yang Li, Abaci Robot

./drivers/net/hyperv/rndis_filter.c: linux/slab.h is included more than once.

Reported-by: Abaci Robot <abaci@linux.alibaba.com>
Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=5462
Signed-off-by: Yang Li <yang.lee@linux.alibaba.com>
---
 drivers/net/hyperv/rndis_filter.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/net/hyperv/rndis_filter.c b/drivers/net/hyperv/rndis_filter.c
index af95947a87c5..ecc2128ca9b7 100644
--- a/drivers/net/hyperv/rndis_filter.c
+++ b/drivers/net/hyperv/rndis_filter.c
@@ -21,7 +21,6 @@
 #include <linux/rtnetlink.h>
 #include <linux/ucs2_string.h>
 #include <linux/string.h>
-#include <linux/slab.h>
 
 #include "hyperv_net.h"
 #include "netvsc_trace.h"
-- 
2.20.1.7.g153144c


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH net-next 3/3] tools: ynl: Remove duplicated include in devlink-user.c
  2023-06-09  8:52 [PATCH net-next 3/3] tools: ynl: Remove duplicated include in devlink-user.c Yang Li
  2023-06-09  8:52 ` [PATCH net-next 2/3] tools: ynl: Remove duplicated include in handshake-user.c Yang Li
  2023-06-09  8:52 ` [PATCH net-next 1/3] net: hv_netvsc: Remove duplicated include in rndis_filter.c Yang Li
@ 2023-06-09 10:40 ` patchwork-bot+netdevbpf
  2023-06-09 11:38 ` Jiri Pirko
  3 siblings, 0 replies; 6+ messages in thread
From: patchwork-bot+netdevbpf @ 2023-06-09 10:40 UTC (permalink / raw)
  To: Yang Li
  Cc: simon.horman, davem, edumazet, kuba, pabeni, netdev, linux-hyperv,
	linux-kernel, abaci

Hello:

This series was applied to netdev/net-next.git (main)
by David S. Miller <davem@davemloft.net>:

On Fri,  9 Jun 2023 16:52:47 +0800 you wrote:
> ./tools/net/ynl/generated/devlink-user.c: stdlib.h is included more than once.
> 
> Reported-by: Abaci Robot <abaci@linux.alibaba.com>
> Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=5464
> Signed-off-by: Yang Li <yang.lee@linux.alibaba.com>
> ---
>  tools/net/ynl/generated/devlink-user.c | 1 -
>  1 file changed, 1 deletion(-)

Here is the summary with links:
  - [net-next,3/3] tools: ynl: Remove duplicated include in devlink-user.c
    (no matching commit)
  - [net-next,2/3] tools: ynl: Remove duplicated include in handshake-user.c
    https://git.kernel.org/netdev/net-next/c/e7c5433c5aaa
  - [net-next,1/3] net: hv_netvsc: Remove duplicated include in rndis_filter.c
    (no matching commit)

You are awesome, thank you!
-- 
Deet-doot-dot, I am a bot.
https://korg.docs.kernel.org/patchwork/pwbot.html



^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH net-next 3/3] tools: ynl: Remove duplicated include in devlink-user.c
  2023-06-09  8:52 [PATCH net-next 3/3] tools: ynl: Remove duplicated include in devlink-user.c Yang Li
                   ` (2 preceding siblings ...)
  2023-06-09 10:40 ` [PATCH net-next 3/3] tools: ynl: Remove duplicated include in devlink-user.c patchwork-bot+netdevbpf
@ 2023-06-09 11:38 ` Jiri Pirko
  2023-06-09 18:04   ` Jakub Kicinski
  3 siblings, 1 reply; 6+ messages in thread
From: Jiri Pirko @ 2023-06-09 11:38 UTC (permalink / raw)
  To: Yang Li
  Cc: simon.horman, davem, edumazet, kuba, pabeni, netdev, linux-hyperv,
	linux-kernel, Abaci Robot

Fri, Jun 09, 2023 at 10:52:47AM CEST, yang.lee@linux.alibaba.com wrote:
>./tools/net/ynl/generated/devlink-user.c: stdlib.h is included more than once.
>
>Reported-by: Abaci Robot <abaci@linux.alibaba.com>
>Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=5464
>Signed-off-by: Yang Li <yang.lee@linux.alibaba.com>
>---
> tools/net/ynl/generated/devlink-user.c | 1 -
> 1 file changed, 1 deletion(-)
>
>diff --git a/tools/net/ynl/generated/devlink-user.c b/tools/net/ynl/generated/devlink-user.c
>index c3204e20b971..18157afd7c73 100644
>--- a/tools/net/ynl/generated/devlink-user.c
>+++ b/tools/net/ynl/generated/devlink-user.c

You are patching generated file, as the path suggests.
See what the file header says:
/* Do not edit directly, auto-generated from: */
/*      Documentation/netlink/specs/devlink.yaml */


>@@ -8,7 +8,6 @@
> #include "ynl.h"
> #include <linux/devlink.h>
> 
>-#include <stdlib.h>
> #include <stdio.h>
> #include <string.h>
> #include <libmnl/libmnl.h>
>-- 
>2.20.1.7.g153144c
>
>

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH net-next 3/3] tools: ynl: Remove duplicated include in devlink-user.c
  2023-06-09 11:38 ` Jiri Pirko
@ 2023-06-09 18:04   ` Jakub Kicinski
  0 siblings, 0 replies; 6+ messages in thread
From: Jakub Kicinski @ 2023-06-09 18:04 UTC (permalink / raw)
  To: Jiri Pirko
  Cc: Yang Li, simon.horman, davem, edumazet, pabeni, netdev,
	linux-hyperv, linux-kernel, Abaci Robot

On Fri, 9 Jun 2023 13:38:21 +0200 Jiri Pirko wrote:
> You are patching generated file, as the path suggests.
> See what the file header says:
> /* Do not edit directly, auto-generated from: */
> /*      Documentation/netlink/specs/devlink.yaml */

And the full fix is already on the list :(
https://lore.kernel.org/all/20230608211200.1247213-2-kuba@kernel.org/
Reverted...

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2023-06-09 18:04 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-06-09  8:52 [PATCH net-next 3/3] tools: ynl: Remove duplicated include in devlink-user.c Yang Li
2023-06-09  8:52 ` [PATCH net-next 2/3] tools: ynl: Remove duplicated include in handshake-user.c Yang Li
2023-06-09  8:52 ` [PATCH net-next 1/3] net: hv_netvsc: Remove duplicated include in rndis_filter.c Yang Li
2023-06-09 10:40 ` [PATCH net-next 3/3] tools: ynl: Remove duplicated include in devlink-user.c patchwork-bot+netdevbpf
2023-06-09 11:38 ` Jiri Pirko
2023-06-09 18: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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).