Re: [PATCH v2 10/16] arm64: kernel: Survive corrected RAS errors notified by SError

2017-09-13 Thread Baicar, Tyler
On 7/28/2017 8:10 AM, James Morse wrote: On v8.0, SError is an uncontainable fatal exception. The v8.2 RAS extensions use SError to notify software about RAS errors, these can be contained by the ESB instruction. An ACPI system with firmware-first may use SError as its 'SEI' notification.

Re: [PATCH V17 01/11] acpi: apei: read ack upon ghes record consumption

2017-06-30 Thread Baicar, Tyler
On 6/30/2017 4:10 AM, Robert Richter wrote: Tyler, On 19.05.17 14:32:03, Tyler Baicar wrote: 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

Re: [PATCH V16 08/11] efi: print unrecognized CPER section

2017-05-16 Thread Baicar, Tyler
On 5/16/2017 10:55 AM, Borislav Petkov wrote: On Tue, May 16, 2017 at 10:44:43AM -0600, Baicar, Tyler wrote: I meant to respond to this comment after I sent the v16 patch series, but you beat me to it :) These prefixes are common to all the GHES/CPER printing to the kernel logs. I don't mean

Re: [PATCH V16 08/11] efi: print unrecognized CPER section

2017-05-16 Thread Baicar, Tyler
On 5/16/2017 8:29 AM, Borislav Petkov wrote: On Mon, May 15, 2017 at 03:27:57PM -0600, 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

Re: [PATCH V15 06/11] acpi: apei: handle SEA notification type for ARMv8

2017-05-08 Thread Baicar, Tyler
On 5/8/2017 11:28 AM, James Morse wrote: Hi Tyler, On 19/04/17 00:05, 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

Re: [PATCH V15 11/11] arm/arm64: KVM: add guest SEA support

2017-05-08 Thread Baicar, Tyler
On 5/8/2017 11:40 AM, Borislav Petkov wrote: On Tue, Apr 18, 2017 at 05:05:23PM -0600, Tyler Baicar wrote: 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. When an SEA occurs in

Re: [PATCH V15 06/11] acpi: apei: handle SEA notification type for ARMv8

2017-04-25 Thread Baicar, Tyler
On 4/25/2017 11:21 AM, Borislav Petkov wrote: On Tue, Apr 18, 2017 at 05:05:18PM -0600, 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

Re: [PATCH V15 04/11] efi: parse ARM processor error

2017-04-21 Thread Baicar, Tyler
On 4/21/2017 11:55 AM, Borislav Petkov wrote: On Tue, Apr 18, 2017 at 05:05:16PM -0600, Tyler Baicar wrote: 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

Re: [PATCH V15 03/11] cper: add timestamp print to CPER status printing

2017-04-21 Thread Baicar, Tyler
On 4/21/2017 11:26 AM, Borislav Petkov wrote: On Fri, Apr 21, 2017 at 10:04:35AM -0600, Baicar, Tyler wrote: This is basically what I already had in v14...you asked to move it into a different if-statement? https://lkml.org/lkml/2017/4/12/397 Well, clearly I've been smoking some nasty potent

Re: [PATCH V15 03/11] cper: add timestamp print to CPER status printing

2017-04-21 Thread Baicar, Tyler
On 4/21/2017 6:21 AM, Borislav Petkov wrote: On Tue, Apr 18, 2017 at 05:05:15PM -0600, Tyler Baicar wrote: The ACPI 6.1 spec added a timestamp to the HEST generic data HEST? I see the timestamp in Table 18-343 Generic Error Data Entry where those things are "One or more Generic Error Data

Re: [PATCH V15 01/11] acpi: apei: read ack upon ghes record consumption

2017-04-19 Thread Baicar, Tyler
On 4/19/2017 12:31 PM, Borislav Petkov wrote: On Tue, Apr 18, 2017 at 05:05:13PM -0600, Tyler Baicar wrote: 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.

Re: [PATCH v3 1/8] trace: ras: add ARM processor error information trace event

2017-04-17 Thread Baicar, Tyler
On 4/16/2017 9:16 PM, Xie XiuQi wrote: On 2017/4/17 11:08, Xie XiuQi wrote: On 3/30/2017 4:31 AM, Xie XiuQi wrote: Add a new trace event for ARM processor error information, so that the user will know what error occurred. With this information the user may take appropriate action. These trace

Re: [PATCH v3 1/8] trace: ras: add ARM processor error information trace event

