Skip to content

Commit

Permalink
x32: Fix coding style violations in the x32 VDSO code
Browse files Browse the repository at this point in the history
Move the prototype for x32_setup_additional_pages() to a header file,
and adjust the coding style to match standard.

Signed-off-by: H. Peter Anvin <hpa@linux.intel.com>
Cc: H. J. Lu <hjl.tools@gmail.com>
  • Loading branch information
H. Peter Anvin committed Feb 21, 2012
1 parent 1a21d4e commit 22e842d
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 6 deletions.
2 changes: 2 additions & 0 deletions arch/x86/include/asm/elf.h
Original file line number Diff line number Diff line change
Expand Up @@ -331,6 +331,8 @@ struct linux_binprm;
#define ARCH_HAS_SETUP_ADDITIONAL_PAGES 1
extern int arch_setup_additional_pages(struct linux_binprm *bprm,
int uses_interp);
extern int x32_setup_additional_pages(struct linux_binprm *bprm,
int uses_interp);

extern int syscall32_setup_pages(struct linux_binprm *, int exstack);
#define compat_arch_setup_additional_pages syscall32_setup_pages
Expand Down
3 changes: 1 addition & 2 deletions arch/x86/vdso/vdso32-setup.c
Original file line number Diff line number Diff line change
Expand Up @@ -318,9 +318,8 @@ int arch_setup_additional_pages(struct linux_binprm *bprm, int uses_interp)
bool compat;

#ifdef CONFIG_X86_X32_ABI
extern int x32_setup_additional_pages(struct linux_binprm *, int);
if (test_thread_flag(TIF_X32))
return x32_setup_additional_pages (bprm, uses_interp);
return x32_setup_additional_pages(bprm, uses_interp);
#endif

if (vdso_enabled == VDSO_DISABLED)
Expand Down
8 changes: 4 additions & 4 deletions arch/x86/vdso/vma.c
Original file line number Diff line number Diff line change
Expand Up @@ -187,15 +187,15 @@ static int setup_additional_pages(struct linux_binprm *bprm,

int arch_setup_additional_pages(struct linux_binprm *bprm, int uses_interp)
{
return setup_additional_pages (bprm, uses_interp, vdso_pages,
vdso_size);
return setup_additional_pages(bprm, uses_interp, vdso_pages,
vdso_size);
}

#ifdef CONFIG_X86_X32_ABI
int x32_setup_additional_pages(struct linux_binprm *bprm, int uses_interp)
{
return setup_additional_pages (bprm, uses_interp, vdsox32_pages,
vdsox32_size);
return setup_additional_pages(bprm, uses_interp, vdsox32_pages,
vdsox32_size);
}
#endif

Expand Down

0 comments on commit 22e842d

Please sign in to comment.