From: Bjorn Helgaas <helgaas@kernel.org>
To: Tyler Baicar <tbaicar@codeaurora.org>
Cc: rjw@rjwysocki.net, lenb@kernel.org, bhelgaas@google.com,
paul.gortmaker@windriver.com, sudipm.mukherjee@gmail.com,
axboe@fb.com, izumi.taku@jp.fujitsu.com,
linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-pci@vger.kernel.org, bp@suse.de
Subject: Re: [PATCH V2 0/2] Correct AER severity for printing AER information
Date: Tue, 20 Sep 2016 14:41:03 -0500 [thread overview]
Message-ID: <20160920194103.GD4941@localhost> (raw)
In-Reply-To: <1473887686-24420-1-git-send-email-tbaicar@codeaurora.org>
On Wed, Sep 14, 2016 at 03:14:44PM -0600, Tyler Baicar wrote:
> AER severity handling has two issues that cause the AER information to
> be printed incorrectly. The first issue is that the function to calculate
> the AER severity is called twice in the code path to print the AER
> information. The second issue is that the original call to calculate the
> AER severity expects the CPER severity but is recieving the GHES
> severity.
>
> V2: Fix minor typo in commit text.
>
> V1: https://lkml.org/lkml/2016/9/12/1075
>
> Tyler Baicar (2):
> PCI/AER: Remove duplicate AER severity translation
> acpi: apei: send correct severity to calculate AER severity
>
> drivers/acpi/apei/ghes.c | 2 +-
> drivers/pci/pcie/aer/aerdrv_errprint.c | 6 ++----
> include/linux/aer.h | 2 +-
> 3 files changed, 4 insertions(+), 6 deletions(-)
I applied both to pci/aer for v4.9, thanks!
Rafael, the second one is strictly ACPI, and I'd be happy to let you
deal with it if you prefer.
Bjorn
next prev parent reply other threads:[~2016-09-20 19:41 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-09-14 21:14 [PATCH V2 0/2] Correct AER severity for printing AER information Tyler Baicar
2016-09-14 21:14 ` [PATCH V2 1/2] PCI/AER: Remove duplicate AER severity translation Tyler Baicar
2016-09-14 21:14 ` [PATCH V2 2/2] acpi: apei: send correct severity to calculate AER severity Tyler Baicar
2016-09-20 19:41 ` Bjorn Helgaas [this message]
2016-09-21 0:27 ` [PATCH V2 0/2] Correct AER severity for printing AER information Rafael J. Wysocki
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20160920194103.GD4941@localhost \
--to=helgaas@kernel.org \
--cc=axboe@fb.com \
--cc=bhelgaas@google.com \
--cc=bp@suse.de \
--cc=izumi.taku@jp.fujitsu.com \
--cc=lenb@kernel.org \
--cc=linux-acpi@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-pci@vger.kernel.org \
--cc=paul.gortmaker@windriver.com \
--cc=rjw@rjwysocki.net \
--cc=sudipm.mukherjee@gmail.com \
--cc=tbaicar@codeaurora.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).