diff options
author | Arnd Bergmann <arnd@arndb.de> | 2018-04-04 08:12:39 -0400 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2018-04-04 12:53:37 -0400 |
commit | 87248d31d1055b56e01a62d9320b4e118bc84e0e (patch) | |
tree | b7d21f239e3898708e84d24f7deeec639fbbbbeb | |
parent | 458bd99e49742c225f75501591573959c7ef50a2 (diff) |
netdevsim: remove incorrect __net_initdata annotations
The __net_initdata section cannot currently be used for structures that
get cleaned up in an exitcall using unregister_pernet_operations:
WARNING: vmlinux.o(.text+0x868c34): Section mismatch in reference from the function nsim_devlink_exit() to the (unknown reference) .init.data:(unknown)
The function nsim_devlink_exit() references
the (unknown reference) __initdata (unknown).
This is often because nsim_devlink_exit lacks a __initdata
annotation or the annotation of (unknown) is wrong.
WARNING: vmlinux.o(.text+0x868c64): Section mismatch in reference from the function nsim_devlink_init() to the (unknown reference) .init.data:(unknown)
WARNING: vmlinux.o(.text+0x8692bc): Section mismatch in reference from the function nsim_fib_exit() to the (unknown reference) .init.data:(unknown)
WARNING: vmlinux.o(.text+0x869300): Section mismatch in reference from the function nsim_fib_init() to the (unknown reference) .init.data:(unknown)
As that warning tells us, discarding the structure after a module is
loaded would lead to a undefined behavior when that module is removed.
It might be possible to change that annotation so it has no effect for
loadable modules, but I have not figured out exactly how to do that, and
we want this to be fixed in -rc1.
This just removes the annotations, just like we do for all other such
modules.
Fixes: 37923ed6b8ce ("netdevsim: Add simple FIB resource controller via devlink")
Signed-off-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: David S. Miller <davem@davemloft.net>
-rw-r--r-- | drivers/net/netdevsim/devlink.c | 2 | ||||
-rw-r--r-- | drivers/net/netdevsim/fib.c | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/drivers/net/netdevsim/devlink.c b/drivers/net/netdevsim/devlink.c index 27ae05c5fdaf..1dba47936456 100644 --- a/drivers/net/netdevsim/devlink.c +++ b/drivers/net/netdevsim/devlink.c | |||
@@ -267,7 +267,7 @@ static int __net_init nsim_devlink_netns_init(struct net *net) | |||
267 | return 0; | 267 | return 0; |
268 | } | 268 | } |
269 | 269 | ||
270 | static struct pernet_operations nsim_devlink_net_ops __net_initdata = { | 270 | static struct pernet_operations nsim_devlink_net_ops = { |
271 | .init = nsim_devlink_netns_init, | 271 | .init = nsim_devlink_netns_init, |
272 | .id = &nsim_devlink_id, | 272 | .id = &nsim_devlink_id, |
273 | .size = sizeof(bool), | 273 | .size = sizeof(bool), |
diff --git a/drivers/net/netdevsim/fib.c b/drivers/net/netdevsim/fib.c index 0d105bafa261..9bfe9e151e13 100644 --- a/drivers/net/netdevsim/fib.c +++ b/drivers/net/netdevsim/fib.c | |||
@@ -230,7 +230,7 @@ static int __net_init nsim_fib_netns_init(struct net *net) | |||
230 | return 0; | 230 | return 0; |
231 | } | 231 | } |
232 | 232 | ||
233 | static struct pernet_operations nsim_fib_net_ops __net_initdata = { | 233 | static struct pernet_operations nsim_fib_net_ops = { |
234 | .init = nsim_fib_netns_init, | 234 | .init = nsim_fib_netns_init, |
235 | .id = &nsim_fib_net_id, | 235 | .id = &nsim_fib_net_id, |
236 | .size = sizeof(struct nsim_fib_data), | 236 | .size = sizeof(struct nsim_fib_data), |