From 6c1487c4e15948af480a4ad0b977428faf1f1f54 Mon Sep 17 00:00:00 2001 From: Patrick Farrell Date: Fri, 27 Oct 2023 16:39:22 -0400 Subject: [PATCH] EX-8421 llite: remove LBUG() from ll_readpage() This LBUG() has been hitting sometimes in sanity-PCC, which means EX-8421 is not completely fixed. Until we can fully sort out EX-8421, we don't want to have this LBUG enabled on customer systems. The underlying bug has been present for some time and the first attempt at an EX-8421 fix improves the situation. So, remove the LBUG(), with the intent of putting it back later once EX-8421 is fixed for real. Fixes: 3a701bf587 ("EX-8421 llite: disable kernel readahead for pcc mmap") Signed-off-by: Patrick Farrell Change-Id: I21f951d38f67b37626f33068d2a4b64377f4c46a Reviewed-on: https://review.whamcloud.com/c/ex/lustre-release/+/52858 Tested-by: Andreas Dilger Reviewed-by: Andreas Dilger --- lustre/llite/rw.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/lustre/llite/rw.c b/lustre/llite/rw.c index 00b4d3d..837c9c5 100644 --- a/lustre/llite/rw.c +++ b/lustre/llite/rw.c @@ -2091,8 +2091,9 @@ int ll_readpage(struct file *file, struct page *vmpage) ra_assert = true; } #endif + /* See EX-8421 if (ra_assert) - LBUG(); + LBUG();*/ /** * Direct read can fall back to buffered read, but DIO is done * with lockless i/o, and buffered requires LDLM locking, so in -- 1.8.3.1