On Thu, 17 Oct 2013, Ming Lei wrote:
> Date: Thu, 17 Oct 2013 20:28:46 +0800
> From: Ming Lei <ming....@canonical.com>
> To: linux-kernel@vger.kernel.org
> Cc: Jan Kara <j...@suse.cz>, Ming Lei <ming....@canonical.com>,
> Ted Tso <ty...@mit.edu>, linux-e...@vger.kernel.org,
> "linux-fsde...@vger.kernel.org" <linux-fsde...@vger.kernel.org>
> Subject: [PATCH] ext4: pair trace_ext4_writepages &
> trace_ext4_writepages_result
>
> Pair the two trace events to make troubeshooting writepages
> easier, and it should be more convinient to write a simple script
> to parse the traces.
>
> Cc: Ted Tso <ty...@mit.edu>
> Cc: linux-e...@vger.kernel.org
> Cc: "linux-fsde...@vger.kernel.org" <linux-fsde...@vger.kernel.org>
> Cc: Jan Kara <j...@suse.cz>
> Signed-off-by: Ming Lei <ming....@canonical.com>
It looks good.
Thanks!
Reviewed-by: Lukas Czerner <lczer...@redhat.com>
> ---
> fs/ext4/inode.c | 11 ++++++-----
> 1 file changed, 6 insertions(+), 5 deletions(-)
>
> diff --git a/fs/ext4/inode.c b/fs/ext4/inode.c
> index 32beaa4..0ad73d4 100644
> --- a/fs/ext4/inode.c
> +++ b/fs/ext4/inode.c
> @@ -2420,16 +2420,15 @@ static int ext4_writepages(struct address_space
> *mapping,
> * because that could violate lock ordering on umount
> */
> if (!mapping->nrpages || !mapping_tagged(mapping, PAGECACHE_TAG_DIRTY))
> - return 0;
> + goto out_writepages;
>
> if (ext4_should_journal_data(inode)) {
> struct blk_plug plug;
> - int ret;
>
> blk_start_plug(&plug);
> ret = write_cache_pages(mapping, wbc, __writepage, mapping);
> blk_finish_plug(&plug);
> - return ret;
> + goto out_writepages;
> }
>
> /*
> @@ -2442,8 +2441,10 @@ static int ext4_writepages(struct address_space
> *mapping,
> * *never* be called, so if that ever happens, we would want
> * the stack trace.
> */
> - if (unlikely(sbi->s_mount_flags & EXT4_MF_FS_ABORTED))
> - return -EROFS;
> + if (unlikely(sbi->s_mount_flags & EXT4_MF_FS_ABORTED)) {
> + ret = -EROFS;
> + goto out_writepages;
> + }
>
> if (ext4_should_dioread_nolock(inode)) {
> /*
>
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
