Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758515Ab1DAROz (ORCPT ); Fri, 1 Apr 2011 13:14:55 -0400 Received: from rcsinet10.oracle.com ([148.87.113.121]:55933 "EHLO rcsinet10.oracle.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758448Ab1DAROy (ORCPT ); Fri, 1 Apr 2011 13:14:54 -0400 Message-ID: <4D9607E8.2040500@kernel.org> Date: Fri, 01 Apr 2011 10:14:16 -0700 From: Yinghai Lu User-Agent: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.14) Gecko/20110221 SUSE/3.1.8 Thunderbird/3.1.8 MIME-Version: 1.0 To: "H. Peter Anvin" CC: Stefano Stabellini , Ingo Molnar , "Rafael J. Wysocki" , Michael Leun , "linux-kernel@vger.kernel.org" , Greg Kroah-Hartman , Mike Pagano Subject: Re: 2.6.38.2 breaks suspend to disk References: <20110330203215.0a1a41a7@xenia.leun.net> <20110331090524.07ad0069@xenia.leun.net> <201103312348.53678.rjw@sisk.pl> <4D94FE37.8070109@kernel.org> <4D95F80D.7070201@kernel.org> <4D95FBDD.9050901@zytor.com> In-Reply-To: <4D95FBDD.9050901@zytor.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit X-Source-IP: acsmt356.oracle.com [141.146.40.156] X-Auth-Type: Internal IP X-CT-RefId: str=0001.0A090208.4D9607F8.002F,ss=1,fgs=0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1734 Lines: 52 On 04/01/2011 09:22 AM, H. Peter Anvin wrote: > On 04/01/2011 09:06 AM, Yinghai Lu wrote: >> Index: linux-2.6/arch/x86/kernel/setup.c >> =================================================================== >> --- linux-2.6.orig/arch/x86/kernel/setup.c >> +++ linux-2.6/arch/x86/kernel/setup.c >> @@ -294,10 +294,17 @@ static void __init init_gbpages(void) >> else >> direct_gbpages = 0; >> } >> +static void __init store_mmu_cr4(void) >> +{ >> + mmu_cr4_features = read_cr4(); >> +} >> #else >> static inline void init_gbpages(void) >> { >> } >> +static void __init store_mmu_cr4(void) >> +{ >> +} >> static void __init cleanup_highmap(void) >> { >> } >> @@ -929,6 +936,8 @@ void __init setup_arch(char **cmdline_p) >> /* max_pfn_mapped is updated here */ >> max_low_pfn_mapped = init_memory_mapping(0, max_low_pfn<> max_pfn_mapped = max_low_pfn_mapped; >> + /* after init_memory_mapping updating cr4*/ >> + store_mmu_cr4(); >> >> #ifdef CONFIG_X86_64 >> if (max_pfn> max_low_pfn) { > > This looks really, really, really wrong. > > Why the heck should we save and restore CR4 only for x86-64? Peter, this patch just restoring old sequence. If you think 32 bit should do same thing, we can do that in another patch. Also after closing looking, Stefano's v1 patch should be ok too. because init_memory_mapping is using set_in_cr4(...) and it will update mmu_cr4_features at the same time. Thanks Yinghai -- 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/