Re: [PATCH] bitops: remove "optimizations"

From: Ingo Molnar
Date: Tue Apr 29 2008 - 10:22:59 EST



* David Miller <davem@xxxxxxxxxxxxx> wrote:

> From: David Miller <davem@xxxxxxxxxxxxx>
> Date: Tue, 29 Apr 2008 03:03:32 -0700 (PDT)
>
> > From: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> > Date: Tue, 29 Apr 2008 12:01:02 +0200 (CEST)
> >
> > > Signed-off-by: Thomas Gleixner <tglx@xxxxxxxxxxxxx>
> >
> > Acked-by: David S. Miller <davem@xxxxxxxxxxxxx>
>
> Ironically, I just bisected sparc64 bootup failures to the following
> changeset. It's very late here, and I haven't looked into the
> details, but it seems to wedge in free_area_init_nodes() on a non-NUMA
> system with CONFIG_NUMA disabled.

hm. I guess the next thing you plan to check is whether Thomas's patch
fixes it?

> Isn't it funny that this optimization not only was useless, but also
> broke things. :-/

yeah and sorry :-/ It would still be nice to understand why this
happened, it looks like a weird (and unexpected) interaction.

Ingo
--
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/