Message ID | 1365498867-27782-7-git-send-email-lczerner@redhat.com |
---|---|
State | Superseded, archived |
Headers | show |
On Tue 09-04-13 11:14:15, Lukas Czerner wrote: > ->invalidatepage() aop now accepts range to invalidate so we can make > use of it in ocfs2_invalidatepage(). Looks good. You can add: Reviewed-by: Jan Kara <jack@suse.cz> Honza > > Signed-off-by: Lukas Czerner <lczerner@redhat.com> > Cc: Joel Becker <jlbec@evilplan.org> > --- > fs/ocfs2/aops.c | 3 +-- > 1 files changed, 1 insertions(+), 2 deletions(-) > > diff --git a/fs/ocfs2/aops.c b/fs/ocfs2/aops.c > index 7c47755..79736a2 100644 > --- a/fs/ocfs2/aops.c > +++ b/fs/ocfs2/aops.c > @@ -608,8 +608,7 @@ static void ocfs2_invalidatepage(struct page *page, unsigned int offset, > { > journal_t *journal = OCFS2_SB(page->mapping->host->i_sb)->journal->j_journal; > > - jbd2_journal_invalidatepage(journal, page, offset, > - PAGE_CACHE_SIZE - offset); > + jbd2_journal_invalidatepage(journal, page, offset, length); > } > > static int ocfs2_releasepage(struct page *page, gfp_t wait) > -- > 1.7.7.6 > > -- > To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html
On Tue, Apr 09, 2013 at 11:14:15AM +0200, Lukas Czerner wrote: > ->invalidatepage() aop now accepts range to invalidate so we can make > use of it in ocfs2_invalidatepage(). > > Signed-off-by: Lukas Czerner <lczerner@redhat.com> > Cc: Joel Becker <jlbec@evilplan.org> +Mark Fasheh, ocfs2-devel To the ocfs2 development team, Since half of this patch series modifies ext4 extensively, and changes to the other file systems are relatively small, I plan to carry the invalidatepage patch set in the ext4 tree for the next development cycle (i.e., not the upcoming merge window, but the next one). To that end, it would be great if you take a look at this patch set and send us an Acked-by signoff. Thanks!! - Ted -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Acked-by: Joel Becker <jlbec@evilplan.org> On Tue, Apr 23, 2013 at 10:16:04AM -0400, Theodore Ts'o wrote: > On Tue, Apr 09, 2013 at 11:14:15AM +0200, Lukas Czerner wrote: > > ->invalidatepage() aop now accepts range to invalidate so we can make > > use of it in ocfs2_invalidatepage(). > > > > Signed-off-by: Lukas Czerner <lczerner@redhat.com> > > Cc: Joel Becker <jlbec@evilplan.org> > > +Mark Fasheh, ocfs2-devel > > To the ocfs2 development team, > > Since half of this patch series modifies ext4 extensively, and changes > to the other file systems are relatively small, I plan to carry the > invalidatepage patch set in the ext4 tree for the next development > cycle (i.e., not the upcoming merge window, but the next one). To > that end, it would be great if you take a look at this patch set and > send us an Acked-by signoff. > > Thanks!! > > - Ted
diff --git a/fs/ocfs2/aops.c b/fs/ocfs2/aops.c index 7c47755..79736a2 100644 --- a/fs/ocfs2/aops.c +++ b/fs/ocfs2/aops.c @@ -608,8 +608,7 @@ static void ocfs2_invalidatepage(struct page *page, unsigned int offset, { journal_t *journal = OCFS2_SB(page->mapping->host->i_sb)->journal->j_journal; - jbd2_journal_invalidatepage(journal, page, offset, - PAGE_CACHE_SIZE - offset); + jbd2_journal_invalidatepage(journal, page, offset, length); } static int ocfs2_releasepage(struct page *page, gfp_t wait)
->invalidatepage() aop now accepts range to invalidate so we can make use of it in ocfs2_invalidatepage(). Signed-off-by: Lukas Czerner <lczerner@redhat.com> Cc: Joel Becker <jlbec@evilplan.org> --- fs/ocfs2/aops.c | 3 +-- 1 files changed, 1 insertions(+), 2 deletions(-)