Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752770AbbKLOeL (ORCPT ); Thu, 12 Nov 2015 09:34:11 -0500 Received: from mail.bmw-carit.de ([62.245.222.98]:56994 "EHLO mail.bmw-carit.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751043AbbKLOeJ (ORCPT ); Thu, 12 Nov 2015 09:34:09 -0500 X-CTCH-RefID: str=0001.0A0C0205.5644A359.01A0,ss=1,re=0.000,recu=0.000,reip=0.000,cl=1,cld=1,fgs=0 Subject: Re: [PATCH] smpboot: Add smpboot state variables instead of reusing CPU hotplug states To: References: <1444908764-9057-1-git-send-email-daniel.wagner@bmw-carit.de> <20151107071723.GH3879@linux.vnet.ibm.com> CC: , , "Thomas Gleixner" , Peter Zijlstra From: Daniel Wagner Message-ID: <5644A358.6090806@bmw-carit.de> Date: Thu, 12 Nov 2015 15:34:00 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: <20151107071723.GH3879@linux.vnet.ibm.com> Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1364 Lines: 33 Hi Paul, On 11/07/2015 08:17 AM, Paul E. McKenney wrote: > On Thu, Oct 15, 2015 at 01:32:44PM +0200, Daniel Wagner wrote: >> The cpu hotplug state machine in smpboot.c is reusing the states from >> cpu.h. That is confusing when it comes to the CPU_DEAD_FROZEN usage. >> Paul explained to me that he was in need of an additional state >> for destinguishing between a CPU error states. For this he just >> picked CPU_DEAD_FROZEN. >> >> 8038dad7e888581266c76df15d70ca457a3c5910 smpboot: Add common code for notification from dying CPU >> 2a442c9c6453d3d043dfd89f2e03a1deff8a6f06 x86: Use common outgoing-CPU-notification code >> >> Instead of reusing the states, let's add new definition inside >> the smpboot.c file with explenation what those states >> mean. Thanks Paul for providing them. >> >> Signed-off-by: Daniel Wagner > > Apologies for the delay, I didn't realize that you were waiting on me. No apology needed. I didn't express my wish explicitly. I'll post it again after the merge window is closed. > Reviewed-by: Paul E. McKenney Thanks, Daniel -- 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/