| lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
|
Open Source and information security mailing list archives
| ||
|
Message-ID: <40772a8d-e541-2f72-3c47-c2595172c3b6@loongson.cn>
Date: Mon, 24 Jun 2024 10:35:08 +0800
From: maobibo <maobibo@...ngson.cn>
To: Huacai Chen <chenhuacai@...nel.org>
Cc: Tianrui Zhao <zhaotianrui@...ngson.cn>, WANG Xuerui <kernel@...0n.name>,
Sean Christopherson <seanjc@...gle.com>, kvm@...r.kernel.org,
loongarch@...ts.linux.dev, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 1/6] LoongArch: KVM: Delay secondary mmu tlb flush
until guest entry
On 2024/6/24 上午9:58, Huacai Chen wrote:
> On Mon, Jun 24, 2024 at 9:23 AM maobibo <maobibo@...ngson.cn> wrote:
>>
>>
>>
>> On 2024/6/23 下午3:54, Huacai Chen wrote:
>>> Hi, Bibo,
>>>
>>> On Wed, Jun 19, 2024 at 4:09 PM Bibo Mao <maobibo@...ngson.cn> wrote:
>>>>
>>>> If there is page fault for secondary mmu, there needs tlb flush
>>> What does "secondary mmu" in this context mean? Maybe "guest mmu"?
>> "secondary mmu" following x86 concepts, the weblink is:
>> https://lwn.net/Articles/977945/
>>
>> It is called stage-2 mmu on ARM64 also. "guest mmu" cannot represent
>> whether it is gva to gpa, or gpa to hpa, or gva to hpa directly.
> Then it is better to explain "what is secondary mmu" in the commit
> message when it appears at the first time.
Sure, will do in next patch.
Regards
Bibo Mao
>
> Huacai
>
>>
>> Regards
>> Bibo Mao
>>
>>>
>>> Huacai
>>>
>>>> operation indexed with fault gpa address and VMID. VMID is stored
>>>> at register CSR_GSTAT and will be reload or recalculated during
>>>> guest entry.
>>>>
>>>> Currently CSR_GSTAT is not saved and restored during vcpu context
>>>> switch, it is recalculated during guest entry. So CSR_GSTAT is in
>>>> effect only when vcpu runs in guest mode, however it may be not in
>>>> effected if vcpu exits to host mode, since register CSR_GSTAT may
>>>> be stale, it maybe records VMID of last scheduled vcpu, rather than
>>>> current vcpu.
>>>>
>>>> Function kvm_flush_tlb_gpa() should be called with its real VMID,
>>>> here move it to guest entrance. Also arch specific request id
>>>> KVM_REQ_TLB_FLUSH_GPA is added to flush tlb, and it can be optimized
>>>> if VMID is updated, since all guest tlb entries will be invalid if
>>>> VMID is updated.
>>>>
>>>> Signed-off-by: Bibo Mao <maobibo@...ngson.cn>
>>>> ---
>>>> arch/loongarch/include/asm/kvm_host.h | 2 ++
>>>> arch/loongarch/kvm/main.c | 1 +
>>>> arch/loongarch/kvm/mmu.c | 4 ++--
>>>> arch/loongarch/kvm/tlb.c | 5 +----
>>>> arch/loongarch/kvm/vcpu.c | 18 ++++++++++++++++++
>>>> 5 files changed, 24 insertions(+), 6 deletions(-)
>>>>
>>>> diff --git a/arch/loongarch/include/asm/kvm_host.h b/arch/loongarch/include/asm/kvm_host.h
>>>> index c87b6ea0ec47..32c4948f534f 100644
>>>> --- a/arch/loongarch/include/asm/kvm_host.h
>>>> +++ b/arch/loongarch/include/asm/kvm_host.h
>>>> @@ -30,6 +30,7 @@
>>>> #define KVM_PRIVATE_MEM_SLOTS 0
>>>>
>>>> #define KVM_HALT_POLL_NS_DEFAULT 500000
>>>> +#define KVM_REQ_TLB_FLUSH_GPA KVM_ARCH_REQ(0)
>>>>
>>>> #define KVM_GUESTDBG_SW_BP_MASK \
>>>> (KVM_GUESTDBG_ENABLE | KVM_GUESTDBG_USE_SW_BP)
>>>> @@ -190,6 +191,7 @@ struct kvm_vcpu_arch {
>>>>
>>>> /* vcpu's vpid */
>>>> u64 vpid;
>>>> + gpa_t flush_gpa;
>>>>
>>>> /* Frequency of stable timer in Hz */
>>>> u64 timer_mhz;
>>>> diff --git a/arch/loongarch/kvm/main.c b/arch/loongarch/kvm/main.c
>>>> index 86a2f2d0cb27..844736b99d38 100644
>>>> --- a/arch/loongarch/kvm/main.c
>>>> +++ b/arch/loongarch/kvm/main.c
>>>> @@ -242,6 +242,7 @@ void kvm_check_vpid(struct kvm_vcpu *vcpu)
>>>> kvm_update_vpid(vcpu, cpu);
>>>> trace_kvm_vpid_change(vcpu, vcpu->arch.vpid);
>>>> vcpu->cpu = cpu;
>>>> + kvm_clear_request(KVM_REQ_TLB_FLUSH_GPA, vcpu);
>>>> }
>>>>
>>>> /* Restore GSTAT(0x50).vpid */
>>>> diff --git a/arch/loongarch/kvm/mmu.c b/arch/loongarch/kvm/mmu.c
>>>> index 98883aa23ab8..9e39d28fec35 100644
>>>> --- a/arch/loongarch/kvm/mmu.c
>>>> +++ b/arch/loongarch/kvm/mmu.c
>>>> @@ -908,8 +908,8 @@ int kvm_handle_mm_fault(struct kvm_vcpu *vcpu, unsigned long gpa, bool write)
>>>> return ret;
>>>>
>>>> /* Invalidate this entry in the TLB */
>>>> - kvm_flush_tlb_gpa(vcpu, gpa);
>>>> -
>>>> + vcpu->arch.flush_gpa = gpa;
>>>> + kvm_make_request(KVM_REQ_TLB_FLUSH_GPA, vcpu);
>>>> return 0;
>>>> }
>>>>
>>>> diff --git a/arch/loongarch/kvm/tlb.c b/arch/loongarch/kvm/tlb.c
>>>> index 02535df6b51f..ebdbe9264e9c 100644
>>>> --- a/arch/loongarch/kvm/tlb.c
>>>> +++ b/arch/loongarch/kvm/tlb.c
>>>> @@ -23,10 +23,7 @@ void kvm_flush_tlb_all(void)
>>>>
>>>> void kvm_flush_tlb_gpa(struct kvm_vcpu *vcpu, unsigned long gpa)
>>>> {
>>>> - unsigned long flags;
>>>> -
>>>> - local_irq_save(flags);
>>>> + lockdep_assert_irqs_disabled();
>>>> gpa &= (PAGE_MASK << 1);
>>>> invtlb(INVTLB_GID_ADDR, read_csr_gstat() & CSR_GSTAT_GID, gpa);
>>>> - local_irq_restore(flags);
>>>> }
>>>> diff --git a/arch/loongarch/kvm/vcpu.c b/arch/loongarch/kvm/vcpu.c
>>>> index 9e8030d45129..b747bd8bc037 100644
>>>> --- a/arch/loongarch/kvm/vcpu.c
>>>> +++ b/arch/loongarch/kvm/vcpu.c
>>>> @@ -51,6 +51,16 @@ static int kvm_check_requests(struct kvm_vcpu *vcpu)
>>>> return RESUME_GUEST;
>>>> }
>>>>
>>>> +static void kvm_late_check_requests(struct kvm_vcpu *vcpu)
>>>> +{
>>>> + lockdep_assert_irqs_disabled();
>>>> + if (kvm_check_request(KVM_REQ_TLB_FLUSH_GPA, vcpu))
>>>> + if (vcpu->arch.flush_gpa != INVALID_GPA) {
>>>> + kvm_flush_tlb_gpa(vcpu, vcpu->arch.flush_gpa);
>>>> + vcpu->arch.flush_gpa = INVALID_GPA;
>>>> + }
>>>> +}
>>>> +
>>>> /*
>>>> * Check and handle pending signal and vCPU requests etc
>>>> * Run with irq enabled and preempt enabled
>>>> @@ -101,6 +111,13 @@ static int kvm_pre_enter_guest(struct kvm_vcpu *vcpu)
>>>> /* Make sure the vcpu mode has been written */
>>>> smp_store_mb(vcpu->mode, IN_GUEST_MODE);
>>>> kvm_check_vpid(vcpu);
>>>> +
>>>> + /*
>>>> + * Called after function kvm_check_vpid()
>>>> + * Since it updates csr_gstat used by kvm_flush_tlb_gpa(),
>>>> + * also it may clear KVM_REQ_TLB_FLUSH_GPA pending bit
>>>> + */
>>>> + kvm_late_check_requests(vcpu);
>>>> vcpu->arch.host_eentry = csr_read64(LOONGARCH_CSR_EENTRY);
>>>> /* Clear KVM_LARCH_SWCSR_LATEST as CSR will change when enter guest */
>>>> vcpu->arch.aux_inuse &= ~KVM_LARCH_SWCSR_LATEST;
>>>> @@ -994,6 +1011,7 @@ int kvm_arch_vcpu_create(struct kvm_vcpu *vcpu)
>>>> struct loongarch_csrs *csr;
>>>>
>>>> vcpu->arch.vpid = 0;
>>>> + vcpu->arch.flush_gpa = INVALID_GPA;
>>>>
>>>> hrtimer_init(&vcpu->arch.swtimer, CLOCK_MONOTONIC, HRTIMER_MODE_ABS_PINNED);
>>>> vcpu->arch.swtimer.function = kvm_swtimer_wakeup;
>>>> --
>>>> 2.39.3
>>>>
>>
>>
Powered by blists - more mailing lists