From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S935502AbYD1POG (ORCPT ); Mon, 28 Apr 2008 11:14:06 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S934178AbYD1PNu (ORCPT ); Mon, 28 Apr 2008 11:13:50 -0400 Received: from rtr.ca ([76.10.145.34]:4996 "EHLO mail.rtr.ca" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932198AbYD1PNt (ORCPT ); Mon, 28 Apr 2008 11:13:49 -0400 Message-ID: <4815E9AD.50601@rtr.ca> Date: Mon, 28 Apr 2008 11:13:49 -0400 From: Mark Lord Organization: Real-Time Remedies Inc. User-Agent: Thunderbird 2.0.0.12 (X11/20080213) MIME-Version: 1.0 To: Pavel Machek Cc: Jens Axboe , linux-arch@vger.kernel.org, linux-kernel@vger.kernel.org, npiggin@suse.de, torvalds@linux-foundation.org Subject: Re: [PATCH 1/11] Add generic helpers for arch IPI function calls References: <1208851058-8500-1-git-send-email-jens.axboe@oracle.com> <1208851058-8500-2-git-send-email-jens.axboe@oracle.com> <480E70ED.3030701@rtr.ca> <20080423072432.GX12774@kernel.dk> <480F3CBC.60305@rtr.ca> <20080423135110.GO12774@kernel.dk> <480F4BD9.8090003@rtr.ca> <20080424105908.GW12774@kernel.dk> <20080426080415.GB3891@ucw.cz> In-Reply-To: <20080426080415.GB3891@ucw.cz> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Pavel Machek wrote: > Hi! > >>>>> The second bug, is that for the halt case at least, >>>>> nobody waits for the other CPU to actually halt >>>>> before continuing.. so we sometimes enter the shutdown >>>>> code while other CPUs are still active. >>>>> >>>>> This causes some machines to hang at shutdown, >>>>> unless CPU_HOTPLUG is configured and takes them offline >>>>> before we get here. >>>> I'm guessing there's a reason it doesn't pass '1' as the last argument, >>>> because that would fix that issue? >>> Undoubtedly -- perhaps the called CPU halts, and therefore cannot reply. :) >> Uhm yes, I guess stop_this_cpu() does exactly what the name implies :-) >> >>> But some kind of pre-halt ack, perhaps plus a short delay by the caller >>> after receipt of the ack, would probably suffice to kill that bug. >>> >>> But I really haven't studied this code enough to know, >>> other than that it historically has been a sticky area >>> to poke around in. >> Something like this will close the window to right up until the point >> where the other CPUs have 'almost' called halt(). > > Now I took a look at context... why not simply use same trick swsusp > uses, and do a hot unplug of all cpus at the end of shutdown? .. That's the current existing workaround for this bug, but not everybody has cpu hotplug in their config, and this bug should still get fixed. Cheers