linux-ext4.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Eric Sandeen <sandeen@redhat.com>
To: Ben Hutchings <ben@decadent.org.uk>
Cc: "Theodore Ts'o" <tytso@mit.edu>,
	Andreas Dilger <adilger.kernel@dilger.ca>,
	linux-ext4@vger.kernel.org, Russell Coker <russell@coker.com.au>,
	654206@bugs.debian.org
Subject: Re: [PATCH] ext4: Report max_batch_time option correctly
Date: Tue, 03 Jan 2012 09:46:24 -0600	[thread overview]
Message-ID: <4F0322D0.2070403@redhat.com> (raw)
In-Reply-To: <1325513582.13595.213.camel@deadeye>

On 1/2/12 8:13 AM, Ben Hutchings wrote:
> Currently the value reported for max_batch_time is really the
> value of min_batch_time.
> 
> Reported-by: Russell Coker <russell@coker.com.au>
> Signed-off-by: Ben Hutchings <ben@decadent.org.uk>

whoops, yep!

Reviewed-by: Eric Sandeen <sandeen@redhat.com>

> ---
>  fs/ext4/super.c |    2 +-
>  1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/fs/ext4/super.c b/fs/ext4/super.c
> index 3e1329e..410e993 100644
> --- a/fs/ext4/super.c
> +++ b/fs/ext4/super.c
> @@ -1096,7 +1096,7 @@ static int ext4_show_options(struct seq_file *seq, struct vfsmount *vfs)
>  	}
>  	if (sbi->s_max_batch_time != EXT4_DEF_MAX_BATCH_TIME) {
>  		seq_printf(seq, ",max_batch_time=%u",
> -			   (unsigned) sbi->s_min_batch_time);
> +			   (unsigned) sbi->s_max_batch_time);
>  	}
>  
>  	/*


  reply	other threads:[~2012-01-03 15:46 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-01-02 14:13 [PATCH] ext4: Report max_batch_time option correctly Ben Hutchings
2012-01-03 15:46 ` Eric Sandeen [this message]
2012-01-05  2:23 ` Ted Ts'o

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=4F0322D0.2070403@redhat.com \
    --to=sandeen@redhat.com \
    --cc=654206@bugs.debian.org \
    --cc=adilger.kernel@dilger.ca \
    --cc=ben@decadent.org.uk \
    --cc=linux-ext4@vger.kernel.org \
    --cc=russell@coker.com.au \
    --cc=tytso@mit.edu \
    /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: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).