All of lore.kernel.org
 help / color / mirror / Atom feed
From: Joel Becker <Joel.Becker@oracle.com>
To: Linus Torvalds <torvalds@linux-foundation.org>,
	Dave Chinner <david@fromorbit.com>,
	Linux Kernel <linux-kernel@vger.kernel.org>,
	ocfs2-devel@oss.oracle.com, Tao Ma <tao.ma@oracle.com>,
	Dave Chinner <dchinner@redhat.com>,
	Christoph Hellwig <hch@lst.de>, Mark Fasheh <mfasheh@suse.com>
Subject: [Ocfs2-devel] [PATCH] Revert "writeback: limit write_cache_pages integrity scanning to current EOF"
Date: Tue, 29 Jun 2010 18:30:53 -0700	[thread overview]
Message-ID: <20100630013053.GK4150@mail.oracle.com> (raw)
In-Reply-To: <20100629081611.GB4150@mail.oracle.com>

On Tue, Jun 29, 2010 at 01:16:11AM -0700, Joel Becker wrote:
> On Mon, Jun 28, 2010 at 07:20:33PM -0700, Linus Torvalds wrote:
> > Your call, I guess.  If any ocfs2 fix looks scary, and you'd prefer to
> > have an -rc4 (in a few days - not today) with just the revert, I'm ok
> > with that. Even if it's only a "at least no worse than 2.6.34"
> > situation rather than a real fix.
> 
> 	If -rc4 isn't coming for a couple of days, can we hold off on
> the decision until we get a chance to think about a paper-over solution
> for it?  Then we can avoid the revert.

Linus,
	I'm going to withdraw the revert request for now.  Our proposed
paper-over solution is too big, and we're just going to focus on the
actual fix.  This will be for .35.  Yes, .35-rc will have a BUG_ON with
refcount trees until we get the fix in, but I'd rather avoid the churn
when the final .35 should have Dave's patch and our fix.

Joel

-- 

"The first thing we do, let's kill all the lawyers."
                                        -Henry VI, IV:ii

Joel Becker
Consulting Software Developer
Oracle
E-mail: joel.becker at oracle.com
Phone: (650) 506-8127

WARNING: multiple messages have this Message-ID (diff)
From: Joel Becker <Joel.Becker@oracle.com>
To: Linus Torvalds <torvalds@linux-foundation.org>,
	Dave Chinner <david@fromorbit.com>,
	Linux Kernel <linux-kernel@vger.kernel.org>,
	ocfs2-devel@oss.oracle.com, Tao Ma <tao.ma@oracle.com>,
	Dave Chinner <dchinner@redhat.com>,
	Christoph Hellwig <hch@lst.de>, Mark Fasheh <mfasheh@suse.com>
Subject: Re: [Ocfs2-devel] [PATCH] Revert "writeback: limit write_cache_pages integrity scanning to current EOF"
Date: Tue, 29 Jun 2010 18:30:53 -0700	[thread overview]
Message-ID: <20100630013053.GK4150@mail.oracle.com> (raw)
In-Reply-To: <20100629081611.GB4150@mail.oracle.com>

On Tue, Jun 29, 2010 at 01:16:11AM -0700, Joel Becker wrote:
> On Mon, Jun 28, 2010 at 07:20:33PM -0700, Linus Torvalds wrote:
> > Your call, I guess.  If any ocfs2 fix looks scary, and you'd prefer to
> > have an -rc4 (in a few days - not today) with just the revert, I'm ok
> > with that. Even if it's only a "at least no worse than 2.6.34"
> > situation rather than a real fix.
> 
> 	If -rc4 isn't coming for a couple of days, can we hold off on
> the decision until we get a chance to think about a paper-over solution
> for it?  Then we can avoid the revert.

Linus,
	I'm going to withdraw the revert request for now.  Our proposed
paper-over solution is too big, and we're just going to focus on the
actual fix.  This will be for .35.  Yes, .35-rc will have a BUG_ON with
refcount trees until we get the fix in, but I'd rather avoid the churn
when the final .35 should have Dave's patch and our fix.

Joel

-- 

"The first thing we do, let's kill all the lawyers."
                                        -Henry VI, IV:ii

Joel Becker
Consulting Software Developer
Oracle
E-mail: joel.becker@oracle.com
Phone: (650) 506-8127

  reply	other threads:[~2010-06-30  1:30 UTC|newest]

