Thread (23 messages) 23 messages, 7 authors, 2025-03-12

Re: [PATCH v2] mm: alloc_pages_bulk: remove assumption of populating only NULL elements

From: Dave Chinner <david@fromorbit.com>
Date: 2025-03-08 06:43:58
Also in: kvm, linux-btrfs, linux-mm, linux-nfs, linux-xfs, lkml, virtualization

On Tue, Mar 04, 2025 at 08:09:35PM +0800, Yunsheng Lin wrote:
On 2025/3/4 16:18, Dave Chinner wrote:

...
quoted
quoted
1. https://lore.kernel.org/all/bd8c2f5c-464d-44ab-b607-390a87ea4cd5@huawei.com/ (local)
2. https://lore.kernel.org/all/20250212092552.1779679-1-linyunsheng@huawei.com/ (local)
CC: Jesper Dangaard Brouer <hawk@kernel.org>
CC: Luiz Capitulino <redacted>
CC: Mel Gorman <redacted>
CC: Dave Chinner <david@fromorbit.com>
CC: Chuck Lever <redacted>
Signed-off-by: Yunsheng Lin <redacted>
Acked-by: Jeff Layton <jlayton@kernel.org>
---
V2:
1. Drop RFC tag and rebased on latest linux-next.
2. Fix a compile error for xfs.
And you still haven't tested the code changes to XFS, because
this patch is also broken.
I tested XFS using the below cmd and testcase, testing seems
to be working fine, or am I missing something obvious here
as I am not realy familiar with fs subsystem yet:
That's hardly what I'd call a test. It barely touches the filesystem
at all, and it is not exercising memory allocation failure paths at
all.

Go look up fstests and use that to test the filesystem changes you
are making. You can use that to test btrfs and NFS, too.

-Dave.

-- 
Dave Chinner
david@fromorbit.com
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help