On Tuesday, March 14, 2017 9:51:52 PM EDT Arend van Spriel wrote: > To support network namespace the driver must assure all created > network interfaces are in the same namespace as the wiphy instance. > > Reported-by: Mark Asselstine <mark.asselstine@xxxxxxxxxxxxx> > Signed-off-by: Arend van Spriel <arend.vanspriel@xxxxxxxxxxxx> > --- > Hi Mark, > > Please check this patch. I can not say I am an expert when it comes > to using namespaces. So let me know if it works and I can change > Reported-by into Tested-by. Seems to pass the tests I threw at it. Seems happy with namespaces. Mark > > Regards, > Arend > --- > drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c | 3 ++- > drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.c | 5 ++++- > 2 files changed, 6 insertions(+), 2 deletions(-) > > diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c > b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c index > 3856de6..e0d65df 100644 > --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c > +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c > @@ -6450,7 +6450,8 @@ static int brcmf_setup_wiphy(struct wiphy *wiphy, > struct brcmf_if *ifp) BIT(NL80211_BSS_SELECT_ATTR_BAND_PREF) | > BIT(NL80211_BSS_SELECT_ATTR_RSSI_ADJUST); > > - wiphy->flags |= WIPHY_FLAG_PS_ON_BY_DEFAULT | > + wiphy->flags |= WIPHY_FLAG_NETNS_OK | > + WIPHY_FLAG_PS_ON_BY_DEFAULT | > WIPHY_FLAG_OFFCHAN_TX | > WIPHY_FLAG_HAS_REMAIN_ON_CHANNEL; > if (brcmf_feat_is_enabled(ifp, BRCMF_FEAT_TDLS)) > diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.c > b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.c index > 22b4883..74ede27 100644 > --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.c > +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/core.c > @@ -463,6 +463,7 @@ static int brcmf_netdev_open(struct net_device *ndev) > int brcmf_net_attach(struct brcmf_if *ifp, bool rtnl_locked) > { > struct brcmf_pub *drvr = ifp->drvr; > + struct wiphy *wiphy; > struct net_device *ndev; > s32 err; > > @@ -476,8 +477,10 @@ int brcmf_net_attach(struct brcmf_if *ifp, bool > rtnl_locked) ndev->needed_headroom += drvr->hdrlen; > ndev->ethtool_ops = &brcmf_ethtool_ops; > > - /* set the mac address */ > + /* set the mac address & netns */ > memcpy(ndev->dev_addr, ifp->mac_addr, ETH_ALEN); > + wiphy = cfg_to_wiphy(drvr->config); > + dev_net_set(ndev, wiphy_net(wiphy)); > > INIT_WORK(&ifp->multicast_work, _brcmf_set_multicast_list); > INIT_WORK(&ifp->ndoffload_work, _brcmf_update_ndtable);