linux-kernel - Re: [PATCH 5/5] tile: Define AT_VECTOR_SIZE_ARCH for ARCH

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:	Mon, 25 Jul 2016 12:37:50 -0400
From:	Chris Metcalf <cmetcalf@...lanox.com>
To:	James Hogan <james.hogan@...tec.com>, <linux-arch@...r.kernel.org>
CC:	<linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 5/5] tile: Define AT_VECTOR_SIZE_ARCH for ARCH_DLINFO

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@...tec.com>
> Cc: Chris Metcalf<cmetcalf@...lanox.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.
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?

-- 
Chris Metcalf, Mellanox Technologies
http://www.mellanox.com