Re: [PATCH v3 0/3] selftests/vDSO: fix clang build errors, and Makefile cleanup

From: John Hubbard
Date: Fri Jun 14 2024 - 19:35:27 EST


On 6/14/24 4:31 PM, John Hubbard wrote:
Hi,

Jason A. Donenfeld, I've added you because I ended up looking through
your latest "implement getrandom() in vDSO" series [1], which also
touches this Makefile, so just a heads up about upcoming (minor) merge
conflicts.

Belatedly adding Carlos Llamas and Thomas Gleixner to Cc.


Changes since v2:

1. Added two patches, both of which apply solely to the Makefile.
These provide a smaller, cleaner, and more accurate Makefile.

2. Added Reviewed-by and Tested-by tags for the original patch, which
fixes all of the clang errors and warnings for this selftest.

3. Removed an obsolete blurb from the commit description of the original
patch, now that Valentin Obst LLVM build fix has been merged.


[1] https://lore.kernel.org/20240614190646.2081057-1-Jason@xxxxxxxxx


John Hubbard (3):
selftests/vDSO: fix clang build errors and warnings
selftests/mm: remove partially duplicated "all:" target in Makefile
selftests/vDSO: remove duplicate compiler invocations from Makefile

tools/testing/selftests/vDSO/Makefile | 29 ++++++++-----------
tools/testing/selftests/vDSO/parse_vdso.c | 16 ++++++----
.../selftests/vDSO/vdso_standalone_test_x86.c | 18 ++++++++++--
3 files changed, 39 insertions(+), 24 deletions(-)


base-commit: 2ccbdf43d5e758f8493a95252073cf9078a5fea5

thanks,
--
John Hubbard
NVIDIA