lkml.org 
[lkml]   [2007]   [May]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH] [16/34] i386: fix mtrr sections
I noticed that I can just use call graphs to make more clear what I want to say.

There is otherwise nothing new in this mail and I'll submit a new patch
which reverts this patch if my review came too late, if in doubt, just
ignore this mail, I may just refer to it later then.

> NACK - obsolete, replaced by:
> Jeremy Fitzhardinge - "x86: clean up identify_cpu"
> http://lkml.org/lkml/2007/4/7/113 [it's also patch 11/26 in this thread]
>
> It's a leftover of "__init to __cpuinit in mtrr code" (Prarit Bhargava):
> http://lkml.org/lkml/2007/2/28/198 - (acked by Bhavana Nagendra) which
> itself is rejected from Andrews tree already.
>
> Description:
> ------------
>
> This function tree (as of 2.6.21) here explains it all:
>
> __cpuinit identify_cpu()
> -> if BSP is inited, so only at __init time, these functions are called:
> __init mtrr_bp_init()
> __init get_mtrr_state()
> __initdata show_mtrr;
> __init print_fixed()

That was around 2.6.15 till 2.6.21.

Prarit Bhargava's patch "__init to __cpuinit in mtrr code" did this:

__cpuinit identify_cpu()
-> if BSP is inited, so only at __init time, these functions are called:
- __init mtrr_bp_init()
- __init get_mtrr_state()
+ __cpuinit mtrr_bp_init()
+ __cpuinit get_mtrr_state()
(plus half a dozen other functions)
__initdata show_mtrr;
__init print_fixed()

and Randy's patch (on top of Prarit Bhargava's, which is removed now) does:

- __initdata show_mtrr;
+ show_mtrr;
- __init print_fixed()
+ __cpuinit print_fixed()

While Jeremy Fitzhardinge - "x86: clean up identify_cpu" cleans it up:

- __cpuinit identify_cpu()
+ __init identify_boot_cpu() (only does the boot-only calls)
__init mtrr_bp_init()
__init get_mtrr_state()
(and half a dozen other functions also keep __init)
__initdata show_mtrr;
__init print_fixed()

which is best and Randy's patch "fix mtrr sections" is also obsolete then,
show_mtrr and print_fixed() can keep the __init/__initdata prefix.

As Andy already submitted it to Linus with http://lkml.org/lkml/2007/5/2/346
we may have to create a patch which reverts this patch.

I'm also planning to add some function headers to the MTRR functions
and also fix the existing and upcoming function headers to be complant
to kernel-doc.

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

\
 
 \ /
  Last update: 2007-05-03 13:19    [W:0.088 / U:0.464 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site