Message ID | 20200410114639.32844-1-gengdongjiu@huawei.com (mailing list archive) |
---|---|
Headers | show |
Series | Add ARMv8 RAS virtualization support in QEMU | expand |
On Thu, 16 Apr 2020 at 14:54, gengdongjiu <gengdongjiu@huawei.com> wrote: > > ping.... Hi; this is on my to-review queue, but so are 25 other patchsets. (I've built up a bit of a backlog due to concentrating on work for the 5.0 release while we're in the freeze period.) I will get to it eventually if nobody else does first... thanks -- PMM
On Fri, 10 Apr 2020 at 12:46, Dongjiu Geng <gengdongjiu@huawei.com> wrote: > > In the ARMv8 platform, the CPU error types includes synchronous external abort(SEA) > and SError Interrupt (SEI). If exception happens in guest, host does not know the detailed > information of guest, so it is expected that guest can do the recovery. For example, if an > exception happens in a guest user-space application, host does not know which application > encounters errors, only guest knows it. > > 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 to 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. Hi. I left a comment on patch 1. The other 3 patches unreviewed are 5, 6 and 8, which are all ACPI core code, so that's for MST, Igor or Shannon to review. Once those have been reviewed, please ping me if you want this to go via target-arm.next. thanks -- PMM
On 2020/4/17 21:32, Peter Maydell wrote: > On Fri, 10 Apr 2020 at 12:46, Dongjiu Geng <gengdongjiu@huawei.com> wrote: >> >> In the ARMv8 platform, the CPU error types includes synchronous external abort(SEA) >> and SError Interrupt (SEI). If exception happens in guest, host does not know the detailed >> information of guest, so it is expected that guest can do the recovery. For example, if an >> exception happens in a guest user-space application, host does not know which application >> encounters errors, only guest knows it. >> >> 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 to 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. > > Hi. I left a comment on patch 1. The other 3 patches unreviewed > are 5, 6 and 8, which are all ACPI core code, so that's for > MST, Igor or Shannon to review. Hi MST, Igor or Shannon when you have time, could you review 5, 6 and 8? thanks very much in advance. It seems this series of patches lasted a long time, hope they can be applied soon. > > Once those have been reviewed, please ping me if you want this > to go via target-arm.next> > thanks > -- PMM > > . >
On 2020/4/17 21:32, Peter Maydell wrote: > On Fri, 10 Apr 2020 at 12:46, Dongjiu Geng <gengdongjiu@huawei.com> wrote: >> >> In the ARMv8 platform, the CPU error types includes synchronous external abort(SEA) >> and SError Interrupt (SEI). If exception happens in guest, host does not know the detailed >> information of guest, so it is expected that guest can do the recovery. For example, if an >> exception happens in a guest user-space application, host does not know which application >> encounters errors, only guest knows it. >> >> 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 to 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. > > Hi. I left a comment on patch 1. The other 3 patches unreviewed > are 5, 6 and 8, which are all ACPI core code, so that's for > MST, Igor or Shannon to review. Ping MST, Igor and Shannon, sorry for the noise. > > Once those have been reviewed, please ping me if you want this > to go via target-arm.next. > > thanks > -- PMM > > . >
On Thu, 30 Apr 2020 11:56:24 +0800 gengdongjiu <gengdongjiu@huawei.com> wrote: > On 2020/4/17 21:32, Peter Maydell wrote: > > On Fri, 10 Apr 2020 at 12:46, Dongjiu Geng <gengdongjiu@huawei.com> wrote: > >> > >> In the ARMv8 platform, the CPU error types includes synchronous external abort(SEA) > >> and SError Interrupt (SEI). If exception happens in guest, host does not know the detailed > >> information of guest, so it is expected that guest can do the recovery. For example, if an > >> exception happens in a guest user-space application, host does not know which application > >> encounters errors, only guest knows it. > >> > >> 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 to 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. > > > > Hi. I left a comment on patch 1. The other 3 patches unreviewed > > are 5, 6 and 8, which are all ACPI core code, so that's for > > MST, Igor or Shannon to review. > > Ping MST, Igor and Shannon, sorry for the noise. I put it on my review queue > > > > > Once those have been reviewed, please ping me if you want this > > to go via target-arm.next. > > > > thanks > > -- PMM > > > > . > > >
On 2020/4/17 21:32, Peter Maydell wrote: > On Fri, 10 Apr 2020 at 12:46, Dongjiu Geng <gengdongjiu@huawei.com> wrote: >> >> In the ARMv8 platform, the CPU error types includes synchronous external abort(SEA) >> and SError Interrupt (SEI). If exception happens in guest, host does not know the detailed >> information of guest, so it is expected that guest can do the recovery. For example, if an >> exception happens in a guest user-space application, host does not know which application >> encounters errors, only guest knows it. >> >> 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 to 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. > > Hi. I left a comment on patch 1. The other 3 patches unreviewed > are 5, 6 and 8, which are all ACPI core code, so that's for > MST, Igor or Shannon to review. > > Once those have been reviewed, please ping me if you want this > to go via target-arm.next. Hi Peter, Igor have reviewed all ACPI core code. whether you can apply this series to target-arm.next? I can make another patches to solve your comments on patch1 and another APCI comment. Thanks very much in advance. > > thanks > -- PMM > > . >
On Wed, May 06, 2020 at 07:42:19PM +0800, gengdongjiu wrote: > On 2020/4/17 21:32, Peter Maydell wrote: > > On Fri, 10 Apr 2020 at 12:46, Dongjiu Geng <gengdongjiu@huawei.com> wrote: > >> > >> In the ARMv8 platform, the CPU error types includes synchronous external abort(SEA) > >> and SError Interrupt (SEI). If exception happens in guest, host does not know the detailed > >> information of guest, so it is expected that guest can do the recovery. For example, if an > >> exception happens in a guest user-space application, host does not know which application > >> encounters errors, only guest knows it. > >> > >> 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 to 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. > > > > Hi. I left a comment on patch 1. The other 3 patches unreviewed > > are 5, 6 and 8, which are all ACPI core code, so that's for > > MST, Igor or Shannon to review. > > > > Once those have been reviewed, please ping me if you want this > > to go via target-arm.next. > > Hi Peter, > Igor have reviewed all ACPI core code. whether you can apply this series to target-arm.next? I can make another patches to solve your comments on patch1 and another APCI comment. > Thanks very much in advance. Given it all starts with patch 1, it's probably easier to address the comment and repost. > > > > thanks > > -- PMM > > > > . > >
On 2020/5/7 4:25, Michael S. Tsirkin wrote: > On Wed, May 06, 2020 at 07:42:19PM +0800, gengdongjiu wrote: >> On 2020/4/17 21:32, Peter Maydell wrote: >>> On Fri, 10 Apr 2020 at 12:46, Dongjiu Geng <gengdongjiu@huawei.com> wrote: >>>> >>>> In the ARMv8 platform, the CPU error types includes synchronous external abort(SEA) >>>> and SError Interrupt (SEI). If exception happens in guest, host does not know the detailed >>>> information of guest, so it is expected that guest can do the recovery. For example, if an >>>> exception happens in a guest user-space application, host does not know which application >>>> encounters errors, only guest knows it. >>>> >>>> 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 to 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. >>> >>> Hi. I left a comment on patch 1. The other 3 patches unreviewed >>> are 5, 6 and 8, which are all ACPI core code, so that's for >>> MST, Igor or Shannon to review. >>> >>> Once those have been reviewed, please ping me if you want this >>> to go via target-arm.next. >> >> Hi Peter, >> Igor have reviewed all ACPI core code. whether you can apply this series to target-arm.next I can make another patches to solve your comments on patch1 and another APCI comment. >> Thanks very much in advance. > > Given it all starts with patch 1, it's probably easier to address the > comment and repost. Ok, I will do it. thanks. > > >>> >>> thanks >>> -- PMM >>> >>> . >>> > > . >
On 2020/5/7 4:25, Michael S. Tsirkin wrote: > On Wed, May 06, 2020 at 07:42:19PM +0800, gengdongjiu wrote: >> On 2020/4/17 21:32, Peter Maydell wrote: >>> On Fri, 10 Apr 2020 at 12:46, Dongjiu Geng <gengdongjiu@huawei.com> wrote: >>>> >>>> In the ARMv8 platform, the CPU error types includes synchronous external abort(SEA) >>>> and SError Interrupt (SEI). If exception happens in guest, host does not know the detailed >>>> information of guest, so it is expected that guest can do the recovery. For example, if an >>>> exception happens in a guest user-space application, host does not know which application >>>> encounters errors, only guest knows it. >>>> >>>> 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 to 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. >>> >>> Hi. I left a comment on patch 1. The other 3 patches unreviewed >>> are 5, 6 and 8, which are all ACPI core code, so that's for >>> MST, Igor or Shannon to review. >>> >>> Once those have been reviewed, please ping me if you want this >>> to go via target-arm.next. >> >> Hi Peter, >> Igor have reviewed all ACPI core code. whether you can apply this series to target-arm.next I can make another patches to solve your comments on patch1 and another APCI comment. >> Thanks very much in advance. > > Given it all starts with patch 1, it's probably easier to address the > comment and repost. Done. Hi Peter, Please review the patch 1 in the patchset v26. Thanks. > > >>> >>> thanks >>> -- PMM >>> >>> . >>> > > . >