* [PATCH 2.6.18.2] EFI: mapping memory region of runtime services when using memmap kernel parameter
@ 2006-11-12 15:45 Myaskouvskey, Artiom
2006-11-12 18:46 ` Randy Dunlap
0 siblings, 1 reply; 2+ messages in thread
From: Myaskouvskey, Artiom @ 2006-11-12 15:45 UTC (permalink / raw)
To: davej, hpa
Cc: linux-kernel, Narayanan, Chandramouli, Jiossy, Rami, Satt, Shai
From: Artiom Myaskouvskey <artiom.myaskouvskey@intel.com>
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 <artiom.myaskouvskey@intel.com>
---
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);
+
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;
^ permalink raw reply [flat|nested] 2+ messages in thread* Re: [PATCH 2.6.18.2] EFI: mapping memory region of runtime services when using memmap kernel parameter
2006-11-12 15:45 [PATCH 2.6.18.2] EFI: mapping memory region of runtime services when using memmap kernel parameter Myaskouvskey, Artiom
@ 2006-11-12 18:46 ` Randy Dunlap
0 siblings, 0 replies; 2+ messages in thread
From: Randy Dunlap @ 2006-11-12 18:46 UTC (permalink / raw)
To: Myaskouvskey, Artiom
Cc: davej, hpa, linux-kernel, Narayanan, Chandramouli, Jiossy, Rami,
Satt, Shai
On Sun, 12 Nov 2006 17:45:26 +0200 Myaskouvskey, Artiom wrote:
>
> From: Artiom Myaskouvskey <artiom.myaskouvskey@intel.com>
>
> 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 <artiom.myaskouvskey@intel.com>
> ---
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: stable@kernel.org .
> 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
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2006-11-12 18:46 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2006-11-12 15:45 [PATCH 2.6.18.2] EFI: mapping memory region of runtime services when using memmap kernel parameter Myaskouvskey, Artiom
2006-11-12 18:46 ` Randy Dunlap
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®