Re: [PATCH v2] acpi: apei: Add SEI notification type support for ARMv8

2017-09-28 Thread gengdongjiu
Hi Stephen, Tyler On 2017/9/27 23:52, Stephen Boyd wrote: Something to be aware of, this patch will conflict with https://lkml.org/lkml/2017/9/14/663 It may make sense to just remove the conditions for the NMI configs as part of this patch or in a series with this patch to

Re: [PATCH v2] acpi: apei: Add SEI notification type support for ARMv8

2017-09-28 Thread gengdongjiu
Hi Stephen, Tyler On 2017/9/27 23:52, Stephen Boyd wrote: Something to be aware of, this patch will conflict with https://lkml.org/lkml/2017/9/14/663 It may make sense to just remove the conditions for the NMI configs as part of this patch or in a series with this patch to

Re: [PATCH v2] acpi: apei: Add SEI notification type support for ARMv8

2017-09-27 Thread Stephen Boyd
On 09/27, Tyler Baicar wrote: > On 9/27/2017 6:05 AM, gengdongjiu wrote: > >Tyler, Stephen > > > >On 2017/9/27 3:23, Tyler Baicar wrote: > >>>Signed-off-by: Dongjiu Geng > >>Tested-by: Tyler Baicar > >> > >>Tested this functionality using SEA

Re: [PATCH v2] acpi: apei: Add SEI notification type support for ARMv8

2017-09-27 Thread Stephen Boyd
On 09/27, Tyler Baicar wrote: > On 9/27/2017 6:05 AM, gengdongjiu wrote: > >Tyler, Stephen > > > >On 2017/9/27 3:23, Tyler Baicar wrote: > >>>Signed-off-by: Dongjiu Geng > >>Tested-by: Tyler Baicar > >> > >>Tested this functionality using SEA support. > >> > >>++Stephen, > >> > >>Something to be

Re: [PATCH v2] acpi: apei: Add SEI notification type support for ARMv8

2017-09-27 Thread Tyler Baicar
On 9/27/2017 6:05 AM, gengdongjiu wrote: Tyler, Stephen On 2017/9/27 3:23, Tyler Baicar wrote: Signed-off-by: Dongjiu Geng Tested-by: Tyler Baicar Tested this functionality using SEA support. ++Stephen, Something to be aware of, this patch

Re: [PATCH v2] acpi: apei: Add SEI notification type support for ARMv8

2017-09-27 Thread Tyler Baicar
On 9/27/2017 6:05 AM, gengdongjiu wrote: Tyler, Stephen On 2017/9/27 3:23, Tyler Baicar wrote: Signed-off-by: Dongjiu Geng Tested-by: Tyler Baicar Tested this functionality using SEA support. ++Stephen, Something to be aware of, this patch will conflict with

Re: [PATCH v2] acpi: apei: Add SEI notification type support for ARMv8

2017-09-27 Thread gengdongjiu
Tyler, Stephen On 2017/9/27 3:23, Tyler Baicar wrote: >> Signed-off-by: Dongjiu Geng > Tested-by: Tyler Baicar > > Tested this functionality using SEA support. > > ++Stephen, > > Something to be aware of, this patch will conflict with >

Re: [PATCH v2] acpi: apei: Add SEI notification type support for ARMv8

2017-09-27 Thread gengdongjiu
Tyler, Stephen On 2017/9/27 3:23, Tyler Baicar wrote: >> Signed-off-by: Dongjiu Geng > Tested-by: Tyler Baicar > > Tested this functionality using SEA support. > > ++Stephen, > > Something to be aware of, this patch will conflict with > https://lkml.org/lkml/2017/9/14/663 > It may make

Re: [PATCH v2] acpi: apei: Add SEI notification type support for ARMv8

2017-09-26 Thread gengdongjiu
Hi Tyler, Thank you very much for your test and comments. On 2017/9/27 3:23, Tyler Baicar wrote: >> should identify the address to a invalid value. >> >> Signed-off-by: Dongjiu Geng > Tested-by: Tyler Baicar > > Tested this functionality

Re: [PATCH v2] acpi: apei: Add SEI notification type support for ARMv8

2017-09-26 Thread gengdongjiu
Hi Tyler, Thank you very much for your test and comments. On 2017/9/27 3:23, Tyler Baicar wrote: >> should identify the address to a invalid value. >> >> Signed-off-by: Dongjiu Geng > Tested-by: Tyler Baicar > > Tested this functionality using SEA support. Thanks for your test. > >

Re: [PATCH v2] acpi: apei: Add SEI notification type support for ARMv8

2017-09-26 Thread Tyler Baicar
On 9/14/2017 1:22 AM, Dongjiu Geng wrote: ARMv8.2 requires implementation of the RAS extension, in this extension it adds SEI(SError Interrupt) notification type, this patch adds new GHES error source SEI handling functions. Because this error source parsing and handling methods are similar with

Re: [PATCH v2] acpi: apei: Add SEI notification type support for ARMv8

2017-09-26 Thread Tyler Baicar
On 9/14/2017 1:22 AM, Dongjiu Geng wrote: ARMv8.2 requires implementation of the RAS extension, in this extension it adds SEI(SError Interrupt) notification type, this patch adds new GHES error source SEI handling functions. Because this error source parsing and handling methods are similar with