Re: [PATCH v2 2/4] brcmfmac: p2p and normal ap access are not always possible at the same time
On 10-3-2017 9:08, Hans de Goede wrote: > Hi, > > On 08-03-17 12:16, Hans de Goede wrote: >> Hi, >> >> On 08-03-17 11:15, Arend Van Spriel wrote: >>> On 8-3-2017 10:26, Arend Van Spriel wrote: On 8-3-2017 9:24, Hans de Goede wrote: > Hi, > > On 07-03-17 11:03, Arend Van Spriel wrote: >> >> >> On 27-2-2017 22:45, Hans de Goede wrote: >>> The firmware responding with -EBUSY when trying to add an extra >>> virtual-if >>> is a normal thing, do not print an error for this. >> >> This may be something we need to look into. It seems to me the >> interface >> combinations needs to be fixed so we do not try to provision >> firmware. >> Can you explain the scenario here? > > I'm not doing anything special just connecting to my isp provided > accesspoint > using NetworkManager. Ok. So it is probably the P2P_DEVICE interface as that is the only interface being created in that scenario. Can you provide a log with brcmfmac driver debug=0x1416. >>> >>> I guess you are seeing this with 43362. Is that correct? > > Attached is a dmesg log from the brcmfmac driver build with debugging > enabled and brcmfmac.debug set to 0x1416. I suspect the P2P_DEVICE interface is assigned the same address as wlp1s0. Can you apply the change below and create the log again. Regards, Arend diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c b/drivers/ne index de19c7c..43fa9f45 100644 --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c @@ -2158,7 +2158,8 @@ struct wireless_dev *brcmf_p2p_add_vif(struct wiphy *wiphy if (brcmf_cfg80211_vif_event_armed(cfg)) return ERR_PTR(-EBUSY); - brcmf_dbg(INFO, "adding vif \"%s\" (type=%d)\n", name, type); + brcmf_dbg(INFO, "adding vif \"%s\" (type=%d, addr=%pM)\n", name, type, + params->macaddr); switch (type) { case NL80211_IFTYPE_P2P_CLIENT:
Re: [PATCH v2 2/4] brcmfmac: p2p and normal ap access are not always possible at the same time
Hi, On 08-03-17 11:15, Arend Van Spriel wrote: On 8-3-2017 10:26, Arend Van Spriel wrote: On 8-3-2017 9:24, Hans de Goede wrote: Hi, On 07-03-17 11:03, Arend Van Spriel wrote: On 27-2-2017 22:45, Hans de Goede wrote: The firmware responding with -EBUSY when trying to add an extra virtual-if is a normal thing, do not print an error for this. This may be something we need to look into. It seems to me the interface combinations needs to be fixed so we do not try to provision firmware. Can you explain the scenario here? I'm not doing anything special just connecting to my isp provided accesspoint using NetworkManager. Ok. So it is probably the P2P_DEVICE interface as that is the only interface being created in that scenario. Can you provide a log with brcmfmac driver debug=0x1416. I guess you are seeing this with 43362. Is that correct? Maybe, it is with the same card as this bug: https://bugzilla.kernel.org/show_bug.cgi?id=185661 Regards, Hans
Re: [PATCH v2 2/4] brcmfmac: p2p and normal ap access are not always possible at the same time
On 8-3-2017 9:24, Hans de Goede wrote: > Hi, > > On 07-03-17 11:03, Arend Van Spriel wrote: >> >> >> On 27-2-2017 22:45, Hans de Goede wrote: >>> The firmware responding with -EBUSY when trying to add an extra >>> virtual-if >>> is a normal thing, do not print an error for this. >> >> This may be something we need to look into. It seems to me the interface >> combinations needs to be fixed so we do not try to provision firmware. >> Can you explain the scenario here? > > I'm not doing anything special just connecting to my isp provided > accesspoint > using NetworkManager. Ok. So it is probably the P2P_DEVICE interface as that is the only interface being created in that scenario. Can you provide a log with brcmfmac driver debug=0x1416. Regards, Arend > Regards, > > Hans > > >> >> Regards, >> Arend >> >>> Signed-off-by: Hans de Goede >>> --- >>> .../net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c| 14 >>> ++ >>> drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c | 5 - >>> 2 files changed, 14 insertions(+), 5 deletions(-) >>> >>> diff --git >>> a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c >>> b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c >>> index 7ffc4ab..c54e8b4 100644 >>> --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c >>> +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c >>> @@ -688,11 +688,17 @@ static struct wireless_dev >>> *brcmf_cfg80211_add_iface(struct wiphy *wiphy, >>> return ERR_PTR(-EINVAL); >>> } >>> >>> -if (IS_ERR(wdev)) >>> -brcmf_err("add iface %s type %d failed: err=%d\n", >>> - name, type, (int)PTR_ERR(wdev)); >>> -else >>> +if (IS_ERR(wdev)) { >>> +err = PTR_ERR(wdev); >>> +if (err != -EBUSY) >>> +brcmf_err("add iface %s type %d failed: err=%d\n", >>> + name, type, err); >>> +else >>> +brcmf_dbg(INFO, "add iface %s type %d failed: err=%d\n", >>> + name, type, err); >>> +} else { >>> brcmf_cfg80211_update_proto_addr_mode(wdev); >>> +} >>> >>> return wdev; >>> } >>> diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c >>> b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c >>> index de19c7c..b5df0a0 100644 >>> --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c >>> +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c >>> @@ -2090,7 +2090,10 @@ static struct wireless_dev >>> *brcmf_p2p_create_p2pdev(struct brcmf_p2p_info *p2p, >>> /* Initialize P2P Discovery in the firmware */ >>> err = brcmf_fil_iovar_int_set(pri_ifp, "p2p_disc", 1); >>> if (err < 0) { >>> -brcmf_err("set p2p_disc error\n"); >>> +if (err != -EBUSY) >>> +brcmf_err("set p2p_disc error\n"); >>> +else >>> +brcmf_dbg(INFO, "set p2p_disc error\n"); >>> brcmf_fweh_p2pdev_setup(pri_ifp, false); >>> brcmf_cfg80211_arm_vif_event(p2p->cfg, NULL); >>> goto fail; >>>
Re: [PATCH v2 2/4] brcmfmac: p2p and normal ap access are not always possible at the same time
On 8-3-2017 10:26, Arend Van Spriel wrote: > On 8-3-2017 9:24, Hans de Goede wrote: >> Hi, >> >> On 07-03-17 11:03, Arend Van Spriel wrote: >>> >>> >>> On 27-2-2017 22:45, Hans de Goede wrote: The firmware responding with -EBUSY when trying to add an extra virtual-if is a normal thing, do not print an error for this. >>> >>> This may be something we need to look into. It seems to me the interface >>> combinations needs to be fixed so we do not try to provision firmware. >>> Can you explain the scenario here? >> >> I'm not doing anything special just connecting to my isp provided >> accesspoint >> using NetworkManager. > > Ok. So it is probably the P2P_DEVICE interface as that is the only > interface being created in that scenario. Can you provide a log with > brcmfmac driver debug=0x1416. I guess you are seeing this with 43362. Is that correct? Regards, Arend
Re: [PATCH v2 2/4] brcmfmac: p2p and normal ap access are not always possible at the same time
Hi, On 07-03-17 11:03, Arend Van Spriel wrote: On 27-2-2017 22:45, Hans de Goede wrote: The firmware responding with -EBUSY when trying to add an extra virtual-if is a normal thing, do not print an error for this. This may be something we need to look into. It seems to me the interface combinations needs to be fixed so we do not try to provision firmware. Can you explain the scenario here? I'm not doing anything special just connecting to my isp provided accesspoint using NetworkManager. Regards, Hans Regards, Arend Signed-off-by: Hans de Goede --- .../net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c| 14 ++ drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c | 5 - 2 files changed, 14 insertions(+), 5 deletions(-) diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c index 7ffc4ab..c54e8b4 100644 --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c @@ -688,11 +688,17 @@ static struct wireless_dev *brcmf_cfg80211_add_iface(struct wiphy *wiphy, return ERR_PTR(-EINVAL); } - if (IS_ERR(wdev)) - brcmf_err("add iface %s type %d failed: err=%d\n", - name, type, (int)PTR_ERR(wdev)); - else + if (IS_ERR(wdev)) { + err = PTR_ERR(wdev); + if (err != -EBUSY) + brcmf_err("add iface %s type %d failed: err=%d\n", + name, type, err); + else + brcmf_dbg(INFO, "add iface %s type %d failed: err=%d\n", + name, type, err); + } else { brcmf_cfg80211_update_proto_addr_mode(wdev); + } return wdev; } diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c index de19c7c..b5df0a0 100644 --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c @@ -2090,7 +2090,10 @@ static struct wireless_dev *brcmf_p2p_create_p2pdev(struct brcmf_p2p_info *p2p, /* Initialize P2P Discovery in the firmware */ err = brcmf_fil_iovar_int_set(pri_ifp, "p2p_disc", 1); if (err < 0) { - brcmf_err("set p2p_disc error\n"); + if (err != -EBUSY) + brcmf_err("set p2p_disc error\n"); + else + brcmf_dbg(INFO, "set p2p_disc error\n"); brcmf_fweh_p2pdev_setup(pri_ifp, false); brcmf_cfg80211_arm_vif_event(p2p->cfg, NULL); goto fail;
Re: [PATCH v2 2/4] brcmfmac: p2p and normal ap access are not always possible at the same time
On 27-2-2017 22:45, Hans de Goede wrote: > The firmware responding with -EBUSY when trying to add an extra virtual-if > is a normal thing, do not print an error for this. This may be something we need to look into. It seems to me the interface combinations needs to be fixed so we do not try to provision firmware. Can you explain the scenario here? Regards, Arend > Signed-off-by: Hans de Goede > --- > .../net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c| 14 > ++ > drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c | 5 - > 2 files changed, 14 insertions(+), 5 deletions(-) > > diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c > b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c > index 7ffc4ab..c54e8b4 100644 > --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c > +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c > @@ -688,11 +688,17 @@ static struct wireless_dev > *brcmf_cfg80211_add_iface(struct wiphy *wiphy, > return ERR_PTR(-EINVAL); > } > > - if (IS_ERR(wdev)) > - brcmf_err("add iface %s type %d failed: err=%d\n", > - name, type, (int)PTR_ERR(wdev)); > - else > + if (IS_ERR(wdev)) { > + err = PTR_ERR(wdev); > + if (err != -EBUSY) > + brcmf_err("add iface %s type %d failed: err=%d\n", > + name, type, err); > + else > + brcmf_dbg(INFO, "add iface %s type %d failed: err=%d\n", > + name, type, err); > + } else { > brcmf_cfg80211_update_proto_addr_mode(wdev); > + } > > return wdev; > } > diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c > b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c > index de19c7c..b5df0a0 100644 > --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c > +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c > @@ -2090,7 +2090,10 @@ static struct wireless_dev > *brcmf_p2p_create_p2pdev(struct brcmf_p2p_info *p2p, > /* Initialize P2P Discovery in the firmware */ > err = brcmf_fil_iovar_int_set(pri_ifp, "p2p_disc", 1); > if (err < 0) { > - brcmf_err("set p2p_disc error\n"); > + if (err != -EBUSY) > + brcmf_err("set p2p_disc error\n"); > + else > + brcmf_dbg(INFO, "set p2p_disc error\n"); > brcmf_fweh_p2pdev_setup(pri_ifp, false); > brcmf_cfg80211_arm_vif_event(p2p->cfg, NULL); > goto fail; >
[PATCH v2 2/4] brcmfmac: p2p and normal ap access are not always possible at the same time
The firmware responding with -EBUSY when trying to add an extra virtual-if is a normal thing, do not print an error for this. Signed-off-by: Hans de Goede --- .../net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c| 14 ++ drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c | 5 - 2 files changed, 14 insertions(+), 5 deletions(-) diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c index 7ffc4ab..c54e8b4 100644 --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/cfg80211.c @@ -688,11 +688,17 @@ static struct wireless_dev *brcmf_cfg80211_add_iface(struct wiphy *wiphy, return ERR_PTR(-EINVAL); } - if (IS_ERR(wdev)) - brcmf_err("add iface %s type %d failed: err=%d\n", - name, type, (int)PTR_ERR(wdev)); - else + if (IS_ERR(wdev)) { + err = PTR_ERR(wdev); + if (err != -EBUSY) + brcmf_err("add iface %s type %d failed: err=%d\n", + name, type, err); + else + brcmf_dbg(INFO, "add iface %s type %d failed: err=%d\n", + name, type, err); + } else { brcmf_cfg80211_update_proto_addr_mode(wdev); + } return wdev; } diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c index de19c7c..b5df0a0 100644 --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c @@ -2090,7 +2090,10 @@ static struct wireless_dev *brcmf_p2p_create_p2pdev(struct brcmf_p2p_info *p2p, /* Initialize P2P Discovery in the firmware */ err = brcmf_fil_iovar_int_set(pri_ifp, "p2p_disc", 1); if (err < 0) { - brcmf_err("set p2p_disc error\n"); + if (err != -EBUSY) + brcmf_err("set p2p_disc error\n"); + else + brcmf_dbg(INFO, "set p2p_disc error\n"); brcmf_fweh_p2pdev_setup(pri_ifp, false); brcmf_cfg80211_arm_vif_event(p2p->cfg, NULL); goto fail; -- 2.9.3