From: Christoffer Dall <cdall@linaro.org>
To: gengdongjiu <gengdongjiu@huawei.com>
Cc: James Morse <james.morse@arm.com>,
gengdongjiu <gengdj.1984@gmail.com>,
Tyler Baicar <tbaicar@codeaurora.org>,
Christoffer Dall <christoffer.dall@linaro.org>,
Marc Zyngier <marc.zyngier@arm.com>,
pbonzini@redhat.com, rkrcmar@redhat.com, linux@armlinux.org.uk,
catalin.marinas@arm.com, will.deacon@arm.com, rjw@rjwysocki.net,
Len Brown <lenb@kernel.org>,
matt@codeblueprint.co.uk, robert.moore@intel.com,
lv.zheng@intel.com, nkaje@codeaurora.org, zjzhang@codeaurora.org,
mark.rutland@arm.com, akpm@linux-foundation.org,
eun.taik.lee@samsung.com,
Sandeepa Prabhu <sandeepa.s.prabhu@gmail.com>,
labbott@redhat.com, shijie.huang@arm.com,
rruigrok@codeaurora.org, paul.gortmaker@windriver.com,
tn@semihalf.com, Fu Wei <fu.wei@linaro.org>,
rostedt@goodmis.org, bristot@redhat.com,
linux-arm-kernel@lists.infradead.org,
kvmarm@lists.cs.columbia.edu, kvm@vger.kernel.org,
linux-kernel@vger.kernel.org, linux-acpi@vger.kernel.org,
linux-efi@vger.kernel.org, devel@acpica.org,
Suzuki.Poulose@arm.com, Punit Agrawal <punit.agrawal@arm.com>,
astone@redhat.com, harba@codeaurora.org,
Hanjun Guo <hanjun.guo@linaro.org>,
John Garry <john.garry@huawei.com>,
Shiju Jose <shiju.jose@huawei.com>,
joe@perches.com, Xiongfeng Wang <wangxiongfeng2@huawei.com>
Subject: Re: [PATCH v3 3/3] arm/arm64: signal SIBGUS and inject SEA Error
Date: Wed, 10 May 2017 14:20:55 +0200 [thread overview]
Message-ID: <20170510122055.GE28721@cbox> (raw)
In-Reply-To: <e8614b38-9c17-bcfc-2239-307ed0f98c94@huawei.com>
On Wed, May 10, 2017 at 05:15:04PM +0800, gengdongjiu wrote:
> Thanks James's explanation.
>
> Hi Christoffer,
>
> On 2017/5/9 22:28, James Morse wrote:
> > Hi Christoffer,
> >
> > On 08/05/17 18:54, Christoffer Dall wrote:
> >> On Mon, May 08, 2017 at 06:28:02PM +0100, James Morse wrote:
> >> I must admit I am losing track of exactly what this proposed API was
> >> supposed to do.
> >
> > There are two, and we keep jumping between them!
> > This is about two notification methods APEI has for arm64, 'SEA' and 'SEI'.
> >
> > SEA is synchronous and looks like a data abort. Qemu/kvmtool can inject these
> > today using the KVM_GET/SET_ONE_REG API whenever it wants to.
> >
> > SEI uses SError, is asynchronous and can be masked. In addition these need to be
> > consumed/synchronised by the ESB instruction, even when executed by a guest.
> > Hardware has the necessary bits to drive all this, we need to expose an API to
> > drive it.
> >
> > (I try to spell them out each time so I don't confuse SEI with something
> > synchronous!)
> >
> >
> > This patch was about SEA. I think you've answered our question:
>
> we are talking about the SEA(synchronous data abort) injection two methods:
>
> (1)change vcpu registers in the Qemu/kvmtools and using the KVM_GET/SET_ONE_REG API to set.
Yes, if this is possible, why would you want something more?
> (2)using existed in-kernel API "kvm_inject_dabt" to inject through IOCTL command from Qemu.
>
I'm not really going to consider this, because "use internal API from
userspace" doesn't work.
So this should be:
(2) Introduce a new API to do X.
I still think you know what my preference is; use the existing API if at
all possible.
Thanks,
-Christoffer
next prev parent reply other threads:[~2017-05-10 12:21 UTC|newest]
Thread overview: 13+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-05-04 16:52 [PATCH v3 3/3] arm/arm64: signal SIBGUS and inject SEA Error gengdongjiu
2017-05-05 12:31 ` gengdongjiu
2017-05-12 17:24 ` James Morse
2017-05-21 8:24 ` gengdongjiu
2017-05-08 17:28 ` James Morse
2017-05-08 17:54 ` Christoffer Dall
2017-05-09 14:28 ` James Morse
2017-05-10 9:15 ` gengdongjiu
2017-05-10 12:20 ` Christoffer Dall [this message]
2017-05-10 12:37 ` gengdongjiu
2017-05-10 8:44 ` gengdongjiu
2017-05-12 17:25 ` James Morse
2017-05-21 9:23 ` gengdongjiu
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=20170510122055.GE28721@cbox \
--to=cdall@linaro.org \
--cc=Suzuki.Poulose@arm.com \
--cc=akpm@linux-foundation.org \
--cc=astone@redhat.com \
--cc=bristot@redhat.com \
--cc=catalin.marinas@arm.com \
--cc=christoffer.dall@linaro.org \
--cc=devel@acpica.org \
--cc=eun.taik.lee@samsung.com \
--cc=fu.wei@linaro.org \
--cc=gengdj.1984@gmail.com \
--cc=gengdongjiu@huawei.com \
--cc=hanjun.guo@linaro.org \
--cc=harba@codeaurora.org \
--cc=james.morse@arm.com \
--cc=joe@perches.com \
--cc=john.garry@huawei.com \
--cc=kvm@vger.kernel.org \
--cc=kvmarm@lists.cs.columbia.edu \
--cc=labbott@redhat.com \
--cc=lenb@kernel.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-efi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux@armlinux.org.uk \
--cc=lv.zheng@intel.com \
--cc=marc.zyngier@arm.com \
--cc=mark.rutland@arm.com \
--cc=matt@codeblueprint.co.uk \
--cc=nkaje@codeaurora.org \
--cc=paul.gortmaker@windriver.com \
--cc=pbonzini@redhat.com \
--cc=punit.agrawal@arm.com \
--cc=rjw@rjwysocki.net \
--cc=rkrcmar@redhat.com \
--cc=robert.moore@intel.com \
--cc=rostedt@goodmis.org \
--cc=rruigrok@codeaurora.org \
--cc=sandeepa.s.prabhu@gmail.com \
--cc=shijie.huang@arm.com \
--cc=shiju.jose@huawei.com \
--cc=tbaicar@codeaurora.org \
--cc=tn@semihalf.com \
--cc=wangxiongfeng2@huawei.com \
--cc=will.deacon@arm.com \
--cc=zjzhang@codeaurora.org \
/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).