Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753131AbdHKNRz (ORCPT ); Fri, 11 Aug 2017 09:17:55 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:35678 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752986AbdHKNRv (ORCPT ); Fri, 11 Aug 2017 09:17:51 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 0954B60115 Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=tbaicar@codeaurora.org Subject: Re: [PATCH] acpi: apei: fix GHES estatus iteration To: gengdongjiu , "Rafael J. Wysocki" , Will Deacon Cc: lenb@kernel.org, james.morse@arm.com, bp@suse.de, shiju.jose@huawei.com, geliangtang@gmail.com, andriy.shevchenko@linux.intel.com, tony.luck@intel.com, linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org, austinwc@codeaurora.org References: <1501795945-5331-1-git-send-email-tbaicar@codeaurora.org> <20170808163219.GB2163@arm.com> <1971058.u9kdacGNiq@aspire.rjw.lan> <5591b641-6eb0-0cb6-e74d-7e55cad5ee33@huawei.com> From: "Baicar, Tyler" Message-ID: Date: Fri, 11 Aug 2017 07:17:48 -0600 User-Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:52.0) Gecko/20100101 Thunderbird/52.2.1 MIME-Version: 1.0 In-Reply-To: <5591b641-6eb0-0cb6-e74d-7e55cad5ee33@huawei.com> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Content-Language: en-US Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2210 Lines: 53 On 8/10/2017 4:04 PM, gengdongjiu wrote: > > On 2017/8/9 8:52, Rafael J. Wysocki wrote: >> On Tuesday, August 8, 2017 6:32:20 PM CEST Will Deacon wrote: >>> On Thu, Aug 03, 2017 at 03:32:25PM -0600, Tyler Baicar wrote: >>>> Currently iterating through the GHES estatus blocks does not >>>> take into account the new generic data v3 structure size. This >>>> can result in garbage non-standard trace events to be triggered >>>> since the loop will not properly iterate through the estatus >>>> blocks and not properly terminate. >>>> >>>> Update the GHES estatus iteration to properly increment through >>>> the estatus blocks similar to how the CPER estatus printing >>>> iterates through them. >>>> >>>> Fixes: bbcc2e7b642e ("ras: acpi/apei: cper: add support for generic data v3 structure") > I do not think it has relationship with this "bbcc2e7b642e", the root cause is that it > does not consider the acpi_hest_generic_data_v300, this code exists long long time ago. bbcc2e7b642e ("ras: acpi/apei: cper: add support for generic data v3 structure") is the change that introduced the generic data v300 structure and this patch does fix the issue with this iteration not taking into account the size difference of the v300 structure. > > so please also review this patch: > https://lkml.org/lkml/2017/8/10/747 > > previously I do not see Tyler's this modification, just know see it. > > >>>> Signed-off-by: Tyler Baicar >>>> Tested-by: Austin Christ >>>> --- >>>> drivers/acpi/apei/apei-internal.h | 5 ----- >>>> drivers/acpi/apei/ghes.c | 8 +++++++- >>>> 2 files changed, 7 insertions(+), 6 deletions(-) >>> Whilst much of the initial code here went through the arm64 tree in the >>> previous merge window, I'm assuming that Boris will take this fix via his >>> tree (likewise for "[PATCH V2] acpi: apei: clear error status before >>> acknowledging the error"). >> Actually I will if Boris ACKs these. >> >> Thanks, >> Rafael >> >> >> . >> -- Qualcomm Datacenter Technologies, Inc. as an affiliate of Qualcomm Technologies, Inc. Qualcomm Technologies, Inc. is a member of the Code Aurora Forum, a Linux Foundation Collaborative Project.