732852 – sys-kernel/gentoo-sources: always print loaded firmware name patch

Bug 732852 - sys-kernel/gentoo-sources: always print loaded firmware name patch

Summary: sys-kernel/gentoo-sources: always print loaded firmware name patch

Status:	RESOLVED FIXED

Alias:	None

Product:	Gentoo Linux
Classification:	Unclassified
Component:	Current packages (show other bugs)
Hardware:	All Linux

Importance:	Normal normal (vote)
Assignee:	Gentoo Kernel Bug Wranglers and Kernel Maintainers

URL:
Whiteboard:	5.13.13, 5.10.61
Keywords:	InVCS, PATCH

Depends on:
Blocks:

Reported:	2020-07-16 07:20 UTC by Georgy Yakovlev
Modified:	2023-08-08 18:27 UTC (History)
CC List:	3 users (show)

See Also:	https://bugzilla.kernel.org/show_bug.cgi?id=217734
Package list:
Runtime testing required:	---

Attachments
0001-always-print-firmware-file-name.patch (0001-always-print-firmware-file-name.patch,850 bytes, patch) 2020-07-16 07:20 UTC, Georgy Yakovlev	Details \| Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.

Description Georgy Yakovlev archtester

2020-07-16 07:20:49 UTC

Created attachment 649432 [details, diff]
0001-always-print-firmware-file-name.patch

Very simple patch I've been running for more than a year. Does not require a rebase, and helps a lot with figuring out which exactly FW file is needed.

By default linux does not print the filename. With this patch it does.


here's an example dmesg
...

[    2.739040] Loading firmware: amdgpu/polaris10_mc.bin
[    2.740303] Loading firmware: amdgpu/polaris10_pfp_2.bin
[    2.740306] Loading firmware: amdgpu/polaris10_me_2.bin
[    2.740308] Loading firmware: amdgpu/polaris10_ce_2.bin
[    2.740311] Loading firmware: amdgpu/polaris10_rlc.bin
[    2.740321] Loading firmware: amdgpu/polaris10_mec_2.bin
[    2.740323] Loading firmware: amdgpu/polaris10_mec2_2.bin
[    2.742261] Loading firmware: amdgpu/polaris10_sdma.bin
[    2.742264] Loading firmware: amdgpu/polaris10_sdma1.bin
[    2.742400] Loading firmware: amdgpu/polaris10_uvd.bin
[    2.745343] Loading firmware: amdgpu/polaris10_vce.bin
[   15.720945] Loading firmware: cxgb4/t5fw.bin
[   16.421347] Loading firmware: cxgb4/t5-config.txt


very easy to see what kernel tries to load.

Comment 1 Thomas Deutschmann (RETIRED) gentoo-dev

2020-08-11 16:25:51 UTC

See also: https://lkml.org/lkml/2019/11/17/215

Comment 2 Martin 2021-02-09 17:27:47 UTC

Thanks for an excellent patch!

Can that be added as a useflag option for the Gentoo kernel?


Aside: Whatever became of the lkml patch?

Comment 3 Alice Ferrazzi Gentoo Infrastructure

2021-03-29 08:06:08 UTC

ping ? any news ?

Comment 4 Alice Ferrazzi Gentoo Infrastructure

2021-03-29 08:07:29 UTC

Linking to Greg kh question "why don't just use ftrace ?"

Comment 5 Georgy Yakovlev archtester

2021-06-03 06:28:05 UTC

meanwhile patch still applies and works =)

I haven't got time to make something upstream-able, but still intend to.

Comment 6 Alice Ferrazzi Gentoo Infrastructure

2021-06-03 10:18:13 UTC

we will wait for this patch to reach upstream.

Comment 7 Patrick Op de Beeck 2021-06-16 21:10:21 UTC

Great idea ! Must be included in kernel source !

Comment 8 Mike Pagano gentoo-dev

2021-08-26 15:55:15 UTC

This is now in the following gentoo-sources:

>=5.13.13
>=5.10.61
>=5.4.143
>=4.19.205
>=4.14.245
>=4.9.281
>=4.4.282