Re: [PATCH] cfq-iosched: Fixed boot warning with BLK_CGROUP=y andCFQ_GROUP_IOSCHED=n

From: Vivek Goyal
Date: Mon Jun 21 2010 - 10:51:15 EST


On Mon, Jun 21, 2010 at 09:05:17AM +0200, Ingo Molnar wrote:
>
> * Vivek Goyal <vgoyal@xxxxxxxxxx> wrote:
>
> > On Sun, Jun 20, 2010 at 10:10:16AM +0200, Ingo Molnar wrote:
> > >
> > > * Vivek Goyal <vgoyal@xxxxxxxxxx> wrote:
> > >
> > > > Hi Jens,
> > > >
> > > > Few days back Ingo noticed a CFQ boot time warning. This patch fixes it.
> > >
> > > Hm, have you tested it? This bit:
> >
> > I did compile and boot testing. Booted with CFQ_GROUP_IOSCHED=n to make
> > sure warning seen by you is gone. [...]
>
> It was not a build warning, it was a build failure. (Even if it built the
> recursion would cause a crash on bootup.) Will try your newer patch too.

I think compiler is doing some optimizations. On my system, it just
compiles fine and even boots fine. But when I introduce a trace_printk(),
statement in the function, then compilation is fine but system hangs
during boot.

I am using gcc version 4.4.4 20100503.

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