2017-04-14 Thread Baicar, Tyler
On 3/30/2017 4:31 AM, Xie XiuQi wrote: Add a new trace event for ARM processor error information, so that the user will know what error occurred. With this information the user may take appropriate action. These trace events are consistent with the ARM processor error information table which

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

2017-04-13 Thread Baicar, Tyler
On 4/13/2017 2:47 PM, Borislav Petkov wrote: On Thu, Apr 13, 2017 at 02:30:21PM -0600, Baicar, Tyler wrote: I do not agree with this. The struct being passed to this function is already named acpi_hest_generic_data in the existing code and all over this code is named gdata not just d. And I'm

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

2017-04-13 Thread Baicar, Tyler
On 4/12/2017 10:51 AM, Borislav Petkov wrote: On Tue, Mar 28, 2017 at 01:30:33PM -0600, Tyler Baicar wrote: 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

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

2017-04-13 Thread Baicar, Tyler
On 4/12/2017 7:34 AM, Borislav Petkov wrote: Subject: [PATCH V14 02/10] ras: acpi/apei: cper: generic error data entry v3 per ACPI 6.1 Use a verb in your patch subjects: "Add support for ..." or so. Hello Boris, Will do in the next version. On Tue, Mar 28, 2017 at 01:30:32PM -0600, Tyler

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

2017-04-13 Thread Baicar, Tyler
On 4/11/2017 11:15 AM, Borislav Petkov wrote: On Tue, Mar 28, 2017 at 01:30:31PM -0600, Tyler Baicar wrote: 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.

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

2017-03-28 Thread Baicar, Tyler
Hello Christoffer, On 3/28/2017 2:26 PM, Christoffer Dall wrote: On Tue, Mar 28, 2017 at 01:30:40PM -0600, Tyler Baicar wrote: 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.

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

