* [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