[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <20251129090122.2457896-3-zhangshida@kylinos.cn>
Date: Sat, 29 Nov 2025 17:01:15 +0800
From: zhangshida <starzhangzsd@...il.com>
To: Johannes.Thumshirn@....com,
hch@...radead.org,
agruenba@...hat.com,
ming.lei@...hat.com,
hsiangkao@...ux.alibaba.com,
csander@...estorage.com
Cc: linux-block@...r.kernel.org,
linux-bcache@...r.kernel.org,
nvdimm@...ts.linux.dev,
virtualization@...ts.linux.dev,
ntfs3@...ts.linux.dev,
linux-xfs@...r.kernel.org,
linux-kernel@...r.kernel.org,
zhangshida@...inos.cn,
starzhangzsd@...il.com
Subject: [PATCH v3 2/9] block: prohibit calls to bio_chain_endio
From: Shida Zhang <zhangshida@...inos.cn>
Now that all potential callers of bio_chain_endio have been
eliminated, completely prohibit any future calls to this function.
Suggested-by: Ming Lei <ming.lei@...hat.com>
Suggested-by: Andreas Gruenbacher <agruenba@...hat.com>
Suggested-by: Christoph Hellwig <hch@...radead.org>
Signed-off-by: Shida Zhang <zhangshida@...inos.cn>
---
block/bio.c | 6 +++++-
1 file changed, 5 insertions(+), 1 deletion(-)
diff --git a/block/bio.c b/block/bio.c
index b3a79285c27..1b5e4577f4c 100644
--- a/block/bio.c
+++ b/block/bio.c
@@ -320,9 +320,13 @@ static struct bio *__bio_chain_endio(struct bio *bio)
return parent;
}
+/**
+ * This function should only be used as a flag and must never be called.
+ * If execution reaches here, it indicates a serious programming error.
+ */
static void bio_chain_endio(struct bio *bio)
{
- bio_endio(__bio_chain_endio(bio));
+ BUG_ON(1);
}
/**
--
2.34.1
Powered by blists - more mailing lists