linux-hyperv.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH 1/7] net: 8021q: remove unneeded MODULE_VERSION() usage
@ 2020-12-02 12:49 Enrico Weigelt, metux IT consult
  2020-12-02 12:49 ` [PATCH 2/7] net: batman-adv: " Enrico Weigelt, metux IT consult
                   ` (7 more replies)
  0 siblings, 8 replies; 16+ messages in thread
From: Enrico Weigelt, metux IT consult @ 2020-12-02 12:49 UTC (permalink / raw)
  To: linux-kernel
  Cc: davem, kuba, mareklindner, sw, a, sven, marcel, johan.hedberg,
	roopa, nikolay, edumazet, kuznet, yoshfuji, jmaloy, ying.xue,
	kafai, songliubraving, yhs, john.fastabend, kpsingh, netdev,
	linux-bluetooth, tipc-discussion, linux-hyperv, bpf

Remove MODULE_VERSION(), as it isn't needed at all: the only version
making sense is the kernel version.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
---
 net/8021q/vlan.c | 8 +-------
 1 file changed, 1 insertion(+), 7 deletions(-)

diff --git a/net/8021q/vlan.c b/net/8021q/vlan.c
index f292e0267bb9..683e9e825b9e 100644
--- a/net/8021q/vlan.c
+++ b/net/8021q/vlan.c
@@ -36,15 +36,10 @@
 #include "vlan.h"
 #include "vlanproc.h"
 
-#define DRV_VERSION "1.8"
-
 /* Global VLAN variables */
 
 unsigned int vlan_net_id __read_mostly;
 
