From: Ingo Molnar <mingo@elte.hu>
To: Pekka Enberg <penberg@cs.helsinki.fi>
Cc: Stephen Rothwell <sfr@canb.auug.org.au>,
Christoph Lameter <cl@linux-foundation.org>,
linux-next@vger.kernel.org,
Eduard - Gabriel Munteanu <eduard.munteanu@linux360.ro>,
Thomas Gleixner <tglx@linutronix.de>,
"H. Peter Anvin" <hpa@zytor.com>
Subject: Re: linux-next: manual merge of the slab tree with the ftrace tree
Date: Fri, 20 Feb 2009 10:53:55 +0100 [thread overview]
Message-ID: <20090220095355.GL24555@elte.hu> (raw)
In-Reply-To: <1235122897.29813.28.camel@penberg-laptop>
* Pekka Enberg <penberg@cs.helsinki.fi> wrote:
> Hi Ingo,
>
> On Fri, 2009-02-20 at 10:37 +0100, Ingo Molnar wrote:
> > * Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> >
> > > Hi all,
> > >
> > > On Fri, 20 Feb 2009 16:57:28 +1100 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
> > > >
> > > > @@@ -2723,18 -2689,9 +2727,19 @@@ static void *kmalloc_large_node(size_t
> > > > void *__kmalloc_node(size_t size, gfp_t flags, int node)
> > > > {
> > > > struct kmem_cache *s;
> > > > + void *ret;
> > > >
> > > > - if (unlikely(size > SLUB_MAX_SIZE))
> > > > - return kmalloc_large_node(size, flags, node);
> > > > ++ if (unlikely(size > SLUB_MAX_SIZE)) {
> > > > + if (unlikely(size > PAGE_SIZE)) {
> > >
> > > Except I screwed that up. I meant to delete the last line
> > > above. I will add a patch to the end of linux-next for today.
> >
> > Hm, i'd love to eliminate the conflict, but it would either mean
> > us to pull the slab tree into the tracing tree, or the other way
> > around - and both have quite many items queued up to make this
> > impractical.
>
> Is it a big problem, though? We could do the
> s/PAGE_SIZE/SLUB_MAX_SIZE/g rename as a separate preparational
> patch (without any of the functional changes) and see if Linus
> merges it to mainline...
Linus does not have to merge the changes - it's enough if both
you and me merge it, then there will be no conflicts if the two
trees are combined - Git will sort it all out.
Could you prepare such a patch please, merge it into a separate
(and append-only) branch your tree and then send me a pull
request after a bit of testing so i can merge it into the
tracing tree?
The only constraint is that this commit should never be rebased
after that point - but neither of us will do that.
Ingo
next prev parent reply other threads:[~2009-02-20 9:54 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-02-20 5:57 linux-next: manual merge of the slab tree with the ftrace tree Stephen Rothwell
2009-02-20 8:25 ` Pekka Enberg
2009-02-20 8:59 ` Stephen Rothwell
2009-02-20 9:37 ` Ingo Molnar
2009-02-20 9:41 ` Pekka Enberg
2009-02-20 9:53 ` Ingo Molnar [this message]
2009-02-20 11:12 ` Pekka Enberg
2009-02-20 11:16 ` Ingo Molnar
2009-02-20 23:07 ` Stephen Rothwell
2009-02-20 22:58 ` Stephen Rothwell
-- strict thread matches above, loose matches on Subject: below --
2009-01-20 3:57 Stephen Rothwell
2009-01-20 6:01 ` Matt Mackall
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20090220095355.GL24555@elte.hu \
--to=mingo@elte.hu \
--cc=cl@linux-foundation.org \
--cc=eduard.munteanu@linux360.ro \
--cc=hpa@zytor.com \
--cc=linux-next@vger.kernel.org \
--cc=penberg@cs.helsinki.fi \
--cc=sfr@canb.auug.org.au \
--cc=tglx@linutronix.de \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.