RE: [PATCH] EDAC/igen6: Fix the flood of invalid error reports

From: Zhuo, Qiuxu
Date: Wed Feb 12 2025 - 08:48:58 EST


> From: Markus Elfring <Markus.Elfring@xxxxxx>
> [...]
> > @@ -785,13 +785,22 @@ static u64 ecclog_read_and_clear(struct
> > igen6_imc *imc) {
>
> > + if (ecclog == ~0)
>
> > + if (!(ecclog & (ECC_ERROR_LOG_CE | ECC_ERROR_LOG_UE)))
>
>
> May these condition checks be combined without repeating the statement
> “return 0;”?

Thanks for the suggestion. Yes.
However, I think that using separate "return 0" statements makes the
code more readable.

-Qiuxu