[PATCH] EDAC/versalnet: Pass flags=0 to memory_failure() for UE errors

Shubhrajyoti Datta <[email protected]>
Newsgroups org.kernel.vger.linux-edac,org.kernel.vger.linux-kernel
Message-ID <[email protected]>
The MF_ACTION_REQUIRED flag is intended for synchronous exceptions
where the consuming process must be killed. For EDAC-reported errors
discovered asynchronously via polling/interrupt, flags=0 is the
correct choice, matching how other EDAC drivers invoke
memory_failure().

Signed-off-by: Shubhrajyoti Datta <[email protected]>
---

 drivers/edac/versalnet_edac.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/edac/versalnet_edac.c b/drivers/edac/versalnet_edac.c
index 97ec05d68bbb..d1af5e175f7e 100644
--- a/drivers/edac/versalnet_edac.c
+++ b/drivers/edac/versalnet_edac.c
@@ -464,7 +464,7 @@ static void handle_error(struct mc_priv  *priv, struct ecc_status *stat,
 		pfn = PHYS_PFN(pa);
 
 		if (IS_ENABLED(CONFIG_MEMORY_FAILURE)) {
-			err = memory_failure(pfn, MF_ACTION_REQUIRED);
+			err = memory_failure(pfn, 0);
 			if (err)
 				edac_dbg(2, "memory_failure() error: %d", err);
 			else
-- 
2.49.1
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.