* Mel Gorman <[EMAIL PROTECTED]> wrote:
> On (02/10/07 14:15), Ingo Molnar didst pronounce:
> >
> > * Mel Gorman <[EMAIL PROTECTED]> wrote:
> >
> > > Dirt. Booting with "profile=sleep,2" is broken in 2.6.23-rc9 and
> > > 2.6.23-rc8 but working in 2.6.22. I was checking it out as part of a
> > > discussion in another thread and noticed it broken in -mm as well
> > > (2.6.23-rc8-mm2). Bisect is in progress but suggestions as to the
> > > prime candidates are welcome or preferably, pointing out that I'm an
> > > idiot because I missed twiddling some config change.
> >
> > Mel, does the patch below fix this bug for you? (Note: you will need to
> > enable CONFIG_SCHEDSTATS=y too.)
>
> Nice one Ingo - got it first try. The problem commit was
> dd41f596cda0d7d6e4a8b139ffdfabcefdd46528 and it's clear that the code
> removed in this commit is put back by this latest patch. When
> applied, profile=sleep works as long as CONFIG_SCHEDSTAT is set.
great - thanks for testing it. I'm glad you caught it as sleep=profile
is pretty useful in "why is my system so slow" tests. (which problems
are usually reported _after_ a stable kernel is released ...)
Ingo
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
