Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758582AbZFQBjw (ORCPT ); Tue, 16 Jun 2009 21:39:52 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752413AbZFQBjp (ORCPT ); Tue, 16 Jun 2009 21:39:45 -0400 Received: from terminus.zytor.com ([198.137.202.10]:56119 "EHLO terminus.zytor.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751403AbZFQBjo (ORCPT ); Tue, 16 Jun 2009 21:39:44 -0400 Message-ID: <4A384914.1090900@zytor.com> Date: Tue, 16 Jun 2009 18:38:28 -0700 From: "H. Peter Anvin" User-Agent: Thunderbird 2.0.0.21 (X11/20090320) MIME-Version: 1.0 To: Huang Ying CC: Cliff Wickman , "linux-kernel@vger.kernel.org" , "mingo@elte.hu" , "yinghai@kernel.org" Subject: Re: [PATCH] x86: efi/e820 table merge fix References: <1245201005.11965.4.camel@yhuang-dev.sh.intel.com> In-Reply-To: <1245201005.11965.4.camel@yhuang-dev.sh.intel.com> Content-Type: text/plain; charset=ISO-8859-1 Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1258 Lines: 38 Huang Ying wrote: > On Wed, 2009-06-17 at 05:43 +0800, Cliff Wickman wrote: >> From: Cliff Wickman >> --- linux.orig/arch/x86/kernel/efi.c >> +++ linux/arch/x86/kernel/efi.c >> @@ -240,10 +240,35 @@ static void __init do_add_efi_memmap(voi >> unsigned long long size = md->num_pages << EFI_PAGE_SHIFT; >> int e820_type; >> >> - if (md->attribute & EFI_MEMORY_WB) >> - e820_type = E820_RAM; >> - else >> + switch (md->type) { >> + case EFI_LOADER_CODE: >> + case EFI_LOADER_DATA: >> + case EFI_BOOT_SERVICES_CODE: >> + case EFI_BOOT_SERVICES_DATA: >> + case EFI_CONVENTIONAL_MEMORY: >> + if (md->attribute & EFI_MEMORY_WB) >> + e820_type = E820_RAM; >> + else >> + e820_type = E820_RESERVED; >> + break; > > Why does BIOS mark memory region without EFI_MEMORY_WB as these types? > Any example? > Probably not, but if it does, it's broken, and the memory should be ignored. The original code had the EFI_MEMORY_WB check already, so it seems prudent to keep it. -hpa -- 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/