From: "Heiko Stübner" <heiko@sntech.de>
To: linux-kernel@vger.kernel.org, linux-riscv@lists.infradead.org
Cc: "Atish Patra" <atish.patra@wdc.com>,
"Anup Patel" <anup.patel@wdc.com>,
"Atish Patra" <atishp@rivosinc.com>,
"Albert Ou" <aou@eecs.berkeley.edu>,
"Heinrich Schuchardt" <xypron.glpk@gmx.de>,
"Kefeng Wang" <wangkefeng.wang@huawei.com>,
kvm-riscv@lists.infradead.org, kvm@vger.kernel.org,
linux-riscv@lists.infradead.org,
"Palmer Dabbelt" <palmer@dabbelt.com>,
"Paul Walmsley" <paul.walmsley@sifive.com>,
"Atish Patra" <atishp@rivosinc.com>,
"Heiko Stübner" <heiko@sntech.de>
Subject: Re: [PATCH v5 1/5] RISC-V: KVM: Mark the existing SBI implementation as v01
Date: Tue, 04 Jan 2022 14:27:14 +0100 [thread overview]
Message-ID: <8323751.1mJVJdxAKN@diego> (raw)
In-Reply-To: <6615284.qex3tTltCR@diego>
Am Dienstag, 4. Januar 2022, 14:19:41 CET schrieb Heiko Stübner:
> Hi Atish,
>
> Am Donnerstag, 18. November 2021, 09:39:08 CET schrieb Atish Patra:
> > From: Atish Patra <atish.patra@wdc.com>
> >
> > The existing SBI specification impelementation follows v0.1
> > specification. The latest specification allows more
> > scalability and performance improvements.
> >
> > Rename the existing implementation as v01 and provide a way to allow
> > future extensions.
> >
> > Reviewed-by: Anup Patel <anup.patel@wdc.com>
> > Signed-off-by: Atish Patra <atish.patra@wdc.com>
> > Signed-off-by: Atish Patra <atishp@rivosinc.com>
> > ---
>
> > diff --git a/arch/riscv/kvm/vcpu_sbi.c b/arch/riscv/kvm/vcpu_sbi.c
> > index eb3c045edf11..32376906ff20 100644
> > --- a/arch/riscv/kvm/vcpu_sbi.c
> > +++ b/arch/riscv/kvm/vcpu_sbi.c
> > @@ -1,5 +1,5 @@
> > // SPDX-License-Identifier: GPL-2.0
> > -/**
> > +/*
> > * Copyright (c) 2019 Western Digital Corporation or its affiliates.
> > *
> > * Authors:
>
> This got already fixed by [0]
> commit 0e2e64192100 ("riscv: kvm: fix non-kernel-doc comment block")
> so this patch doesn't apply cleanly anymore.
hmm, just found Anup's "I've queued this..." message after
writing my reply, so scratch the above ;-) .
@Anup: I've looked at git.kernel.org but didn't find a tree
there, can you tell me where this did go to?
Thanks
Heiko
next prev parent reply other threads:[~2022-01-04 13:27 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-11-18 8:39 [PATCH v5 0/5] Add SBI v0.2 support for KVM Atish Patra
2021-11-18 8:39 ` [PATCH v5 1/5] RISC-V: KVM: Mark the existing SBI implementation as v01 Atish Patra
2021-11-23 8:02 ` Anup Patel
2022-01-04 13:19 ` Heiko Stübner
2022-01-04 13:27 ` Heiko Stübner [this message]
2022-01-04 13:59 ` Anup Patel
2021-11-18 8:39 ` [PATCH v5 2/5] RISC-V: KVM: Reorganize SBI code by moving SBI v0.1 to its own file Atish Patra
2021-11-23 8:02 ` Anup Patel
2021-11-18 8:39 ` [PATCH v5 3/5] RISC-V: KVM: Add SBI v0.2 base extension Atish Patra
2021-11-23 8:03 ` Anup Patel
2021-11-18 8:39 ` [PATCH v5 4/5] RISC-V: KVM: Add v0.1 replacement SBI extensions defined in v02 Atish Patra
2021-11-23 8:03 ` Anup Patel
2021-11-18 8:39 ` [PATCH v5 5/5] RISC-V: KVM: Add SBI HSM extension in KVM Atish Patra
2021-11-23 8:06 ` Anup Patel
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=8323751.1mJVJdxAKN@diego \
--to=heiko@sntech.de \
--cc=anup.patel@wdc.com \
--cc=aou@eecs.berkeley.edu \
--cc=atish.patra@wdc.com \
--cc=atishp@rivosinc.com \
--cc=kvm-riscv@lists.infradead.org \
--cc=kvm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-riscv@lists.infradead.org \
--cc=palmer@dabbelt.com \
--cc=paul.walmsley@sifive.com \
--cc=wangkefeng.wang@huawei.com \
--cc=xypron.glpk@gmx.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).