2017-03-28 Thread Baicar, Tyler
Hello, On 3/27/2017 10:19 PM, gengdongjiu wrote: Hi Tyler, I have a question for below code. On 2017/3/25 0:01, Christoffer Dall wrote: is_iabt = kvm_vcpu_trap_is_iabt(vcpu); - if (unlikely(!is_iabt && kvm_vcpu_dabt_isextabt(vcpu))) { + if (unlikely(!is_iabt &&

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

2017-03-28 Thread Baicar, Tyler
Hello, On 3/28/2017 3:53 AM, gengdongjiu wrote: Hi, On 2017/3/22 6:47, Tyler Baicar wrote: + fault_ipa = kvm_vcpu_get_fault_ipa(vcpu); + + /* The host kernel will handle the synchronous external abort. There +* is no need to pass the error into the guest. +*/ +

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

2017-03-28 Thread Baicar, Tyler
Hello Christoffer, On 3/24/2017 10:01 AM, Christoffer Dall wrote: On Tue, Mar 21, 2017 at 04:47:05PM -0600, Tyler Baicar wrote: 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.

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

2017-03-28 Thread Baicar, Tyler
Hello Marc, On 3/24/2017 8:03 AM, Marc Zyngier wrote: On 21/03/17 22:47, Tyler Baicar wrote: 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

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

2017-03-21 Thread Baicar, Tyler
Hello Xie XiuQi, On 3/21/2017 8:48 PM, Xie XiuQi wrote: Reviewed-by: Xie XiuQi Thank you! Tyler On 2017/3/22 6:47, 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

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

2017-03-21 Thread Baicar, Tyler
Hello James, On 3/17/2017 10:43 AM, James Morse wrote: On 06/03/17 20:44, 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,

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

2017-03-14 Thread Baicar, Tyler
Hello Xie XiUQi, On 3/12/2017 8:31 PM, Xie XiuQi wrote: Hi Baicar Tyler, On 2017/3/11 2:23, Baicar, Tyler wrote: Hello Xie XiuQi, On 3/9/2017 2:41 AM, Xie XiuQi wrote: On 2017/3/7 4:45, Tyler Baicar wrote: Currently there are trace events for the various RAS errors with the exception

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

2017-03-10 Thread Baicar, Tyler
Hello Xie XiuQi, On 3/9/2017 2:41 AM, Xie XiuQi wrote: On 2017/3/7 4:45, 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

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

2017-03-10 Thread Baicar, Tyler
Hello James, On 3/8/2017 9:09 AM, James Morse wrote: On 07/03/17 17:58, Baicar, Tyler wrote: On 3/7/2017 4:48 AM, James Morse wrote: On 06/03/17 20:45, Tyler Baicar wrote: Currently external aborts are unsupported by the guest abort handling. Add handling for SEAs so that the host kernel

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

2017-03-07 Thread Baicar, Tyler
Hello James, On 3/7/2017 4:48 AM, James Morse wrote: On 06/03/17 20:45, Tyler Baicar wrote: 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. diff --git

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

2017-03-07 Thread Baicar, Tyler
Hello James, On 3/7/2017 4:37 AM, James Morse wrote: On 06/03/17 20:44, 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,

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

2017-03-07 Thread Baicar, Tyler
On 3/6/2017 2:05 PM, Joe Perches wrote: On Mon, 2017-03-06 at 13:45 -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

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

2017-03-07 Thread Baicar, Tyler
On 3/7/2017 4:37 AM, James Morse wrote: Hi Tyler, On 06/03/17 20:44, Tyler Baicar wrote: 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 notify

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

2017-03-06 Thread Baicar, Tyler
Hello James, On 3/6/2017 3:28 AM, James Morse wrote: On 28/02/17 19:43, Baicar, Tyler wrote: On 2/24/2017 3:42 AM, James Morse wrote: On 21/02/17 21:22, Tyler Baicar wrote: Currently external aborts are unsupported by the guest abort handling. Add handling for SEAs so that the host kernel

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

2017-03-03 Thread Baicar, Tyler
Hello Shiju, On 3/3/2017 8:34 AM, Shiju Jose wrote: diff --git a/arch/arm64/mm/fault.c b/arch/arm64/mm/fault.c index b2d57fc..403277b 100644 --- a/arch/arm64/mm/fault.c +++ b/arch/arm64/mm/fault.c @@ -602,6 +602,24 @@ static const char *fault_name(unsigned int esr) } /* + * Handle

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

2017-03-01 Thread Baicar, Tyler
Hello Xie XiuQi, On 3/1/2017 12:42 AM, Xie XiuQi wrote: Hi Tyler, On 2017/2/22 5:21, 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

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

2017-02-28 Thread Baicar, Tyler
Hello James, On 2/24/2017 3:42 AM, James Morse wrote: On 21/02/17 21:22, Tyler Baicar wrote: 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. diff --git

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

2017-02-21 Thread Baicar, Tyler
Hello James, On 2/21/2017 12:10 PM, James Morse wrote: Hi Tyler, On 15/02/17 19:51, 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

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

2017-02-21 Thread Baicar, Tyler
Hello Ard, On 2/16/2017 11:32 AM, Ard Biesheuvel wrote: On 15 February 2017 at 19:51, 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

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

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

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" <zjzh...@codeaurora.org> Even if an error status block's severity is f

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" <tbai...@codeaurora.org> 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

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; +

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

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

2017-02-13 Thread Baicar, Tyler
Hello James, On 2/9/2017 3:48 AM, James Morse wrote: Hi Jonathan, Tyler, 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. With the

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

2017-02-03 Thread Baicar, Tyler
Hello James, On 2/3/2017 9:00 AM, James Morse wrote: On 01/02/17 17: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,

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

2017-02-03 Thread Baicar, Tyler
Hello James, On 2/3/2017 8:59 AM, James Morse wrote: On 01/02/17 17:16, Tyler Baicar wrote: 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

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

2017-02-03 Thread Baicar, Tyler
Hello Steve, On 2/1/2017 8:15 PM, Steven Rostedt wrote: On Wed, 1 Feb 2017 10:16:52 -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

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

2017-01-24 Thread Baicar, Tyler
On 1/24/2017 10:55 AM, James Morse wrote: Hi Tyler, On 20/01/17 20:58, Baicar, Tyler wrote: On 1/19/2017 10:57 AM, James Morse wrote: On 18/01/17 23:51, Baicar, Tyler wrote: On 1/18/2017 7:50 AM, James Morse wrote: On 12/01/17 18:15, Tyler Baicar wrote: diff --git a/drivers/acpi/apei

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

2017-01-24 Thread Baicar, Tyler
On 1/23/2017 3:01 AM, James Morse wrote: Hi Tyler, On 20/01/17 20:35, Baicar, Tyler wrote: On 1/19/2017 10:55 AM, James Morse wrote: On 18/01/17 23:26, Baicar, Tyler wrote: On 1/17/2017 3:31 AM, James Morse wrote: On 12/01/17 18:15, Tyler Baicar wrote: +info.si_addr = (void __user

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

2017-01-20 Thread Baicar, Tyler
On 1/19/2017 10:57 AM, James Morse wrote: Hi Tyler, On 18/01/17 23:51, Baicar, Tyler wrote: On 1/18/2017 7:50 AM, James Morse wrote: On 12/01/17 18:15, Tyler Baicar wrote: ARM APEI extension proposal added SEA (Synchrounous External Abort) notification type for ARMv8. Add a new GHES error

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

2017-01-20 Thread Baicar, Tyler
On 1/19/2017 10:55 AM, James Morse wrote: Hi Tyler, On 18/01/17 23:26, Baicar, Tyler wrote: On 1/17/2017 3:31 AM, James Morse wrote: On 12/01/17 18:15, Tyler Baicar wrote: SEA exceptions are often caused by an uncorrected hardware error, and are handled when data abort and instruction abort

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

2017-01-18 Thread Baicar, Tyler
Hello James, On 1/18/2017 7:50 AM, James Morse wrote: Hi Tyler, On 12/01/17 18:15, Tyler Baicar wrote: ARM APEI extension proposal added SEA (Synchrounous External Nit: Synchronous I'll fix that :) Abort) notification type for ARMv8. Add a new GHES error source handling function for SEA.

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

2017-01-18 Thread Baicar, Tyler
Hello James, On 1/17/2017 3:31 AM, James Morse wrote: Hi Tyler, On 12/01/17 18:15, Tyler Baicar wrote: 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.

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

2017-01-18 Thread Baicar, Tyler
On 1/17/2017 3:27 AM, Will Deacon wrote: On Mon, Jan 16, 2017 at 01:09:22PM -0700, Baicar, Tyler wrote: On 1/16/2017 4:53 AM, Will Deacon wrote: On Thu, Jan 12, 2017 at 11:15:18AM -0700, Tyler Baicar wrote: SEA exceptions are often caused by an uncorrected hardware error, and are handled when

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

2017-01-18 Thread Baicar, Tyler
On 1/17/2017 3:23 AM, James Morse wrote: Hi Tyler, On 16/01/17 11:53, Will Deacon wrote: On Thu, Jan 12, 2017 at 11:15:18AM -0700, Tyler Baicar wrote: SEA exceptions are often caused by an uncorrected hardware error, and are handled when data abort and instruction abort exception classes have

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

2017-01-16 Thread Baicar, Tyler
Hello Marc, On 1/16/2017 4:58 AM, Marc Zyngier wrote: Hi Tyler, On 12/01/17 18:15, Tyler Baicar wrote: 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

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

2017-01-16 Thread Baicar, Tyler
Hello Will, On 1/16/2017 4:53 AM, Will Deacon wrote: On Thu, Jan 12, 2017 at 11:15:18AM -0700, Tyler Baicar wrote: 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

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

2017-01-10 Thread Baicar, Tyler
Hello James, On 1/6/2017 3:43 AM, James Morse wrote: On 05/01/17 22:31, Baicar, Tyler wrote: On 12/20/2016 8:29 AM, James Morse wrote: On 07/12/16 21:48, Tyler Baicar wrote: ARM APEI extension proposal added SEA (Synchrounous External Abort) notification type for ARMv8. Add a new GHES error

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

2017-01-06 Thread Baicar, Tyler
Hi Will, On 1/4/2017 6:54 AM, Will Deacon wrote: On Wed, Dec 07, 2016 at 02:48:17PM -0700, Tyler Baicar wrote: 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

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

2017-01-05 Thread Baicar, Tyler
On 12/20/2016 8:29 AM, James Morse wrote: Hi Tyler, On 07/12/16 21:48, Tyler Baicar wrote: ARM APEI extension proposal added SEA (Synchrounous 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

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

2017-01-05 Thread Baicar, Tyler
On 12/15/2016 7:08 AM, James Morse wrote: Hi Tyler, On 07/12/16 21:48, Tyler Baicar wrote: 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

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

2016-12-13 Thread Baicar, Tyler
Hello Shiju, Great! Thank you for testing! :) Tyler On 12/13/2016 4:10 AM, Shiju Jose wrote: Hi Tyler, We have tested V6 patch set on our platform. It worked fine. Thanks, Shiju -Original Message- From: Tyler Baicar [mailto:tbai...@codeaurora.org] Sent: 07 December 2016 21:48 To:

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

2016-11-29 Thread Baicar, Tyler
On 11/29/2016 4:29 AM, Shiju Jose wrote: @@ -451,12 +484,12 @@ void cper_estatus_print(const char *pfx, printk("%s""event severity: %s\n", pfx, cper_severity_str(severity)); data_len = estatus->data_length; gdata = (struct acpi_hest_generic_data *)(estatus + 1); +

Re: [PATCH V5 03/10] efi: parse ARMv8 processor error

2016-11-29 Thread Baicar, Tyler
Hello James, On 11/25/2016 11:23 AM, James Morse wrote: Hi Tyler, On 21/11/16 22:35, Tyler Baicar wrote: Add support for ARMv8 Common Platform Error Record (CPER). UEFI 2.6 specification adds support for ARMv8 specific processor error information to be reported as part of the CPER records.

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

2016-11-28 Thread Baicar, Tyler
Hello James, On 11/25/2016 11:20 AM, James Morse wrote: Hi Tyler, On 21/11/16 22:35, Tyler Baicar wrote: 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

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

2016-11-28 Thread Baicar, Tyler
Hello James, Thank you for your feedback! On 11/25/2016 11:19 AM, James Morse wrote: Hi Tyler, On 21/11/16 22:35, Tyler Baicar wrote: A RAS (Reliability, Availability, Serviceability) controller may be a separate processor running in parallel with OS execution, and may generate error records

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

2016-10-24 Thread Baicar, Tyler
On 10/24/2016 3:50 AM, Suzuki K Poulose wrote: On 21/10/16 18:30, Tyler Baicar wrote: 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

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

2016-10-24 Thread Baicar, Tyler
On 10/24/2016 2:51 AM, Suzuki K Poulose wrote: On 21/10/16 18:30, Tyler Baicar wrote: 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

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

2016-10-24 Thread Baicar, Tyler
On 10/21/2016 2:34 PM, Steven Rostedt wrote: On Fri, 21 Oct 2016 11:30:12 -0600 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

Re: [PATCH V3 10/10] arm64: KVM: add guest SEA support

2016-10-14 Thread Baicar, Tyler
On 10/14/2016 2:38 AM, Punit Agrawal wrote: "Baicar, Tyler" <tbai...@codeaurora.org> writes: Hello Punit, On 10/13/2016 7:14 AM, Punit Agrawal wrote: Hi Tyler, I know I've had my last comment already ;), but I thought I'd rather raise the question than stay confused... Ty

Re: [PATCH V3 10/10] arm64: KVM: add guest SEA support

2016-10-13 Thread Baicar, Tyler
Hello Punit, On 10/13/2016 7:14 AM, Punit Agrawal wrote: Hi Tyler, I know I've had my last comment already ;), but I thought I'd rather raise the question than stay confused... Tyler Baicar writes: Currently external aborts are unsupported by the guest abort

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

2016-10-13 Thread Baicar, Tyler
Hello Suzuki, On 10/13/2016 2:50 AM, Suzuki K Poulose wrote: On 12/10/16 23:10, Baicar, Tyler wrote: Hello Suzuki, Thank you for the feedback! Responses below. On 10/11/2016 11:28 AM, Suzuki K Poulose wrote: On 07/10/16 22:31, Tyler Baicar wrote: Currently when a RAS error is reported

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

2016-10-13 Thread Baicar, Tyler
Hello Punit, On 10/12/2016 11:46 AM, Punit Agrawal wrote: Hi Tyler, A couple of hopefully not bike shedding comments below. Tyler Baicar writes: SEA exceptions are often caused by an uncorrected hardware error, and are handled when data abort and instruction abort

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

2016-10-13 Thread Baicar, Tyler
Hello Punit, On 10/12/2016 12:00 PM, Punit Agrawal wrote: Tyler Baicar writes: ARM APEI extension proposal added SEA (Synchrounous External Abort) notification type for ARMv8. Add a new GHES error source handling function for SEA. If an error source's notification

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

2016-10-13 Thread Baicar, Tyler
Hello Punit, Thank you for the feedback! Responses below On 10/12/2016 9:39 AM, Punit Agrawal wrote: Hi Tyler, A few comments below. Tyler Baicar writes: A RAS (Reliability, Availability, Serviceability) controller may be a separate processor running in parallel

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

2016-10-12 Thread Baicar, Tyler
Hello Suzuki, Thank you for the feedback! Responses below. On 10/11/2016 11:28 AM, Suzuki K Poulose wrote: On 07/10/16 22:31, Tyler Baicar wrote: 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