Re: [PATCH] KVM: TDX: Remove always true ifdef

[Binbin Wu]

On 9/18/2026 7:36 PM, Nikolay Borisov wrote:
> 
> 
> On 26.08.26 г. 13:03 ч., Nikolay Borisov wrote:
>> CONFIG_KVM_INTEL_TDX depends on KVM_TDX_HOST, which in turn depends on
>> CONFIG_X86_64. So it's guaranteed that any code under ifdef
>> CONFIG_KVM_INTEL_TDX is also running under CONFIG_X86_64 being true,
>> remove pointless ifdef guard.
>>
>> Signed-off-by: Nikolay Borisov <nik.borisov@xxxxxxxx>

Reviewed-by: Binbin Wu <binbin.wu@xxxxxxxxxxxxxxx>

> 
> Gentle ping.

BTW, I find the patch has been applied to kvm-x86 next branch already.

> 
> 
>> ---
>>   arch/x86/kvm/vmx/main.c | 2 --
>>   1 file changed, 2 deletions(-)
>>
>> diff --git a/arch/x86/kvm/vmx/main.c b/arch/x86/kvm/vmx/main.c
>> index f204a0fc0a57..e9cca303c855 100644
>> --- a/arch/x86/kvm/vmx/main.c
>> +++ b/arch/x86/kvm/vmx/main.c
>> @@ -799,7 +799,6 @@ static void vt_write_tsc_multiplier(struct kvm_vcpu *vcpu)
>>       vmx_write_tsc_multiplier(vcpu);
>>   }
>>   -#ifdef CONFIG_X86_64
>>   static int vt_set_hv_timer(struct kvm_vcpu *vcpu, u64 guest_deadline_tsc,
>>                     bool *expired)
>>   {
>> @@ -818,7 +817,6 @@ static void vt_cancel_hv_timer(struct kvm_vcpu *vcpu)
>>         vmx_cancel_hv_timer(vcpu);
>>   }
>> -#endif
>>     static void vt_setup_mce(struct kvm_vcpu *vcpu)
>>   {
>