* [Cluster-devel] [PATCH v2 08/18] gfs2: use ->invalidatepage() length argument [not found] <1360055531-26309-1-git-send-email-lczerner@redhat.com> @ 2013-02-05 9:12 ` Lukas Czerner 2013-02-05 9:47 ` Steven Whitehouse 0 siblings, 1 reply; 2+ messages in thread From: Lukas Czerner @ 2013-02-05 9:12 UTC (permalink / raw) To: cluster-devel.redhat.com ->invalidatepage() aop now accepts range to invalidate so we can make use of it in gfs2_invalidatepage(). Signed-off-by: Lukas Czerner <lczerner@redhat.com> Cc: cluster-devel at redhat.com --- fs/gfs2/aops.c | 9 +++++++-- 1 files changed, 7 insertions(+), 2 deletions(-) diff --git a/fs/gfs2/aops.c b/fs/gfs2/aops.c index 5bd558c..3cf3dc8 100644 --- a/fs/gfs2/aops.c +++ b/fs/gfs2/aops.c @@ -949,24 +949,29 @@ static void gfs2_invalidatepage(struct page *page, unsigned int offset, unsigned int length) { struct gfs2_sbd *sdp = GFS2_SB(page->mapping->host); + unsigned int stop = offset + length; + int partial_page = (offset || length < PAGE_CACHE_SIZE); struct buffer_head *bh, *head; unsigned long pos = 0; BUG_ON(!PageLocked(page)); - if (offset == 0) + if (!partial_page) ClearPageChecked(page); if (!page_has_buffers(page)) goto out; bh = head = page_buffers(page); do { + if (pos + bh->b_size > stop) + return; + if (offset <= pos) gfs2_discard(sdp, bh); pos += bh->b_size; bh = bh->b_this_page; } while (bh != head); out: - if (offset == 0) + if (!partial_page) try_to_release_page(page, 0); } -- 1.7.7.6 ^ permalink raw reply related [flat|nested] 2+ messages in thread
* [Cluster-devel] [PATCH v2 08/18] gfs2: use ->invalidatepage() length argument 2013-02-05 9:12 ` [Cluster-devel] [PATCH v2 08/18] gfs2: use ->invalidatepage() length argument Lukas Czerner @ 2013-02-05 9:47 ` Steven Whitehouse 0 siblings, 0 replies; 2+ messages in thread From: Steven Whitehouse @ 2013-02-05 9:47 UTC (permalink / raw) To: cluster-devel.redhat.com Hi, Acked-by: Steven Whitehouse <swhiteho@redhat.com> Steve. On Tue, 2013-02-05 at 10:12 +0100, Lukas Czerner wrote: > ->invalidatepage() aop now accepts range to invalidate so we can make > use of it in gfs2_invalidatepage(). > > Signed-off-by: Lukas Czerner <lczerner@redhat.com> > Cc: cluster-devel at redhat.com > --- > fs/gfs2/aops.c | 9 +++++++-- > 1 files changed, 7 insertions(+), 2 deletions(-) > > diff --git a/fs/gfs2/aops.c b/fs/gfs2/aops.c > index 5bd558c..3cf3dc8 100644 > --- a/fs/gfs2/aops.c > +++ b/fs/gfs2/aops.c > @@ -949,24 +949,29 @@ static void gfs2_invalidatepage(struct page *page, unsigned int offset, > unsigned int length) > { > struct gfs2_sbd *sdp = GFS2_SB(page->mapping->host); > + unsigned int stop = offset + length; > + int partial_page = (offset || length < PAGE_CACHE_SIZE); > struct buffer_head *bh, *head; > unsigned long pos = 0; > > BUG_ON(!PageLocked(page)); > - if (offset == 0) > + if (!partial_page) > ClearPageChecked(page); > if (!page_has_buffers(page)) > goto out; > > bh = head = page_buffers(page); > do { > + if (pos + bh->b_size > stop) > + return; > + > if (offset <= pos) > gfs2_discard(sdp, bh); > pos += bh->b_size; > bh = bh->b_this_page; > } while (bh != head); > out: > - if (offset == 0) > + if (!partial_page) > try_to_release_page(page, 0); > } > ^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2013-02-05 9:47 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <1360055531-26309-1-git-send-email-lczerner@redhat.com>
2013-02-05 9:12 ` [Cluster-devel] [PATCH v2 08/18] gfs2: use ->invalidatepage() length argument Lukas Czerner
2013-02-05 9:47 ` Steven Whitehouse
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox