On Fri, 2006-04-28 at 16:23 -0700, Andrew Morton wrote:
> Chandra Seetharaman <[email protected]> wrote:
> >
> > Looks like the patches I provided is a step backward from where Ashok &
> > Andrew were taking the register_cpu_notifier stuff to.
> >
> > After some discussions with Ashok we both think the following would be
> > the right direction:
> > 1 revert the changes i pushed recently
> > 2 make all usages of register_cpu_notifier to be _init and
> > __initdata (if hotplug cpu is defined these are removed)
> > 3 export the symbols register_cpu_notifier and
> > unregister_cpu_notifier only in CONFIG_HOTPLUG_CPU is defined
> > 4 move the hot plug cpu based usages of register_cpu_notifier
> > inside #ifdef CONFIG_HOTPLUF_CPU(like xfs's usage).
> >
> > I have few questions:
> > - any problems with the above direction (mainly 3) ?
> > - Should we proceed in this direction ?
> > - is it too late for 2.6.17 ? if not late how much time do we have ?
>
> hm. I'm leaning more towards doing something expedient and obvious for
> 2.6.17. It's pretty late in the cycle, and the only downside is the loss
> of a kbyte or two. Plus I'll be at linuxtag next week and won't be around to
> help out.
>
> So if it's OK, can we do something minimal, revisit it after 2.6.17?
- if we are ok with a loss of a kbyte or two, 2.6.17 is fine as is
(with my incorrect patches in).
- if we want to save that memory, we can revert the two patches and fix
xfs to make the register calls only when hotplug cpu is defined. This
change is also minimal. It is a step in the right direction.
Only downside i can see in reverting my patch is that if there is any
other modules that are doing the same as what xfs was doing, we might
trip in a similar oops.
chandra
--
----------------------------------------------------------------------
Chandra Seetharaman | Be careful what you choose....
- [email protected] | .......you may get it.
----------------------------------------------------------------------
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
