On 5/31/2024 11:42 AM, Baochen Qiang wrote: >> as noted above does it make more sense to get the netdev associated with the >> arvif and then use in6_dev_get(net_device) to get the inet6_dev rather than >> caching the pointer from the ipv6_addr_changed() callback? > Ah.. I didn't note that we can get inet6_dev in such a way, just thought the only way is to cache it in ipv6_changed() callback. > > will get it using the following in next version: > struct ieee80211_vif *vif = container_of(arvif) sorry, should be: struct ieee80211_vif *vif = arvif->vif > struct ieee80211_sub_if_data *sub_if_data = container_of(vif) > struct net_dev *ndev = sub_if_data->dev > struct inet6_dev *idev = in6_dev_get(ndev)