Re: [RESEND PATCH v18 1/2] ACPI: APEI: send SIGBUS to current task if synchronous memory error not recovered

Shuai Xue
2025-07-15T02:03:35Z

在 2025/7/15 01:30, Rafael J. Wysocki 写道:
> Hi,
>
> On Mon, Jul 14, 2025 at 1:54 PM Shuai Xue <xueshuai@linux.alibaba.com> wrote:
>>
>> 在 2025/7/1 21:56, Rafael J. Wysocki 写道:
>>> On Tue, Jul 1, 2025 at 1:00 PM Shuai Xue <xueshuai@linux.alibaba.com> wrote:
>>>>
>>>> >在 2025/4/28 23:23, Will Deacon 写道:
>>>> >> On Fri, Apr 25, 2025 at 09:10:09AM +0800, Shuai Xue wrote:
>>>> >>> 在 2025/4/25 09:00, Hanjun Guo 写道:
>>>> >>>> Call force_sig(SIGBUS) directly in ghes_do_proc() is not my favourite,
>>>> >>>> but I can bear that, please add
>>>> >>>>
>>>> >>>> Reviewed-by: Hanjun Guo <guohanjun@huawei.com>
>>>> >>>>
>>>> >>>> Thanks
>>>> >>>> Hanjun
>>>> >>>
>>>> >>> Thanks. Hanjun.
>>>> >>>
>>>> >>> @Rafael, @Catalin,
>>>> >>>
>>>> >>> Both patch 1 and 2 have reviewed-by tag from the arm64 ACPI
>>>> maintainers, Hanjun,
>>>> >>> now. Are you happpy to pick and queue this patch set to acpi tree
>>>> or arm tree?
>>>> >>
>>>> >> Since this primarily touches drivers/acpi/apei/ghes.c, I think it should
>>>> >> go via the ACPI tree and not the arm64 one.
>>>> >>
>>>> >> Will
>>>> >
>>>> >Hi, Will,
>>>> >
>>>> >Thank you for your confirmation :)
>>>> >
>>>> >@Rafael, do you have more comments on this patch set?
>>>> >
>>>> >Thanks you.
>>>> >
>>>> >Best Regards,
>>>> >Shuai
>>>>
>>>> Hi, all,
>>>>
>>>> Gentle ping.
>>>>
>>>> Does ACPI or APEI tree still active? Looking forward to any response.
>>>
>>> For APEI changes, you need an ACK from one of the reviewers listed in
>>> the MAINTAINERS entry for APEI.
>>>
>>> Thanks!
>>
>> Hi, Rafael
>>
>> Sorry, I missed your email which goes in span (:
>>
>> ARM maintain @Catalin points that:
>>
>> > James Morse is listed as reviewer of the ACPI APEI code but he's busy
>> > with resctrl/MPAM. Adding Lorenzo, Sudeep and Hanjun as arm64 ACPI
>> > maintainers, hopefully they can help.
>>
>> And Hanjun explictly gived his Reviewed-by tag in this thread, is that
>> happy for you for merge?
>
> Not really.
>
> I need an ACK or R-by from a reviewer listed in the APEI entry in MAINTAINERS.

Hi Rafael,

I understand your requirement for an ACK/R-by from the APEI reviewers
listed in MAINTAINERS.

So, @Tony, @James, @Borislav, @Len,

Gentle ping, we need your help to review and ack this patch set.

If I recall correctly, Rafael has mentioned this issue at least three
times in previous emails, but we still haven't received explict response
from the APEI maintainer.

>
> If James Morse is not able to fill that role (and AFAICS he's not been
> for quite some time now), I'd expect someone else to step up.
>
> Thanks!

Thank you for the clarification.

I'd like to volunteer to help with APEI code reviews. I have been
working with APEI-related code and am familiar with the ACPI error
handling mechanisms.

I'm willing to start by contributing reviews and help move pending APEI
patches forward. If the community finds my contributions valuable and
believes I have the necessary expertise, I would welcome the opportunity
to be formally set up as an APEI reviewer in the future.

Thanks for considering this, and I look forward to your guidance.

Thanks.
Shuai