2020-06-10 07:10:20

by Zhenzhong Duan

Subject: [PATCH] EDAC, pnd2: set MCE_PRIO_EDAC priority for pnd2_mce_dec notifier

...or else it has MCE_PRIO_LOWEST priority by default.

Signed-off-by: Zhenzhong Duan <[email protected]>
---
drivers/edac/pnd2_edac.c | 1 +
1 file changed, 1 insertion(+)

diff --git a/drivers/edac/pnd2_edac.c b/drivers/edac/pnd2_edac.c
index bc47328..368fae3 100644
--- a/drivers/edac/pnd2_edac.c
+++ b/drivers/edac/pnd2_edac.c
@@ -1434,6 +1434,7 @@ static int pnd2_mce_check_error(struct notifier_block *nb, unsigned long val, vo

static struct notifier_block pnd2_mce_dec = {
.notifier_call = pnd2_mce_check_error,
+ .priority = MCE_PRIO_EDAC,
};

#ifdef CONFIG_EDAC_DEBUG
--
1.8.3.1

2020-06-10 17:30:52

by Tony Luck

Subject: Re: [PATCH] EDAC, pnd2: set MCE_PRIO_EDAC priority for pnd2_mce_dec notifier

On Wed, Jun 10, 2020 at 02:58:45PM +0800, Zhenzhong Duan wrote:
> ...or else it has MCE_PRIO_LOWEST priority by default.
>
> Signed-off-by: Zhenzhong Duan <[email protected]>
> ---
> drivers/edac/pnd2_edac.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/drivers/edac/pnd2_edac.c b/drivers/edac/pnd2_edac.c
> index bc47328..368fae3 100644
> --- a/drivers/edac/pnd2_edac.c
> +++ b/drivers/edac/pnd2_edac.c
> @@ -1434,6 +1434,7 @@ static int pnd2_mce_check_error(struct notifier_block *nb, unsigned long val, vo
>
> static struct notifier_block pnd2_mce_dec = {
> .notifier_call = pnd2_mce_check_error,
> + .priority = MCE_PRIO_EDAC,
> };
>
> #ifdef CONFIG_EDAC_DEBUG
> --

Looks OK. I'll queue this next week (after Linus releases v5.8-rc1). It should
be merged into v5.9

Thanks

-Tony