Messages in this thread | | | Subject | Re: [GIT PULL] Early SLAB fixes for 2.6.31 | From | Pekka Enberg <> | Date | Fri, 12 Jun 2009 16:45:06 +0300 |
| |
Hi Ben,
On Fri, 2009-06-12 at 16:25 +0300, Pekka J Enberg wrote: > > + /* Lets avoid crashing in early boot code. */ > > + if (WARN_ON_ONCE((local_flags & ~slab_gfp_mask) != 0)) > > + local_flags &= slab_gfp_mask; > > +
On Fri, 2009-06-12 at 23:38 +1000, Benjamin Herrenschmidt wrote: > I still object to the WARN_ON, see all the previous discussions on that > subject...
Yes, I know that!
I am not proposing this as the final fix to powerpc problems. But if we were to do the masking you suggested, this is not the proper way to do it at all. We need to mask the flags much earlier than this which is why I am keeping the WARN_ON() there.
The patch is a stop-gap measure until we've reached an agreement how to fix things properly. And I do think my patch is the right thing to do here while we wait for Linus' opinion on this.
Pekka
| |