lkml.org 
[lkml]   [2013]   [Jan]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v7u1 20/31] x86, kexec: replace ident_mapping_init and init_level4_page
On Thu, Jan 03, 2013 at 04:48:40PM -0800, Yinghai Lu wrote:
> static int init_pgtable(struct kimage *image, unsigned long start_pgtable)
> {
> + struct x86_mapping_info info = {
> + .alloc_pgt_page = alloc_pgt_page,
> + .context = image,
> + .pmd_flag = __PAGE_KERNEL_LARGE_EXEC,
> + };

This is leaving ->kernel_mapping uninitialized to contain a random,
previous stack value. I don't think we want that.

> unsigned long mstart, mend;
> pgd_t *level4p;
> int result;
> int i;
>
> level4p = (pgd_t *)__va(start_pgtable);
> - result = init_level4_page(image, level4p, 0, max_pfn << PAGE_SHIFT);
> + clear_page(level4p);
> + result = kernel_ident_mapping_init(&info, level4p,
> + 0, max_pfn << PAGE_SHIFT);
> if (result)
> return result;
>
> @@ -225,7 +115,8 @@ static int init_pgtable(struct kimage *image, unsigned long start_pgtable)
> mstart = image->segment[i].mem;
> mend = mstart + image->segment[i].memsz;
>
> - result = ident_mapping_init(image, level4p, mstart, mend);
> + result = kernel_ident_mapping_init(&info,
> + level4p, mstart, mend);
>
> if (result)
> return result;
> --
> 1.7.10.4
>
>

--
Regards/Gruss,
Boris.


\
 
 \ /
  Last update: 2013-01-04 22:41    [W:0.839 / U:0.328 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site