From patchwork Mon Feb 26 23:04:17 2018 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Arend Van Spriel X-Patchwork-Id: 10243917 X-Patchwork-Delegate: kvalo@adurom.com Return-Path: Received: from mail.wl.linuxfoundation.org (pdx-wl-mail.web.codeaurora.org [172.30.200.125]) by pdx-korg-patchwork.web.codeaurora.org (Postfix) with ESMTP id 3A3EF602A0 for ; Mon, 26 Feb 2018 23:04:25 +0000 (UTC) Received: from mail.wl.linuxfoundation.org (localhost [127.0.0.1]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 281862A471 for ; Mon, 26 Feb 2018 23:04:25 +0000 (UTC) Received: by mail.wl.linuxfoundation.org (Postfix, from userid 486) id 187292A474; Mon, 26 Feb 2018 23:04:25 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.3.1 (2010-03-16) on pdx-wl-mail.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-7.0 required=2.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID, DKIM_VALID_AU, RCVD_IN_DNSWL_HI autolearn=ham version=3.3.1 Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.wl.linuxfoundation.org (Postfix) with ESMTP id 5FB162A471 for ; Mon, 26 Feb 2018 23:04:24 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751552AbeBZXEW (ORCPT ); Mon, 26 Feb 2018 18:04:22 -0500 Received: from mail-qt0-f178.google.com ([209.85.216.178]:36149 "EHLO mail-qt0-f178.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751340AbeBZXEV (ORCPT ); Mon, 26 Feb 2018 18:04:21 -0500 Received: by mail-qt0-f178.google.com with SMTP id c7so20907175qtn.3 for ; Mon, 26 Feb 2018 15:04:21 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=broadcom.com; s=google; h=subject:to:references:cc:from:message-id:date:user-agent :mime-version:in-reply-to:content-transfer-encoding; bh=nOm8m35zLHB+x2S4awLpfjp9WfBLXh0lofbdrbqgQb4=; b=OYsxYg3HUbCIf6rnnajpNt9vCfARQGhGQ6+hJvoBrMK+NB/1mP1HUdKIRIY69GKsvT UiLLIKeH7lPRJGnpXnzxLdcuEQajvlN7DKVho0NGYq1hguK5Ju9+5vzSvL+rvRGqWqxJ bw9uss4/NUZdWURMWrXhVWZq9/khgy8iu0Oog= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:subject:to:references:cc:from:message-id:date :user-agent:mime-version:in-reply-to:content-transfer-encoding; bh=nOm8m35zLHB+x2S4awLpfjp9WfBLXh0lofbdrbqgQb4=; b=qw8b4hjsKjz+YrG8WOnwFc1IlEinZlzHAuiyHuxD3/7N32W6HpEM3RbDETxWB2psI+ gUA6m13kXDdzN/3rzDczExJCgC0f13TYlLr93r/AvXN28ErA0hGQ0oAgJMf2ozkuEIY9 FLqvPbYAaT+JcrFYTmsHsn7OSxgUxkdoBlIKasB466DWFtD0mTI8+SG1tQ4cH3jJn9IZ ysFHdkVIWGu6ClRGLJ4p3il8Cinh3PCPs5LOZeBB7PO1eNsfU/jclaDN2rJFfc45cUIC flrC3TnWiSzuLdzREHfCmPIXTauC5+5bFDkSVkfbZyS9JOJRq93cD3tkmyxUvypxK6YL vRoA== X-Gm-Message-State: APf1xPAGj0BVVLk3kIKGt3sYCVxyQkzzWcjdl3vo7qVW+DroMjGhuQRe R95mg/Ur4B5wXUHN3WeHMm4vqA== X-Google-Smtp-Source: AG47ELuR5sHlxJbVqRPRC67IVSgEuNSC1hRFfsADTFXQIJ/GWTDjfR26VfTUKU8FR9fRtFjPYkKjNA== X-Received: by 10.200.53.137 with SMTP id k9mr20500909qtb.37.1519686260554; Mon, 26 Feb 2018 15:04:20 -0800 (PST) Received: from [10.177.251.142] ([192.19.248.250]) by smtp.gmail.com with ESMTPSA id t68sm4079752qkf.62.2018.02.26.15.04.18 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Mon, 26 Feb 2018 15:04:20 -0800 (PST) Subject: Re: [PATCH resend] brcmfmac: p2p and normal ap access are not always possible at the same time To: Hans de Goede , Franky Lin , Hante Meuleman , Kalle Valo References: <20170526105747.16874-1-hdegoede@redhat.com> <20170526105747.16874-2-hdegoede@redhat.com> <8a61e3ad-3949-68dc-bed9-09f61ab6b647@redhat.com> <5A93DFF1.2030208@broadcom.com> <5A93E8FC.10606@broadcom.com> <5ee4884b-5278-472b-8da1-6bef8b9c61a9@redhat.com> Cc: linux-wireless@vger.kernel.org, brcm80211-dev-list.pdl@broadcom.com From: Arend van Spriel Message-ID: <5A949271.3090406@broadcom.com> Date: Tue, 27 Feb 2018 00:04:17 +0100 User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:38.0) Gecko/20100101 Thunderbird/38.5.0 MIME-Version: 1.0 In-Reply-To: <5ee4884b-5278-472b-8da1-6bef8b9c61a9@redhat.com> Sender: linux-wireless-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-wireless@vger.kernel.org X-Virus-Scanned: ClamAV using ClamSMTP On 2/26/2018 12:22 PM, Hans de Goede wrote: > Hi, > > On 26-02-18 12:01, Arend van Spriel wrote: >> On 2/26/2018 11:29 AM, Hans de Goede wrote: >>> Hi, >>> >>> On 26-02-18 11:22, Arend van Spriel wrote: >>>> On 2/25/2018 3:52 PM, Hans de Goede wrote: >>>>> Hi, >>>>> >>>>> On 26-05-17 12:57, 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. >>>>>> >>>>>> Signed-off-by: Hans de Goede >>>>> >>>>> I'm now seeing this on another device too, but this time the error >>>>> thrown is -EBADE, this seems to be new with recent kernels: >>>> >>>> Yup. Before we were passing firmware errors up to user-space, which >>>> was confusing and potentially be misinterpreted. However, looking at >>>> the output below it would have been good to log the firmware error as >>>> well. And staring at it some more I suddenly realize I broke the >>>> feature detection module with this change. Actually only the GSCAN >>>> feature detection. >>>> >>>>> [root@localhost ~]# dmesg | grep brcmfmac >>>>> [ 34.265950] usbcore: registered new interface driver brcmfmac >>>>> [ 34.266059] brcmfmac 0000:01:00.0: enabling device (0000 -> 0002) >>>>> [ 34.376468] brcmfmac: brcmf_fw_map_chip_to_name: using >>>>> brcm/brcmfmac4356-pcie.bin for chip 0x004356(17238) rev 0x000002 >>>>> [ 34.855143] brcmfmac 0000:01:00.0: Direct firmware load for >>>>> brcm/brcmfmac4356-pcie.clm_blob failed with error -2 >>>>> [ 34.855147] brcmfmac: brcmf_c_process_clm_blob: no clm_blob >>>>> available(err=-2), device may have limited channels available >>>>> [ 34.857029] brcmfmac: brcmf_c_preinit_dcmds: Firmware version = >>>>> wl0: >>>>> Jun 4 2017 16:50:07 version 7.35.101.6 (r702795) FWID 01-5e8eb735 >>>>> [ 34.938854] brcmfmac 0000:01:00.0 wlp1s0: renamed from wlan0 >>>>> [ 37.086420] brcmfmac: brcmf_p2p_create_p2pdev: set p2p_disc error >>>>> [ 37.086431] brcmfmac: brcmf_cfg80211_add_iface: add iface >>>>> p2p-dev-wlp1s0 type 10 failed: err=-52 >>>>> >>>>> [root@localhost ~]# strings /lib/firmware/brcm/brcmfmac4356-pcie.bin | >>>>> tail -n 1 >>>>> 4356a2-roml/pcie-ag-msgbuf-splitrx-p2p-pno-aoe-pktfilter-keepalive-sr-mchan-pktctx-proptxstatus-ampduhostreorder-lpc-pwropt-txbf-wl11u-mfp-tdls-amsdutx-sarctrl-proxd-hs20sta-rcc-wepso-ndoe-linkstat-gscan-hchk-logtrace-roamexp-rmon >>>>> >>>>> >>>>> Version: 7.35.101.6 (r702795) CRC: 4f3f65c5 Date: Sun 2017-06-04 >>>>> 16:51:38 PDT Ucode Ver: 963.316 FWID: 01-5e8eb735 >>>>> >>>>> It would be good if we can silence these errors, or maybe at a >>>>> minimum lower their log-level from error to warning? >>>> >>>> I had a look at it and it seems to be a difference in firmware api >>>> that we need to support in the driver. Need to do a bit more digging, >>>> but it seems an actual issue. You could silence it for now, but I >>>> prefer to wait for the fix. >>> >>> Ok, what is the ETA of a fix for this? >> >> Actually went back to an old log you sent and noticed: >> >> [ 15.714569] brcmfmac: brcmf_attach Enter >> [ 15.714756] brcmfmac: brcmf_fweh_register event handler registered >> for PSM_WATCHDOG >> [ 15.714757] brcmfmac: brcmf_proto_attach Enter >> [ 15.716598] brcmfmac: brcmf_bus_started >> [ 15.716603] brcmfmac: brcmf_add_if Enter, bsscfgidx=0, ifidx=0 >> [ 15.716604] brcmfmac: brcmf_add_if allocate netdev interface >> [ 15.716622] brcmfmac: brcmf_add_if ==== pid:2a, if:wlan%d >> (00:00:00:00:00:00) created === >> [ 15.716624] brcmfmac: brcmf_bus_change_state 0 -> 1 >> [ 15.717841] brcmfmac: brcmf_fil_iovar_data_get ifidx=0, >> name=cur_etheraddr, len=6 >> [ 15.717843] brcmutil: data >> [ 15.717847] 00000000: 44 2c 05 9e c9 02 D,.... >> >> So mac address of the device is 44:2c:05:9e:c9. However, further down >> I see: >> >> [ 17.819113] brcmfmac: brcmf_netdev_set_mac_address Enter, bsscfgidx=0 >> [ 17.819122] brcmfmac: brcmf_fil_iovar_data_set ifidx=0, >> name=cur_etheraddr, len=6 >> [ 17.819127] brcmutil: data >> [ 17.819135] 00000000: aa 3e 81 77 bc 40 .>.w.@ >> [ 17.819864] brcmfmac: brcmf_netdev_set_mac_address updated to >> aa:3e:81:77:bc:40 >> >> So the mac address in a local admin variant. > > Right, this is likely NetworkManager randomizing the mac for privacy > reasons. > >> Now our firmware has a requirement for the p2p-dev interface that it >> should be different from the mac address of the primary interface, ie. >> wlp1s0 in this log. In brcmfmac we try to do that by setting the local >> admin bit, but... as it is already set we end up using the same mac >> address hence the -EBUSY. > > Ah, that is good to know, so how can we fix this? Can userspace specify a > different mac-address when it asks for the p2p-dev intf to be created? Or > should we do something about this in the kernel? So this is the patch I tested. Maybe you can verify it works for you as well. Regards, Arend --- *dev_addr) { @@ -471,12 +471,12 @@ static void brcmf_p2p_generate_bss_mac(struct brcmf_p2p_info *p2p, u8 *dev_addr) bool local_admin = false; if (!dev_addr || is_zero_ether_addr(dev_addr)) { - dev_addr = pri_ifp->mac_addr; + dev_addr = pri_ifp->drvr->mac; local_admin = true; } /* Generate the P2P Device Address. This consists of the device's - * primary MAC address with the locally administered bit set. + * permanent ethernet address with the locally administered bit set. */ memcpy(p2p->dev_addr, dev_addr, ETH_ALEN); if (local_admin) @@ -486,7 +486,7 @@ static void brcmf_p2p_generate_bss_mac(struct brcmf_p2p_info *p2p, u8 *dev_addr) * BSSCFGs need to simultaneously co-exist, then this address must be * different from the P2P Device Address, but also locally administered. */ - memcpy(p2p->int_addr, p2p->dev_addr, ETH_ALEN); + memcpy(p2p->int_addr, dev_addr, ETH_ALEN); p2p->int_addr[0] |= 0x02; p2p->int_addr[4] ^= 0x80; } diff --git a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c index 2ee5413..ddbb386 100644 --- a/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c +++ b/drivers/net/wireless/broadcom/brcm80211/brcmfmac/p2p.c @@ -462,8 +462,8 @@ static int brcmf_p2p_set_firmware(struct brcmf_if *ifp, u8 *p2p_mac) * @dev_addr: optional device address. * * P2P needs mac addresses for P2P device and interface. If no device - * address it specified, these are derived from the primary net device, ie. - * the permanent ethernet address of the device. + * address it specified, these are derived from the permanent ethernet + * address of the device. */ static void brcmf_p2p_generate_bss_mac(struct brcmf_p2p_info *p2p, u8