On 2025-11-06, John Ogness <[email protected]> wrote: >> I think that we should do the following: >> >> diff --git a/kernel/printk/printk_ringbuffer.c >> b/kernel/printk/printk_ringbuffer.c >> index 839f504db6d3..78e02711872e 100644 >> --- a/kernel/printk/printk_ringbuffer.c >> +++ b/kernel/printk/printk_ringbuffer.c >> @@ -1260,9 +1260,8 @@ static const char *get_data(struct prb_data_ring >> *data_ring, >> return NULL; >> } >> >> - /* Regular data block: @begin less than @next and in same wrap. */ >> - if (!is_blk_wrapped(data_ring, blk_lpos->begin, blk_lpos->next) && >> - blk_lpos->begin < blk_lpos->next) { >> + /* Regular data block: @begin and @next in same wrap. */ >> + if (!is_blk_wrapped(data_ring, blk_lpos->begin, blk_lpos->next)) { >> db = to_block(data_ring, blk_lpos->begin); >> *data_size = blk_lpos->next - blk_lpos->begin;
Upon further consideration, your suggestion here is better. The wrapping data block detection should continue to make sure there is exactly one 1 wrap. The size check will not catch the case where there are multiple wraps. John _______________________________________________ Linux-f2fs-devel mailing list [email protected] https://lists.sourceforge.net/lists/listinfo/linux-f2fs-devel
