Re: [PATCH V9 05/10] acpi: apei: handle SEA notification type for ARMv8

2017-02-15 Thread kbuild test robot
Hi Tyler, [auto build test WARNING on pm/linux-next] [also build test WARNING on v4.10-rc8 next-20170215] [if your patch is applied to the wrong git tree, please drop us a note to help improve the system] url: https://github.com/0day-ci/linux/commits/Tyler-Baicar/Add-UEFI-2-6-and-ACPI-6-1

Re: [PATCH V10 07/10] efi: print unrecognized CPER section

2017-02-15 Thread Baicar, Tyler
Hello Joe, On 2/15/2017 1:07 PM, Joe Perches wrote: On Wed, 2017-02-15 at 12:51 -0700, Tyler Baicar wrote: UEFI spec allows for non-standard section in Common Platform Error Record. This is defined in section N.2.3 of UEFI version 2.5. Currently if the CPER section's type (UUID) does not matc

Re: [PATCH V10 07/10] efi: print unrecognized CPER section

2017-02-15 Thread Joe Perches
On Wed, 2017-02-15 at 12:51 -0700, Tyler Baicar wrote: > UEFI spec allows for non-standard section in Common Platform Error > Record. This is defined in section N.2.3 of UEFI version 2.5. > > Currently if the CPER section's type (UUID) does not match with > one of the section types that the kernel

[PATCH V10 10/10] arm/arm64: KVM: add guest SEA support

2017-02-15 Thread Tyler Baicar
Currently external aborts are unsupported by the guest abort handling. Add handling for SEAs so that the host kernel reports SEAs which occur in the guest kernel. Signed-off-by: Tyler Baicar --- arch/arm/include/asm/kvm_arm.h | 1 + arch/arm/include/asm/system_misc.h | 5 + arch/ar

[PATCH V10 08/10] ras: acpi / apei: generate trace event for unrecognized CPER section

2017-02-15 Thread Tyler Baicar
UEFI spec allows for non-standard section in Common Platform Error Record. This is defined in section N.2.3 of UEFI version 2.5. Currently if the CPER section's type (UUID) does not match with any section type that the kernel knows how to parse, trace event is not generated for such section. And t

[PATCH V10 07/10] efi: print unrecognized CPER section

2017-02-15 Thread Tyler Baicar
UEFI spec allows for non-standard section in Common Platform Error Record. This is defined in section N.2.3 of UEFI version 2.5. Currently if the CPER section's type (UUID) does not match with one of the section types that the kernel knows how to parse, the section is skipped. Therefore, user is n

[PATCH V10 09/10] trace, ras: add ARM processor error trace event

2017-02-15 Thread Tyler Baicar
Currently there are trace events for the various RAS errors with the exception of ARM processor type errors. Add a new trace event for such errors so that the user will know when they occur. These trace events are consistent with the ARM processor error section type defined in UEFI 2.6 spec section

[PATCH V10 04/10] arm64: exception: handle Synchronous External Abort

2017-02-15 Thread Tyler Baicar
SEA exceptions are often caused by an uncorrected hardware error, and are handled when data abort and instruction abort exception classes have specific values for their Fault Status Code. When SEA occurs, before killing the process, report the error in the kernel logs. Update fault_info[] with spec

[PATCH V10 02/10] ras: acpi/apei: cper: generic error data entry v3 per ACPI 6.1

2017-02-15 Thread Tyler Baicar
Currently when a RAS error is reported it is not timestamped. The ACPI 6.1 spec adds the timestamp field to the generic error data entry v3 structure. The timestamp of when the firmware generated the error is now being reported. Signed-off-by: Tyler Baicar Signed-off-by: Jonathan (Zhixiong) Zhang

[PATCH V10 05/10] acpi: apei: handle SEA notification type for ARMv8

2017-02-15 Thread Tyler Baicar
ARM APEI extension proposal added SEA (Synchronous External Abort) notification type for ARMv8. Add a new GHES error source handling function for SEA. If an error source's notification type is SEA, then this function can be registered into the SEA exception handler. That way GHES will parse and rep

