[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v20 0/5] Add ARMv8 RAS virtualization support in QEMU
From: |
Xiang Zheng |
Subject: |
Re: [PATCH v20 0/5] Add ARMv8 RAS virtualization support in QEMU |
Date: |
Mon, 28 Oct 2019 11:16:51 +0800 |
User-agent: |
Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:68.0) Gecko/20100101 Thunderbird/68.1.0 |
On 2019/10/27 18:17, Michael S. Tsirkin wrote:
> On Sat, Oct 26, 2019 at 11:24:42AM +0800, Xiang Zheng wrote:
>> In the ARMv8 platform, the CPU error types are synchronous external
>> abort(SEA)
>> and SError Interrupt (SEI). If exception happens in guest, sometimes it's
>> better
>> for guest to perform the recovery, because host does not know the detailed
>> information of guest. For example, if an exception happens in a user-space
>> application within guest, host does not know which application encounters
>> errors.
>>
>> For the ARMv8 SEA/SEI, KVM or host kernel delivers SIGBUS to notify
>> userspace.
>> After user space gets the notification, it will record the CPER into guest
>> GHES
>> buffer and inject an exception or IRQ into guest.
>>
>> In the current implementation, if the type of SIGBUS is BUS_MCEERR_AR, we
>> will
>> treat it as a synchronous exception, and notify guest with ARMv8 SEA
>> notification type after recording CPER into guest.
>>
>> This series of patches are based on Qemu 4.1, which include two parts:
>> 1. Generate APEI/GHES table.
>> 2. Handle the SIGBUS signal, record the CPER in runtime and fill it into
>> guest
>> memory, then notify guest according to the type of SIGBUS.
>>
>> The whole solution was suggested by James(address@hidden); The solution of
>> APEI section was suggested by Laszlo(address@hidden).
>> Show some discussions in [1].
>>
>> This series of patches have already been tested on ARM64 platform with RAS
>> feature enabled:
>> Show the APEI part verification result in [2].
>> Show the BUS_MCEERR_AR SIGBUS handling verification result in [3].
>
>
> This looks mostly OK to me. I sent some minor style comments but they
> can be addressed by follow up patches.
>
> Maybe it's a good idea to merge this before soft freeze to make sure it
> gets some testing. I'll leave this decision to the ARM maintainer. For
> ACPI parts:
>
> Reviewed-by: Michael S. Tsirkin <address@hidden>
>
Hi Peter,
I can address the style comments and send the series of patches before soft
freeze if needed. And if there is no window before soft freeze, I can also
address them by follow up patches. :)
--
Thanks,
Xiang
- [PATCH v20 0/5] Add ARMv8 RAS virtualization support in QEMU, Xiang Zheng, 2019/10/25
- [PATCH v20 4/5] KVM: Move hwpoison page related functions into kvm-all.c, Xiang Zheng, 2019/10/25
- [PATCH v20 1/5] hw/arm/virt: Introduce a RAS machine option, Xiang Zheng, 2019/10/25
- [PATCH v20 2/5] docs: APEI GHES generation and CPER record description, Xiang Zheng, 2019/10/25
- [PATCH v20 3/5] ACPI: Add APEI GHES table generation support, Xiang Zheng, 2019/10/25
- [PATCH v20 5/5] target-arm: kvm64: handle SIGBUS signal from kernel or KVM, Xiang Zheng, 2019/10/25
- Re: [PATCH v20 0/5] Add ARMv8 RAS virtualization support in QEMU, Michael S. Tsirkin, 2019/10/27
- Re: [PATCH v20 0/5] Add ARMv8 RAS virtualization support in QEMU,
Xiang Zheng <=
- Re: [PATCH v20 0/5] Add ARMv8 RAS virtualization support in QEMU, gengdongjiu, 2019/10/28
- Re: [PATCH v20 0/5] Add ARMv8 RAS virtualization support in QEMU, Michael S. Tsirkin, 2019/10/28
- Re: [PATCH v20 0/5] Add ARMv8 RAS virtualization support in QEMU, gengdongjiu, 2019/10/28
- Re: [PATCH v20 0/5] Add ARMv8 RAS virtualization support in QEMU, Michael S. Tsirkin, 2019/10/28
- Re: [PATCH v20 0/5] Add ARMv8 RAS virtualization support in QEMU, gengdongjiu, 2019/10/28
- Re: [PATCH v20 0/5] Add ARMv8 RAS virtualization support in QEMU, Peter Maydell, 2019/10/28
- Re: [PATCH v20 0/5] Add ARMv8 RAS virtualization support in QEMU, gengdongjiu, 2019/10/29
Re: [PATCH v20 0/5] Add ARMv8 RAS virtualization support in QEMU, Peter Maydell, 2019/10/28