Thread overview: 78+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-06-28 17:35 [Ocfs2-devel] [PATCH] Revert "writeback: limit write_cache_pages integrity scanning to current EOF" Joel Becker
2010-06-28 17:35 ` Joel Becker
2010-06-29  0:24 ` [Ocfs2-devel] " Dave Chinner
2010-06-29  0:24   ` Dave Chinner
2010-06-29  0:54   ` [Ocfs2-devel] " Joel Becker
2010-06-29  0:54     ` Joel Becker
2010-06-29  1:12     ` [Ocfs2-devel] " Linus Torvalds
2010-06-29  1:12       ` Linus Torvalds
2010-06-29  1:58       ` [Ocfs2-devel] " Joel Becker
2010-06-29  1:58         ` Joel Becker
2010-06-29  2:20         ` Linus Torvalds
2010-06-29  2:20           ` Linus Torvalds
2010-06-29  2:44           ` Dave Chinner
2010-06-29  2:44             ` Dave Chinner
2010-06-29  8:16           ` Joel Becker
2010-06-29  8:16             ` Joel Becker
2010-06-30  1:30             ` Joel Becker [this message]
2010-06-30  1:30               ` Joel Becker
2010-07-06 19:06         ` Joel Becker
2010-07-06 19:06           ` Joel Becker
2010-06-29  1:56     ` Dave Chinner
2010-06-29  1:56       ` Dave Chinner
2010-06-29  2:04       ` [Ocfs2-devel] " Joel Becker
2010-06-29  2:04         ` Joel Becker
2010-06-29  2:27         ` [Ocfs2-devel] " Dave Chinner
2010-06-29  2:27           ` Dave Chinner
2010-06-29  7:18           ` [Ocfs2-devel] " Joel Becker
2010-06-29  7:18             ` Joel Becker
2010-07-02 22:49             ` [Ocfs2-devel] [PATCH] ocfs2: Zero the tail cluster when extending past i_size Joel Becker
2010-07-02 22:49               ` Joel Becker
2010-07-03 21:32               ` [Ocfs2-devel] [PATCH 1/2] ocfs2: Zero the tail cluster when extending past i_size v2 Joel Becker
2010-07-03 21:32                 ` Joel Becker
2010-07-03 21:33                 ` [Ocfs2-devel] [PATCH 2/2] ocfs2: No need to zero pages past i_size. " Joel Becker
2010-07-03 21:33                   ` Joel Becker
2010-07-04 15:13                   ` [Ocfs2-devel] " Tao Ma
2010-07-04 15:13                     ` Tao Ma
2010-07-05  1:38                     ` [Ocfs2-devel] " Tao Ma
2010-07-05  1:38                       ` Tao Ma
2010-07-06  7:10                       ` [Ocfs2-devel] " Joel Becker
2010-07-06  7:10                         ` Joel Becker
2010-07-06  7:09                     ` [Ocfs2-devel] " Joel Becker
2010-07-06  7:09                       ` Joel Becker
2010-07-06 18:39                       ` [Ocfs2-devel] " Joel Becker
2010-07-06 18:39                         ` Joel Becker
2010-07-05  3:51                 ` [Ocfs2-devel] [PATCH 1/2] ocfs2: Zero the tail cluster when extending past " Tao Ma
2010-07-05  3:51                   ` Tao Ma
2010-07-06  7:17                   ` [Ocfs2-devel] " Joel Becker
2010-07-06  7:17                     ` Joel Becker
2010-07-06  7:54                     ` [Ocfs2-devel] " Tao Ma
2010-07-06  7:54                       ` Tao Ma
2010-07-06 11:58                       ` [Ocfs2-devel] " Joel Becker
2010-07-06 11:58                         ` Joel Becker
2010-07-07  0:42                         ` [Ocfs2-devel] " Tao Ma
2010-07-07  0:42                           ` Tao Ma
2010-07-07  2:03                           ` [Ocfs2-devel] " Joel Becker
2010-07-07  2:03                             ` Joel Becker
2010-07-06 18:48                   ` [Ocfs2-devel] " Joel Becker
2010-07-06 18:48                     ` Joel Becker
2010-07-06 18:57                   ` [Ocfs2-devel] " Joel Becker
2010-07-06 18:57                     ` Joel Becker
2010-07-07 11:16                 ` [Ocfs2-devel] [PATCH 0/3] ocfs2: Tail zeroing fixes Joel Becker
2010-07-07 11:16                   ` Joel Becker
2010-07-12 22:45                   ` [Ocfs2-devel] " Joel Becker
2010-07-12 22:45                     ` Joel Becker
2010-07-07 11:16                 ` [Ocfs2-devel] [PATCH 1/3] ocfs2: When zero extending, do it by page Joel Becker
2010-07-07 11:16                   ` Joel Becker
2010-07-07 15:19                   ` [Ocfs2-devel] " Tao Ma
2010-07-07 15:19                     ` Tao Ma
2010-07-07 20:04                     ` [Ocfs2-devel] " Joel Becker
2010-07-07 20:04                       ` Joel Becker
2010-07-08  3:44                   ` [Ocfs2-devel] " Tao Ma
2010-07-08  3:44                     ` Tao Ma
2010-07-08  9:51                     ` [Ocfs2-devel] " Joel Becker
2010-07-08  9:51                       ` Joel Becker
2010-07-07 11:16                 ` [Ocfs2-devel] [PATCH 2/3] ocfs2: Zero the tail cluster when extending past i_size Joel Becker
2010-07-07 11:16                   ` Joel Becker
2010-07-07 11:16                 ` [Ocfs2-devel] [PATCH 3/3] ocfs2: No need to zero pages " Joel Becker
2010-07-07 11:16                   ` Joel Becker

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=20100630013053.GK4150@mail.oracle.com \
    --to=joel.becker@oracle.com \
    --cc=david@fromorbit.com \
    --cc=dchinner@redhat.com \
    --cc=hch@lst.de \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mfasheh@suse.com \
    --cc=ocfs2-devel@oss.oracle.com \
    --cc=tao.ma@oracle.com \
    --cc=torvalds@linux-foundation.org \
    /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 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.