qemu-devel.nongnu.org archive mirror
 help / color / mirror / Atom feed
From: Laszlo Ersek <lersek@redhat.com>
To: James Morse <james.morse@arm.com>, gengdongjiu <gengdongjiu@huawei.com>
Cc: Igor Mammedov <imammedo@redhat.com>,
	"Michael S. Tsirkin" <mst@redhat.com>,
	Wuquanming <wuquanming@huawei.com>,
	QEMU Developers <qemu-devel@nongnu.org>,
	Huangshaoyu <huangshaoyu@huawei.com>,
	Andrew Jones <drjones@redhat.com>
Subject: Re: [Qemu-devel] 答复:  using which notification for guest about GHES error
Date: Mon, 16 Oct 2017 19:25:18 +0200	[thread overview]
Message-ID: <69f10790-f178-0383-3f74-0c7e5d0afce8@redhat.com> (raw)
In-Reply-To: <59E4E57D.4040304@arm.com>

On 10/16/17 18:59, James Morse wrote:
> Hi gengdongjiu, Igor,
> 
> On 16/10/17 15:33, gengdongjiu wrote:
>>> On Mon, 16 Oct 2017 14:10:05 +0800
>>> gengdongjiu <gengdongjiu@huawei.com> wrote:
>>>> Now we use Qemu to create APEI table and record CPER for guest, After
>>>> QEMU recorded a asynchronous CPER error, we needs to notify guest using interrupt or Polled notification.
>>>> For the asynchronous error. I think using GPIO-signaled notification may be better in the Qemu, and also which is suggested by APEI spec.
>>>> James worried that old guest OS may not support GPIO or GSIV notification for GHES, because GPIO or GSIV notification is supported in OS
>>> since about kernel version 4.10.
>>>
>>> How APEI support is fairly new on ARM (kernel), isn't it still in state of development?
> 
> The NMI-like notifications, (SEA, SEI, SDEI) are still being worked on, but the
> less exotic Polled and many-flavours-of-interrupt should have exactly the same
> meaning/behaviour as on x86. (it should be possible to emulate/configure these
> with common user-space code too)
> 
> 
>>> Do we really care about old guests in this case?
> 
> I think the scenario here is the host kernel has some RAS support, Qemu has RAS
> support and has advertised its CPER regions via the HEST, but the guest doesn't
> doesn't support RAS. (booted via DT, wasn't configured for APEI, the kernel
> pre-dates the support etc).
> 
> What should Qemu do in response to 'action optional' memory errors?
> 
> My suggestion is whatever action Qemu takes, it shouldn't kill a guest that
> doesn't support RAS. Using NOTIFY_SEI for an action-optional memory error will
> do this. A guest that doesn't know about NOTIFY_SEI will take this as a fatal
> SError.

As far as I understand, this QEMU feature is not enabled by default; it
takes at least one specific -device option (maybe some other options
too). If that's the case, then adding the option, and then seeing a
guest (without supporting the feature) hang itself, is okay.

It is no different from the guest OS lacking drivers for other important
-- such as boot -- devices, such as virtio drivers in general, or
virtio-1.0 drivers for modern-only virtio devices.

Such knowledge (= what OS has built-in drivers or other kind of support
for what devices) is maintained in libosinfo.

> I think you shouldn't expect host kernel, guest kernel and Qemu version to all
> pair up nicely.

If I understand correctly, the things that are expected to match up are:
- guest kernel (more generally, guest OS image),
- and QEMU command line (more generally, virtual hardware
  configuration).


Just my two cents anyway; I haven't been following this discussion closely.

Thanks
Laszlo

  reply	other threads:[~2017-10-16 17:25 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <8ba7e693-4e32-7873-70bf-4efbf57f9cf5@huawei.com>
     [not found] ` <20170928011407-mutt-send-email-mst@kernel.org>
     [not found]   ` <405bbc99-1a43-aa8b-37e9-9599480f4c06@huawei.com>
     [not found]     ` <20171001063010-mutt-send-email-mst@kernel.org>
2017-10-16  6:10       ` [Qemu-devel] using which notification for guest about GHES error gengdongjiu
2017-10-16  9:33         ` Igor Mammedov
2017-10-16 14:33           ` [Qemu-devel] 答复: " gengdongjiu
2017-10-16 16:59             ` James Morse
2017-10-16 17:25               ` Laszlo Ersek [this message]
2017-10-17  1:31               ` [Qemu-devel] " 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=69f10790-f178-0383-3f74-0c7e5d0afce8@redhat.com \
    --to=lersek@redhat.com \
    --cc=drjones@redhat.com \
    --cc=gengdongjiu@huawei.com \
    --cc=huangshaoyu@huawei.com \
    --cc=imammedo@redhat.com \
    --cc=james.morse@arm.com \
    --cc=mst@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=wuquanming@huawei.com \
    /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).