Re: [PATCH v2] nfs: drop the incorrect assertion in nfs_swap_rw()

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jun 18, 2024 at 8:57 AM Barry Song <21cnbao@xxxxxxxxx> wrote:
>
> From: Christoph Hellwig <hch@xxxxxx>
>
> Since commit 2282679fb20b ("mm: submit multipage write for SWP_FS_OPS
> swap-space"), we can plug multiple pages then unplug them all together.
> That means iov_iter_count(iter) could be way bigger than PAGE_SIZE, it
> actually equals the size of iov_iter_npages(iter, INT_MAX).
>
> Note this issue has nothing to do with large folios as we don't support
> THP_SWPOUT to non-block devices.
>
> Fixes: 2282679fb20b ("mm: submit multipage write for SWP_FS_OPS swap-space")
> Reported-by: Christoph Hellwig <hch@xxxxxx>
> Closes: https://lore.kernel.org/linux-mm/20240617053201.GA16852@xxxxxx/
> Cc: NeilBrown <neilb@xxxxxxx>
> Cc: Anna Schumaker <anna@xxxxxxxxxx>
> Cc: Steve French <sfrench@xxxxxxxxx>
> Cc: Trond Myklebust <trondmy@xxxxxxxxxx>
> Cc: Chuanhua Han <hanchuanhua@xxxxxxxx>
> Cc: Ryan Roberts <ryan.roberts@xxxxxxx>
> Cc: Chris Li <chrisl@xxxxxxxxxx>
> Cc: "Huang, Ying" <ying.huang@xxxxxxxxx>
> Cc: Jeff Layton <jlayton@xxxxxxxxxx>
> Cc: <stable@xxxxxxxxxxxxxxx>
> Cc: Matthew Wilcox <willy@xxxxxxxxxxxxx>
> Cc: Martin Wege <martin.l.wege@xxxxxxxxx>
> Signed-off-by: Christoph Hellwig <hch@xxxxxx>
> [Barry: figure out the cause and correct the commit message]
> Signed-off-by: Barry Song <v-songbaohua@xxxxxxxx>
> ---
>  fs/nfs/direct.c | 2 --
>  1 file changed, 2 deletions(-)
>
> diff --git a/fs/nfs/direct.c b/fs/nfs/direct.c
> index bb2f583eb28b..90079ca134dd 100644
> --- a/fs/nfs/direct.c
> +++ b/fs/nfs/direct.c

Please add a

Reviewed-by: Martin Wege <martin.l.wege@xxxxxxxxx>

Thanks,
Martin





[Index of Archives]     [Linux Filesystem Development]     [Linux USB Development]     [Linux Media Development]     [Video for Linux]     [Linux NILFS]     [Linux Audio Users]     [Yosemite Info]     [Linux SCSI]

  Powered by Linux