lkml.org 
[lkml]   [2014]   [Mar]   [12]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 3/3] x86, vdso32: handle 32 bit vDSO larger one page
On 03/12/2014 03:51 PM, Stefani Seibold wrote:
> This patch enables 32 bit vDSO which are larger than a page. Currently
> two pages are reserved, this should be enough for future improvements.
>
> Signed-off-by: Stefani Seibold <stefani@seibold.net>
> ---
> arch/x86/include/asm/fixmap.h | 4 +++-
> arch/x86/vdso/vdso32-setup.c | 29 +++++++++++++++++++----------
> 2 files changed, 22 insertions(+), 11 deletions(-)
>
> diff --git a/arch/x86/include/asm/fixmap.h b/arch/x86/include/asm/fixmap.h
> index 094d0cc..f513f14 100644
> --- a/arch/x86/include/asm/fixmap.h
> +++ b/arch/x86/include/asm/fixmap.h
> @@ -43,6 +43,8 @@ extern unsigned long __FIXADDR_TOP;
>
> #define FIXADDR_USER_START __fix_to_virt(FIX_VDSO)
> #define FIXADDR_USER_END __fix_to_virt(FIX_VDSO - 1)
> +
> +#define MAX_VDSO_PAGES 2
> #else
> #define FIXADDR_TOP (VSYSCALL_END-PAGE_SIZE)
>
> @@ -74,7 +76,7 @@ extern unsigned long __FIXADDR_TOP;
> enum fixed_addresses {
> #ifdef CONFIG_X86_32
> FIX_HOLE,
> - FIX_VDSO,
> + FIX_VDSO = MAX_VDSO_PAGES,
> VVAR_PAGE,
> VSYSCALL_HPET,

Can we make this FIX_HOLE + MAX_VDSO_PAGES or at least a comment
explaning the constraint here... otherwise I fear random breakage.

> + for(i = 0; i != vdso32_size; ++i)
> + vdso32_pages[i] = virt_to_page(vdso_pages + VDSO_OFFSET(i));

Stylistic note:

for (i = 0; i < vdso32_size; i++)

Otherwise it really seems simple enough.



\
 
 \ /
  Last update: 2014-03-13 01:01    [W:0.088 / U:0.184 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site