Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1638149

Re: [PATCH v3 3/3] arm/arm64: signal SIBGUS and inject SEA Error

From James Morse <james.morse@arm.com>
Newsgroups linux.kernel
Subject Re: [PATCH v3 3/3] arm/arm64: signal SIBGUS and inject SEA Error
Date 2017-05-09 16:30 +0200
Message-ID <tFejf-2TS-1@gated-at.bofh.it> (permalink)
References <tDsgF-6aT-15@gated-at.bofh.it> <tEUNz-6V3-1@gated-at.bofh.it> <tEV6V-71B-1@gated-at.bofh.it>
Organization linux.* mail to news gateway

Show all headers | View raw


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:

> However, if it's a question about setting up VCPU registers to a certain
> state and potentially modifying memory, then I think experience has
> shown us (psci) that emulating something in the kernel that userspace
> can have fine-grained control over is a bad idea, and should be left to
> userspace using as generic APIs as possible.
> 
> Furthermore, if I understand what injecting a SEA requires, it is very
> similar to resetting the CPU and loading data into guest memory, which
> QEMU already does today, and there is no reason to introduce additional
> APIs if it can be done using KVM_GET/SET_ONE_REG ioctls.


Thanks,

James

Back to linux.kernel | Previous | NextPrevious in thread | Next in thread | Find similar | Unroll thread


Thread

Re: [PATCH v3 3/3] arm/arm64: signal SIBGUS and inject SEA Error gengdongjiu <gengdj.1984@gmail.com> - 2017-05-04 19:00 +0200
  Re: [PATCH v3 3/3] arm/arm64: signal SIBGUS and inject SEA Error gengdongjiu <gengdj.1984@gmail.com> - 2017-05-05 14:40 +0200
  Re: [PATCH v3 3/3] arm/arm64: signal SIBGUS and inject SEA Error James Morse <james.morse@arm.com> - 2017-05-08 19:40 +0200
    Re: [PATCH v3 3/3] arm/arm64: signal SIBGUS and inject SEA Error Christoffer Dall <cdall@linaro.org> - 2017-05-08 20:00 +0200
      Re: [PATCH v3 3/3] arm/arm64: signal SIBGUS and inject SEA Error James Morse <james.morse@arm.com> - 2017-05-09 16:30 +0200
        Re: [PATCH v3 3/3] arm/arm64: signal SIBGUS and inject SEA Error gengdongjiu <gengdongjiu@huawei.com> - 2017-05-10 11:30 +0200
          Re: [PATCH v3 3/3] arm/arm64: signal SIBGUS and inject SEA Error Christoffer Dall <cdall@linaro.org> - 2017-05-10 14:30 +0200
            Re: [PATCH v3 3/3] arm/arm64: signal SIBGUS and inject SEA Error gengdongjiu <gengdongjiu@huawei.com> - 2017-05-10 14:50 +0200
    Re: [PATCH v3 3/3] arm/arm64: signal SIBGUS and inject SEA Error gengdongjiu <gengdongjiu@huawei.com> - 2017-05-10 10:50 +0200

csiph-web