Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753069Ab3EPQPA (ORCPT ); Thu, 16 May 2013 12:15:00 -0400 Received: from relay3.sgi.com ([192.48.152.1]:57096 "EHLO relay.sgi.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1750733Ab3EPQO6 (ORCPT ); Thu, 16 May 2013 12:14:58 -0400 From: Robin Holt To: Andrew Morton Cc: "H. Peter Anvin" , Russ Anderson , Robin Holt , Russell King , Guan Xuetao , Linux Kernel Mailing List , the arch/x86 maintainers , Arm Mailing List Subject: [PATCH -v11 resend 00/11] Shutdown from reboot_cpuid without stopping other cpus. Date: Thu, 16 May 2013 11:14:39 -0500 Message-Id: <1368720890-57711-1-git-send-email-holt@sgi.com> X-Mailer: git-send-email 1.8.2.1 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2996 Lines: 93 We recently noticed that reboot of a 1024 cpu machine takes approx 16 minutes of just stopping the cpus. The slowdown was tracked to commit f96972f. The current implementation does all the work of hot removing the cpus before halting the system. We are switching to just migrating to the reboot_cpu and then continuing with shutdown/reboot. The patch set is broken into eleven parts. The first two are planned for the stable release. The others move the halt/shutdown/reboot related functions to their own kernel/reboot.c file and then move the handling of the kernel reboot= kernel parameter to generic kernel code. Changes since -v10 - Added Russell's Acked-by for arm. - Fixed an accidentally commented out line in an arm header file. Changes since -v9 - Added Ingo's Acked-by for x86. - Added Guan's Acked-by for unicore32. - Replaced first patch with updated patch from Srivatsa S. Bhat. This compiles for alpha allmodconfig, all arm defconfigs, and a few test x86_64 defconfigs. I have not tried more. Changes since -v8 - Changes reboot_cpu on stack to cpu to fix bug noticed by Russell King. - Switched unicore32 and arm from using REBOOT_WARM/COLD to HARD/SOFT. - Fixed case statement bug. - Went to using simple_strtoul for parsing reboot_cpu=smp###. - Made parsing of reboot= not use any #ifdef'd code. Changes since -v7. - Fixed authorship for first patch. - Rebased to Linus' current tree (51a26ae7a). Changes since -v6. - Cross compiled all arm architectures (using v3.9 kernel. Fails with current). - Added a #define for non-hotplug case. - Add #define for PF_THREAD_BOUND as compatibility to make stable easier. - Fixup s/reboot_cpu_id/reboot_cpu/ - Add include of linux/uaccess.h to allow building on arm. - Removed last remaining checkpatch.pl line length warning on kernel/reboot.c. - Fixed the duplicate handling or the reboot= kernel parameter. Changes since -v5. - Moved the arch/x86 reboot= up to the generic kernel code. Changes since -v4. - Integrated Srivatsa S. Bhat creating cpu_hotplug_disable() function - Integrated comments by Srivatsa S. Bhat. - Made one more comment consistent with others in function. Changes since -v3. - Added a tested-by for the original reporter. - Fix compile failure found by Joe Perches. - Integrated comments by Joe Perches. To: Andrew Morton Cc: H. Peter Anvin Cc: Russ Anderson Cc: Robin Holt Cc: Russell King Cc: Guan Xuetao Cc: Linux Kernel Mailing List Cc: the arch/x86 maintainers Cc: Arm Mailing List -- 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/