[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <6f6c2933-b0a7-4fac-6342-3860f022229f@linux.alibaba.com>
Date: Wed, 20 Oct 2021 10:35:29 +0800
From: Xuesong Chen <xuesong.chen@...ux.alibaba.com>
To: Bjorn Helgaas <helgaas@...nel.org>
Cc: catalin.marinas@....com, lorenzo.pieralisi@....com,
james.morse@....com, will@...nel.org, rafael@...nel.org,
tony.luck@...el.com, bp@...en8.de, mingo@...nel.org,
bhelgaas@...gle.com, linux-pci@...r.kernel.org,
linux-acpi@...r.kernel.org, linux-arm-kernel@...ts.infradead.org,
linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3 2/2] ACPI: APEI: Filter the PCI MCFG address with an
arch-agnostic method
On 19/10/2021 23:04, Bjorn Helgaas wrote:
> On Tue, Oct 19, 2021 at 12:50:33PM +0800, Xuesong Chen wrote:
>> The commit d91525eb8ee6 ("ACPI, EINJ: Enhance error injection tolerance
>> level") fixes the issue that the ACPI/APEI can not access the PCI MCFG
>> address on x86 platform, but this issue can also happen on other
>> architectures, for instance, we got below error message on arm64 platform:
>> ...
>> APEI: Can not request [mem 0x50100000-0x50100003] for APEI EINJ Trigger registers
>> ...
>>
>> This patch will try to handle this case in a more common way instead of the
>> original 'arch' specific solution, which will be beneficial to all the
>> APEI-dependent platforms after that.
>>
>> Signed-off-by: Xuesong Chen <xuesong.chen@...ux.alibaba.com>
>> Reported-by: kernel test robot <lkp@...el.com>
>
> The purpose of this patch is not to fix a problem reported by the
> kernel test robot, so remove this tag.
Yes, will do
>
> I know the robot found a problem with a previous version of this
> patch, but we treat that the same as a code review comment. We
> normally don't explicitly credit reviewers unless it was something
> major, and then it would go in the commit log, not a "Reported-by"
> tag.
>
> It makes sense to credit the kernel test robot for things found in
> Linus' tree, but it's a little too aggressive about suggesting the tag
> for problems with unmerged changes.
>
>> Reviewed-by: Lorenzo Pieralisi <lorenzo.pieralisi@....com>
>
> This tag can only be added when Lorenzo explicitly supplies it
> himself. I do not see that on the mailing list, so please remove this
> tag as well. After Lorenzo supplies it, you can include it in future
> postings as long as you don't make significant changes to the patch.
En, Lorenzo does have comments on the patch#2 and I also update that patch
according to his feedback, so why the tag is here. OK, I'll add this tag
if Lorenzo can supply it explicitly before I send the next version.
>
> Bjorn
>
Powered by blists - more mailing lists