Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754537AbYGOENX (ORCPT ); Tue, 15 Jul 2008 00:13:23 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751315AbYGOENO (ORCPT ); Tue, 15 Jul 2008 00:13:14 -0400 Received: from smtp1.linux-foundation.org ([140.211.169.13]:37015 "EHLO smtp1.linux-foundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750783AbYGOENN (ORCPT ); Tue, 15 Jul 2008 00:13:13 -0400 Date: Mon, 14 Jul 2008 21:12:38 -0700 (PDT) From: Linus Torvalds To: Max Krasnyansky cc: Dmitry Adamushko , Vegard Nossum , Paul Menage , Paul Jackson , Peter Zijlstra , miaox@cn.fujitsu.com, rostedt@goodmis.org, Thomas Gleixner , Ingo Molnar , Linux Kernel Subject: Re: current linux-2.6.git: cpusets completely broken In-Reply-To: <487C1374.8020404@qualcomm.com> Message-ID: References: <20080712031736.GA3040@damson.getinternet.no> <487C1374.8020404@qualcomm.com> User-Agent: Alpine 1.10 (LFD 962 2008-03-14) MIME-Version: 1.0 Content-Type: TEXT/PLAIN; charset=US-ASCII Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1678 Lines: 42 On Mon, 14 Jul 2008, Max Krasnyansky wrote: > > Did you guys an updated patch ? Dmitry pointed out several things that Linus > missed in his original version. I guess I can go through the thread and > reconstruct that but if you have a patch I can try let me know. I didn't update it, and right now I'm just merging too much (and discussing the merges) to have time. The patch really needs to have some scheduler person look at the use fo cpu_active_map - I was kind of hoping that Ingo would. The other thing I wish somebody would at least double-check (and preferably just remove) is the !CPU_HOTPLUG special case. For that case, I literally did #define cpu_active_map cpu_online_map and just depended on the fact that the few cases that touched cpu_online_map even when CPU_HOTPLUG was disabled would do if (cpu_online(cpu)) set_cpu(cpu, cpu_active_map); (or something - I ended up discaring the patch as I was doing the whole 2.6.26 release, so I don't even have it in front of me right now, just in my email archives and in my memory). And quite frankly, I don't think that was a good idea. It was exactly the sort of "subtle cleverness" that I myself argue against. So I think that aliasing of cpu_active/online_map was a mistake - it was kind of cute, but there's really no good reason to do it. It's not like that one CPU map wastes any memory, and the special-case simply isn't worth it. Linus -- 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/