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