On 6/8/2026 2:39 PM, Binbin Wu wrote:
> On 5/22/2026 7:16 AM, Lisa Wang wrote:
>
> [...]
>
>> +
>> +static inline u64 kvm_get_default_cr4(void)
>> +{
>> + u64 cr4 = X86_CR4_PAE | X86_CR4_OSFXSR;
>> +
>> + if (kvm_cpu_has(X86_FEATURE_XSAVE))
>> + cr4 |= X86_CR4_OSXSAVE;
>> + return cr4;
>> +}
>> +
>
> [...]
>
>> @@ -647,16 +643,12 @@ static void vcpu_init_sregs(struct kvm_vm *vm, struct
>> kvm_vcpu *vcpu)
>> vcpu_sregs_get(vcpu, &sregs);
>>
>> sregs.idt.base = vm->arch.idt;
>> - sregs.idt.limit = NUM_INTERRUPTS * sizeof(struct idt_entry) - 1;
>> + sregs.idt.limit = kvm_get_default_idt_limit();
>> sregs.gdt.base = vm->arch.gdt;
>> - sregs.gdt.limit = getpagesize() - 1;
>> -
>> - sregs.cr0 = X86_CR0_PE | X86_CR0_NE | X86_CR0_PG;
>> - sregs.cr4 |= X86_CR4_PAE | X86_CR4_OSFXSR;
>> - if (kvm_cpu_has(X86_FEATURE_XSAVE))
>> - sregs.cr4 |= X86_CR4_OSXSAVE;
>> - if (vm->mmu.pgtable_levels == 5)
>> - sregs.cr4 |= X86_CR4_LA57;
>
> I guess the 5-level paging thing is dropped unexpectedly during rebase?
>
>
>> + sregs.gdt.limit = kvm_get_default_gdt_limit();
>>
>> + sregs.cr0 = kvm_get_default_cr0();
>> + sregs.cr4 |= kvm_get_default_cr4();
>> sregs.efer |= (EFER_LME | EFER_LMA | EFER_NX);
Also, sregs.efer |= kvm_get_default_efer() is dropped unexpectedly during
rebase.
>>
>> kvm_seg_set_unusable(&sregs.ldt);
>>
>