Hello, Damien Zammit, le ven. 02 janv. 2026 04:15:11 +0000, a ecrit: > --- > i386/i386/gdt.c | 8 ++------ > 1 file changed, 2 insertions(+), 6 deletions(-) > > diff --git a/i386/i386/gdt.c b/i386/i386/gdt.c > index 4edd3ec5..9409c2fe 100644 > --- a/i386/i386/gdt.c > +++ b/i386/i386/gdt.c > @@ -60,6 +60,7 @@ gdt_fill(int cpu, struct real_descriptor *mygdt) > #ifndef MACH_PV_DESCRIPTORS > _fill_gdt_descriptor(mygdt, LINEAR_DS, 0, 0, ACC_PL_K|ACC_DATA_W, > SZ_64); > #endif /* MACH_PV_DESCRIPTORS */ > + _fill_gdt_descriptor(mygdt, PERCPU_DS, 0, 0, ACC_PL_K|ACC_DATA_W, > SZ_64); > #else > _fill_gdt_descriptor(mygdt, KERNEL_CS, > LINEAR_MIN_KERNEL_ADDRESS - VM_MIN_KERNEL_ADDRESS, > @@ -79,12 +80,7 @@ gdt_fill(int cpu, struct real_descriptor *mygdt) > _fill_gdt_descriptor(mygdt, PERCPU_DS, > thiscpu, > thiscpu + sizeof(struct percpu) - 1, > -#ifdef __x86_64__ > - ACC_PL_K|ACC_DATA_W, SZ_64 > -#else > - ACC_PL_K|ACC_DATA_W, SZ_32 > -#endif > - ); > + ACC_PL_K|ACC_DATA_W, SZ_32);
Better move that part out of ifdef: we do want the thiscpu computation part too, and better have it factorized. Samuel
