niansa-misc/linux
1
0
Fork 0
mirror of synced 2025-03-06 20:59:54 +01:00
Code Issues Projects Releases Packages Wiki Activity

dpll: fix build failure due to rcu_dereference_check() on unknown type

Browse source 
Tasmiya reports that their compiler complains that we deref
a pointer to unknown type with rcu_dereference_rtnl():

include/linux/rcupdate.h:439:9: error: dereferencing pointer to incomplete type ‘struct dpll_pin’

Unclear what compiler it is, at the moment, and we can't report
but since DPLL can't be a module - move the code from the header
into the source file.

Fixes: 0d60d8df6f ("dpll: rely on rcu for netdev_dpll_pin()")
Reported-by: Tasmiya Nalatwad <tasmiya@linux.vnet.ibm.com>
Link: https://lore.kernel.org/all/3fcf3a2c-1c1b-42c1-bacb-78fdcd700389@linux.vnet.ibm.com/
Signed-off-by: Eric Dumazet <edumazet@google.com>
Link: https://lore.kernel.org/r/20240229190515.2740221-1-kuba@kernel.org
Signed-off-by: Jakub Kicinski <kuba@kernel.org>
This commit is contained in:
Eric Dumazet 2024-02-29 11:05:15 -08:00 • committed by Jakub Kicinski
parent 3d6423ef8d
commit 640f41ed33
2 changed files with 9 additions and 4 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
drivers/dpll/dpll_core.c
View file

@ -42,6 +42,11 @@ struct dpll_pin_registration {
void *priv; void *priv;
}; };
struct dpll_pin *netdev_dpll_pin(const struct net_device *dev)
{
return rcu_dereference_rtnl(dev->dpll_pin);
}
struct dpll_device *dpll_device_get_by_id(int id) struct dpll_device *dpll_device_get_by_id(int id)
{ {
if (xa_get_mark(&dpll_device_xa, id, DPLL_REGISTERED)) if (xa_get_mark(&dpll_device_xa, id, DPLL_REGISTERED))

8
include/linux/dpll.h
View file

@ -169,13 +169,13 @@ int dpll_device_change_ntf(struct dpll_device *dpll);
int dpll_pin_change_ntf(struct dpll_pin *pin); int dpll_pin_change_ntf(struct dpll_pin *pin);
#if !IS_ENABLED(CONFIG_DPLL)
static inline struct dpll_pin *netdev_dpll_pin(const struct net_device *dev) static inline struct dpll_pin *netdev_dpll_pin(const struct net_device *dev)
{ {
#if IS_ENABLED(CONFIG_DPLL)
return rcu_dereference_rtnl(dev->dpll_pin);
#else
return NULL; return NULL;
#endif
} }
#else
struct dpll_pin *netdev_dpll_pin(const struct net_device *dev);
#endif
#endif #endif