Hi Andi
On Mon, Sep 05, 2005 at 06:48:21AM +0200, Andi Kleen wrote:
> On Sat, Sep 03, 2005 at 02:33:26PM -0700, [email protected] wrote:
> >
> > From: Ashok Raj <[email protected]>
> >
> > No need to enforce_max_cpus when hotplug code is enabled. This nukes out
> > cpu_present_map and cpu_possible_map making it impossible to add new cpus in
> > the system.
>
> I see the point, but the implementation is wrong. If anything
> we shouldn't do it neither for the !HOTPLUG_CPU case.Why did
> you not do it unconditionally?
>
> I would prefer to keep the special cases for hotplug to be
> as narrow as possible.
Link to earlier discussion below
http://marc.theaimsgroup.com/?l=linux-kernel&m=112317327529855&w=2
I had suggested that we remove it completely in our discussion but i didnt
hear anything from you after that, so i thought that was acceptable.
You had suggested in that discussion that it would be better to add an
option for startup. Iam opposed to adding any option, when we certainly know
there are no users. Earlier based on your suggestion i added a startup
option to choose ipi broadcast mode, which you promptly removed when you
put physflat changes. I think its better to not add any option without
real need. Do you agree?
Please reply if you want me to remove the !HOTPLUG case which is my
preference as well, and maybe while the memory is fresh, we can stick
with it this time when we are in the same page :-(
>
> -Andi
--
Cheers,
Ashok Raj
- Open Source Technology Center
-
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]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
|
|