Re: [PATCH v2 1/6] fuse: limit the length of ITER_KVEC dio by max_pages

From: Miklos Szeredi
Date: Fri Mar 01 2024 - 08:45:04 EST


On Wed, 28 Feb 2024 at 15:40, Hou Tao <houtao@xxxxxxxxxxxxxxx> wrote:

> So instead of limiting both the values of max_read and max_write in
> kernel, capping the maximal length of kvec iter IO by using max_pages in
> fuse_direct_io() just like it does for ubuf/iovec iter IO. Now the max
> value for max_pages is 256, so on host with 4KB page size, the maximal
> size passed to kmalloc() in copy_args_to_argbuf() is about 1MB+40B. The
> allocation of 2MB of physically contiguous memory will still incur
> significant stress on the memory subsystem, but the warning is fixed.
> Additionally, the requirement for huge physically contiguous memory will
> be removed in the following patch.

So the issue will be fixed properly by following patches?

In that case this patch could be omitted, right?

Thanks,
Miklos