lkml.org 
[lkml]   [2016]   [Jul]   [25]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 5/5] tile: Define AT_VECTOR_SIZE_ARCH for ARCH_DLINFO
On Mon, Jul 25, 2016 at 12:37:50PM -0400, Chris Metcalf wrote:
> On 7/25/2016 11:59 AM, James Hogan wrote:
> > AT_VECTOR_SIZE_ARCH should be defined with the maximum number of
> > NEW_AUX_ENT entries that ARCH_DLINFO can contain, but it wasn't defined
> > for tile at all even though ARCH_DLINFO will contain one NEW_AUX_ENT for
> > the VDSO address.
> >
> > This shouldn't be a problem as AT_VECTOR_SIZE_BASE includes space for
> > AT_BASE_PLATFORM which tile doesn't use, but lets define it now and add
> > the comment above ARCH_DLINFO as found in several other architectures to
> > remind future modifiers of ARCH_DLINFO to keep AT_VECTOR_SIZE_ARCH up to
> > date.
> >
> > Fixes: 4a556f4f56da ("tile: implement gettimeofday() via vDSO")
> > Signed-off-by: James Hogan<james.hogan@imgtec.com>
> > Cc: Chris Metcalf<cmetcalf@mellanox.com>
> > ---
> > arch/tile/include/asm/elf.h | 1 +
> > arch/tile/include/uapi/asm/auxvec.h | 2 ++
> > 2 files changed, 3 insertions(+)
>
> Thanks, taken into the tile tree.
>
> I'm a little dubious that AT_VECTOR_SIZE_ARCH should be being defined
> in the uapi version of auxvec.h, since I don't see why userspace should
> be interested in it, but that does seem to be the current convention.

Yes, I agree.

> Ugh, there are even CONFIG_xxx ifdefs in the x86 and sh uapi/asm/auxvec.h ...
> Maybe the define could be moved to arch/*/include/asm/elf.h?

That sounds like a sensible place for it to me.

Cheers
James
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2016-07-25 22:01    [W:0.030 / U:0.332 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site