Received-SPF: pass (google.com: domain of linux-wireless-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
From:   Arend van Spriel <aspriel@gmail.com>
To:     Ryutaroh Matsumoto <ryutaroh@ict.e.titech.ac.jp>,
        <arend.vanspriel@broadcom.com>
CC:     <linux-rpi-kernel@lists.infradead.org>,
        <linux-wireless@vger.kernel.org>,
        <brcm80211-dev-list.pdl@broadcom.com>,
        <SHA-cyfmac-dev-list@infineon.com>, <franky.lin@broadcom.com>,
        <hante.meuleman@broadcom.com>, <chi-hsien.lin@infineon.com>,
        <wright.feng@infineon.com>, <chung-hsien.hsu@infineon.com>
Date:   Tue, 17 Aug 2021 07:42:12 +0200
Message-ID: <17b52a1ab20.279b.9696ff82abe5fb6502268bdc3b0467d4@gmail.com>
In-Reply-To: <20210817.093658.33467107987117119.ryutaroh@ict.e.titech.ac.jp>
References: <20210816.084210.1700916388797835755.ryutaroh@ict.e.titech.ac.jp>
 <85b31c5a-eb4a-48a0-ad94-e46db00af016@broadcom.com>
 <20210817.093658.33467107987117119.ryutaroh@ict.e.titech.ac.jp>
User-Agent: AquaMail/1.30.1-1828 (build: 103001000)
Subject: Re: 5.10.58 UBSAN from brcmf_sdio_dpc+0xa50/0x128c [brcmfmac]
MIME-Version: 1.0
Content-Type: text/plain; format=flowed; charset="us-ascii"
Content-Transfer-Encoding: 8bit
Precedence: bulk

Using different email to avoid disclaimers...


On August 17, 2021 2:39:56 AM Ryutaroh Matsumoto 
<ryutaroh@ict.e.titech.ac.jp> wrote:

> Hi Arend, thank you for paying attention to this.
>
>> Line 2016 in skbuff.h is inline function __skb_queue_before() and as
>> far as I can tell brcmfmac is not using that direct or indirect. Maybe
>> I am reading the line info incorrectly?
>
> I am unsure of it. On the other hand, I have also seen somewhat similar
> UBSAN from a header file "include/net/flow.h" as reported at
> https://lore.kernel.org/netdev/20210813.081908.1574714532738245424.ryutaroh@ict.e.titech.ac.jp/
>
> All UBSANs that I have seen come from *.h compiled with clang...
>
>> Would you be able to provide information as to what line
>> brcmf_sdio_dpc+0xa50 refers to.
>
> I'd like to do, but I do not know how to let kernel UBSAN include a line 
> number,
> though I know it with user-space applications...

If you enable CONFIG_DEBUG_INFO in your kernel .config and recompile 
brcmfmac you can load the module in gdb:

gdb> add-symbol-file brcmfmac.ko [address]
gdb> l *brcmf_sdio_dpc+0xa50

The [address] is not very important so just fill in a nice value. The 'l' 
command should provide the line number.

Regards,
Arend