lkml.org 
[lkml]   [2007]   [Jan]   [9]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: 2.6.20-rc4: known unfixed regressions (v2)
On Tue, Jan 09, 2007 at 09:58:19AM -0800, Linus Torvalds wrote:
>
>
> On Tue, 9 Jan 2007, Adrian Bunk wrote:
> >
> > Subject : BUG: at mm/truncate.c:60 cancel_dirty_page() (reiserfs)
> > References : http://lkml.org/lkml/2007/1/7/117
> > Submitter : Malte Schröder <MalteSch@gmx.de>
> > Status : unknown
>
> Adrian, this is also available as
>
> http://lkml.org/lkml/2007/1/5/308

The latter was in my list as:

Subject : BUG: at mm/truncate.c:60 cancel_dirty_page() (XFS)
References : http://lkml.org/lkml/2007/1/5/308
Submitter : Sami Farin <7atbggg02@sneakemail.com>
Handled-By : David Chinner <dgc@sgi.com>
Patch : http://lkml.org/lkml/2007/1/7/201
Status : patch available

>...
> So I think this does show some confusion in reiserfs, but it's not
> anything new. The only new thing is that the _message_ happens.
>
> So I don't personally consider this a regression. Just a sign of old and
> preexisting confusion that is now uncovered by new code (and it will print
> out the scary message at most four times, and then stop complaining about
> it. So apart from the scary message, nothing new and bad has really
> happened).

At least the printing of scary messages is a regression.

Unless we want to be buried in bug reports after 2.6.20 got released,
the minimum fix is to temporarily remove the WARN_ON().

> Linus

cu
Adrian

--

"Is there not promise of rain?" Ling Tan asked suddenly out
of the darkness. There had been need of rain for many days.
"Only a promise," Lao Er said.
Pearl S. Buck - Dragon Seed

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2007-01-09 22:07    [from the cache]
©2003-2014 Jasper Spaans. Advertise on this site