[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <cdbf7880-a515-6394-fc24-6a4498cace2c@huawei.com>
Date: Wed, 18 Oct 2017 18:21:38 +0800
From: gengdongjiu <gengdongjiu@...wei.com>
To: James Morse <james.morse@....com>, Borislav Petkov <bp@...e.de>
CC: <tbaicar@...eaurora.org>, <will.deacon@....com>,
<rjw@...ysocki.net>, <lenb@...nel.org>, <robert.moore@...el.com>,
<lv.zheng@...el.com>, <mark.rutland@....com>,
<kristina.martsenko@....com>, <mingo@...nel.org>,
<punit.agrawal@....com>, <stephen.boyd@...aro.org>,
<kamensky@...co.com>, <prarit@...hat.com>, <shiju.jose@...wei.com>,
<linux-arm-kernel@...ts.infradead.org>,
<linux-kernel@...r.kernel.org>, <linux-acpi@...r.kernel.org>,
<devel@...ica.org>, Huangshaoyu <huangshaoyu@...wei.com>,
Wuquanming <wuquanming@...wei.com>
Subject: Re: [PATCH v5 2/2] acpi: apei: Add SEI notification type support for
ARMv8
On 2017/10/18 17:44, James Morse wrote:
>> The thing is abbreviated as "SEI" and apparently means "System Error
>> Interrupt". Nothing else.
> ARM has 'external abort', which are either synchronous or asynchronous, both are
> delivered as different types of exception.
>
> Asynchronous external abort is treated as a special kind of interrupt, 'SError
> Interrupt', (where SError stands for System Error, but its rarely written like
> that). 'SEI' is a relatively new abbreviation for SError interrupt.
>
>
> What should we call this thing? In the ACPI code I'd prefer 'SEI' as that is
> what the ACPI spec calls it. Here we are talking about an GHES notification.
>
> But in the arm64 arch code this should be called SError Interrupt as this is
> what the ARM-ARM calls it. This code cares about exception routing and interrupt
> masking.
>
>
> But, I don't really care.
Thanks very much James's clear explanation.
I agree with James.
In the ACPI sepc, we usually call SEI as SError Interrupt, we rarely call SError to System Error,
Anyway I will explain clearly about the abbreviations in my next version patch.
Powered by blists - more mailing lists