Re: [xen-tip:linux-next 12/12] WARNING: modpost: vmlinux: section mismatch in reference: mc_debug_data+0x0 (section: .data) -> mc_debug_data_early (section: .init.data)

["Jürgen Groß" <jgross@suse.com>]

[-- Attachment #1.1.1: Type: text/plain, Size: 825 bytes --]

On 27.03.25 15:40, Borislav Petkov wrote:
> On Thu, Mar 27, 2025 at 03:21:45PM +0100, Jürgen Groß wrote:
>> Well, that is wasting nearly 3kB of the data section.
>>
>> Maybe not a big deal, but still...
> 
> We could do it until the proper fix is in place, no?
> 
> 3K is meh, especially for the hypervisor kernel, I'd say...
> 

Yeah, that was my thinking.

Another approach could be to have:

-static DEFINE_PER_CPU(struct mc_debug_data *, mc_debug_data) =
-	&mc_debug_data_early;
+static DEFINE_PER_CPU(struct mc_debug_data *, mc_debug_data);

and to use an inline access function returning &mc_debug_data_early
if the percpu variable is NULL. This access function could be __ref.

It is a debug feature after all, so having a few additional instructions
isn't the end of the world.


Juergen

[-- Attachment #1.1.2: OpenPGP public key --]
[-- Type: application/pgp-keys, Size: 3743 bytes --]

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 495 bytes --]