ath10k.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
* Re: ath10k_pci logs errors about missing pre-cal and cal firmware on a laptop
       [not found] <7158e5e9-8bdc-4660-ac5d-5e3f81cf6158@molgen.mpg.de>
@ 2023-11-14 17:32 ` Jeff Johnson
  2023-11-14 18:19   ` Jeff Johnson
  0 siblings, 1 reply; 4+ messages in thread
From: Jeff Johnson @ 2023-11-14 17:32 UTC (permalink / raw)
  To: Paul Menzel, Kalle Valo; +Cc: linux-wireless, LKML, ath10k

On 11/12/2023 4:13 AM, Paul Menzel wrote:
> Dear Linux folks,
> 
> 
> Onn the Dell XPS 13 9360 with Debian sid/unstable, Linux 6.5.10 logs the 
> error below:
> 
>      $ sudo dmesg | grep -e "DMI:" -e "Linux version" -e microcode
>      [    0.000000] microcode: updated early: 0xf0 -> 0xf4, date = 
> 2023-02-22
>      [    0.000000] Linux version 6.5.0-4-amd64 
> (debian-kernel@lists.debian.org) (gcc-13 (Debian 13.2.0-6) 13.2.0, GNU 
> ld (GNU Binutils for Debian) 2.41) #1 SMP PREEMPT_DYNAMIC Debian 
> 6.5.10-1 (2023-11-03)
>      [    0.000000] DMI: Dell Inc. XPS 13 9360/0596KF, BIOS 2.21.0 
> 06/02/2022
>      [    0.580689] microcode: Microcode Update Driver: v2.2.
> 
>      $ sudo dmesg --level alert,crit,err
>      [    0.053566] x86/cpu: SGX disabled by BIOS.
>      [    1.942183] psmouse serio1: synaptics: Unable to query device: -5
>      [   14.401112] ath10k_pci 0000:3a:00.0: firmware: failed to load 
> ath10k/pre-cal-pci-0000:3a:00.0.bin (-2)
>      [   14.401143] firmware_class: See https://wiki.debian.org/Firmware 
> for information about missing firmware
>      [   14.401185] ath10k_pci 0000:3a:00.0: firmware: failed to load 
> ath10k/pre-cal-pci-0000:3a:00.0.bin (-2)
>      [   14.401233] ath10k_pci 0000:3a:00.0: firmware: failed to load 
> ath10k/cal-pci-0000:3a:00.0.bin (-2)
>      [   14.401273] ath10k_pci 0000:3a:00.0: firmware: failed to load 
> ath10k/cal-pci-0000:3a:00.0.bin (-2)
> 
> As it’s logged with error level, I’d like to address the ath10k_pci 
> errors, but I am unable to find the firmware in the Linux firmware 
> archive [1].
> 
> What can I do about this?
> 
> 
> Kind regards,
> 
> Paul
> 
> 
> [1]: 
> https://git.kernel.org/pub/scm/linux/kernel/git/firmware/linux-firmware.git/

Adding the ath10k list so this isn't lost in the noise

_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: ath10k_pci logs errors about missing pre-cal and cal firmware on a laptop
  2023-11-14 17:32 ` ath10k_pci logs errors about missing pre-cal and cal firmware on a laptop Jeff Johnson
@ 2023-11-14 18:19   ` Jeff Johnson
  2023-11-14 19:58     ` Paul Menzel
  0 siblings, 1 reply; 4+ messages in thread
From: Jeff Johnson @ 2023-11-14 18:19 UTC (permalink / raw)
  To: Paul Menzel, Kalle Valo; +Cc: linux-wireless, LKML, ath10k

On 11/14/2023 9:32 AM, Jeff Johnson wrote:
> On 11/12/2023 4:13 AM, Paul Menzel wrote:
>> Dear Linux folks,
>>
>>
>> Onn the Dell XPS 13 9360 with Debian sid/unstable, Linux 6.5.10 logs the 
>> error below:
>>
>>      $ sudo dmesg | grep -e "DMI:" -e "Linux version" -e microcode
>>      [    0.000000] microcode: updated early: 0xf0 -> 0xf4, date = 
>> 2023-02-22
>>      [    0.000000] Linux version 6.5.0-4-amd64 
>> (debian-kernel@lists.debian.org) (gcc-13 (Debian 13.2.0-6) 13.2.0, GNU 
>> ld (GNU Binutils for Debian) 2.41) #1 SMP PREEMPT_DYNAMIC Debian 
>> 6.5.10-1 (2023-11-03)
>>      [    0.000000] DMI: Dell Inc. XPS 13 9360/0596KF, BIOS 2.21.0 
>> 06/02/2022
>>      [    0.580689] microcode: Microcode Update Driver: v2.2.
>>
>>      $ sudo dmesg --level alert,crit,err
>>      [    0.053566] x86/cpu: SGX disabled by BIOS.
>>      [    1.942183] psmouse serio1: synaptics: Unable to query device: -5
>>      [   14.401112] ath10k_pci 0000:3a:00.0: firmware: failed to load 
>> ath10k/pre-cal-pci-0000:3a:00.0.bin (-2)
>>      [   14.401143] firmware_class: See https://wiki.debian.org/Firmware 
>> for information about missing firmware
>>      [   14.401185] ath10k_pci 0000:3a:00.0: firmware: failed to load 
>> ath10k/pre-cal-pci-0000:3a:00.0.bin (-2)
>>      [   14.401233] ath10k_pci 0000:3a:00.0: firmware: failed to load 
>> ath10k/cal-pci-0000:3a:00.0.bin (-2)
>>      [   14.401273] ath10k_pci 0000:3a:00.0: firmware: failed to load 
>> ath10k/cal-pci-0000:3a:00.0.bin (-2)
>>
>> As it’s logged with error level, I’d like to address the ath10k_pci 
>> errors, but I am unable to find the firmware in the Linux firmware 
>> archive [1].
>>
>> What can I do about this?
>>
>>
>> Kind regards,
>>
>> Paul
>>
>>
>> [1]: 
>> https://git.kernel.org/pub/scm/linux/kernel/git/firmware/linux-firmware.git/
> 
> Adding the ath10k list so this isn't lost in the noise

The calibration files are optional and you should not see any warnings
if they are not present.

Note ath10k explicitly calls firmware_request_nowarn() when it loads the
files since it doesn't want any warnings from the firmware loader if the
files aren't present.
<https://elixir.bootlin.com/linux/v6.5.10/source/drivers/net/wireless/ath/ath10k/core.c#L929>

And ath10k itself explicitly doesn't warn:
<https://elixir.bootlin.com/linux/v6.5.10/source/drivers/net/wireless/ath/ath10k/core.c#L1259>

So I'm confused where these warnings originate.

Based upon the message:
[   14.401143] firmware_class: See https://wiki.debian.org/Firmware
for information about missing firmware

it seems you are not running a stock kernel. So perhaps Debian has
modified the firmware loading such that it ignores the FW_OPT_NO_WARN
flag and warns even when told not to do so? This does not appear to be
an upstream kernel issue.

/jeff

_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: ath10k_pci logs errors about missing pre-cal and cal firmware on a laptop
  2023-11-14 18:19   ` Jeff Johnson
@ 2023-11-14 19:58     ` Paul Menzel
  2023-11-14 21:49       ` Diederik de Haas
  0 siblings, 1 reply; 4+ messages in thread
From: Paul Menzel @ 2023-11-14 19:58 UTC (permalink / raw)
  To: Jeff Johnson, debian-kernel
  Cc: Kalle Valo, linux-wireless, LKML, ath10k, Ben Hutchings

[To: +debian-kernel@lists.debian.org]

Dear Jeff, dear Debian Linux kernel team,


Am 14.11.23 um 19:19 schrieb Jeff Johnson:
> On 11/14/2023 9:32 AM, Jeff Johnson wrote:
>> On 11/12/2023 4:13 AM, Paul Menzel wrote:

>>> On the Dell XPS 13 9360 with Debian sid/unstable, Linux 6.5.10 logs the
>>> error below:
>>>
>>>       $ sudo dmesg | grep -e "DMI:" -e "Linux version" -e microcode
>>>       [    0.000000] microcode: updated early: 0xf0 -> 0xf4, date = 2023-02-22
>>>       [    0.000000] Linux version 6.5.0-4-amd64 (debian-kernel@lists.debian.org) (gcc-13 (Debian 13.2.0-6) 13.2.0, GNU ld (GNU Binutils for Debian) 2.41) #1 SMP PREEMPT_DYNAMIC Debian 6.5.10-1 (2023-11-03)
>>>       [    0.000000] DMI: Dell Inc. XPS 13 9360/0596KF, BIOS 2.21.0 06/02/2022
>>>       [    0.580689] microcode: Microcode Update Driver: v2.2.
>>>
>>>       $ sudo dmesg --level alert,crit,err
>>>       [    0.053566] x86/cpu: SGX disabled by BIOS.
>>>       [    1.942183] psmouse serio1: synaptics: Unable to query device: -5
>>>       [   14.401112] ath10k_pci 0000:3a:00.0: firmware: failed to load ath10k/pre-cal-pci-0000:3a:00.0.bin (-2)
>>>       [   14.401143] firmware_class: See https://wiki.debian.org/Firmware for information about missing firmware
>>>       [   14.401185] ath10k_pci 0000:3a:00.0: firmware: failed to load ath10k/pre-cal-pci-0000:3a:00.0.bin (-2)
>>>       [   14.401233] ath10k_pci 0000:3a:00.0: firmware: failed to load ath10k/cal-pci-0000:3a:00.0.bin (-2)
>>>       [   14.401273] ath10k_pci 0000:3a:00.0: firmware: failed to load ath10k/cal-pci-0000:3a:00.0.bin (-2)
>>>
>>> As it’s logged with error level, I’d like to address the ath10k_pci
>>> errors, but I am unable to find the firmware in the Linux firmware
>>> archive [1].
>>>
>>> What can I do about this?

