Re: [RFC PATCH v7 17/37] x86/apic: Unionize apic regs for 32bit/64bit access w/o type casting
From: Neeraj Upadhyay
Date: Tue Jun 24 2025 - 21:16:22 EST
On 6/24/2025 3:58 PM, Huang, Kai wrote:
>
>> +struct apic_page {
>> + union {
>> + u64 regs64[PAGE_SIZE / 8];
>> + u32 regs[PAGE_SIZE / 4];
>> + u8 bytes[PAGE_SIZE];
>> + };
>> +} __aligned(PAGE_SIZE);
>
> The 'bytes' field is never used. Should it be removed?
>
It's used in patch 25/37, in get_reg_bitmap(). I will describe
it in the commit log of this patch. Thanks for the review!
- Neeraj