Message ID | 20230609090832.2131037-30-zhaotianrui@loongson.cn |
---|---|
State | New |
Headers |
Return-Path: <linux-kernel-owner@vger.kernel.org> Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:994d:0:b0:3d9:f83d:47d9 with SMTP id k13csp821796vqr; Fri, 9 Jun 2023 02:36:12 -0700 (PDT) X-Google-Smtp-Source: ACHHUZ72Qhs/FkiFCkh0sitAP9VEJTkFdFvCpDI5cFxHq/tG55moLgPhhfb+Z6fO4zmN42pxZUXo X-Received: by 2002:a05:6359:d1d:b0:128:6cdd:962 with SMTP id gp29-20020a0563590d1d00b001286cdd0962mr900835rwb.1.1686303371931; Fri, 09 Jun 2023 02:36:11 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1686303371; cv=none; d=google.com; s=arc-20160816; b=KG3hfV95QZLUZpXxoU7wyYOj9yoj5qUPtxYhyPG03gdK80EwTM5GNdqR6KZsRi0lP7 pmdV/Z+IprTzeHHzYy+NGuwfB7QUAGd3mpuLUdp3r31O6eMjkQfk0cJZ7gfQJ0EoFJpQ mdC1IgG7Wy5/Ni4jqNV3GBJ2p80rqw1uqUaHOCqcMZ1+FJF9hzw1RqTA7F/AsUxmZ0om AmnLluz+Elts0XDQdmRp80LpLUvGhwV5H4ChjrTJAk0GT0T9l2PjHyq+HjrXN0KtsSXq kGzkvGHLSDQUxaeCvanTLVayZ6i8fIXTLgEs3rDoXf+3wyoBEep/bIScW1mdIEozSoKV T3Xw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from; bh=clOq/Jcgq+CTpuezcPJsitLPEXV+1HloD52k+7NHWHk=; b=ezdkEpyO7TiE7SOn/tZKkedrlJDydAfjuCR8mCGIWSlMfxC82chvoYVcr1LWuF6hv/ bE4eTv2EvzgrB/7d/3Vb9YpXLbd1B/NUzeFMzzKK39aLGqfZ40ikfkrEXDcT3YfJw9mJ KWZlZwGd/ROn9O/wwbr8eOxAxbXNXLJ/gzPBbdCTGIOxmuAvkMJlCRrZPExR5ZRGTLZj 7fCqo7jq9Uj+Yr7rtYSeWMntA50byEZ+2/jHbzuBgYHG3ChhQP9QnUSnnbuCLlNuSLuF tR4aPBBnTuX5cVouTRf7ttHGo/HYBKhUwYH3U3T5QzwxH6aHJIPJEh3Dw6ZvN1TsxOR2 i8ZA== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id i10-20020a639d0a000000b00534849cb1afsi2333734pgd.593.2023.06.09.02.35.58; Fri, 09 Jun 2023 02:36:11 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S241092AbjFIJcT (ORCPT <rfc822;liningstudo@gmail.com> + 99 others); Fri, 9 Jun 2023 05:32:19 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:52154 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S241609AbjFIJbz (ORCPT <rfc822;linux-kernel@vger.kernel.org>); Fri, 9 Jun 2023 05:31:55 -0400 Received: from mail.loongson.cn (mail.loongson.cn [114.242.206.163]) by lindbergh.monkeyblade.net (Postfix) with ESMTP id 0FAD855A2; Fri, 9 Jun 2023 02:26:47 -0700 (PDT) Received: from loongson.cn (unknown [10.2.5.185]) by gateway (Coremail) with SMTP id _____8Dx_+sd7IJkC_4AAA--.3088S3; Fri, 09 Jun 2023 17:08:45 +0800 (CST) Received: from localhost.localdomain (unknown [10.2.5.185]) by localhost.localdomain (Coremail) with SMTP id AQAAf8DxFOQQ7IJkzHQKAA--.31790S31; Fri, 09 Jun 2023 17:08:44 +0800 (CST) From: Tianrui Zhao <zhaotianrui@loongson.cn> To: linux-kernel@vger.kernel.org, kvm@vger.kernel.org Cc: Paolo Bonzini <pbonzini@redhat.com>, Huacai Chen <chenhuacai@kernel.org>, WANG Xuerui <kernel@xen0n.name>, Greg Kroah-Hartman <gregkh@linuxfoundation.org>, loongarch@lists.linux.dev, Jens Axboe <axboe@kernel.dk>, Mark Brown <broonie@kernel.org>, Alex Deucher <alexander.deucher@amd.com>, Oliver Upton <oliver.upton@linux.dev>, maobibo@loongson.cn, Xi Ruoyao <xry111@xry111.site>, zhaotianrui@loongson.cn, tangyouling@loongson.cn Subject: [PATCH v13 29/30] LoongArch: KVM: Supplement kvm document about LoongArch-specific part Date: Fri, 9 Jun 2023 17:08:31 +0800 Message-Id: <20230609090832.2131037-30-zhaotianrui@loongson.cn> X-Mailer: git-send-email 2.39.1 In-Reply-To: <20230609090832.2131037-1-zhaotianrui@loongson.cn> References: <20230609090832.2131037-1-zhaotianrui@loongson.cn> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-CM-TRANSID: AQAAf8DxFOQQ7IJkzHQKAA--.31790S31 X-CM-SenderInfo: p2kd03xldq233l6o00pqjv00gofq/ X-Coremail-Antispam: 1Uk129KBjDUn29KB7ZKAUJUUUUU529EdanIXcx71UUUUU7KY7 ZEXasCq-sGcSsGvfJ3UbIjqfuFe4nvWSU5nxnvy29KBjDU0xBIdaVrnUUvcSsGvfC2Kfnx nUUI43ZEXa7xR_UUUUUUUUU== X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,SPF_HELO_NONE, SPF_PASS,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: <linux-kernel.vger.kernel.org> X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1768217244541753925?= X-GMAIL-MSGID: =?utf-8?q?1768217244541753925?= |
Series |
Add KVM LoongArch support
|
|
Commit Message
zhaotianrui
June 9, 2023, 9:08 a.m. UTC
Supplement kvm document about LoongArch-specific part, such as add
api introduction for GET/SET_ONE_REG, GET/SET_FPU, GET/SET_MP_STATE,
etc.
Signed-off-by: Tianrui Zhao <zhaotianrui@loongson.cn>
---
Documentation/virt/kvm/api.rst | 71 +++++++++++++++++++++++++++++-----
1 file changed, 62 insertions(+), 9 deletions(-)
Comments
Hi, Tianrui, On Fri, Jun 9, 2023 at 5:09 PM Tianrui Zhao <zhaotianrui@loongson.cn> wrote: > > Supplement kvm document about LoongArch-specific part, such as add > api introduction for GET/SET_ONE_REG, GET/SET_FPU, GET/SET_MP_STATE, > etc. > > Signed-off-by: Tianrui Zhao <zhaotianrui@loongson.cn> > --- > Documentation/virt/kvm/api.rst | 71 +++++++++++++++++++++++++++++----- > 1 file changed, 62 insertions(+), 9 deletions(-) > > diff --git a/Documentation/virt/kvm/api.rst b/Documentation/virt/kvm/api.rst > index add067793b90..ad8e13eab48d 100644 > --- a/Documentation/virt/kvm/api.rst > +++ b/Documentation/virt/kvm/api.rst > @@ -416,6 +416,12 @@ Reads the general purpose registers from the vcpu. > __u64 pc; > }; > > + /* LoongArch */ > + struct kvm_regs { Add a " /* out (KVM_GET_REGS) / in (KVM_SET_REGS) */" line as others here? > + unsigned long gpr[32]; > + unsigned long pc; > + }; > + > > 4.12 KVM_SET_REGS > ----------------- > @@ -506,7 +512,7 @@ translation mode. > ------------------ > > :Capability: basic > -:Architectures: x86, ppc, mips, riscv > +:Architectures: x86, ppc, mips, riscv, loongarch > :Type: vcpu ioctl > :Parameters: struct kvm_interrupt (in) > :Returns: 0 on success, negative on failure. > @@ -592,6 +598,14 @@ b) KVM_INTERRUPT_UNSET > > This is an asynchronous vcpu ioctl and can be invoked from any thread. > > +LOONGARCH: > +^^^^^^^^^^ > + > +Queues an external interrupt to be injected into the virtual CPU. A negative > +interrupt number dequeues the interrupt. > + > +This is an asynchronous vcpu ioctl and can be invoked from any thread. > + > > 4.17 KVM_DEBUG_GUEST > -------------------- > @@ -737,7 +751,7 @@ signal mask. > ---------------- > > :Capability: basic > -:Architectures: x86 > +:Architectures: x86, loongarch > :Type: vcpu ioctl > :Parameters: struct kvm_fpu (out) > :Returns: 0 on success, -1 on error > @@ -746,7 +760,7 @@ Reads the floating point state from the vcpu. > > :: > > - /* for KVM_GET_FPU and KVM_SET_FPU */ > + /* x86: for KVM_GET_FPU and KVM_SET_FPU */ > struct kvm_fpu { > __u8 fpr[8][16]; > __u16 fcw; > @@ -761,12 +775,22 @@ Reads the floating point state from the vcpu. > __u32 pad2; > }; > > + /* LoongArch: for KVM_GET_FPU and KVM_SET_FPU */ > + struct kvm_fpu { > + __u32 fcsr; > + __u32 none; Maybe use pad1 as x86 is better. Huacai > + __u64 fcc; > + struct kvm_fpureg { > + __u64 val64[4]; > + }fpr[32]; > + }; > + > > 4.23 KVM_SET_FPU > ---------------- > > :Capability: basic > -:Architectures: x86 > +:Architectures: x86, loongarch > :Type: vcpu ioctl > :Parameters: struct kvm_fpu (in) > :Returns: 0 on success, -1 on error > @@ -775,7 +799,7 @@ Writes the floating point state to the vcpu. > > :: > > - /* for KVM_GET_FPU and KVM_SET_FPU */ > + /* x86: for KVM_GET_FPU and KVM_SET_FPU */ > struct kvm_fpu { > __u8 fpr[8][16]; > __u16 fcw; > @@ -790,6 +814,16 @@ Writes the floating point state to the vcpu. > __u32 pad2; > }; > > + /* LoongArch: for KVM_GET_FPU and KVM_SET_FPU */ > + struct kvm_fpu { > + __u32 fcsr; > + __u32 none; > + __u64 fcc; > + struct kvm_fpureg { > + __u64 val64[4]; > + }fpr[32]; > + }; > + > > 4.24 KVM_CREATE_IRQCHIP > ----------------------- > @@ -1387,7 +1421,7 @@ documentation when it pops into existence). > ------------------- > > :Capability: KVM_CAP_ENABLE_CAP > -:Architectures: mips, ppc, s390, x86 > +:Architectures: mips, ppc, s390, x86, loongarch > :Type: vcpu ioctl > :Parameters: struct kvm_enable_cap (in) > :Returns: 0 on success; -1 on error > @@ -1442,7 +1476,7 @@ for vm-wide capabilities. > --------------------- > > :Capability: KVM_CAP_MP_STATE > -:Architectures: x86, s390, arm64, riscv > +:Architectures: x86, s390, arm64, riscv, loongarch > :Type: vcpu ioctl > :Parameters: struct kvm_mp_state (out) > :Returns: 0 on success; -1 on error > @@ -1460,7 +1494,7 @@ Possible values are: > > ========================== =============================================== > KVM_MP_STATE_RUNNABLE the vcpu is currently running > - [x86,arm64,riscv] > + [x86,arm64,riscv,loongarch] > KVM_MP_STATE_UNINITIALIZED the vcpu is an application processor (AP) > which has not yet received an INIT signal [x86] > KVM_MP_STATE_INIT_RECEIVED the vcpu has received an INIT signal, and is > @@ -1516,11 +1550,14 @@ For riscv: > The only states that are valid are KVM_MP_STATE_STOPPED and > KVM_MP_STATE_RUNNABLE which reflect if the vcpu is paused or not. > > +On LoongArch, only the KVM_MP_STATE_RUNNABLE state is used to reflect > +whether the vcpu is runnable. > + > 4.39 KVM_SET_MP_STATE > --------------------- > > :Capability: KVM_CAP_MP_STATE > -:Architectures: x86, s390, arm64, riscv > +:Architectures: x86, s390, arm64, riscv, loongarch > :Type: vcpu ioctl > :Parameters: struct kvm_mp_state (in) > :Returns: 0 on success; -1 on error > @@ -1538,6 +1575,9 @@ For arm64/riscv: > The only states that are valid are KVM_MP_STATE_STOPPED and > KVM_MP_STATE_RUNNABLE which reflect if the vcpu should be paused or not. > > +On LoongArch, only the KVM_MP_STATE_RUNNABLE state is used to reflect > +whether the vcpu is runnable. > + > 4.40 KVM_SET_IDENTITY_MAP_ADDR > ------------------------------ > > @@ -2839,6 +2879,19 @@ Following are the RISC-V D-extension registers: > 0x8020 0000 0600 0020 fcsr Floating point control and status register > ======================= ========= ============================================= > > +LoongArch registers are mapped using the lower 32 bits. The upper 16 bits of > +that is the register group type. > + > +LoongArch csr registers are used to control guest cpu or get status of guest > +cpu, and they have the following id bit patterns:: > + > + 0x9030 0000 0001 00 <reg:5> <sel:3> (64-bit) > + > +LoongArch KVM control registers are used to implement some new defined functions > +such as set vcpu counter or reset vcpu, and they have the following id bit patterns:: > + > + 0x9030 0000 0002 <reg:16> > + > > 4.69 KVM_GET_ONE_REG > -------------------- > -- > 2.39.1 > >
在 2023/6/15 下午5:42, Huacai Chen 写道: > Hi, Tianrui, > > On Fri, Jun 9, 2023 at 5:09 PM Tianrui Zhao <zhaotianrui@loongson.cn> wrote: >> Supplement kvm document about LoongArch-specific part, such as add >> api introduction for GET/SET_ONE_REG, GET/SET_FPU, GET/SET_MP_STATE, >> etc. >> >> Signed-off-by: Tianrui Zhao <zhaotianrui@loongson.cn> >> --- >> Documentation/virt/kvm/api.rst | 71 +++++++++++++++++++++++++++++----- >> 1 file changed, 62 insertions(+), 9 deletions(-) >> >> diff --git a/Documentation/virt/kvm/api.rst b/Documentation/virt/kvm/api.rst >> index add067793b90..ad8e13eab48d 100644 >> --- a/Documentation/virt/kvm/api.rst >> +++ b/Documentation/virt/kvm/api.rst >> @@ -416,6 +416,12 @@ Reads the general purpose registers from the vcpu. >> __u64 pc; >> }; >> >> + /* LoongArch */ >> + struct kvm_regs { > Add a " /* out (KVM_GET_REGS) / in (KVM_SET_REGS) */" line as others here? Thanks, I will add this comment here. Tianrui Zhao >> + unsigned long gpr[32]; >> + unsigned long pc; >> + }; >> + >> >> 4.12 KVM_SET_REGS >> ----------------- >> @@ -506,7 +512,7 @@ translation mode. >> ------------------ >> >> :Capability: basic >> -:Architectures: x86, ppc, mips, riscv >> +:Architectures: x86, ppc, mips, riscv, loongarch >> :Type: vcpu ioctl >> :Parameters: struct kvm_interrupt (in) >> :Returns: 0 on success, negative on failure. >> @@ -592,6 +598,14 @@ b) KVM_INTERRUPT_UNSET >> >> This is an asynchronous vcpu ioctl and can be invoked from any thread. >> >> +LOONGARCH: >> +^^^^^^^^^^ >> + >> +Queues an external interrupt to be injected into the virtual CPU. A negative >> +interrupt number dequeues the interrupt. >> + >> +This is an asynchronous vcpu ioctl and can be invoked from any thread. >> + >> >> 4.17 KVM_DEBUG_GUEST >> -------------------- >> @@ -737,7 +751,7 @@ signal mask. >> ---------------- >> >> :Capability: basic >> -:Architectures: x86 >> +:Architectures: x86, loongarch >> :Type: vcpu ioctl >> :Parameters: struct kvm_fpu (out) >> :Returns: 0 on success, -1 on error >> @@ -746,7 +760,7 @@ Reads the floating point state from the vcpu. >> >> :: >> >> - /* for KVM_GET_FPU and KVM_SET_FPU */ >> + /* x86: for KVM_GET_FPU and KVM_SET_FPU */ >> struct kvm_fpu { >> __u8 fpr[8][16]; >> __u16 fcw; >> @@ -761,12 +775,22 @@ Reads the floating point state from the vcpu. >> __u32 pad2; >> }; >> >> + /* LoongArch: for KVM_GET_FPU and KVM_SET_FPU */ >> + struct kvm_fpu { >> + __u32 fcsr; >> + __u32 none; > Maybe use pad1 as x86 is better. > > Huacai Thanks, this 'none' variable is meaningless, and I will remove it. As this have already been removed in the actually kvm_fpu structure. Tianrui Zhao >> + __u64 fcc; >> + struct kvm_fpureg { >> + __u64 val64[4]; >> + }fpr[32]; >> + }; >> + >> >> 4.23 KVM_SET_FPU >> ---------------- >> >> :Capability: basic >> -:Architectures: x86 >> +:Architectures: x86, loongarch >> :Type: vcpu ioctl >> :Parameters: struct kvm_fpu (in) >> :Returns: 0 on success, -1 on error >> @@ -775,7 +799,7 @@ Writes the floating point state to the vcpu. >> >> :: >> >> - /* for KVM_GET_FPU and KVM_SET_FPU */ >> + /* x86: for KVM_GET_FPU and KVM_SET_FPU */ >> struct kvm_fpu { >> __u8 fpr[8][16]; >> __u16 fcw; >> @@ -790,6 +814,16 @@ Writes the floating point state to the vcpu. >> __u32 pad2; >> }; >> >> + /* LoongArch: for KVM_GET_FPU and KVM_SET_FPU */ >> + struct kvm_fpu { >> + __u32 fcsr; >> + __u32 none; I will also remove this 'none' variable. Tianrui Zhao >> + __u64 fcc; >> + struct kvm_fpureg { >> + __u64 val64[4]; >> + }fpr[32]; >> + }; >> + >> >> 4.24 KVM_CREATE_IRQCHIP >> ----------------------- >> @@ -1387,7 +1421,7 @@ documentation when it pops into existence). >> ------------------- >> >> :Capability: KVM_CAP_ENABLE_CAP >> -:Architectures: mips, ppc, s390, x86 >> +:Architectures: mips, ppc, s390, x86, loongarch >> :Type: vcpu ioctl >> :Parameters: struct kvm_enable_cap (in) >> :Returns: 0 on success; -1 on error >> @@ -1442,7 +1476,7 @@ for vm-wide capabilities. >> --------------------- >> >> :Capability: KVM_CAP_MP_STATE >> -:Architectures: x86, s390, arm64, riscv >> +:Architectures: x86, s390, arm64, riscv, loongarch >> :Type: vcpu ioctl >> :Parameters: struct kvm_mp_state (out) >> :Returns: 0 on success; -1 on error >> @@ -1460,7 +1494,7 @@ Possible values are: >> >> ========================== =============================================== >> KVM_MP_STATE_RUNNABLE the vcpu is currently running >> - [x86,arm64,riscv] >> + [x86,arm64,riscv,loongarch] >> KVM_MP_STATE_UNINITIALIZED the vcpu is an application processor (AP) >> which has not yet received an INIT signal [x86] >> KVM_MP_STATE_INIT_RECEIVED the vcpu has received an INIT signal, and is >> @@ -1516,11 +1550,14 @@ For riscv: >> The only states that are valid are KVM_MP_STATE_STOPPED and >> KVM_MP_STATE_RUNNABLE which reflect if the vcpu is paused or not. >> >> +On LoongArch, only the KVM_MP_STATE_RUNNABLE state is used to reflect >> +whether the vcpu is runnable. >> + >> 4.39 KVM_SET_MP_STATE >> --------------------- >> >> :Capability: KVM_CAP_MP_STATE >> -:Architectures: x86, s390, arm64, riscv >> +:Architectures: x86, s390, arm64, riscv, loongarch >> :Type: vcpu ioctl >> :Parameters: struct kvm_mp_state (in) >> :Returns: 0 on success; -1 on error >> @@ -1538,6 +1575,9 @@ For arm64/riscv: >> The only states that are valid are KVM_MP_STATE_STOPPED and >> KVM_MP_STATE_RUNNABLE which reflect if the vcpu should be paused or not. >> >> +On LoongArch, only the KVM_MP_STATE_RUNNABLE state is used to reflect >> +whether the vcpu is runnable. >> + >> 4.40 KVM_SET_IDENTITY_MAP_ADDR >> ------------------------------ >> >> @@ -2839,6 +2879,19 @@ Following are the RISC-V D-extension registers: >> 0x8020 0000 0600 0020 fcsr Floating point control and status register >> ======================= ========= ============================================= >> >> +LoongArch registers are mapped using the lower 32 bits. The upper 16 bits of >> +that is the register group type. >> + >> +LoongArch csr registers are used to control guest cpu or get status of guest >> +cpu, and they have the following id bit patterns:: >> + >> + 0x9030 0000 0001 00 <reg:5> <sel:3> (64-bit) >> + >> +LoongArch KVM control registers are used to implement some new defined functions >> +such as set vcpu counter or reset vcpu, and they have the following id bit patterns:: >> + >> + 0x9030 0000 0002 <reg:16> >> + >> >> 4.69 KVM_GET_ONE_REG >> -------------------- >> -- >> 2.39.1 >> >>
On Fri, Jun 16, 2023 at 10:39 AM zhaotianrui <zhaotianrui@loongson.cn> wrote: > > > 在 2023/6/15 下午5:42, Huacai Chen 写道: > > Hi, Tianrui, > > > > On Fri, Jun 9, 2023 at 5:09 PM Tianrui Zhao <zhaotianrui@loongson.cn> wrote: > >> Supplement kvm document about LoongArch-specific part, such as add > >> api introduction for GET/SET_ONE_REG, GET/SET_FPU, GET/SET_MP_STATE, > >> etc. > >> > >> Signed-off-by: Tianrui Zhao <zhaotianrui@loongson.cn> > >> --- > >> Documentation/virt/kvm/api.rst | 71 +++++++++++++++++++++++++++++----- > >> 1 file changed, 62 insertions(+), 9 deletions(-) > >> > >> diff --git a/Documentation/virt/kvm/api.rst b/Documentation/virt/kvm/api.rst > >> index add067793b90..ad8e13eab48d 100644 > >> --- a/Documentation/virt/kvm/api.rst > >> +++ b/Documentation/virt/kvm/api.rst > >> @@ -416,6 +416,12 @@ Reads the general purpose registers from the vcpu. > >> __u64 pc; > >> }; > >> > >> + /* LoongArch */ > >> + struct kvm_regs { > > Add a " /* out (KVM_GET_REGS) / in (KVM_SET_REGS) */" line as others here? > > Thanks, I will add this comment here. > > Tianrui Zhao > > >> + unsigned long gpr[32]; > >> + unsigned long pc; > >> + }; > >> + > >> > >> 4.12 KVM_SET_REGS > >> ----------------- > >> @@ -506,7 +512,7 @@ translation mode. > >> ------------------ > >> > >> :Capability: basic > >> -:Architectures: x86, ppc, mips, riscv > >> +:Architectures: x86, ppc, mips, riscv, loongarch > >> :Type: vcpu ioctl > >> :Parameters: struct kvm_interrupt (in) > >> :Returns: 0 on success, negative on failure. > >> @@ -592,6 +598,14 @@ b) KVM_INTERRUPT_UNSET > >> > >> This is an asynchronous vcpu ioctl and can be invoked from any thread. > >> > >> +LOONGARCH: > >> +^^^^^^^^^^ > >> + > >> +Queues an external interrupt to be injected into the virtual CPU. A negative > >> +interrupt number dequeues the interrupt. > >> + > >> +This is an asynchronous vcpu ioctl and can be invoked from any thread. > >> + > >> > >> 4.17 KVM_DEBUG_GUEST > >> -------------------- > >> @@ -737,7 +751,7 @@ signal mask. > >> ---------------- > >> > >> :Capability: basic > >> -:Architectures: x86 > >> +:Architectures: x86, loongarch > >> :Type: vcpu ioctl > >> :Parameters: struct kvm_fpu (out) > >> :Returns: 0 on success, -1 on error > >> @@ -746,7 +760,7 @@ Reads the floating point state from the vcpu. > >> > >> :: > >> > >> - /* for KVM_GET_FPU and KVM_SET_FPU */ > >> + /* x86: for KVM_GET_FPU and KVM_SET_FPU */ > >> struct kvm_fpu { > >> __u8 fpr[8][16]; > >> __u16 fcw; > >> @@ -761,12 +775,22 @@ Reads the floating point state from the vcpu. > >> __u32 pad2; > >> }; > >> > >> + /* LoongArch: for KVM_GET_FPU and KVM_SET_FPU */ > >> + struct kvm_fpu { > >> + __u32 fcsr; > >> + __u32 none; > > Maybe use pad1 as x86 is better. > > > > Huacai > > Thanks, this 'none' variable is meaningless, and I will remove it. As > this have already been removed in the actually kvm_fpu structure. Why remove? I think it is for alignment, just like pad1 for x86. Huacai > > Tianrui Zhao > > >> + __u64 fcc; > >> + struct kvm_fpureg { > >> + __u64 val64[4]; > >> + }fpr[32]; > >> + }; > >> + > >> > >> 4.23 KVM_SET_FPU > >> ---------------- > >> > >> :Capability: basic > >> -:Architectures: x86 > >> +:Architectures: x86, loongarch > >> :Type: vcpu ioctl > >> :Parameters: struct kvm_fpu (in) > >> :Returns: 0 on success, -1 on error > >> @@ -775,7 +799,7 @@ Writes the floating point state to the vcpu. > >> > >> :: > >> > >> - /* for KVM_GET_FPU and KVM_SET_FPU */ > >> + /* x86: for KVM_GET_FPU and KVM_SET_FPU */ > >> struct kvm_fpu { > >> __u8 fpr[8][16]; > >> __u16 fcw; > >> @@ -790,6 +814,16 @@ Writes the floating point state to the vcpu. > >> __u32 pad2; > >> }; > >> > >> + /* LoongArch: for KVM_GET_FPU and KVM_SET_FPU */ > >> + struct kvm_fpu { > >> + __u32 fcsr; > >> + __u32 none; > > I will also remove this 'none' variable. > > Tianrui Zhao > > >> + __u64 fcc; > >> + struct kvm_fpureg { > >> + __u64 val64[4]; > >> + }fpr[32]; > >> + }; > >> + > >> > >> 4.24 KVM_CREATE_IRQCHIP > >> ----------------------- > >> @@ -1387,7 +1421,7 @@ documentation when it pops into existence). > >> ------------------- > >> > >> :Capability: KVM_CAP_ENABLE_CAP > >> -:Architectures: mips, ppc, s390, x86 > >> +:Architectures: mips, ppc, s390, x86, loongarch > >> :Type: vcpu ioctl > >> :Parameters: struct kvm_enable_cap (in) > >> :Returns: 0 on success; -1 on error > >> @@ -1442,7 +1476,7 @@ for vm-wide capabilities. > >> --------------------- > >> > >> :Capability: KVM_CAP_MP_STATE > >> -:Architectures: x86, s390, arm64, riscv > >> +:Architectures: x86, s390, arm64, riscv, loongarch > >> :Type: vcpu ioctl > >> :Parameters: struct kvm_mp_state (out) > >> :Returns: 0 on success; -1 on error > >> @@ -1460,7 +1494,7 @@ Possible values are: > >> > >> ========================== =============================================== > >> KVM_MP_STATE_RUNNABLE the vcpu is currently running > >> - [x86,arm64,riscv] > >> + [x86,arm64,riscv,loongarch] > >> KVM_MP_STATE_UNINITIALIZED the vcpu is an application processor (AP) > >> which has not yet received an INIT signal [x86] > >> KVM_MP_STATE_INIT_RECEIVED the vcpu has received an INIT signal, and is > >> @@ -1516,11 +1550,14 @@ For riscv: > >> The only states that are valid are KVM_MP_STATE_STOPPED and > >> KVM_MP_STATE_RUNNABLE which reflect if the vcpu is paused or not. > >> > >> +On LoongArch, only the KVM_MP_STATE_RUNNABLE state is used to reflect > >> +whether the vcpu is runnable. > >> + > >> 4.39 KVM_SET_MP_STATE > >> --------------------- > >> > >> :Capability: KVM_CAP_MP_STATE > >> -:Architectures: x86, s390, arm64, riscv > >> +:Architectures: x86, s390, arm64, riscv, loongarch > >> :Type: vcpu ioctl > >> :Parameters: struct kvm_mp_state (in) > >> :Returns: 0 on success; -1 on error > >> @@ -1538,6 +1575,9 @@ For arm64/riscv: > >> The only states that are valid are KVM_MP_STATE_STOPPED and > >> KVM_MP_STATE_RUNNABLE which reflect if the vcpu should be paused or not. > >> > >> +On LoongArch, only the KVM_MP_STATE_RUNNABLE state is used to reflect > >> +whether the vcpu is runnable. > >> + > >> 4.40 KVM_SET_IDENTITY_MAP_ADDR > >> ------------------------------ > >> > >> @@ -2839,6 +2879,19 @@ Following are the RISC-V D-extension registers: > >> 0x8020 0000 0600 0020 fcsr Floating point control and status register > >> ======================= ========= ============================================= > >> > >> +LoongArch registers are mapped using the lower 32 bits. The upper 16 bits of > >> +that is the register group type. > >> + > >> +LoongArch csr registers are used to control guest cpu or get status of guest > >> +cpu, and they have the following id bit patterns:: > >> + > >> + 0x9030 0000 0001 00 <reg:5> <sel:3> (64-bit) > >> + > >> +LoongArch KVM control registers are used to implement some new defined functions > >> +such as set vcpu counter or reset vcpu, and they have the following id bit patterns:: > >> + > >> + 0x9030 0000 0002 <reg:16> > >> + > >> > >> 4.69 KVM_GET_ONE_REG > >> -------------------- > >> -- > >> 2.39.1 > >> > >> >
在 2023/6/16 上午10:59, Huacai Chen 写道: > On Fri, Jun 16, 2023 at 10:39 AM zhaotianrui <zhaotianrui@loongson.cn> wrote: >> >> 在 2023/6/15 下午5:42, Huacai Chen 写道: >>> Hi, Tianrui, >>> >>> On Fri, Jun 9, 2023 at 5:09 PM Tianrui Zhao <zhaotianrui@loongson.cn> wrote: >>>> Supplement kvm document about LoongArch-specific part, such as add >>>> api introduction for GET/SET_ONE_REG, GET/SET_FPU, GET/SET_MP_STATE, >>>> etc. >>>> >>>> Signed-off-by: Tianrui Zhao <zhaotianrui@loongson.cn> >>>> --- >>>> Documentation/virt/kvm/api.rst | 71 +++++++++++++++++++++++++++++----- >>>> 1 file changed, 62 insertions(+), 9 deletions(-) >>>> >>>> diff --git a/Documentation/virt/kvm/api.rst b/Documentation/virt/kvm/api.rst >>>> index add067793b90..ad8e13eab48d 100644 >>>> --- a/Documentation/virt/kvm/api.rst >>>> +++ b/Documentation/virt/kvm/api.rst >>>> @@ -416,6 +416,12 @@ Reads the general purpose registers from the vcpu. >>>> __u64 pc; >>>> }; >>>> >>>> + /* LoongArch */ >>>> + struct kvm_regs { >>> Add a " /* out (KVM_GET_REGS) / in (KVM_SET_REGS) */" line as others here? >> Thanks, I will add this comment here. >> >> Tianrui Zhao >> >>>> + unsigned long gpr[32]; >>>> + unsigned long pc; >>>> + }; >>>> + >>>> >>>> 4.12 KVM_SET_REGS >>>> ----------------- >>>> @@ -506,7 +512,7 @@ translation mode. >>>> ------------------ >>>> >>>> :Capability: basic >>>> -:Architectures: x86, ppc, mips, riscv >>>> +:Architectures: x86, ppc, mips, riscv, loongarch >>>> :Type: vcpu ioctl >>>> :Parameters: struct kvm_interrupt (in) >>>> :Returns: 0 on success, negative on failure. >>>> @@ -592,6 +598,14 @@ b) KVM_INTERRUPT_UNSET >>>> >>>> This is an asynchronous vcpu ioctl and can be invoked from any thread. >>>> >>>> +LOONGARCH: >>>> +^^^^^^^^^^ >>>> + >>>> +Queues an external interrupt to be injected into the virtual CPU. A negative >>>> +interrupt number dequeues the interrupt. >>>> + >>>> +This is an asynchronous vcpu ioctl and can be invoked from any thread. >>>> + >>>> >>>> 4.17 KVM_DEBUG_GUEST >>>> -------------------- >>>> @@ -737,7 +751,7 @@ signal mask. >>>> ---------------- >>>> >>>> :Capability: basic >>>> -:Architectures: x86 >>>> +:Architectures: x86, loongarch >>>> :Type: vcpu ioctl >>>> :Parameters: struct kvm_fpu (out) >>>> :Returns: 0 on success, -1 on error >>>> @@ -746,7 +760,7 @@ Reads the floating point state from the vcpu. >>>> >>>> :: >>>> >>>> - /* for KVM_GET_FPU and KVM_SET_FPU */ >>>> + /* x86: for KVM_GET_FPU and KVM_SET_FPU */ >>>> struct kvm_fpu { >>>> __u8 fpr[8][16]; >>>> __u16 fcw; >>>> @@ -761,12 +775,22 @@ Reads the floating point state from the vcpu. >>>> __u32 pad2; >>>> }; >>>> >>>> + /* LoongArch: for KVM_GET_FPU and KVM_SET_FPU */ >>>> + struct kvm_fpu { >>>> + __u32 fcsr; >>>> + __u32 none; >>> Maybe use pad1 as x86 is better. >>> >>> Huacai >> Thanks, this 'none' variable is meaningless, and I will remove it. As >> this have already been removed in the actually kvm_fpu structure. > Why remove? I think it is for alignment, just like pad1 for x86. > > Huacai This 'none' variable is not for alignment but a unused register named vcsr in kvm_fpu, so I think it could be removed. Tianrui Zhao >> Tianrui Zhao >> >>>> + __u64 fcc; >>>> + struct kvm_fpureg { >>>> + __u64 val64[4]; >>>> + }fpr[32]; >>>> + }; >>>> + >>>> >>>> 4.23 KVM_SET_FPU >>>> ---------------- >>>> >>>> :Capability: basic >>>> -:Architectures: x86 >>>> +:Architectures: x86, loongarch >>>> :Type: vcpu ioctl >>>> :Parameters: struct kvm_fpu (in) >>>> :Returns: 0 on success, -1 on error >>>> @@ -775,7 +799,7 @@ Writes the floating point state to the vcpu. >>>> >>>> :: >>>> >>>> - /* for KVM_GET_FPU and KVM_SET_FPU */ >>>> + /* x86: for KVM_GET_FPU and KVM_SET_FPU */ >>>> struct kvm_fpu { >>>> __u8 fpr[8][16]; >>>> __u16 fcw; >>>> @@ -790,6 +814,16 @@ Writes the floating point state to the vcpu. >>>> __u32 pad2; >>>> }; >>>> >>>> + /* LoongArch: for KVM_GET_FPU and KVM_SET_FPU */ >>>> + struct kvm_fpu { >>>> + __u32 fcsr; >>>> + __u32 none; >> I will also remove this 'none' variable. >> >> Tianrui Zhao >> >>>> + __u64 fcc; >>>> + struct kvm_fpureg { >>>> + __u64 val64[4]; >>>> + }fpr[32]; >>>> + }; >>>> + >>>> >>>> 4.24 KVM_CREATE_IRQCHIP >>>> ----------------------- >>>> @@ -1387,7 +1421,7 @@ documentation when it pops into existence). >>>> ------------------- >>>> >>>> :Capability: KVM_CAP_ENABLE_CAP >>>> -:Architectures: mips, ppc, s390, x86 >>>> +:Architectures: mips, ppc, s390, x86, loongarch >>>> :Type: vcpu ioctl >>>> :Parameters: struct kvm_enable_cap (in) >>>> :Returns: 0 on success; -1 on error >>>> @@ -1442,7 +1476,7 @@ for vm-wide capabilities. >>>> --------------------- >>>> >>>> :Capability: KVM_CAP_MP_STATE >>>> -:Architectures: x86, s390, arm64, riscv >>>> +:Architectures: x86, s390, arm64, riscv, loongarch >>>> :Type: vcpu ioctl >>>> :Parameters: struct kvm_mp_state (out) >>>> :Returns: 0 on success; -1 on error >>>> @@ -1460,7 +1494,7 @@ Possible values are: >>>> >>>> ========================== =============================================== >>>> KVM_MP_STATE_RUNNABLE the vcpu is currently running >>>> - [x86,arm64,riscv] >>>> + [x86,arm64,riscv,loongarch] >>>> KVM_MP_STATE_UNINITIALIZED the vcpu is an application processor (AP) >>>> which has not yet received an INIT signal [x86] >>>> KVM_MP_STATE_INIT_RECEIVED the vcpu has received an INIT signal, and is >>>> @@ -1516,11 +1550,14 @@ For riscv: >>>> The only states that are valid are KVM_MP_STATE_STOPPED and >>>> KVM_MP_STATE_RUNNABLE which reflect if the vcpu is paused or not. >>>> >>>> +On LoongArch, only the KVM_MP_STATE_RUNNABLE state is used to reflect >>>> +whether the vcpu is runnable. >>>> + >>>> 4.39 KVM_SET_MP_STATE >>>> --------------------- >>>> >>>> :Capability: KVM_CAP_MP_STATE >>>> -:Architectures: x86, s390, arm64, riscv >>>> +:Architectures: x86, s390, arm64, riscv, loongarch >>>> :Type: vcpu ioctl >>>> :Parameters: struct kvm_mp_state (in) >>>> :Returns: 0 on success; -1 on error >>>> @@ -1538,6 +1575,9 @@ For arm64/riscv: >>>> The only states that are valid are KVM_MP_STATE_STOPPED and >>>> KVM_MP_STATE_RUNNABLE which reflect if the vcpu should be paused or not. >>>> >>>> +On LoongArch, only the KVM_MP_STATE_RUNNABLE state is used to reflect >>>> +whether the vcpu is runnable. >>>> + >>>> 4.40 KVM_SET_IDENTITY_MAP_ADDR >>>> ------------------------------ >>>> >>>> @@ -2839,6 +2879,19 @@ Following are the RISC-V D-extension registers: >>>> 0x8020 0000 0600 0020 fcsr Floating point control and status register >>>> ======================= ========= ============================================= >>>> >>>> +LoongArch registers are mapped using the lower 32 bits. The upper 16 bits of >>>> +that is the register group type. >>>> + >>>> +LoongArch csr registers are used to control guest cpu or get status of guest >>>> +cpu, and they have the following id bit patterns:: >>>> + >>>> + 0x9030 0000 0001 00 <reg:5> <sel:3> (64-bit) >>>> + >>>> +LoongArch KVM control registers are used to implement some new defined functions >>>> +such as set vcpu counter or reset vcpu, and they have the following id bit patterns:: >>>> + >>>> + 0x9030 0000 0002 <reg:16> >>>> + >>>> >>>> 4.69 KVM_GET_ONE_REG >>>> -------------------- >>>> -- >>>> 2.39.1 >>>> >>>>
diff --git a/Documentation/virt/kvm/api.rst b/Documentation/virt/kvm/api.rst index add067793b90..ad8e13eab48d 100644 --- a/Documentation/virt/kvm/api.rst +++ b/Documentation/virt/kvm/api.rst @@ -416,6 +416,12 @@ Reads the general purpose registers from the vcpu. __u64 pc; }; + /* LoongArch */ + struct kvm_regs { + unsigned long gpr[32]; + unsigned long pc; + }; + 4.12 KVM_SET_REGS ----------------- @@ -506,7 +512,7 @@ translation mode. ------------------ :Capability: basic -:Architectures: x86, ppc, mips, riscv +:Architectures: x86, ppc, mips, riscv, loongarch :Type: vcpu ioctl :Parameters: struct kvm_interrupt (in) :Returns: 0 on success, negative on failure. @@ -592,6 +598,14 @@ b) KVM_INTERRUPT_UNSET This is an asynchronous vcpu ioctl and can be invoked from any thread. +LOONGARCH: +^^^^^^^^^^ + +Queues an external interrupt to be injected into the virtual CPU. A negative +interrupt number dequeues the interrupt. + +This is an asynchronous vcpu ioctl and can be invoked from any thread. + 4.17 KVM_DEBUG_GUEST -------------------- @@ -737,7 +751,7 @@ signal mask. ---------------- :Capability: basic -:Architectures: x86 +:Architectures: x86, loongarch :Type: vcpu ioctl :Parameters: struct kvm_fpu (out) :Returns: 0 on success, -1 on error @@ -746,7 +760,7 @@ Reads the floating point state from the vcpu. :: - /* for KVM_GET_FPU and KVM_SET_FPU */ + /* x86: for KVM_GET_FPU and KVM_SET_FPU */ struct kvm_fpu { __u8 fpr[8][16]; __u16 fcw; @@ -761,12 +775,22 @@ Reads the floating point state from the vcpu. __u32 pad2; }; + /* LoongArch: for KVM_GET_FPU and KVM_SET_FPU */ + struct kvm_fpu { + __u32 fcsr; + __u32 none; + __u64 fcc; + struct kvm_fpureg { + __u64 val64[4]; + }fpr[32]; + }; + 4.23 KVM_SET_FPU ---------------- :Capability: basic -:Architectures: x86 +:Architectures: x86, loongarch :Type: vcpu ioctl :Parameters: struct kvm_fpu (in) :Returns: 0 on success, -1 on error @@ -775,7 +799,7 @@ Writes the floating point state to the vcpu. :: - /* for KVM_GET_FPU and KVM_SET_FPU */ + /* x86: for KVM_GET_FPU and KVM_SET_FPU */ struct kvm_fpu { __u8 fpr[8][16]; __u16 fcw; @@ -790,6 +814,16 @@ Writes the floating point state to the vcpu. __u32 pad2; }; + /* LoongArch: for KVM_GET_FPU and KVM_SET_FPU */ + struct kvm_fpu { + __u32 fcsr; + __u32 none; + __u64 fcc; + struct kvm_fpureg { + __u64 val64[4]; + }fpr[32]; + }; + 4.24 KVM_CREATE_IRQCHIP ----------------------- @@ -1387,7 +1421,7 @@ documentation when it pops into existence). ------------------- :Capability: KVM_CAP_ENABLE_CAP -:Architectures: mips, ppc, s390, x86 +:Architectures: mips, ppc, s390, x86, loongarch :Type: vcpu ioctl :Parameters: struct kvm_enable_cap (in) :Returns: 0 on success; -1 on error @@ -1442,7 +1476,7 @@ for vm-wide capabilities. --------------------- :Capability: KVM_CAP_MP_STATE -:Architectures: x86, s390, arm64, riscv +:Architectures: x86, s390, arm64, riscv, loongarch :Type: vcpu ioctl :Parameters: struct kvm_mp_state (out) :Returns: 0 on success; -1 on error @@ -1460,7 +1494,7 @@ Possible values are: ========================== =============================================== KVM_MP_STATE_RUNNABLE the vcpu is currently running - [x86,arm64,riscv] + [x86,arm64,riscv,loongarch] KVM_MP_STATE_UNINITIALIZED the vcpu is an application processor (AP) which has not yet received an INIT signal [x86] KVM_MP_STATE_INIT_RECEIVED the vcpu has received an INIT signal, and is @@ -1516,11 +1550,14 @@ For riscv: The only states that are valid are KVM_MP_STATE_STOPPED and KVM_MP_STATE_RUNNABLE which reflect if the vcpu is paused or not. +On LoongArch, only the KVM_MP_STATE_RUNNABLE state is used to reflect +whether the vcpu is runnable. + 4.39 KVM_SET_MP_STATE --------------------- :Capability: KVM_CAP_MP_STATE -:Architectures: x86, s390, arm64, riscv +:Architectures: x86, s390, arm64, riscv, loongarch :Type: vcpu ioctl :Parameters: struct kvm_mp_state (in) :Returns: 0 on success; -1 on error @@ -1538,6 +1575,9 @@ For arm64/riscv: The only states that are valid are KVM_MP_STATE_STOPPED and KVM_MP_STATE_RUNNABLE which reflect if the vcpu should be paused or not. +On LoongArch, only the KVM_MP_STATE_RUNNABLE state is used to reflect +whether the vcpu is runnable. + 4.40 KVM_SET_IDENTITY_MAP_ADDR ------------------------------ @@ -2839,6 +2879,19 @@ Following are the RISC-V D-extension registers: 0x8020 0000 0600 0020 fcsr Floating point control and status register ======================= ========= ============================================= +LoongArch registers are mapped using the lower 32 bits. The upper 16 bits of +that is the register group type. + +LoongArch csr registers are used to control guest cpu or get status of guest +cpu, and they have the following id bit patterns:: + + 0x9030 0000 0001 00 <reg:5> <sel:3> (64-bit) + +LoongArch KVM control registers are used to implement some new defined functions +such as set vcpu counter or reset vcpu, and they have the following id bit patterns:: + + 0x9030 0000 0002 <reg:16> + 4.69 KVM_GET_ONE_REG --------------------