From: wufan <wufan@codeaurora.org>
To: 'Borislav Petkov' <bp@alien8.de>
Cc: mchehab@kernel.org, james.morse@arm.com, baicar.tyler@gmail.com,
linux-edac@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-arm-kernel@lists.infradead.org, john.garry@huawei.com,
toshi.kani@hpe.com, tanxiaofei@huawei.com,
wanghuiqiang@huawei.com, shiju.jose@huawei.com
Subject: [v3] EDAC, ghes: use CPER module handles to locate DIMMs
Date: Tue, 4 Sep 2018 07:22:25 -0600 [thread overview]
Message-ID: <003601d44452$531d4060$f957c120$@codeaurora.org> (raw)
> -----Original Message-----
> From: Borislav Petkov <bp@alien8.de>
> Sent: Tuesday, September 4, 2018 1:29 AM
> To: Fan Wu <wufan@codeaurora.org>
> Cc: mchehab@kernel.org; james.morse@arm.com; baicar.tyler@gmail.com;
> linux-edac@vger.kernel.org; linux-kernel@vger.kernel.org; linux-arm-
> kernel@lists.infradead.org; john.garry@huawei.com; toshi.kani@hpe.com;
> tanxiaofei@huawei.com; wanghuiqiang@huawei.com;
> shiju.jose@huawei.com
> Subject: Re: [PATCH v3] EDAC, ghes: use CPER module handles to locate
> DIMMs
>
> On Tue, Sep 04, 2018 at 03:50:55AM +0000, Fan Wu wrote:
> > For platforms whose firmwares provide valid module handles (SMBIOS
> > type 17) in error records, this patch uses the module handles to
> > locate corresponding DIMMs and enables per-DIMM error counter update.
> >
> > Signed-off-by: Fan Wu <wufan@codeaurora.org>
> > Reviewed-by: Tyler Baicar <baicar.tyler@gmail.com>
> > Tested-by: Toshi Kani <toshi.kani@hpe.com>
>
> Those two tags I did see being given to you ...
>
> > Reviewed-by: Borislav Petkov <bp@alien8.de>
> > Reviewed-by: James Morse <james.morse@arm.com>
> > Reviewed-by: tanxiaofei <tanxiaofei@huawei.com>
>
> ... but how in the world did you come up with those?
Sorry. Will remove these in v4.
Thanks,
Fan
> Do you understand how Reviewed-by works or do you need to look at
> Documentation/process/submitting-patches.rst, section 13 ?
>
> --
> Regards/Gruss,
> Boris.
>
> Good mailing practices for 400: avoid top-posting and trim the reply.
WARNING: multiple messages have this Message-ID (diff)
From: wufan@codeaurora.org (wufan)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH v3] EDAC, ghes: use CPER module handles to locate DIMMs
Date: Tue, 4 Sep 2018 07:22:25 -0600 [thread overview]
Message-ID: <003601d44452$531d4060$f957c120$@codeaurora.org> (raw)
In-Reply-To: <20180904072858.GA32615@zn.tnic>
> -----Original Message-----
> From: Borislav Petkov <bp@alien8.de>
> Sent: Tuesday, September 4, 2018 1:29 AM
> To: Fan Wu <wufan@codeaurora.org>
> Cc: mchehab at kernel.org; james.morse at arm.com; baicar.tyler at gmail.com;
> linux-edac at vger.kernel.org; linux-kernel at vger.kernel.org; linux-arm-
> kernel at lists.infradead.org; john.garry at huawei.com; toshi.kani at hpe.com;
> tanxiaofei at huawei.com; wanghuiqiang at huawei.com;
> shiju.jose at huawei.com
> Subject: Re: [PATCH v3] EDAC, ghes: use CPER module handles to locate
> DIMMs
>
> On Tue, Sep 04, 2018 at 03:50:55AM +0000, Fan Wu wrote:
> > For platforms whose firmwares provide valid module handles (SMBIOS
> > type 17) in error records, this patch uses the module handles to
> > locate corresponding DIMMs and enables per-DIMM error counter update.
> >
> > Signed-off-by: Fan Wu <wufan@codeaurora.org>
> > Reviewed-by: Tyler Baicar <baicar.tyler@gmail.com>
> > Tested-by: Toshi Kani <toshi.kani@hpe.com>
>
> Those two tags I did see being given to you ...
>
> > Reviewed-by: Borislav Petkov <bp@alien8.de>
> > Reviewed-by: James Morse <james.morse@arm.com>
> > Reviewed-by: tanxiaofei <tanxiaofei@huawei.com>
>
> ... but how in the world did you come up with those?
Sorry. Will remove these in v4.
Thanks,
Fan
> Do you understand how Reviewed-by works or do you need to look at
> Documentation/process/submitting-patches.rst, section 13 ?
>
> --
> Regards/Gruss,
> Boris.
>
> Good mailing practices for 400: avoid top-posting and trim the reply.
WARNING: multiple messages have this Message-ID (diff)
From: "wufan" <wufan@codeaurora.org>
To: "'Borislav Petkov'" <bp@alien8.de>
Cc: <mchehab@kernel.org>, <james.morse@arm.com>,
<baicar.tyler@gmail.com>, <linux-edac@vger.kernel.org>,
<linux-kernel@vger.kernel.org>,
<linux-arm-kernel@lists.infradead.org>, <john.garry@huawei.com>,
<toshi.kani@hpe.com>, <tanxiaofei@huawei.com>,
<wanghuiqiang@huawei.com>, <shiju.jose@huawei.com>
Subject: RE: [PATCH v3] EDAC, ghes: use CPER module handles to locate DIMMs
Date: Tue, 4 Sep 2018 07:22:25 -0600 [thread overview]
Message-ID: <003601d44452$531d4060$f957c120$@codeaurora.org> (raw)
In-Reply-To: <20180904072858.GA32615@zn.tnic>
> -----Original Message-----
> From: Borislav Petkov <bp@alien8.de>
> Sent: Tuesday, September 4, 2018 1:29 AM
> To: Fan Wu <wufan@codeaurora.org>
> Cc: mchehab@kernel.org; james.morse@arm.com; baicar.tyler@gmail.com;
> linux-edac@vger.kernel.org; linux-kernel@vger.kernel.org; linux-arm-
> kernel@lists.infradead.org; john.garry@huawei.com; toshi.kani@hpe.com;
> tanxiaofei@huawei.com; wanghuiqiang@huawei.com;
> shiju.jose@huawei.com
> Subject: Re: [PATCH v3] EDAC, ghes: use CPER module handles to locate
> DIMMs
>
> On Tue, Sep 04, 2018 at 03:50:55AM +0000, Fan Wu wrote:
> > For platforms whose firmwares provide valid module handles (SMBIOS
> > type 17) in error records, this patch uses the module handles to
> > locate corresponding DIMMs and enables per-DIMM error counter update.
> >
> > Signed-off-by: Fan Wu <wufan@codeaurora.org>
> > Reviewed-by: Tyler Baicar <baicar.tyler@gmail.com>
> > Tested-by: Toshi Kani <toshi.kani@hpe.com>
>
> Those two tags I did see being given to you ...
>
> > Reviewed-by: Borislav Petkov <bp@alien8.de>
> > Reviewed-by: James Morse <james.morse@arm.com>
> > Reviewed-by: tanxiaofei <tanxiaofei@huawei.com>
>
> ... but how in the world did you come up with those?
Sorry. Will remove these in v4.
Thanks,
Fan
> Do you understand how Reviewed-by works or do you need to look at
> Documentation/process/submitting-patches.rst, section 13 ?
>
> --
> Regards/Gruss,
> Boris.
>
> Good mailing practices for 400: avoid top-posting and trim the reply.
next reply other threads:[~2018-09-04 13:22 UTC|newest]
Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-09-04 13:22 wufan [this message]
2018-09-04 13:22 ` [PATCH v3] EDAC, ghes: use CPER module handles to locate DIMMs wufan
2018-09-04 13:22 ` wufan
-- strict thread matches above, loose matches on Subject: below --
2018-09-04 7:28 [v3] " Borislav Petkov
2018-09-04 7:28 ` [PATCH v3] " Borislav Petkov
2018-09-04 7:28 ` Borislav Petkov
2018-09-04 3:50 [v3] " wufan
2018-09-04 3:50 ` [PATCH v3] " Fan Wu
2018-09-04 3:50 ` Fan Wu
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='003601d44452$531d4060$f957c120$@codeaurora.org' \
--to=wufan@codeaurora.org \
--cc=baicar.tyler@gmail.com \
--cc=bp@alien8.de \
--cc=james.morse@arm.com \
--cc=john.garry@huawei.com \
--cc=linux-arm-kernel@lists.infradead.org \
--cc=linux-edac@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=mchehab@kernel.org \
--cc=shiju.jose@huawei.com \
--cc=tanxiaofei@huawei.com \
--cc=toshi.kani@hpe.com \
--cc=wanghuiqiang@huawei.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.