linux-mm.kvack.org archive mirror
 help / color / mirror / Atom feed
From: Christoph Lameter <clameter@sgi.com>
To: Pekka J Enberg <penberg@cs.helsinki.fi>
Cc: Andrew Morton <akpm@google.com>, linux-mm@kvack.org
Subject: Re: mm section mismatches
Date: Sun, 8 Oct 2006 14:51:11 -0700 (PDT)	[thread overview]
Message-ID: <Pine.LNX.4.64.0610081444010.23640@schroedinger.engr.sgi.com> (raw)
In-Reply-To: <Pine.LNX.4.64.0610081030100.2562@sbz-30.cs.Helsinki.FI>

On Sun, 8 Oct 2006, Pekka J Enberg wrote:

> setup_cpu_cache is a non-init function that calls set_up_list3s which is 
> init.  However, due to g_cpucache_up, we will never hit the branch in 
> setup_cpu_cache that calls set_up_list3s.

It hits that code during bootstrap. Note that g_cpucache_up is NONE on 
startup and thus we will be hitting that function from kmem_init().

> No idea how to fix the warning. Due to g_cpucache_up, we need some entry 
> point that calls both init and non-init functions... Christoph?

To tell you the truth on of the crappy issues about the 
current slab is the mindboogling way of complexity of the bootstrap. 
Which is due to the inability to statically define a kmem_cache 
structure because we seem to have made an early decision to only work with 
kmem_cache_t handles so that the data structures are opaque from the 
outside.

To fix: Revert the change that made set_up_lists3s non init.
In 2.6.18-mm3 this set_up_list3s is not init.

--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@kvack.org.  For more info on Linux MM,
see: http://www.linux-mm.org/ .
Don't email: <a href=mailto:"dont@kvack.org"> email@kvack.org </a>

      reply	other threads:[~2006-10-08 21:51 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2006-10-07  1:49 Andrew Morton
2006-10-07  4:10 ` Randy Dunlap
2006-10-07  6:46   ` Andrew Morton
2006-10-07 17:58     ` Randy Dunlap
2006-10-09 18:06       ` Christoph Lameter
2006-10-09 19:15         ` Andrew Morton
2006-10-09 14:38     ` Mel Gorman
2006-10-08  7:33 ` Pekka J Enberg
2006-10-08 21:51   ` Christoph Lameter [this message]

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=Pine.LNX.4.64.0610081444010.23640@schroedinger.engr.sgi.com \
    --to=clameter@sgi.com \
    --cc=akpm@google.com \
    --cc=linux-mm@kvack.org \
    --cc=penberg@cs.helsinki.fi \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox