From: Mirsad Todorovac <mtodorovac69@gmail.com>
To: "Thomas Weißschuh" <thomas@t-8ch.de>
Cc: Linux Kernel Build System <linux-kbuild@vger.kernel.org>,
Masahiro Yamada <masahiroy@kernel.org>,
Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
Bagas Sanjaya <bagasdotme@gmail.com>
Subject: Re: [PROBLEM] make bindeb-pkg: When just one source line in a single driver is changed, all driver are rebuilt?
Date: Mon, 25 Nov 2024 20:10:42 +0100 [thread overview]
Message-ID: <d136d201-7aa5-4bab-8437-c44e0c54e0e4@gmail.com> (raw)
In-Reply-To: <b21160de-8f12-4796-8df1-2188b5fc93e0@t-8ch.de>
On 11/23/24 15:57, Thomas Weißschuh wrote:
> Hi Mirsad,
>
> On 2024-11-23 02:53:09+0100, Mirsad Todorovac wrote:
>> Hi, Mr. Weißschuh,
>
> Call me Thomas, please :-)
It will be my honour, thanks :-)
>> On 11/21/24 22:43, Thomas Weißschuh wrote:
>>
>>> On 2024-11-21 22:18:58+0100, Mirsad Todorovac wrote:
>>>> I am perplexed now at 6.12 stable vanilla tree, and I've noticed that after changing just one
>>>> line in ./drivers/gpu/drm/xe/tests/xe_migrate.c:226:5-11, all unrelated drivers are being BTF'd, SIGNED,
>>>> and INSTALL-ed:
>>>>
>>>> SIGN debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/aht10.ko
>>>> INSTALL debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/coretemp.ko
>>>> STRIP debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/atxp1.ko
>>>> INSTALL debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/corsair-cpro.ko
>>>> SIGN debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/adt7475.ko
>>>> STRIP debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/axi-fan-control.ko
>>>> SIGN debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/as370-hwmon.ko
>>>> STRIP debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/chipcap2.ko
>>>> SIGN debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/aquacomputer_d5next.ko
>>>> STRIP debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/coretemp.ko
>>>> SIGN debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/applesmc.ko
>>>> INSTALL debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/corsair-psu.ko
>>>> STRIP debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/corsair-cpro.ko
>>>> SIGN debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/asus_rog_ryujin.ko
>>>> INSTALL debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/cros_ec_hwmon.ko
>>>> SIGN debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/asc7621.ko
>>>> INSTALL debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/da9052-hwmon.ko
>>>> SIGN debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/atxp1.ko
>>>> SIGN debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/axi-fan-control.ko
>>>> STRIP debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/corsair-psu.ko
>>>> INSTALL debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/da9055-hwmon.ko
>>>> SIGN debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/chipcap2.ko
>>>> STRIP debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/cros_ec_hwmon.ko
>>>> STRIP debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/da9052-hwmon.ko
>>>> SIGN debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/coretemp.ko
>>>> SIGN debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/corsair-cpro.ko
>>>> INSTALL debian/linux-image-6.12.0-dirty/lib/modules/6.12.0-dirty/kernel/drivers/hwmon/dell-smm-hwmon.ko
>>> This happens when the vermagic is changed. The vermagic is linked into
>>> each module. And after relinking all the other steps also need to be
>>> done.
>>> Building the Debian package increases the build number which ends up in
>>> the vermagic and therefore all modules.
>>> Also there is CONFIG_LOCALVERSION_AUTO which may be involved.
>>
>> Yes, I use CONFIG_LOCALVERSION_AUTO because it saves a lot of trouble in bisecting.
>>
>> This vermagic manipulation is beyond my grasp. I understand that
>> modules must match kernel version, for internal ABI often changes. But
>> not between builds of the same source? Am I thinking right?
>
> The ABI doesn't change between builds of the same source tree which is
> why there are no actual rebuilds ("CC") of all modules.
That is actually more precise.
> But your usage of bindeb-pkg bumps the version string
> (also without CONFIG_LOCALVERSION_AUTO) and this needs to end up in each
> module file which explains the LD/BTF/SIGN/INSTALL steps.
I see. I hoped there is a way around this.
CONFIG_LOCALVERSION_AUTO is very useful when compared to manually keeping tracks of commits.
Once upon a time, about a year or two ago, I told Mr. Bagas Sanjaya how a list of applied diffs
could be added to the kernel manifest instead of just config-${VERSION} and "-dirty" flag.
Thanks.
Best regards,
Mirsad
> [..]
prev parent reply other threads:[~2024-11-25 19:11 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2024-11-21 21:18 [PROBLEM] make bindeb-pkg: When just one source line in a single driver is changed, all driver are rebuilt? Mirsad Todorovac
2024-11-21 21:43 ` Thomas Weißschuh
2024-11-23 1:53 ` Mirsad Todorovac
2024-11-23 14:57 ` Thomas Weißschuh
2024-11-25 19:10 ` Mirsad Todorovac [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=d136d201-7aa5-4bab-8437-c44e0c54e0e4@gmail.com \
--to=mtodorovac69@gmail.com \
--cc=bagasdotme@gmail.com \
--cc=linux-kbuild@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=masahiroy@kernel.org \
--cc=thomas@t-8ch.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox