On Mon, 13 Nov 2006 13:08:50 +0200 Myaskouvskey, Artiom wrote:
> From: Artiom Myaskouvskey <[email protected]>
>
> When using memmap kernel parameter in EFI boot we should also add to memory map
> memory regions of runtime services to enable their mapping later.
>
> Signed-off-by: Artiom Myaskouvskey <[email protected]>
> ---
Much better (no line wrap), but still has problems with
a. extern not in a header file
b. lines > 80 columns
c. trailing whitespace (in both EFI patches)
(sorry if you receive duplicates, my original was dropped or blocked;
oh, the email address strings were odd)
(third/final attempt)
> diff -uprN linux-2.6.19-rc5-git2.orig/arch/i386/kernel/setup.c linux-2.6.19-rc5-git2/arch/i386/kernel/setup.c
> --- linux-2.6.19-rc5-git2.orig/arch/i386/kernel/setup.c 2006-11-13 11:15:17.000000000 +0200
> +++ linux-2.6.19-rc5-git2/arch/i386/kernel/setup.c 2006-11-13 11:15:42.000000000 +0200
> @@ -346,28 +346,44 @@ static void __init probe_roms(void)
> }
> }
>
> +extern int is_available_memory(efi_memory_desc_t * md);
> +
> static void __init limit_regions(unsigned long long size)
> {
> unsigned long long current_addr = 0;
> - int i;
> + int i , j;
>
> - if (efi_enabled) {
> - efi_memory_desc_t *md;
> - void *p;
> + if (efi_enabled) {
> + efi_memory_desc_t *md, *next_md = 0;
> + void *p, *p1;
>
> - for (p = memmap.map, i = 0; p < memmap.map_end;
> + for (p = memmap.map, i = 0,j = 0, p1 = memmap.map; p < memmap.map_end;
> p += memmap.desc_size, i++) {
> md = p;
> + next_md = p1;
> current_addr = md->phys_addr + (md->num_pages << 12);
> - if (md->type == EFI_CONVENTIONAL_MEMORY) {
> - if (current_addr >= size) {
> - md->num_pages -=
> - (((current_addr-size) + PAGE_SIZE-1) >> PAGE_SHIFT);
> - memmap.nr_map = i + 1;
> - return;
> + if (is_available_memory(md)) {
> + if (md->phys_addr >= size) continue;
> + memcpy(next_md, md, memmap.desc_size);
> + if (current_addr >= size) {
> + next_md->num_pages -= (((current_addr-size) + PAGE_SIZE-1) >> PAGE_SHIFT);
> }
> + p1 += memmap.desc_size;
> + next_md = p1;
> + j++;
> }
> - }
> + else if ((md->attribute & EFI_MEMORY_RUNTIME) == EFI_MEMORY_RUNTIME) {
> + /* In order to make runtime services available we have to include runtime
> + * memory regions in memory map */
> + memcpy(next_md, md, memmap.desc_size);
> + p1 += memmap.desc_size;
> + next_md = p1;
> + j++;
> + }
> + }
> + memmap.nr_map = j;
> + memmap.map_end = memmap.map + (memmap.nr_map * memmap.desc_size);
> + return;
> }
> for (i = 0; i < e820.nr_map; i++) {
> current_addr = e820.map[i].addr + e820.map[i].size;
> -
---
~Randy
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
[Index of Archives]
[Kernel Newbies]
[Netfilter]
[Bugtraq]
[Photo]
[Stuff]
[Gimp]
[Yosemite News]
[MIPS Linux]
[ARM Linux]
[Linux Security]
[Linux RAID]
[Video 4 Linux]
[Linux for the blind]
[Linux Resources]