>>> [1]: https://git.kernel.org/pub/scm/linux/kernel/git/firmware/linux-firmware.git/
>>
>> Adding the ath10k list so this isn't lost in the noise
> 
> The calibration files are optional and you should not see any warnings
> if they are not present.
> 
> Note ath10k explicitly calls firmware_request_nowarn() when it loads the
> files since it doesn't want any warnings from the firmware loader if the
> files aren't present.
> <https://elixir.bootlin.com/linux/v6.5.10/source/drivers/net/wireless/ath/ath10k/core.c#L929>
> 
> And ath10k itself explicitly doesn't warn:
> <https://elixir.bootlin.com/linux/v6.5.10/source/drivers/net/wireless/ath/ath10k/core.c#L1259>
> 
> So I'm confused where these warnings originate.
> 
> Based upon the message:
> [   14.401143] firmware_class: See https://wiki.debian.org/Firmware
> for information about missing firmware
> 
> it seems you are not running a stock kernel. So perhaps Debian has
> modified the firmware loading such that it ignores the FW_OPT_NO_WARN
> flag and warns even when told not to do so? This does not appear to be
> an upstream kernel issue.

Thank you very much for the analysis. It seems to be indeed a Debian 
specific patch [1].

Dear Debian Linux kernel team, is my observation about the error log a 
result of the patch and intended?


Kind regards,

Paul


[1]: 
debian/patches/debian/firmware_class-refer-to-debian-wiki-firmware-page.patch

_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: ath10k_pci logs errors about missing pre-cal and cal firmware on a laptop
  2023-11-14 19:58     ` Paul Menzel
@ 2023-11-14 21:49       ` Diederik de Haas
  0 siblings, 0 replies; 4+ messages in thread
From: Diederik de Haas @ 2023-11-14 21:49 UTC (permalink / raw)
  To: Jeff Johnson, debian-kernel, Paul Menzel
  Cc: Kalle Valo, linux-wireless, LKML, ath10k, Ben Hutchings


[-- Attachment #1.1: Type: text/plain, Size: 1474 bytes --]

FTR: I do NOT speak on behalf of the Debian kernel team.

On Tuesday, 14 November 2023 20:58:58 CET Paul Menzel wrote:
> > Based upon the message:
> > [   14.401143] firmware_class: See https://wiki.debian.org/Firmware
> > for information about missing firmware
> > 
> > it seems you are not running a stock kernel. So perhaps Debian has
> > modified the firmware loading such that it ignores the FW_OPT_NO_WARN
> > flag and warns even when told not to do so? This does not appear to be
> > an upstream kernel issue.
> 
> Thank you very much for the analysis. It seems to be indeed a Debian
> specific patch [1].
> 
> Dear Debian Linux kernel team, is my observation about the error log a
> result of the patch and intended?
> 
> [1]:
> debian/patches/debian/firmware_class-refer-to-debian-wiki-firmware-page.patch

There are a number of Debian patches wrt firmware and *I* think they should get 
a (serious) review. In https://bugs.debian.org/1040738 I requested a review of 
the firmware related patches and described one of the issues I encountered 
myself. I've also been involved in triaging Debian kernel bug issues and there 
I've encountered several more such cases.

AFAIK quite a bit of work has happened upstream to make the firmware messages 
more appropriate and I think the Debian patches haven't been (properly) 
adjusted for those changes.

So they are (very) likely caused by the Debian patches and thus expected, but 
I'm hesitant to call them intended ;-)

[-- Attachment #1.2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 228 bytes --]

[-- Attachment #2: Type: text/plain, Size: 146 bytes --]

_______________________________________________
ath10k mailing list
ath10k@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/ath10k

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2023-11-14 21:50 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
     [not found] <7158e5e9-8bdc-4660-ac5d-5e3f81cf6158@molgen.mpg.de>
2023-11-14 17:32 ` ath10k_pci logs errors about missing pre-cal and cal firmware on a laptop Jeff Johnson
2023-11-14 18:19   ` Jeff Johnson
2023-11-14 19:58     ` Paul Menzel
2023-11-14 21:49       ` Diederik de Haas

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).