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

Message

John Hubbard July 5, 2024, 7:57 p.m. UTC

Hi Shuah,

These are for 6.10, as we just discussed.

Changes since v4:

1) Subject line on patch #2/3: s/mm/vDSO/

2) Added Muhammad's review tag.

Changes since v3:

1. Rebased onto Linux 6.10-rc6+.

Cover letter for v3:

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.

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.

thanks,
John Hubbard
NVIDIA

John Hubbard (3):
  selftests/vDSO: fix clang build errors and warnings
  selftests/vDSO: 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: d270dd21bee023ab627f34cfb77a9b89a688492a

Comments

Shuah Khan July 5, 2024, 8:16 p.m. UTC | #1

On 7/5/24 13:57, John Hubbard wrote:
> Hi Shuah,
> 
> These are for 6.10, as we just discussed.
> 
> Changes since v4:
> 
> 1) Subject line on patch #2/3: s/mm/vDSO/
> 
> 2) Added Muhammad's review tag.
> 

Applied to
https://git.kernel.org/pub/scm/linux/kernel/git/shuah/linux-kselftest.git/?h=fixes
for Linux 6.10

thanks,
-- Shuah