mirror of
https://sourceware.org/git/glibc.git
synced 2024-12-11 22:00:08 +00:00
b118bce87a
The rtld audit support show two problems on aarch64:
1. _dl_runtime_resolve does not preserve x8, the indirect result
location register, which might generate wrong result calls
depending of the function signature.
2. The NEON Q registers pushed onto the stack by _dl_runtime_resolve
were twice the size of D registers extracted from the stack frame by
_dl_runtime_profile.
While 2. might result in wrong information passed on the PLT tracing,
1. generates wrong runtime behaviour.
The aarch64 rtld audit support is changed to:
* Both La_aarch64_regs and La_aarch64_retval are expanded to include
both x8 and the full sized NEON V registers, as defined by the
ABI.
* dl_runtime_profile needed to extract registers saved by
_dl_runtime_resolve and put them into the new correctly sized
La_aarch64_regs structure.
* The LAV_CURRENT check is change to only accept new audit modules
to avoid the undefined behavior of not save/restore x8.
* Different than other architectures, audit modules older than
LAV_CURRENT are rejected (both La_aarch64_regs and La_aarch64_retval
changed their layout and there are no requirements to support multiple
audit interface with the inherent aarch64 issues).
* A new field is also reserved on both La_aarch64_regs and
La_aarch64_retval to support variant pcs symbols.
Similar to x86, a new La_aarch64_vector type to represent the NEON
register is added on the La_aarch64_regs (so each type can be accessed
directly).
Since LAV_CURRENT was already bumped to support bind-now, there is
no need to increase it again.
Checked on aarch64-linux-gnu.
Co-authored-by: Adhemerval Zanella <adhemerval.zanella@linaro.org>
Reviewed-by: Szabolcs Nagy <szabolcs.nagy@arm.com>
Reviewed-by: Carlos O'Donell <carlos@redhat.com>
Tested-by: Carlos O'Donell <carlos@redhat.com>
(cherry picked from commit ce9a68c57c)
Resolved conflicts:
NEWS
elf/rtld.c
65 lines
2.0 KiB
C
65 lines
2.0 KiB
C
/* Check LD_AUDIT for aarch64 ABI specifics.
|
|
Copyright (C) 2022 Free Software Foundation, Inc.
|
|
This file is part of the GNU C Library.
|
|
|
|
The GNU C Library is free software; you can redistribute it and/or
|
|
modify it under the terms of the GNU Lesser General Public
|
|
License as published by the Free Software Foundation; either
|
|
version 2.1 of the License, or (at your option) any later version.
|
|
|
|
The GNU C Library is distributed in the hope that it will be useful,
|
|
but WITHOUT ANY WARRANTY; without even the implied warranty of
|
|
MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
|
|
Lesser General Public License for more details.
|
|
|
|
You should have received a copy of the GNU Lesser General Public
|
|
License along with the GNU C Library; if not, see
|
|
<https://www.gnu.org/licenses/>. */
|
|
|
|
#include <array_length.h>
|
|
#include <string.h>
|
|
#include <support/check.h>
|
|
#include "tst-audit27mod.h"
|
|
|
|
int
|
|
do_test (void)
|
|
{
|
|
{
|
|
float r = tst_audit27_func_float (FUNC_FLOAT_ARG0, FUNC_FLOAT_ARG1,
|
|
FUNC_FLOAT_ARG2, FUNC_FLOAT_ARG3,
|
|
FUNC_FLOAT_ARG4, FUNC_FLOAT_ARG5,
|
|
FUNC_FLOAT_ARG6, FUNC_FLOAT_ARG7);
|
|
if (r != FUNC_FLOAT_RET)
|
|
FAIL_EXIT1 ("tst_audit27_func_float() returned %a, expected %a",
|
|
r, FUNC_FLOAT_RET);
|
|
}
|
|
|
|
{
|
|
double r = tst_audit27_func_double (FUNC_DOUBLE_ARG0, FUNC_DOUBLE_ARG1,
|
|
FUNC_DOUBLE_ARG2, FUNC_DOUBLE_ARG3,
|
|
FUNC_DOUBLE_ARG4, FUNC_DOUBLE_ARG5,
|
|
FUNC_DOUBLE_ARG6, FUNC_DOUBLE_ARG7);
|
|
if (r != FUNC_DOUBLE_RET)
|
|
FAIL_EXIT1 ("tst_audit27_func_double() returned %la, expected %la",
|
|
r, FUNC_DOUBLE_RET);
|
|
}
|
|
|
|
{
|
|
long double r = tst_audit27_func_ldouble (FUNC_LDOUBLE_ARG0,
|
|
FUNC_LDOUBLE_ARG1,
|
|
FUNC_LDOUBLE_ARG2,
|
|
FUNC_LDOUBLE_ARG3,
|
|
FUNC_LDOUBLE_ARG4,
|
|
FUNC_LDOUBLE_ARG5,
|
|
FUNC_LDOUBLE_ARG6,
|
|
FUNC_LDOUBLE_ARG7);
|
|
if (r != FUNC_LDOUBLE_RET)
|
|
FAIL_EXIT1 ("tst_audit27_func_ldouble() returned %La, expected %La",
|
|
r, FUNC_LDOUBLE_RET);
|
|
}
|
|
|
|
return 0;
|
|
}
|
|
|
|
#include <support/test-driver.c>
|