Messages in this thread | | | Subject | Re: [PATCH 4/4] gfs2: ->readpages() fixes | From | Steven Whitehouse <> | Date | Tue, 31 Oct 2006 08:56:45 +0000 |
| |
Hi,
On Tue, 2006-10-31 at 05:05 +0900, OGAWA Hirofumi wrote: > This just ignore the remaining pages, and remove unneeded unlock_pages(). > > Signed-off-by: OGAWA Hirofumi <hirofumi@mail.parknet.co.jp> Acked-by: Steven Whitehouse <swhiteho@redhat.com>
Thanks for fixing this,
Steve.
> --- > > fs/gfs2/ops_address.c | 7 ------- > 1 file changed, 7 deletions(-) > > diff -puN fs/gfs2/ops_address.c~readpages-fixes-gfs2 fs/gfs2/ops_address.c > --- linux-2.6/fs/gfs2/ops_address.c~readpages-fixes-gfs2 2006-10-31 04:26:20.000000000 +0900 > +++ linux-2.6-hirofumi/fs/gfs2/ops_address.c 2006-10-31 04:26:20.000000000 +0900 > @@ -337,13 +337,6 @@ out: > out_noerror: > ret = 0; > out_unlock: > - /* unlock all pages, we can't do any I/O right now */ > - for (page_idx = 0; page_idx < nr_pages; page_idx++) { > - struct page *page = list_entry(pages->prev, struct page, lru); > - list_del(&page->lru); > - unlock_page(page); > - page_cache_release(page); > - } > if (do_unlock) > gfs2_holder_uninit(&gh); > goto out; > _ > - > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/
- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
| |