On Sun, 12 Nov 2006 17:45:26 +0200 Myaskouvskey, Artiom wrote:
>
> From: Artiom Myaskouvskey <[email protected]>
>
> When using memmap kernel parameter in EFI boot we should also map memory
> regions of runtime services to enable their mapping later.
>
> Signed-off-by: Artiom Myaskouvskey <[email protected]>
> ---
Hi,
It's not clear to me why you are sending a patch to 2.6.18.2.
If this should be applied to current mainline, then the patch
should be made against current source tree. OTOH, if the patch
is meant to be made to the stable tree, you need to cc: [email protected] .
> diff -uprN linux-2.6.18.2.orig/arch/i386/kernel/setup.c
> linux-2.6.18.2/arch/i386/kernel/setup.c
> --- linux-2.6.18.2.orig/arch/i386/kernel/setup.c 2006-11-12
> 11:22:22.000000000 +0200
> +++ linux-2.6.18.2/arch/i386/kernel/setup.c 2006-11-12
> 16:31:32.000000000 +0200
> @@ -364,28 +364,44 @@ static void __init probe_roms(void)
> }
> }
>
> +extern int is_available_memory(efi_memory_desc_t * md);
> +
Externs should be in header files. Please do that and also fix
this one:
./arch/i386/mm/init.c:195:extern int is_available_memory(efi_memory_desc_t *);
> 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;
patch: **** malformed patch at line 37: memmap.map_end;
(it seems that the line was split by your mail client or server)
The line is also too long (> 80 characters).
> p += memmap.desc_size, i++) {
> md = p;
> + next_md = p1;
> current_addr = md->phys_addr + (md->num_pages <<
> 12);
patch: **** malformed patch at line 41: 12);
(split by mail client or server)
> - if (md->type == EFI_CONVENTIONAL_MEMORY) {
> - if (current_addr >= size) {
> - md->num_pages -=
> - (((current_addr-size) +
> PAGE_SIZE-1) >> PAGE_SHIFT);
patch: **** malformed patch at line 45: PAGE_SIZE-1) >> PAGE_SHIFT);
(line was split again)
> - 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);
patch: **** malformed patch at line 53: (((current_addr-size) + PAGE_SIZE-1) >> PAGE_SHIFT);
ditto
> }
> + p1 += memmap.desc_size;
> + next_md = p1;
> + j++;
> }
> - }
> + else if ((md->attribute & EFI_MEMORY_RUNTIME) ==
> EFI_MEMORY_RUNTIME) {
patch: **** malformed patch at line 60: EFI_MEMORY_RUNTIME) {
ditto
> + /* In order to make runtime services
> available we have to include runtime
patch: **** malformed patch at line 61: available we have to include runtime
Please observer 80-column limit.
Also the line above ends with whitespace. Please chop that off.
Also, your mailer has split that long line so that the patch is
malformed -- does not apply.
> + * 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);
patch: **** malformed patch at line 70: memmap.desc_size);
ditto
> + 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]
