From: Jan Kara <jack@suse.cz> To: LKML <linux-kernel@vger.kernel.org> Cc: hch@lst.de, linux-fsdevel@vger.kernel.org, Jan Kara <jack@suse.cz>, Felix Blyakher <felixb@sgi.com>, xfs@oss.sgi.com Subject: [PATCH 13/17] xfs: Convert sync_page_range() to simple fdatawrite_range() Date: Fri, 21 Aug 2009 19:24:03 +0200 [thread overview] Message-ID: <1250875447-15622-14-git-send-email-jack@suse.cz> (raw) In-Reply-To: <1250875447-15622-1-git-send-email-jack@suse.cz> Christoph Hellwig says that it is enough for XFS to call filemap_fdatawrite_range() instead of sync_page_range() because we do all the waiting when forcing the log. CC: Felix Blyakher <felixb@sgi.com> CC: xfs@oss.sgi.com CC: Christoph Hellwig <hch@lst.de> Signed-off-by: Jan Kara <jack@suse.cz> --- fs/xfs/linux-2.6/xfs_lrw.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/fs/xfs/linux-2.6/xfs_lrw.c b/fs/xfs/linux-2.6/xfs_lrw.c index 7078974..a68053c 100644 --- a/fs/xfs/linux-2.6/xfs_lrw.c +++ b/fs/xfs/linux-2.6/xfs_lrw.c @@ -817,7 +817,7 @@ write_retry: xfs_iunlock(xip, iolock); if (need_i_mutex) mutex_unlock(&inode->i_mutex); - error2 = sync_page_range(inode, mapping, pos, ret); + error2 = filemap_fdatawrite_range(mapping, pos, pos + ret - 1); if (!error) error = error2; if (need_i_mutex) -- 1.6.0.2
WARNING: multiple messages have this Message-ID (diff)
From: Jan Kara <jack@suse.cz> To: LKML <linux-kernel@vger.kernel.org> Cc: linux-fsdevel@vger.kernel.org, Jan Kara <jack@suse.cz>, hch@lst.de, xfs@oss.sgi.com Subject: [PATCH 13/17] xfs: Convert sync_page_range() to simple fdatawrite_range() Date: Fri, 21 Aug 2009 19:24:03 +0200 [thread overview] Message-ID: <1250875447-15622-14-git-send-email-jack@suse.cz> (raw) In-Reply-To: <1250875447-15622-1-git-send-email-jack@suse.cz> Christoph Hellwig says that it is enough for XFS to call filemap_fdatawrite_range() instead of sync_page_range() because we do all the waiting when forcing the log. CC: Felix Blyakher <felixb@sgi.com> CC: xfs@oss.sgi.com CC: Christoph Hellwig <hch@lst.de> Signed-off-by: Jan Kara <jack@suse.cz> --- fs/xfs/linux-2.6/xfs_lrw.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/fs/xfs/linux-2.6/xfs_lrw.c b/fs/xfs/linux-2.6/xfs_lrw.c index 7078974..a68053c 100644 --- a/fs/xfs/linux-2.6/xfs_lrw.c +++ b/fs/xfs/linux-2.6/xfs_lrw.c @@ -817,7 +817,7 @@ write_retry: xfs_iunlock(xip, iolock); if (need_i_mutex) mutex_unlock(&inode->i_mutex); - error2 = sync_page_range(inode, mapping, pos, ret); + error2 = filemap_fdatawrite_range(mapping, pos, pos + ret - 1); if (!error) error = error2; if (need_i_mutex) -- 1.6.0.2 _______________________________________________ xfs mailing list xfs@oss.sgi.com http://oss.sgi.com/mailman/listinfo/xfs
next prev parent reply other threads:[~2009-08-21 17:28 UTC|newest] Thread overview: 62+ messages / expand[flat|nested] mbox.gz Atom feed top 2009-08-21 17:23 [PATCH 0/17] Make O_SYNC handling use standard syncing path (Version 2) Jan Kara 2009-08-21 17:23 ` [PATCH 01/17] vfs: Introduce filemap_fdatawait_range Jan Kara 2009-08-21 17:23 ` [PATCH 02/17] vfs: Export __generic_file_aio_write() and add some comments Jan Kara 2009-08-21 17:23 ` [Ocfs2-devel] " Jan Kara 2009-08-21 17:23 ` Jan Kara 2009-08-21 17:23 ` [PATCH 03/17] vfs: Remove syncing from generic_file_direct_write() and generic_file_buffered_write() Jan Kara 2009-08-21 17:23 ` [Ocfs2-devel] " Jan Kara 2009-08-21 17:23 ` Jan Kara 2009-08-21 17:23 ` Jan Kara 2009-08-21 17:23 ` [PATCH 04/17] pohmelfs: Use __generic_file_aio_write instead of generic_file_aio_write_nolock Jan Kara 2009-08-21 17:23 ` [PATCH 05/17] ocfs2: " Jan Kara 2009-08-21 17:23 ` [Ocfs2-devel] " Jan Kara 2009-08-21 17:23 ` Jan Kara 2009-08-25 18:58 ` Joel Becker 2009-08-25 18:58 ` [Ocfs2-devel] " Joel Becker 2009-08-25 18:58 ` Joel Becker 2009-08-21 17:23 ` [PATCH 06/17] vfs: Rename generic_file_aio_write_nolock Jan Kara 2009-08-21 17:30 ` Christoph Hellwig 2009-08-21 17:56 ` Jan Kara 2009-08-21 18:07 ` Christoph Hellwig 2009-08-21 17:23 ` [PATCH 07/17] vfs: Introduce new helpers for syncing after writing to O_SYNC file or IS_SYNC inode Jan Kara 2009-08-21 17:24 ` [Ocfs2-devel] " Jan Kara 2009-08-21 17:23 ` Jan Kara 2009-08-27 17:35 ` Christoph Hellwig 2009-08-27 17:35 ` [Ocfs2-devel] " Christoph Hellwig 2009-08-27 17:35 ` Christoph Hellwig 2009-08-30 16:35 ` Jamie Lokier 2009-08-30 16:36 ` [Ocfs2-devel] " Jamie Lokier 2009-08-30 16:35 ` Jamie Lokier 2009-08-30 16:39 ` Christoph Hellwig 2009-08-30 16:40 ` [Ocfs2-devel] " Christoph Hellwig 2009-08-30 16:39 ` Christoph Hellwig 2009-08-30 17:29 ` Jamie Lokier 2009-08-30 17:30 ` [Ocfs2-devel] " Jamie Lokier 2009-08-30 17:29 ` Jamie Lokier 2009-08-21 17:23 ` [PATCH 08/17] ext2: Update comment about generic_osync_inode Jan Kara 2009-08-21 17:23 ` [PATCH 09/17] ext3: Remove syncing logic from ext3_file_write Jan Kara 2009-08-21 17:24 ` [PATCH 10/17] ext4: Remove syncing logic from ext4_file_write Jan Kara 2009-08-21 17:24 ` [PATCH 11/17] ntfs: Use new syncing helpers and update comments Jan Kara 2009-08-21 17:24 ` [PATCH 12/17] ocfs2: Update syncing after splicing to match generic version Jan Kara 2009-08-21 17:24 ` [Ocfs2-devel] " Jan Kara 2009-08-21 17:24 ` Jan Kara 2009-08-24 18:35 ` [Ocfs2-devel] " Mark Fasheh 2009-08-24 18:35 ` Mark Fasheh 2009-08-24 18:40 ` Joel Becker 2009-08-24 18:40 ` Joel Becker 2009-08-25 13:13 ` Jan Kara 2009-08-25 13:13 ` Jan Kara 2009-08-21 17:24 ` Jan Kara [this message] 2009-08-21 17:24 ` [PATCH 13/17] xfs: Convert sync_page_range() to simple fdatawrite_range() Jan Kara 2009-08-21 17:28 ` Christoph Hellwig 2009-08-21 17:28 ` Christoph Hellwig 2009-08-21 17:59 ` Jan Kara 2009-08-21 17:59 ` Jan Kara 2009-08-21 17:24 ` [PATCH 14/17] pohmelfs: Use new syncing helper Jan Kara 2009-08-21 17:24 ` [PATCH 15/17] nfs: Remove reference to generic_osync_inode from a comment Jan Kara 2009-08-21 17:52 ` Trond Myklebust 2009-08-21 17:52 ` Trond Myklebust 2009-08-21 17:24 ` [PATCH 16/17] fat: Opencode sync_page_range_nolock() Jan Kara 2009-08-21 17:24 ` [PATCH 17/17] vfs: Remove generic_osync_inode() and sync_page_range{_nolock}() Jan Kara 2009-08-22 16:27 ` [PATCH 0/17] Make O_SYNC handling use standard syncing path (Version 2) Jamie Lokier 2009-08-24 9:29 ` Jan Kara
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=1250875447-15622-14-git-send-email-jack@suse.cz \ --to=jack@suse.cz \ --cc=felixb@sgi.com \ --cc=hch@lst.de \ --cc=linux-fsdevel@vger.kernel.org \ --cc=linux-kernel@vger.kernel.org \ --cc=xfs@oss.sgi.com \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: linkBe sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.