Re: New brcmfmac errors in 4.12: brcmf_sdio_rxglom: sublen ... not multiple of 8

[Hans de Goede <hdegoede@redhat.com>]

Hi,

On 13-05-17 15:39, Heiner Kallweit wrote:
> Am 13.05.2017 um 14:35 schrieb Hans de Goede:
>> <resend with the author of the commit causing this added>
>>
>> Hi,
>>
>> On 13-05-17 14:19, Hans de Goede wrote:
>>> Hi,
>>>
>>> I've just rebased my personal kernel tree to what will soon be 4.12-rc1
>>> and I'm getting my dmesg log filled with the following errors:
>>>
>>> [   32.528271] brcmfmac: brcmf_sdio_rxglom: sublen 524 not multiple of 8
>>> [   32.528296] brcmfmac: brcmf_sdio_rxglom: sublen 84 not multiple of 8
>>> [   33.063241] brcmfmac: brcmf_sdio_rxglom: sublen 1492 not multiple of 8
>>>
> After a brief look at the code I'm not sure that the check actually checks
> for an error condition. Apart from the error messages:
> Do you face issues with the functionality of the driver?

Yes after a while I get -ETIMEOUT errors for any sdio transfers
to the device. But I'm not sure if this is caused by this commit,
I think I've seen this once with 4.11 too.

I've reverted the commit for now, but I'm fine with instead of
doing the revert dropping the error check if the brcmfmac developers
think that is ok. Currently the ETIMEOUT seems to be gone, so
if dropping the revert causes it to re-appear then we know more.

Regards,

Hans