-const char vlan_fullname[] = "802.1Q VLAN Support";
-const char vlan_version[] = DRV_VERSION;
-
 /* End of global variables definitions. */
 
 static int vlan_group_prealloc_vid(struct vlan_group *vg,
@@ -687,7 +682,7 @@ static int __init vlan_proto_init(void)
 {
 	int err;
 
-	pr_info("%s v%s\n", vlan_fullname, vlan_version);
+	pr_info("802.1Q VLAN Support\n");
 
 	err = register_pernet_subsys(&vlan_net_ops);
 	if (err < 0)
@@ -743,4 +738,3 @@ module_init(vlan_proto_init);
 module_exit(vlan_cleanup_module);
 
 MODULE_LICENSE("GPL");
-MODULE_VERSION(DRV_VERSION);
-- 
2.11.0


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

* [PATCH 2/7] net: batman-adv: remove unneeded MODULE_VERSION() usage
  2020-12-02 12:49 [PATCH 1/7] net: 8021q: remove unneeded MODULE_VERSION() usage Enrico Weigelt, metux IT consult
@ 2020-12-02 12:49 ` Enrico Weigelt, metux IT consult
  2020-12-05  7:06   ` Sven Eckelmann
  2020-12-02 12:49 ` [PATCH 3/7] net: ipv4: " Enrico Weigelt, metux IT consult
                   ` (6 subsequent siblings)
  7 siblings, 1 reply; 16+ messages in thread
From: Enrico Weigelt, metux IT consult @ 2020-12-02 12:49 UTC (permalink / raw)
  To: linux-kernel
  Cc: davem, kuba, mareklindner, sw, a, sven, marcel, johan.hedberg,
	roopa, nikolay, edumazet, kuznet, yoshfuji, jmaloy, ying.xue,
	kafai, songliubraving, yhs, john.fastabend, kpsingh, netdev,
	linux-bluetooth, tipc-discussion, linux-hyperv, bpf

Remove MODULE_VERSION(), as it isn't needed at all: the only version
making sense is the kernel version.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
---
 net/batman-adv/main.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/net/batman-adv/main.c b/net/batman-adv/main.c
index 70fee9b42e25..1c2ccad94bf8 100644
--- a/net/batman-adv/main.c
+++ b/net/batman-adv/main.c
@@ -747,6 +747,5 @@ MODULE_LICENSE("GPL");
 MODULE_AUTHOR(BATADV_DRIVER_AUTHOR);
 MODULE_DESCRIPTION(BATADV_DRIVER_DESC);
 MODULE_SUPPORTED_DEVICE(BATADV_DRIVER_DEVICE);
-MODULE_VERSION(BATADV_SOURCE_VERSION);
 MODULE_ALIAS_RTNL_LINK("batadv");
 MODULE_ALIAS_GENL_FAMILY(BATADV_NL_NAME);
-- 
2.11.0


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

* [PATCH 3/7] net: ipv4: remove unneeded MODULE_VERSION() usage
  2020-12-02 12:49 [PATCH 1/7] net: 8021q: remove unneeded MODULE_VERSION() usage Enrico Weigelt, metux IT consult
  2020-12-02 12:49 ` [PATCH 2/7] net: batman-adv: " Enrico Weigelt, metux IT consult
@ 2020-12-02 12:49 ` Enrico Weigelt, metux IT consult
  2020-12-02 12:49 ` [PATCH 4/7] net: bluetooth: " Enrico Weigelt, metux IT consult
                   ` (5 subsequent siblings)
  7 siblings, 0 replies; 16+ messages in thread
From: Enrico Weigelt, metux IT consult @ 2020-12-02 12:49 UTC (permalink / raw)
  To: linux-kernel
  Cc: davem, kuba, mareklindner, sw, a, sven, marcel, johan.hedberg,
	roopa, nikolay, edumazet, kuznet, yoshfuji, jmaloy, ying.xue,
	kafai, songliubraving, yhs, john.fastabend, kpsingh, netdev,
	linux-bluetooth, tipc-discussion, linux-hyperv, bpf

Remove MODULE_VERSION(), as it isn't needed at all: the only version
making sense is the kernel version.

Signed-off-by: Enrico Weigelt <info@metux.net>
---
 net/ipv4/tcp_cubic.c    | 1 -
 net/ipv4/tcp_illinois.c | 1 -
 net/ipv4/tcp_nv.c       | 1 -
 3 files changed, 3 deletions(-)

diff --git a/net/ipv4/tcp_cubic.c b/net/ipv4/tcp_cubic.c
index c7bf5b26bf0c..c6bcd445df04 100644
--- a/net/ipv4/tcp_cubic.c
+++ b/net/ipv4/tcp_cubic.c
@@ -537,4 +537,3 @@ module_exit(cubictcp_unregister);
 MODULE_AUTHOR("Sangtae Ha, Stephen Hemminger");
 MODULE_LICENSE("GPL");
 MODULE_DESCRIPTION("CUBIC TCP");
-MODULE_VERSION("2.3");
diff --git a/net/ipv4/tcp_illinois.c b/net/ipv4/tcp_illinois.c
index 00e54873213e..8cc9967e82ef 100644
--- a/net/ipv4/tcp_illinois.c
+++ b/net/ipv4/tcp_illinois.c
@@ -355,4 +355,3 @@ module_exit(tcp_illinois_unregister);
 MODULE_AUTHOR("Stephen Hemminger, Shao Liu");
 MODULE_LICENSE("GPL");
 MODULE_DESCRIPTION("TCP Illinois");
-MODULE_VERSION("1.0");
diff --git a/net/ipv4/tcp_nv.c b/net/ipv4/tcp_nv.c
index 95db7a11ba2a..b3879fb24d33 100644
--- a/net/ipv4/tcp_nv.c
+++ b/net/ipv4/tcp_nv.c
@@ -499,4 +499,3 @@ module_exit(tcpnv_unregister);
 MODULE_AUTHOR("Lawrence Brakmo");
 MODULE_LICENSE("GPL");
 MODULE_DESCRIPTION("TCP NV");
-MODULE_VERSION("1.0");
-- 
2.11.0


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

* [PATCH 4/7] net: bluetooth: remove unneeded MODULE_VERSION() usage
  2020-12-02 12:49 [PATCH 1/7] net: 8021q: remove unneeded MODULE_VERSION() usage Enrico Weigelt, metux IT consult
  2020-12-02 12:49 ` [PATCH 2/7] net: batman-adv: " Enrico Weigelt, metux IT consult
  2020-12-02 12:49 ` [PATCH 3/7] net: ipv4: " Enrico Weigelt, metux IT consult
@ 2020-12-02 12:49 ` Enrico Weigelt, metux IT consult
  2020-12-02 12:49 ` [PATCH 5/7] net: bridge: " Enrico Weigelt, metux IT consult
                   ` (4 subsequent siblings)
  7 siblings, 0 replies; 16+ messages in thread
From: Enrico Weigelt, metux IT consult @ 2020-12-02 12:49 UTC (permalink / raw)
  To: linux-kernel
  Cc: davem, kuba, mareklindner, sw, a, sven, marcel, johan.hedberg,
	roopa, nikolay, edumazet, kuznet, yoshfuji, jmaloy, ying.xue,
	kafai, songliubraving, yhs, john.fastabend, kpsingh, netdev,
	linux-bluetooth, tipc-discussion, linux-hyperv, bpf

Remove MODULE_VERSION(), as it isn't needed at all: the only version
making sense is the kernel version.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
---
 net/bluetooth/6lowpan.c      | 3 ---
 net/bluetooth/af_bluetooth.c | 1 -
 net/bluetooth/bnep/core.c    | 1 -
 net/bluetooth/cmtp/core.c    | 1 -
 net/bluetooth/hidp/core.c    | 1 -
 net/bluetooth/rfcomm/core.c  | 1 -
 6 files changed, 8 deletions(-)

diff --git a/net/bluetooth/6lowpan.c b/net/bluetooth/6lowpan.c
index cff4944d5b66..9759515edc6e 100644
--- a/net/bluetooth/6lowpan.c
+++ b/net/bluetooth/6lowpan.c
@@ -21,8 +21,6 @@
 
 #include <net/6lowpan.h> /* for the compression support */
 
-#define VERSION "0.1"
-
 static struct dentry *lowpan_enable_debugfs;
 static struct dentry *lowpan_control_debugfs;
 
@@ -1316,5 +1314,4 @@ module_exit(bt_6lowpan_exit);
 
 MODULE_AUTHOR("Jukka Rissanen <jukka.rissanen@linux.intel.com>");
 MODULE_DESCRIPTION("Bluetooth 6LoWPAN");
-MODULE_VERSION(VERSION);
 MODULE_LICENSE("GPL");
diff --git a/net/bluetooth/af_bluetooth.c b/net/bluetooth/af_bluetooth.c
index 4ef6a54403aa..6d587a3b3c56 100644
--- a/net/bluetooth/af_bluetooth.c
+++ b/net/bluetooth/af_bluetooth.c
@@ -799,6 +799,5 @@ module_exit(bt_exit);
 
 MODULE_AUTHOR("Marcel Holtmann <marcel@holtmann.org>");
 MODULE_DESCRIPTION("Bluetooth Core ver " VERSION);
-MODULE_VERSION(VERSION);
 MODULE_LICENSE("GPL");
 MODULE_ALIAS_NETPROTO(PF_BLUETOOTH);
diff --git a/net/bluetooth/bnep/core.c b/net/bluetooth/bnep/core.c
index 43c284158f63..96f0eb60deb0 100644
--- a/net/bluetooth/bnep/core.c
+++ b/net/bluetooth/bnep/core.c
@@ -764,6 +764,5 @@ MODULE_PARM_DESC(compress_dst, "Compress destination headers");
 
 MODULE_AUTHOR("Marcel Holtmann <marcel@holtmann.org>");
 MODULE_DESCRIPTION("Bluetooth BNEP ver " VERSION);
-MODULE_VERSION(VERSION);
 MODULE_LICENSE("GPL");
 MODULE_ALIAS("bt-proto-4");
diff --git a/net/bluetooth/cmtp/core.c b/net/bluetooth/cmtp/core.c
index 07cfa3249f83..9a6306d7f738 100644
--- a/net/bluetooth/cmtp/core.c
+++ b/net/bluetooth/cmtp/core.c
@@ -511,6 +511,5 @@ module_exit(cmtp_exit);
 
 MODULE_AUTHOR("Marcel Holtmann <marcel@holtmann.org>");
 MODULE_DESCRIPTION("Bluetooth CMTP ver " VERSION);
-MODULE_VERSION(VERSION);
 MODULE_LICENSE("GPL");
 MODULE_ALIAS("bt-proto-5");
diff --git a/net/bluetooth/hidp/core.c b/net/bluetooth/hidp/core.c
index 3b4fa27a44e6..597f5dde434a 100644
--- a/net/bluetooth/hidp/core.c
+++ b/net/bluetooth/hidp/core.c
@@ -1475,6 +1475,5 @@ module_exit(hidp_exit);
 MODULE_AUTHOR("Marcel Holtmann <marcel@holtmann.org>");
 MODULE_AUTHOR("David Herrmann <dh.herrmann@gmail.com>");
 MODULE_DESCRIPTION("Bluetooth HIDP ver " VERSION);
-MODULE_VERSION(VERSION);
 MODULE_LICENSE("GPL");
 MODULE_ALIAS("bt-proto-6");
diff --git a/net/bluetooth/rfcomm/core.c b/net/bluetooth/rfcomm/core.c
index f2bacb464ccf..3384da308a9b 100644
--- a/net/bluetooth/rfcomm/core.c
+++ b/net/bluetooth/rfcomm/core.c
@@ -2240,6 +2240,5 @@ MODULE_PARM_DESC(l2cap_ertm, "Use L2CAP ERTM mode for connection");
 
 MODULE_AUTHOR("Marcel Holtmann <marcel@holtmann.org>");
 MODULE_DESCRIPTION("Bluetooth RFCOMM ver " VERSION);
-MODULE_VERSION(VERSION);
 MODULE_LICENSE("GPL");
 MODULE_ALIAS("bt-proto-3");
-- 
2.11.0


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

* [PATCH 5/7] net: bridge: remove unneeded MODULE_VERSION() usage
  2020-12-02 12:49 [PATCH 1/7] net: 8021q: remove unneeded MODULE_VERSION() usage Enrico Weigelt, metux IT consult
                   ` (2 preceding siblings ...)
  2020-12-02 12:49 ` [PATCH 4/7] net: bluetooth: " Enrico Weigelt, metux IT consult
@ 2020-12-02 12:49 ` Enrico Weigelt, metux IT consult
  2020-12-02 12:49 ` [PATCH 6/7] net: vmw_vsock: " Enrico Weigelt, metux IT consult
                   ` (3 subsequent siblings)
  7 siblings, 0 replies; 16+ messages in thread
From: Enrico Weigelt, metux IT consult @ 2020-12-02 12:49 UTC (permalink / raw)
  To: linux-kernel
  Cc: davem, kuba, mareklindner, sw, a, sven, marcel, johan.hedberg,
	roopa, nikolay, edumazet, kuznet, yoshfuji, jmaloy, ying.xue,
	kafai, songliubraving, yhs, john.fastabend, kpsingh, netdev,
	linux-bluetooth, tipc-discussion, linux-hyperv, bpf

Remove MODULE_VERSION(), as it isn't needed at all: the only version
making sense is the kernel version.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
---
 net/bridge/br.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/net/bridge/br.c b/net/bridge/br.c
index 401eeb9142eb..2502fdcbb8b2 100644
--- a/net/bridge/br.c
+++ b/net/bridge/br.c
@@ -399,5 +399,4 @@ static void __exit br_deinit(void)
 module_init(br_init)
 module_exit(br_deinit)
 MODULE_LICENSE("GPL");
-MODULE_VERSION(BR_VERSION);
 MODULE_ALIAS_RTNL_LINK("bridge");
-- 
2.11.0


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

* [PATCH 6/7] net: vmw_vsock: remove unneeded MODULE_VERSION() usage
  2020-12-02 12:49 [PATCH 1/7] net: 8021q: remove unneeded MODULE_VERSION() usage Enrico Weigelt, metux IT consult
                   ` (3 preceding siblings ...)
  2020-12-02 12:49 ` [PATCH 5/7] net: bridge: " Enrico Weigelt, metux IT consult
@ 2020-12-02 12:49 ` Enrico Weigelt, metux IT consult
  2020-12-02 12:49 ` [PATCH 7/7] net: tipc: " Enrico Weigelt, metux IT consult
                   ` (2 subsequent siblings)
  7 siblings, 0 replies; 16+ messages in thread
From: Enrico Weigelt, metux IT consult @ 2020-12-02 12:49 UTC (permalink / raw)
  To: linux-kernel
  Cc: davem, kuba, mareklindner, sw, a, sven, marcel, johan.hedberg,
	roopa, nikolay, edumazet, kuznet, yoshfuji, jmaloy, ying.xue,
	kafai, songliubraving, yhs, john.fastabend, kpsingh, netdev,
	linux-bluetooth, tipc-discussion, linux-hyperv, bpf

Remove MODULE_VERSION(), as it isn't needed at all: the only version
making sense is the kernel version.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
---
 net/vmw_vsock/af_vsock.c         | 1 -
 net/vmw_vsock/hyperv_transport.c | 1 -
 net/vmw_vsock/vmci_transport.c   | 1 -
 3 files changed, 3 deletions(-)

diff --git a/net/vmw_vsock/af_vsock.c b/net/vmw_vsock/af_vsock.c
index d10916ab4526..cc196ffba3ed 100644
--- a/net/vmw_vsock/af_vsock.c
+++ b/net/vmw_vsock/af_vsock.c
@@ -2238,5 +2238,4 @@ module_exit(vsock_exit);
 
 MODULE_AUTHOR("VMware, Inc.");
 MODULE_DESCRIPTION("VMware Virtual Socket Family");
-MODULE_VERSION("1.0.2.0-k");
 MODULE_LICENSE("GPL v2");
diff --git a/net/vmw_vsock/hyperv_transport.c b/net/vmw_vsock/hyperv_transport.c
index 630b851f8150..dae562f40896 100644
--- a/net/vmw_vsock/hyperv_transport.c
+++ b/net/vmw_vsock/hyperv_transport.c
@@ -929,6 +929,5 @@ module_init(hvs_init);
 module_exit(hvs_exit);
 
 MODULE_DESCRIPTION("Hyper-V Sockets");
-MODULE_VERSION("1.0.0");
 MODULE_LICENSE("GPL");
 MODULE_ALIAS_NETPROTO(PF_VSOCK);
diff --git a/net/vmw_vsock/vmci_transport.c b/net/vmw_vsock/vmci_transport.c
index 8b65323207db..bd39cca58ee6 100644
--- a/net/vmw_vsock/vmci_transport.c
+++ b/net/vmw_vsock/vmci_transport.c
@@ -2140,7 +2140,6 @@ module_exit(vmci_transport_exit);
 
 MODULE_AUTHOR("VMware, Inc.");
 MODULE_DESCRIPTION("VMCI transport for Virtual Sockets");
-MODULE_VERSION("1.0.5.0-k");
 MODULE_LICENSE("GPL v2");
 MODULE_ALIAS("vmware_vsock");
 MODULE_ALIAS_NETPROTO(PF_VSOCK);
-- 
2.11.0


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

* [PATCH 7/7] net: tipc: remove unneeded MODULE_VERSION() usage
  2020-12-02 12:49 [PATCH 1/7] net: 8021q: remove unneeded MODULE_VERSION() usage Enrico Weigelt, metux IT consult
                   ` (4 preceding siblings ...)
  2020-12-02 12:49 ` [PATCH 6/7] net: vmw_vsock: " Enrico Weigelt, metux IT consult
@ 2020-12-02 12:49 ` Enrico Weigelt, metux IT consult
  2020-12-05  0:09 ` [PATCH 1/7] net: 8021q: " Jakub Kicinski
  2020-12-05 11:20 ` Vladimir Oltean
  7 siblings, 0 replies; 16+ messages in thread
From: Enrico Weigelt, metux IT consult @ 2020-12-02 12:49 UTC (permalink / raw)
  To: linux-kernel
  Cc: davem, kuba, mareklindner, sw, a, sven, marcel, johan.hedberg,
	roopa, nikolay, edumazet, kuznet, yoshfuji, jmaloy, ying.xue,
	kafai, songliubraving, yhs, john.fastabend, kpsingh, netdev,
	linux-bluetooth, tipc-discussion, linux-hyperv, bpf

Remove MODULE_VERSION(), as it isn't needed at all: the only version
making sense is the kernel version.

Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
---
 net/tipc/core.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/net/tipc/core.c b/net/tipc/core.c
index c2ff42900b53..8c0c45347c53 100644
--- a/net/tipc/core.c
+++ b/net/tipc/core.c
@@ -227,4 +227,3 @@ module_exit(tipc_exit);
 
 MODULE_DESCRIPTION("TIPC: Transparent Inter Process Communication");
 MODULE_LICENSE("Dual BSD/GPL");
-MODULE_VERSION(TIPC_MOD_VER);
-- 
2.11.0


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

* Re: [PATCH 1/7] net: 8021q: remove unneeded MODULE_VERSION() usage
  2020-12-02 12:49 [PATCH 1/7] net: 8021q: remove unneeded MODULE_VERSION() usage Enrico Weigelt, metux IT consult
                   ` (5 preceding siblings ...)
  2020-12-02 12:49 ` [PATCH 7/7] net: tipc: " Enrico Weigelt, metux IT consult
@ 2020-12-05  0:09 ` Jakub Kicinski
  2020-12-05 10:51   ` Joe Perches
  2020-12-05 11:20 ` Vladimir Oltean
  7 siblings, 1 reply; 16+ messages in thread
From: Jakub Kicinski @ 2020-12-05  0:09 UTC (permalink / raw)
  To: Enrico Weigelt, metux IT consult
  Cc: linux-kernel, davem, mareklindner, sw, a, sven, marcel,
	johan.hedberg, roopa, nikolay, edumazet, kuznet, yoshfuji,
	jmaloy, ying.xue, kafai, songliubraving, yhs, john.fastabend,
	kpsingh, netdev, linux-bluetooth, tipc-discussion, linux-hyperv,
	bpf

On Wed,  2 Dec 2020 13:49:53 +0100 Enrico Weigelt, metux IT consult
wrote:
> Remove MODULE_VERSION(), as it isn't needed at all: the only version
> making sense is the kernel version.
> 
> Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>

Thanks for the patches. Please drop the "metux IT consult" from the
addresses. The from space is supposed to be for your name.

> diff --git a/net/8021q/vlan.c b/net/8021q/vlan.c
> index f292e0267bb9..683e9e825b9e 100644
> --- a/net/8021q/vlan.c
> +++ b/net/8021q/vlan.c
> @@ -36,15 +36,10 @@
>  #include "vlan.h"
>  #include "vlanproc.h"
>  
> -#define DRV_VERSION "1.8"
> -
>  /* Global VLAN variables */
>  
>  unsigned int vlan_net_id __read_mostly;
>  
> -const char vlan_fullname[] = "802.1Q VLAN Support";
> -const char vlan_version[] = DRV_VERSION;

This patches does not build. Please redo it more carefully.

You'll need to fix and resend the entire series. When you do so please
provide a cover letter, even if it only contains a couple of sentences
and separate the patches for bluetooth and batman-adv out as stand
alone patches, so the respective maintainers can pick them up.

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

* Re: [PATCH 2/7] net: batman-adv: remove unneeded MODULE_VERSION() usage
  2020-12-02 12:49 ` [PATCH 2/7] net: batman-adv: " Enrico Weigelt, metux IT consult
@ 2020-12-05  7:06   ` Sven Eckelmann
  2020-12-05 16:56     ` Jakub Kicinski
  2020-12-08  7:48     ` Enrico Weigelt, metux IT consult
  0 siblings, 2 replies; 16+ messages in thread
From: Sven Eckelmann @ 2020-12-05  7:06 UTC (permalink / raw)
  To: linux-kernel, Enrico Weigelt, metux IT consult
  Cc: davem, kuba, mareklindner, sw, a, marcel, johan.hedberg, roopa,
	nikolay, edumazet, kuznet, yoshfuji, jmaloy, ying.xue, kafai,
	songliubraving, yhs, john.fastabend, kpsingh, netdev,
	linux-bluetooth, tipc-discussion, linux-hyperv, bpf,
	Matthias Schiffer

[-- Attachment #1: Type: text/plain, Size: 757 bytes --]

On Wednesday, 2 December 2020 13:49:54 CET Enrico Weigelt, metux IT consult wrote:
> Remove MODULE_VERSION(), as it isn't needed at all: the only version
> making sense is the kernel version.

Is there some explanation besides an opinion? Some kind goal which you want to 
achieve with it maybe?

At least for us it was an easy way to query the release cycle information via 
batctl. Which made it easier for us to roughly figure out what an reporter/
inquirer was using - independent of whether he is using the in-kernel version 
or a backported version.

Loosing this source of information and breaking parts of batctl and other 
tools (respondd, ...) is not the end of the world. But I would at least know 
why this is now necessary.

Kind regards,
	Sven

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

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

* Re: [PATCH 1/7] net: 8021q: remove unneeded MODULE_VERSION() usage
  2020-12-05  0:09 ` [PATCH 1/7] net: 8021q: " Jakub Kicinski
@ 2020-12-05 10:51   ` Joe Perches
  0 siblings, 0 replies; 16+ messages in thread
From: Joe Perches @ 2020-12-05 10:51 UTC (permalink / raw)
  To: Jakub Kicinski, Enrico Weigelt, metux IT consult
  Cc: linux-kernel, davem, mareklindner, sw, a, sven, marcel,
	johan.hedberg, roopa, nikolay, edumazet, kuznet, yoshfuji,
	jmaloy, ying.xue, kafai, songliubraving, yhs, john.fastabend,
	kpsingh, netdev, linux-bluetooth, tipc-discussion, linux-hyperv,
	bpf

On Fri, 2020-12-04 at 16:09 -0800, Jakub Kicinski wrote:
> On Wed,  2 Dec 2020 13:49:53 +0100 Enrico Weigelt, metux IT consult
> wrote:
> > Remove MODULE_VERSION(), as it isn't needed at all: the only version
> > making sense is the kernel version.
> > 
> > Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
> 
> Thanks for the patches. Please drop the "metux IT consult" from the
> addresses. The from space is supposed to be for your name.

If you _really_ want this superfluous 'metux IT consult' content in your
signature, and I don't think you should, use parentheses around it.

Enrico Weigelt (metux IT consult) <info@metux.net>

Using a comma makes copy/paste into an email client think it's two addresses.




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

* Re: [PATCH 1/7] net: 8021q: remove unneeded MODULE_VERSION() usage
  2020-12-02 12:49 [PATCH 1/7] net: 8021q: remove unneeded MODULE_VERSION() usage Enrico Weigelt, metux IT consult
                   ` (6 preceding siblings ...)
  2020-12-05  0:09 ` [PATCH 1/7] net: 8021q: " Jakub Kicinski
@ 2020-12-05 11:20 ` Vladimir Oltean
  2020-12-05 15:53   ` Greg KH
  7 siblings, 1 reply; 16+ messages in thread
From: Vladimir Oltean @ 2020-12-05 11:20 UTC (permalink / raw)
  To: Enrico Weigelt, metux IT consult
  Cc: linux-kernel, davem, kuba, mareklindner, sw, a, sven, marcel,
	johan.hedberg, roopa, nikolay, edumazet, kuznet, yoshfuji,
	jmaloy, ying.xue, kafai, songliubraving, yhs, john.fastabend,
	kpsingh, netdev, linux-bluetooth, tipc-discussion, linux-hyperv,
	bpf

On Wed, Dec 02, 2020 at 01:49:53PM +0100, Enrico Weigelt, metux IT consult wrote:
> Remove MODULE_VERSION(), as it isn't needed at all: the only version
> making sense is the kernel version.
>
> Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
> ---
>  net/8021q/vlan.c | 8 +-------
>  1 file changed, 1 insertion(+), 7 deletions(-)
>
> diff --git a/net/8021q/vlan.c b/net/8021q/vlan.c
> index f292e0267bb9..683e9e825b9e 100644
> --- a/net/8021q/vlan.c
> +++ b/net/8021q/vlan.c
> @@ -36,15 +36,10 @@
>  #include "vlan.h"
>  #include "vlanproc.h"
>
> -#define DRV_VERSION "1.8"
> -
>  /* Global VLAN variables */
>
>  unsigned int vlan_net_id __read_mostly;
>
> -const char vlan_fullname[] = "802.1Q VLAN Support";
> -const char vlan_version[] = DRV_VERSION;
> -
>  /* End of global variables definitions. */
>
>  static int vlan_group_prealloc_vid(struct vlan_group *vg,
> @@ -687,7 +682,7 @@ static int __init vlan_proto_init(void)
>  {
>  	int err;
>
> -	pr_info("%s v%s\n", vlan_fullname, vlan_version);
> +	pr_info("802.1Q VLAN Support\n");

How do we feel about deleting this not really informative message
altogether in a future patch?

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

* Re: [PATCH 1/7] net: 8021q: remove unneeded MODULE_VERSION() usage
  2020-12-05 11:20 ` Vladimir Oltean
@ 2020-12-05 15:53   ` Greg KH
  2020-12-08  8:51     ` Enrico Weigelt, metux IT consult
  0 siblings, 1 reply; 16+ messages in thread
From: Greg KH @ 2020-12-05 15:53 UTC (permalink / raw)
  To: Vladimir Oltean
  Cc: Enrico Weigelt, metux IT consult, linux-kernel, davem, kuba,
	mareklindner, sw, a, sven, marcel, johan.hedberg, roopa, nikolay,
	edumazet, kuznet, yoshfuji, jmaloy, ying.xue, kafai,
	songliubraving, yhs, john.fastabend, kpsingh, netdev,
	linux-bluetooth, tipc-discussion, linux-hyperv, bpf

On Sat, Dec 05, 2020 at 01:20:18PM +0200, Vladimir Oltean wrote:
> On Wed, Dec 02, 2020 at 01:49:53PM +0100, Enrico Weigelt, metux IT consult wrote:
> > Remove MODULE_VERSION(), as it isn't needed at all: the only version
> > making sense is the kernel version.
> >
> > Signed-off-by: Enrico Weigelt, metux IT consult <info@metux.net>
> > ---
> >  net/8021q/vlan.c | 8 +-------
> >  1 file changed, 1 insertion(+), 7 deletions(-)
> >
> > diff --git a/net/8021q/vlan.c b/net/8021q/vlan.c
> > index f292e0267bb9..683e9e825b9e 100644
> > --- a/net/8021q/vlan.c
> > +++ b/net/8021q/vlan.c
> > @@ -36,15 +36,10 @@
> >  #include "vlan.h"
> >  #include "vlanproc.h"
> >
> > -#define DRV_VERSION "1.8"
> > -
> >  /* Global VLAN variables */
> >
> >  unsigned int vlan_net_id __read_mostly;
> >
> > -const char vlan_fullname[] = "802.1Q VLAN Support";
> > -const char vlan_version[] = DRV_VERSION;
> > -
> >  /* End of global variables definitions. */
> >
> >  static int vlan_group_prealloc_vid(struct vlan_group *vg,
> > @@ -687,7 +682,7 @@ static int __init vlan_proto_init(void)
> >  {
> >  	int err;
> >
> > -	pr_info("%s v%s\n", vlan_fullname, vlan_version);
> > +	pr_info("802.1Q VLAN Support\n");
> 
> How do we feel about deleting this not really informative message
> altogether in a future patch?

It too should be removed.  If drivers are working properly, they are
quiet.

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

* Re: [PATCH 2/7] net: batman-adv: remove unneeded MODULE_VERSION() usage
  2020-12-05  7:06   ` Sven Eckelmann
@ 2020-12-05 16:56     ` Jakub Kicinski
  2020-12-08  7:48     ` Enrico Weigelt, metux IT consult
  1 sibling, 0 replies; 16+ messages in thread
From: Jakub Kicinski @ 2020-12-05 16:56 UTC (permalink / raw)
  To: Sven Eckelmann
  Cc: linux-kernel, Enrico Weigelt, metux IT consult, davem,
	mareklindner, sw, a, marcel, johan.hedberg, roopa, nikolay,
	edumazet, kuznet, yoshfuji, jmaloy, ying.xue, kafai,
	songliubraving, yhs, john.fastabend, kpsingh, netdev,
	linux-bluetooth, tipc-discussion, linux-hyperv, bpf,
	Matthias Schiffer

On Sat, 05 Dec 2020 08:06:40 +0100 Sven Eckelmann wrote:
> On Wednesday, 2 December 2020 13:49:54 CET Enrico Weigelt, metux IT consult wrote:
> > Remove MODULE_VERSION(), as it isn't needed at all: the only version
> > making sense is the kernel version.  
> 
> Is there some explanation besides an opinion? Some kind goal which you want to 
> achieve with it maybe?
> 
> At least for us it was an easy way to query the release cycle information via 
> batctl. Which made it easier for us to roughly figure out what an reporter/
> inquirer was using - independent of whether he is using the in-kernel version 
> or a backported version.
> 
> Loosing this source of information and breaking parts of batctl and other 
> tools (respondd, ...) is not the end of the world. But I would at least know 
> why this is now necessary.

No, no, if it breaks your user space we can't do it, let's leave batman
alone, then.

I think this is mostly a clean up. In-tree the kernel version is usually
far more dependable because backports don't include version bumps.

Indeed it would be great if the clear motivation was spelled out in the
cover letter and/or patches.

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

* Re: [PATCH 2/7] net: batman-adv: remove unneeded MODULE_VERSION() usage
  2020-12-05  7:06   ` Sven Eckelmann
  2020-12-05 16:56     ` Jakub Kicinski
@ 2020-12-08  7:48     ` Enrico Weigelt, metux IT consult
  2020-12-08  9:55       ` Sven Eckelmann
  1 sibling, 1 reply; 16+ messages in thread
From: Enrico Weigelt, metux IT consult @ 2020-12-08  7:48 UTC (permalink / raw)
  To: Sven Eckelmann, linux-kernel, Enrico Weigelt, metux IT consult
  Cc: davem, kuba, mareklindner, sw, a, marcel, johan.hedberg, roopa,
	nikolay, edumazet, kuznet, yoshfuji, jmaloy, ying.xue, kafai,
	songliubraving, yhs, john.fastabend, kpsingh, netdev,
	linux-bluetooth, tipc-discussion, linux-hyperv, bpf,
	Matthias Schiffer

On 05.12.20 08:06, Sven Eckelmann wrote:

Hi,

> Is there some explanation besides an opinion? Some kind goal which you want to 
> achieve with it maybe?

Just a cleanup. I've been under the impression that this version is just
an relic from oot times.

> At least for us it was an easy way to query the release cycle information via 
> batctl. Which made it easier for us to roughly figure out what an reporter/
> inquirer was using - independent of whether he is using the in-kernel version 
> or a backported version.

Is the OOT scenario still valid ?

> Loosing this source of information and breaking parts of batctl and other 
> tools (respondd, ...) is not the end of the world. But I would at least know 
> why this is now necessary.

Okay, if this particular information indeed has a practical value, we
should keep it. Taking it as a NAK.

Perhaps we should add a comment what it's used for and make sure, the
version number is properly maintained.

The problem I see w/ those version fields is that we have lots of
changes in the kernel tree, w/o the version number being increased -
making this information at least doubtful.


--mtx

-- 
---
Hinweis: unverschlüsselte E-Mails können leicht abgehört und manipuliert
werden ! Für eine vertrauliche Kommunikation senden Sie bitte ihren
GPG/PGP-Schlüssel zu.
---
Enrico Weigelt, metux IT consult
Free software and Linux embedded engineering
info@metux.net -- +49-151-27565287

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

* Re: [PATCH 1/7] net: 8021q: remove unneeded MODULE_VERSION() usage
  2020-12-05 15:53   ` Greg KH
@ 2020-12-08  8:51     ` Enrico Weigelt, metux IT consult
  0 siblings, 0 replies; 16+ messages in thread
From: Enrico Weigelt, metux IT consult @ 2020-12-08  8:51 UTC (permalink / raw)
  To: Greg KH, Vladimir Oltean
  Cc: Enrico Weigelt, metux IT consult, linux-kernel, davem, kuba,
	mareklindner, sw, a, sven, marcel, johan.hedberg, roopa, nikolay,
	edumazet, kuznet, yoshfuji, jmaloy, ying.xue, kafai,
	songliubraving, yhs, john.fastabend, kpsingh, netdev,
	linux-bluetooth, tipc-discussion, linux-hyperv, bpf

On 05.12.20 16:53, Greg KH wrote:
>> How do we feel about deleting this not really informative message
>> altogether in a future patch?
> 
> It too should be removed.  If drivers are working properly, they are
> quiet.

Just sent a separate patch for removing this message. I'll rebase my
patch queue when this patch went through.


--mtx

-- 
---
Hinweis: unverschlüsselte E-Mails können leicht abgehört und manipuliert
werden ! Für eine vertrauliche Kommunikation senden Sie bitte ihren
GPG/PGP-Schlüssel zu.
---
Enrico Weigelt, metux IT consult
Free software and Linux embedded engineering
info@metux.net -- +49-151-27565287

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

* Re: [PATCH 2/7] net: batman-adv: remove unneeded MODULE_VERSION() usage
  2020-12-08  7:48     ` Enrico Weigelt, metux IT consult
@ 2020-12-08  9:55       ` Sven Eckelmann
  0 siblings, 0 replies; 16+ messages in thread
From: Sven Eckelmann @ 2020-12-08  9:55 UTC (permalink / raw)
  To: linux-kernel, Enrico Weigelt, metux IT consult, davem
  Cc: kuba, mareklindner, sw, a, marcel, johan.hedberg, roopa, nikolay,
	edumazet, kuznet, yoshfuji, jmaloy, ying.xue, kafai,
	songliubraving, yhs, john.fastabend, kpsingh, netdev,
	linux-bluetooth, tipc-discussion, linux-hyperv, bpf,
	Matthias Schiffer

[-- Attachment #1: Type: text/plain, Size: 893 bytes --]

On Tuesday, 8 December 2020 08:48:56 CET Enrico Weigelt, metux IT consult wrote:
> > Is there some explanation besides an opinion? Some kind goal which you want to 
> > achieve with it maybe?
> 
> Just a cleanup. I've been under the impression that this version is just
> an relic from oot times.

There are various entities which are loving to use the distro kernel and 
replace the batman-adv module with a backport from a newer kernel version. 
Similar to what is done in OpenWrt for the wifi drivers.

> > At least for us it was an easy way to query the release cycle information via 
> > batctl. Which made it easier for us to roughly figure out what an reporter/
> > inquirer was using - independent of whether he is using the in-kernel version 
> > or a backported version.
> 
> Is the OOT scenario still valid ?

Since the backport is OOT - yes, it is still valid.

Kind regards,
	Sven

[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 833 bytes --]

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

end of thread, other threads:[~2020-12-08  9:57 UTC | newest]

Thread overview: 16+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-12-02 12:49 [PATCH 1/7] net: 8021q: remove unneeded MODULE_VERSION() usage Enrico Weigelt, metux IT consult
2020-12-02 12:49 ` [PATCH 2/7] net: batman-adv: " Enrico Weigelt, metux IT consult
2020-12-05  7:06   ` Sven Eckelmann
2020-12-05 16:56     ` Jakub Kicinski
2020-12-08  7:48     ` Enrico Weigelt, metux IT consult
2020-12-08  9:55       ` Sven Eckelmann
2020-12-02 12:49 ` [PATCH 3/7] net: ipv4: " Enrico Weigelt, metux IT consult
2020-12-02 12:49 ` [PATCH 4/7] net: bluetooth: " Enrico Weigelt, metux IT consult
2020-12-02 12:49 ` [PATCH 5/7] net: bridge: " Enrico Weigelt, metux IT consult
2020-12-02 12:49 ` [PATCH 6/7] net: vmw_vsock: " Enrico Weigelt, metux IT consult
2020-12-02 12:49 ` [PATCH 7/7] net: tipc: " Enrico Weigelt, metux IT consult
2020-12-05  0:09 ` [PATCH 1/7] net: 8021q: " Jakub Kicinski
2020-12-05 10:51   ` Joe Perches
2020-12-05 11:20 ` Vladimir Oltean
2020-12-05 15:53   ` Greg KH
2020-12-08  8:51     ` Enrico Weigelt, metux IT consult

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).