[PATCH V10 06/10] acpi: apei: panic OS with fatal error status block

2017-02-15 Thread Tyler Baicar
From: "Jonathan (Zhixiong) Zhang" Even if an error status block's severity is fatal, the kernel does not honor the severity level and panic. With the firmware first model, the platform could inform the OS about a fatal hardware error through the non-NMI GHES notification type. The OS should pani

[PATCH V10 03/10] efi: parse ARM processor error

2017-02-15 Thread Tyler Baicar
Add support for ARM Common Platform Error Record (CPER). UEFI 2.6 specification adds support for ARM specific processor error information to be reported as part of the CPER records. This provides more detail on for processor error logs. Signed-off-by: Tyler Baicar Signed-off-by: Jonathan (Zhixion

[PATCH V10 01/10] acpi: apei: read ack upon ghes record consumption

2017-02-15 Thread Tyler Baicar
A RAS (Reliability, Availability, Serviceability) controller may be a separate processor running in parallel with OS execution, and may generate error records for consumption by the OS. If the RAS controller produces multiple error records, then they may be overwritten before the OS has consumed th

[PATCH V10 00/10] Add UEFI 2.6 and ACPI 6.1 updates for RAS on ARM64

2017-02-15 Thread Tyler Baicar
Hi Rafael, Do you think this series should go through your tree like the other recent APEI patches? When a memory error, CPU error, PCIe error, or other type of hardware error that's covered by RAS occurs, firmware should populate the shared GHES memory location with the proper GHES structures to

Re: [PATCH V9 09/10] trace, ras: add ARM processor error trace event

2017-02-15 Thread Baicar, Tyler
On 2/15/2017 10:47 AM, Steven Rostedt wrote: On Wed, 15 Feb 2017 10:44:51 -0700 Tyler Baicar wrote: Currently there are trace events for the various RAS errors with the exception of ARM processor type errors. Add a new trace event for such errors so that the user will know when they occur. The

Re: [PATCH V9 09/10] trace, ras: add ARM processor error trace event

2017-02-15 Thread Steven Rostedt
On Wed, 15 Feb 2017 10:44:51 -0700 Tyler Baicar wrote: > Currently there are trace events for the various RAS > errors with the exception of ARM processor type errors. > Add a new trace event for such errors so that the user > will know when they occur. These trace events are > consistent with th

[PATCH V9 09/10] trace, ras: add ARM processor error trace event

2017-02-15 Thread Tyler Baicar
Currently there are trace events for the various RAS errors with the exception of ARM processor type errors. Add a new trace event for such errors so that the user will know when they occur. These trace events are consistent with the ARM processor error section type defined in UEFI 2.6 spec section

[PATCH V9 08/10] ras: acpi / apei: generate trace event for unrecognized CPER section

2017-02-15 Thread Tyler Baicar
UEFI spec allows for non-standard section in Common Platform Error Record. This is defined in section N.2.3 of UEFI version 2.5. Currently if the CPER section's type (UUID) does not match with any section type that the kernel knows how to parse, trace event is not generated for such section. And t

[PATCH V9 10/10] arm/arm64: KVM: add guest SEA support

2017-02-15 Thread Tyler Baicar
Currently external aborts are unsupported by the guest abort handling. Add handling for SEAs so that the host kernel reports SEAs which occur in the guest kernel. Signed-off-by: Tyler Baicar --- arch/arm/include/asm/kvm_arm.h | 1 + arch/arm/include/asm/system_misc.h | 5 + arch/ar

[PATCH V9 06/10] acpi: apei: panic OS with fatal error status block

2017-02-15 Thread Tyler Baicar
From: "Jonathan (Zhixiong) Zhang" Even if an error status block's severity is fatal, the kernel does not honor the severity level and panic. With the firmware first model, the platform could inform the OS about a fatal hardware error through the non-NMI GHES notification type. The OS should pani

[PATCH V9 07/10] efi: print unrecognized CPER section

2017-02-15 Thread Tyler Baicar
UEFI spec allows for non-standard section in Common Platform Error Record. This is defined in section N.2.3 of UEFI version 2.5. Currently if the CPER section's type (UUID) does not match with one of the section types that the kernel knows how to parse, the section is skipped. Therefore, user is n

[PATCH V9 05/10] acpi: apei: handle SEA notification type for ARMv8

2017-02-15 Thread Tyler Baicar
ARM APEI extension proposal added SEA (Synchronous External Abort) notification type for ARMv8. Add a new GHES error source handling function for SEA. If an error source's notification type is SEA, then this function can be registered into the SEA exception handler. That way GHES will parse and rep

[PATCH V9 04/10] arm64: exception: handle Synchronous External Abort

2017-02-15 Thread Tyler Baicar
SEA exceptions are often caused by an uncorrected hardware error, and are handled when data abort and instruction abort exception classes have specific values for their Fault Status Code. When SEA occurs, before killing the process, report the error in the kernel logs. Update fault_info[] with spec

[PATCH V9 02/10] ras: acpi/apei: cper: generic error data entry v3 per ACPI 6.1

2017-02-15 Thread Tyler Baicar
Currently when a RAS error is reported it is not timestamped. The ACPI 6.1 spec adds the timestamp field to the generic error data entry v3 structure. The timestamp of when the firmware generated the error is now being reported. Signed-off-by: Tyler Baicar Signed-off-by: Jonathan (Zhixiong) Zhang

[PATCH V9 01/10] acpi: apei: read ack upon ghes record consumption

2017-02-15 Thread Tyler Baicar
A RAS (Reliability, Availability, Serviceability) controller may be a separate processor running in parallel with OS execution, and may generate error records for consumption by the OS. If the RAS controller produces multiple error records, then they may be overwritten before the OS has consumed th

[PATCH V9 03/10] efi: parse ARM processor error

2017-02-15 Thread Tyler Baicar
Add support for ARM Common Platform Error Record (CPER). UEFI 2.6 specification adds support for ARM specific processor error information to be reported as part of the CPER records. This provides more detail on for processor error logs. Signed-off-by: Tyler Baicar Signed-off-by: Jonathan (Zhixion

[PATCH V9 00/10] Add UEFI 2.6 and ACPI 6.1 updates for RAS on ARM64

2017-02-15 Thread Tyler Baicar
Hi Rafael, Do you think this series should go through your tree like the other recent APEI patches? When a memory error, CPU error, PCIe error, or other type of hardware error that's covered by RAS occurs, firmware should populate the shared GHES memory location with the proper GHES structures to

Re: [PATCH V8 06/10] acpi: apei: panic OS with fatal error status block

2017-02-15 Thread Baicar, Tyler
On 2/15/2017 5:13 AM, James Morse wrote: Hi Tyler, On 13/02/17 22:45, Baicar, Tyler wrote: On 2/9/2017 3:48 AM, James Morse wrote: On 01/02/17 17:16, Tyler Baicar wrote: From: "Jonathan (Zhixiong) Zhang" Even if an error status block's severity is fatal, the kernel does not honor the severi

Re: [PATCH V8 08/10] ras: acpi / apei: generate trace event for unrecognized CPER section

2017-02-15 Thread Baicar, Tyler
On 2/15/2017 10:03 AM, Steven Rostedt wrote: On Wed, 15 Feb 2017 09:54:09 -0700 "Baicar, Tyler" wrote: \ In v9 I currently have this and the ARM trace event from this series both wrapped in an ifdef verifying that CONFIG_RAS is enabled. This resolves the kbuild failures and will have this code

Re: [PATCH V8 08/10] ras: acpi / apei: generate trace event for unrecognized CPER section

2017-02-15 Thread Steven Rostedt
On Wed, 15 Feb 2017 09:54:09 -0700 "Baicar, Tyler" wrote: \ > > In v9 I currently have this and the ARM trace event from this series > both wrapped in an > ifdef verifying that CONFIG_RAS is enabled. This resolves the kbuild > failures and > will have this code compiled out when that config isn

Re: [PATCH V8 08/10] ras: acpi / apei: generate trace event for unrecognized CPER section

2017-02-15 Thread Baicar, Tyler
On 2/15/2017 8:52 AM, Steven Rostedt wrote: On Wed, 1 Feb 2017 10:16:51 -0700 Tyler Baicar wrote: @@ -452,11 +454,21 @@ static void ghes_do_proc(struct ghes *ghes, { int sev, sec_sev; struct acpi_hest_generic_data *gdata; + uuid_le sec_type; + uuid_le *fru_id = &

Re: [PATCH V8 05/10] acpi: apei: handle SEA notification type for ARMv8

2017-02-15 Thread Zhengqiang
Hi Tyler: On 2017/2/2 1:16, Tyler Baicar wrote: > ARM APEI extension proposal added SEA (Synchronous External Abort) > notification type for ARMv8. > Add a new GHES error source handling function for SEA. If an error > source's notification type is SEA, then this function can be registered > into

Re: [PATCH] KVM: race-free exit from KVM_RUN without POSIX signals

2017-02-15 Thread Paolo Bonzini
On 15/02/2017 16:24, Christian Borntraeger wrote: > On 02/15/2017 03:43 PM, Paolo Bonzini wrote: >> The purpose of the KVM_SET_SIGNAL_MASK API is to let userspace "kick" >> a VCPU out of KVM_RUN through a POSIX signal. A signal is attached >> to a dummy signal handler; by blocking the signal out

Re: [PATCH V8 08/10] ras: acpi / apei: generate trace event for unrecognized CPER section

2017-02-15 Thread Steven Rostedt
On Wed, 1 Feb 2017 10:16:51 -0700 Tyler Baicar wrote: > @@ -452,11 +454,21 @@ static void ghes_do_proc(struct ghes *ghes, > { > int sev, sec_sev; > struct acpi_hest_generic_data *gdata; > + uuid_le sec_type; > + uuid_le *fru_id = &NULL_UUID_LE; > + char *fru_text = ""; >

Re: [PATCH] KVM: race-free exit from KVM_RUN without POSIX signals

2017-02-15 Thread Christian Borntraeger
On 02/15/2017 03:43 PM, Paolo Bonzini wrote: > The purpose of the KVM_SET_SIGNAL_MASK API is to let userspace "kick" > a VCPU out of KVM_RUN through a POSIX signal. A signal is attached > to a dummy signal handler; by blocking the signal outside KVM_RUN and > unblocking it inside, this possible ra

Re: [PATCH V8 05/10] acpi: apei: handle SEA notification type for ARMv8

2017-02-15 Thread Baicar, Tyler
Hello Zhengqiang, On 2/14/2017 11:24 PM, Zhengqiang wrote: Hi Tyler: On 2017/2/2 1:16, Tyler Baicar wrote: ARM APEI extension proposal added SEA (Synchronous External Abort) notification type for ARMv8. Add a new GHES error source handling function for SEA. If an error source's notification t

[PATCH] KVM: race-free exit from KVM_RUN without POSIX signals

2017-02-15 Thread Paolo Bonzini
The purpose of the KVM_SET_SIGNAL_MASK API is to let userspace "kick" a VCPU out of KVM_RUN through a POSIX signal. A signal is attached to a dummy signal handler; by blocking the signal outside KVM_RUN and unblocking it inside, this possible race is closed: VCPU thread

Re: [PATCH V8 06/10] acpi: apei: panic OS with fatal error status block

2017-02-15 Thread James Morse
Hi Tyler, On 13/02/17 22:45, Baicar, Tyler wrote: > On 2/9/2017 3:48 AM, James Morse wrote: >> On 01/02/17 17:16, Tyler Baicar wrote: >>> From: "Jonathan (Zhixiong) Zhang" >>> >>> Even if an error status block's severity is fatal, the kernel does not >>> honor the severity level and panic. >>> >>