On 7/5/22 10:22 AM, xiubli@xxxxxxxxxx wrote:
From: Xiubo Li <xiubli@xxxxxxxxxx> And also the comment said it will drop the folio references but the code was increasing it. Signed-off-by: Xiubo Li <xiubli@xxxxxxxxxx> --- fs/netfs/buffered_read.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/fs/netfs/buffered_read.c b/fs/netfs/buffered_read.c index 5b93e22397fe..a44a5b3b8d4c 100644 --- a/fs/netfs/buffered_read.c +++ b/fs/netfs/buffered_read.c @@ -396,9 +396,6 @@ int netfs_write_begin(struct netfs_inode *ctx, */ ractl._nr_pages = folio_nr_pages(folio); netfs_rreq_expand(rreq, &ractl); - - /* We hold the folio locks, so we can drop the references */ - folio_get(folio); while (readahead_folio(&ractl)) ;
Will drop this patch, the above fix it incorrect. Thanks! -- Linux-cachefs mailing list Linux-cachefs@xxxxxxxxxx https://listman.redhat.com/mailman/listinfo/linux-cachefs
