Re: [PATCH] efi/arm: fix allocation failure when reserving the kernel base

2019-08-20 Thread Chester Lin
On Mon, Aug 19, 2019 at 05:56:51PM +0300, Ard Biesheuvel wrote: > On Mon, 19 Aug 2019 at 11:01, Chester Lin wrote: > > > > Hi Mike and Ard, > > > > On Thu, Aug 15, 2019 at 04:37:39PM +0300, Mike Rapoport wrote: > > > On Thu, Aug 15, 2019 at 02:32:50PM +0300, Ard Biesheuvel wrote: > > > > (adding

Re: [PATCH] efi/arm: fix allocation failure when reserving the kernel base

2019-08-20 Thread Mike Rapoport
On Mon, Aug 19, 2019 at 05:56:51PM +0300, Ard Biesheuvel wrote: > On Mon, 19 Aug 2019 at 11:01, Chester Lin wrote: > > > > Hi Mike and Ard, > > > > On Thu, Aug 15, 2019 at 04:37:39PM +0300, Mike Rapoport wrote: > > > On Thu, Aug 15, 2019 at 02:32:50PM +0300, Ard Biesheuvel wrote: > > > > (adding

Re: [PATCH] efi/arm: fix allocation failure when reserving the kernel base

2019-08-20 Thread Chester Lin
On Tue, Aug 20, 2019 at 10:49:30AM +0300, Mike Rapoport wrote: > On Mon, Aug 19, 2019 at 05:56:51PM +0300, Ard Biesheuvel wrote: > > On Mon, 19 Aug 2019 at 11:01, Chester Lin wrote: > > > > > > Hi Mike and Ard, > > > > > > On Thu, Aug 15, 2019 at 04:37:39PM +0300, Mike Rapoport wrote: > > > > On

Re: [PATCH] efi/arm: fix allocation failure when reserving the kernel base

2019-08-20 Thread Ard Biesheuvel
On Tue, 20 Aug 2019 at 10:49, Mike Rapoport wrote: > > On Mon, Aug 19, 2019 at 05:56:51PM +0300, Ard Biesheuvel wrote: > > On Mon, 19 Aug 2019 at 11:01, Chester Lin wrote: > > > > > > Hi Mike and Ard, > > > > > > On Thu, Aug 15, 2019 at 04:37:39PM +0300, Mike Rapoport wrote: > > > > On Thu, Aug

Re: [PATCH] efi/arm: fix allocation failure when reserving the kernel base

2019-08-20 Thread Russell King - ARM Linux admin
On Sun, Aug 04, 2019 at 10:57:00AM +0300, Ard Biesheuvel wrote: > (The first TEXT_OFFSET bytes are no longer used in practice, which is > why putting a reserved region of 4 KB bytes works at the moment, but > this is fragile). That is not correct for 32-bit ARM. The swapper page table is still

Re: [PATCH] efi/arm: fix allocation failure when reserving the kernel base

2019-08-20 Thread Russell King - ARM Linux admin
On Fri, Aug 02, 2019 at 05:38:54AM +, Chester Lin wrote: > diff --git a/arch/arm/mm/mmu.c b/arch/arm/mm/mmu.c > index f3ce34113f89..909b11ba48d8 100644 > --- a/arch/arm/mm/mmu.c > +++ b/arch/arm/mm/mmu.c > @@ -1184,6 +1184,9 @@ void __init adjust_lowmem_bounds(void) > phys_addr_t

Re: [PATCH] efi/arm: fix allocation failure when reserving the kernel base

2019-08-20 Thread Ard Biesheuvel
On Tue, 20 Aug 2019 at 14:56, Russell King - ARM Linux admin wrote: > > On Fri, Aug 02, 2019 at 05:38:54AM +, Chester Lin wrote: > > diff --git a/arch/arm/mm/mmu.c b/arch/arm/mm/mmu.c > > index f3ce34113f89..909b11ba48d8 100644 > > --- a/arch/arm/mm/mmu.c > > +++ b/arch/arm/mm/mmu.c > > @@

Re: [PATCH] efi/arm: fix allocation failure when reserving the kernel base

2019-08-20 Thread Ard Biesheuvel
On Tue, 20 Aug 2019 at 14:54, Russell King - ARM Linux admin wrote: > > On Sun, Aug 04, 2019 at 10:57:00AM +0300, Ard Biesheuvel wrote: > > (The first TEXT_OFFSET bytes are no longer used in practice, which is > > why putting a reserved region of 4 KB bytes works at the moment, but > > this is

[PATCH 1/6 V2] efi / ras: CCIX Memory error reporting

2019-08-20 Thread Jonathan Cameron
CCIX defines a number of different error types (See CCIX spec 1.0) and UEFI 2.8 defines a CPER record to allow for them to be reported when firmware first handling is in use. The last part of that record is a copy of the CCIX protocol error record which can provide very detailed information. This

[PATCH 3/6 V2] efi / ras: CCIX Address Translation Cache error reporting

2019-08-20 Thread Jonathan Cameron
CCIX devices tend to make heavy use of ATCs. The CCIX base specification defines a protocol error message (PER) that describes errors reported by such caches. The UEFI 2.8 specification includes a CCIX CPER record for firmware first handling to report these errors to the operating system. This

[PATCH 2/6 V2] efi / ras: CCIX Cache error reporting

2019-08-20 Thread Jonathan Cameron
As CCIX Request Agents have fully cache coherent caches, the CCIX 1.0 Base Specification defines detailed error reporting for these caches. A CCIX cache error is reported via a CPER record as defined in the UEFI 2.8 specification. The PER log section is defined in the CCIX 1.0 Base Specification.

[PATCH 0/6 V2] CCIX Protocol error reporting.

2019-08-20 Thread Jonathan Cameron
Note this will clash with: [PATCH RFC 0/4] ACPI: APEI: Add support to notify the vendor specific HW errors so a rebase will be needed. Changes since V1: Addressed comments from James Morse - Dropped kernel logging of vendor data. We just push it to the tracepoints. - Tidied up this cover letter

[PATCH 6/6 V2] efi / ras: CCIX Agent internal error reporting

2019-08-20 Thread Jonathan Cameron
The CCIX 1.0 Base specification defines an internal agent error, for which the specific data present afte the header is vendor defined. Signed-off-by: Jonathan Cameron --- Changes since v1. Drop printing of vendor data to kernel log. drivers/acpi/apei/ghes.c | 4 ++

[PATCH 5/6 V2] efi / ras: CCIX Link error reporting

2019-08-20 Thread Jonathan Cameron
The CCIX 1.0 Base Specification defines a protocol layer CCIX link and related error reporting mechanism. The UEFI 2.8 specification includes a CCIX CPER record for firmware first handling to report these errors to the operating system. This patch is very similar to the support previously added

[PATCH 4/6 V2] efi / ras: CCIX Port error reporting

2019-08-20 Thread Jonathan Cameron
The CCIX 1.0 base specification defines a CCIX protocol layer port. The specification provides a mechanism for detailed error logging for these ports. The UEFI 2.8 specification includes a CCIX CPER record for firmware first handling to report these errors to the operating system. This patch is