From mboxrd@z Thu Jan 1 00:00:00 1970 From: Steven Whitehouse Date: Tue, 31 Oct 2006 08:56:45 +0000 Subject: [Cluster-devel] Re: [PATCH 4/4] gfs2: ->readpages() fixes In-Reply-To: <87slh5lgbo.fsf@duaron.myhome.or.jp> References: <87slh5lgbo.fsf@duaron.myhome.or.jp> Message-ID: <1162285005.27980.278.camel@quoit.chygwyn.com> List-Id: To: cluster-devel.redhat.com MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit 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 Acked-by: Steven Whitehouse 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 at vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/