linux-kernel - Re: [PATCH v13 03/30] LoongArch: KVM: Implement kvm hardware enable, disable interface

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

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <cc73aaf7-85bf-2062-6d1b-f1cbb70242f8@loongson.cn>
Date:   Tue, 13 Jun 2023 20:49:26 +0800
From:   "bibo, mao" <bibo.mao@...il.com>
To:     Tianrui Zhao <zhaotianrui@...ngson.cn>
Cc:     Paolo Bonzini <pbonzini@...hat.com>,
        Huacai Chen <chenhuacai@...nel.org>,
        WANG Xuerui <kernel@...0n.name>,
        Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
        loongarch@...ts.linux.dev, Jens Axboe <axboe@...nel.dk>,
        Mark Brown <broonie@...nel.org>,
        Alex Deucher <alexander.deucher@....com>,
        Oliver Upton <oliver.upton@...ux.dev>,
        Xi Ruoyao <xry111@...111.site>, tangyouling@...ngson.cn,
        linux-kernel@...r.kernel.org, kvm@...r.kernel.org
Subject: Re: [PATCH v13 03/30] LoongArch: KVM: Implement kvm hardware enable,
 disable interface



在 2023/6/9 17:08, Tianrui Zhao 写道:
> Implement kvm hardware enable, disable interface, setting
> the guest config register to enable virtualization features
> when called the interface.
> 
> Signed-off-by: Tianrui Zhao <zhaotianrui@...ngson.cn>
> ---
>   arch/loongarch/kvm/main.c | 64 +++++++++++++++++++++++++++++++++++++++
>   1 file changed, 64 insertions(+)
> 
> diff --git a/arch/loongarch/kvm/main.c b/arch/loongarch/kvm/main.c
> index f98c1619725f..5ebae1ea7565 100644
> --- a/arch/loongarch/kvm/main.c
> +++ b/arch/loongarch/kvm/main.c
> @@ -195,6 +195,70 @@ static void _kvm_init_gcsr_flag(void)
>   	set_gcsr_sw_flag(LOONGARCH_CSR_PERFCNTR3);
>   }
>   
> +void kvm_init_vmcs(struct kvm *kvm)
> +{
> +	kvm->arch.vmcs = vmcs;
> +}
> +
> +long kvm_arch_dev_ioctl(struct file *filp,
> +			unsigned int ioctl, unsigned long arg)
> +{
> +	return -ENOIOCTLCMD;
> +}
> +
> +#ifdef CONFIG_KVM_GENERIC_HARDWARE_ENABLING
Can we remove macro CONFIG_KVM_GENERIC_HARDWARE_ENABLING here since it 
is always enabled? The other looks good to me.

Reviewed-by: Bibo Mao <maobibo@...ngson.cn>

Regards
Bibo, Mao
> +int kvm_arch_hardware_enable(void)
> +{
> +	unsigned long env, gcfg = 0;
> +
> +	env = read_csr_gcfg();
> +	/* First init gtlbc, gcfg, gstat, gintc. All guest use the same config */
> +	clear_csr_gtlbc(CSR_GTLBC_USETGID | CSR_GTLBC_TOTI);
> +	write_csr_gcfg(0);
> +	write_csr_gstat(0);
> +	write_csr_gintc(0);
> +
> +	/*
> +	 * Enable virtualization features granting guest direct control of
> +	 * certain features:
> +	 * GCI=2:       Trap on init or unimplement cache instruction.
> +	 * TORU=0:      Trap on Root Unimplement.
> +	 * CACTRL=1:    Root control cache.
> +	 * TOP=0:       Trap on Previlege.
> +	 * TOE=0:       Trap on Exception.
> +	 * TIT=0:       Trap on Timer.
> +	 */
> +	if (env & CSR_GCFG_GCIP_ALL)
> +		gcfg |= CSR_GCFG_GCI_SECURE;
> +	if (env & CSR_GCFG_MATC_ROOT)
> +		gcfg |= CSR_GCFG_MATC_ROOT;
> +
> +	gcfg |= CSR_GCFG_TIT;
> +	write_csr_gcfg(gcfg);
> +
> +	kvm_flush_tlb_all();
> +
> +	/* Enable using TGID  */
> +	set_csr_gtlbc(CSR_GTLBC_USETGID);
> +	kvm_debug("gtlbc:%llx gintc:%llx gstat:%llx gcfg:%llx",
> +			read_csr_gtlbc(), read_csr_gintc(),
> +			read_csr_gstat(), read_csr_gcfg());
> +
> +	return 0;
> +}
> +
> +void kvm_arch_hardware_disable(void)
> +{
> +	clear_csr_gtlbc(CSR_GTLBC_USETGID | CSR_GTLBC_TOTI);
> +	write_csr_gcfg(0);
> +	write_csr_gstat(0);
> +	write_csr_gintc(0);
> +
> +	/* Flush any remaining guest TLB entries */
> +	kvm_flush_tlb_all();
> +}
> +#endif
> +
>   static int kvm_loongarch_env_init(void)
>   {
>   	struct kvm_context *context;