From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751448AbXCYPK0 (ORCPT ); Sun, 25 Mar 2007 11:10:26 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1751411AbXCYPK0 (ORCPT ); Sun, 25 Mar 2007 11:10:26 -0400 Received: from ug-out-1314.google.com ([66.249.92.175]:19223 "EHLO ug-out-1314.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751448AbXCYPKZ (ORCPT ); Sun, 25 Mar 2007 11:10:25 -0400 DomainKey-Signature: a=rsa-sha1; c=nofws; d=gmail.com; s=beta; h=received:from:to:subject:date:user-agent:cc:references:in-reply-to:mime-version:content-type:content-transfer-encoding:content-disposition:message-id; b=HXyGIlxIIP6QADoEahBIYUchLrC4vSNir1ppyy2EelTp0mgKNASqNhgdG1/YrBrSm8YxiAG3uCK4uN3cLonb9k1dqW8KejMdKM+jy7hWDntOtANmpPSXYmzWjVhe2bJKtRCD4rWNAjHvs7f2u/0YZ1gHMY/U4eWgQCXQFTHn5lw= From: Maxim To: "Rafael J. Wysocki" Subject: Re: [BUG] Code reordering in swsusp breaks suspend on SMP systems Date: Sun, 25 Mar 2007 17:10:12 +0200 User-Agent: KMail/1.9.6 Cc: linux-kernel@vger.kernel.org, Pavel Machek , Tigran Aivazian References: <200703211840.53242.maximlevitsky@gmail.com> <200703250240.39457.maximlevitsky@gmail.com> <200703251413.07799.rjw@sisk.pl> In-Reply-To: <200703251413.07799.rjw@sisk.pl> MIME-Version: 1.0 Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Content-Disposition: inline Message-Id: <200703251710.12842.maximlevitsky@gmail.com> Sender: linux-kernel-owner@vger.kernel.org X-Mailing-List: linux-kernel@vger.kernel.org On Sunday 25 March 2007 14:13:07 Rafael J. Wysocki wrote: > On Sunday, 25 March 2007 01:40, Maxim wrote: > > On Friday 23 March 2007 16:42:44 Rafael J. Wysocki wrote: > > > On Friday, 23 March 2007 00:30, Rafael J. Wysocki wrote: > > > > On Thursday, 22 March 2007 00:53, Rafael J. Wysocki wrote: > > > > > On Thursday, 22 March 2007 00:39, Maxim wrote: > > > > > > On Thursday 22 March 2007 01:24:25 Rafael J. Wysocki wrote: > > > > > > > On Thursday, 22 March 2007 00:09, Maxim wrote: > > > > > > > > On Thursday 22 March 2007 00:39:02 you wrote: > > > > > > > > > On Wednesday, 21 March 2007 23:21, Pavel Machek wrote: > > Hi, > > I confirm that the above patch works, > > > > At least system didn't hang on resume with microcode driver loaded, > > > > I can't really test whenever it did update microcode because I almost sure there is nothing to update > > (I use core 2 duo that I bought a month ago, and an intel motherboard with latest bios ( updated yesterday) ) > > I selected this driver just in case when I compiled kernel. > > OK, thanks for testing. > Hello, I tested this patch, and it does work too, System suspended /resumed correctly, no errors in kernel log > - first_cpu = first_cpu(cpu_present_map); > - if (!cpu_online(first_cpu)) { > - error = _cpu_up(first_cpu); > - if (error) { > - printk(KERN_ERR "Could not bring CPU%d up.\n", > - first_cpu); > - goto out; > - } > - } > - Nice, I once have seen those lines too, and they look ridiculous, but could that be that they are still necessary on some systems, Best regards, Maxim Levitsky