From: Maciej Wieczor-Retman <maciej.wieczor-retman@intel.com>
To: akpm@linux-foundation.org, christian@kellner.me,
fenghua.yu@intel.com, keescook@chromium.org,
ndesaulniers@google.com, coltonlewis@google.com,
dmatlack@google.com, vipinsh@google.com, seanjc@google.com,
brauner@kernel.org, pbonzini@redhat.com, shuah@kernel.org,
hannes@cmpxchg.org, nphamcs@gmail.com, reinette.chatre@intel.com
Cc: ilpo.jarvinen@linux.intel.com, linux-kselftest@vger.kernel.org,
kvm@vger.kernel.org, linux-kernel@vger.kernel.org,
linux-mm@kvack.org
Subject: [PATCH v3 0/8] Add printf attribute to kselftest functions
Date: Fri, 22 Sep 2023 11:05:50 +0200 [thread overview]
Message-ID: <cover.1695373131.git.maciej.wieczor-retman@intel.com> (raw)
Kselftest.h declares many variadic functions that can print some
formatted message while also executing selftest logic. These
declarations don't have any compiler mechanism to verify if passed
arguments are valid in comparison with format specifiers used in
printf() calls.
Attribute addition can make debugging easier, the code more consistent
and prevent mismatched or missing variables.
Add a __printf() macro that validates types of variables passed to the
format string. The macro is similarly used in other tools in the kernel.
Add __printf() attributes to function definitions inside kselftest.h that
use printing.
Adding the __printf() macro exposes some mismatches in format strings
across different selftests.
Fix the mismatched format specifiers in multiple tests.
Series is based on kselftests next branch.
Changelog v3:
- Change git signature from Wieczor-Retman Maciej to Maciej
Wieczor-Retman.
- Add one review tag.
- Rebase onto updated kselftests next branch and change base commit.
Changelog v2:
- Add review and fixes tags to patches.
- Add two patches with mismatch fixes.
- Fix missed attribute in selftests/kvm. (Andrew)
- Fix previously missed issues in selftests/mm (Ilpo)
[v2] https://lore.kernel.org/all/cover.1693829810.git.maciej.wieczor-retman@intel.com/
[v1] https://lore.kernel.org/all/cover.1693216959.git.maciej.wieczor-retman@intel.com/
Maciej Wieczor-Retman (8):
selftests: Add printf attribute to ksefltest prints
selftests/cachestat: Fix print_cachestat format
selftests/openat2: Fix wrong format specifier
selftests/pidfd: Fix ksft print formats
selftests/sigaltstack: Fix wrong format specifier
selftests/kvm: Replace attribute with macro
selftests/mm: Substitute attribute with a macro
selftests/resctrl: Fix wrong format specifier
.../selftests/cachestat/test_cachestat.c | 2 +-
tools/testing/selftests/kselftest.h | 18 ++++++++++--------
.../testing/selftests/kvm/include/test_util.h | 8 ++++----
tools/testing/selftests/mm/mremap_test.c | 2 +-
tools/testing/selftests/mm/pkey-helpers.h | 2 +-
tools/testing/selftests/openat2/openat2_test.c | 2 +-
.../selftests/pidfd/pidfd_fdinfo_test.c | 2 +-
tools/testing/selftests/pidfd/pidfd_test.c | 12 ++++++------
tools/testing/selftests/resctrl/cache.c | 2 +-
tools/testing/selftests/sigaltstack/sas.c | 2 +-
10 files changed, 27 insertions(+), 25 deletions(-)
base-commit: ce9ecca0238b140b88f43859b211c9fdfd8e5b70
--
2.42.0
next reply other threads:[~2023-09-22 9:06 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-09-22 9:05 Maciej Wieczor-Retman [this message]
2023-09-22 9:06 ` [PATCH v3 1/8] selftests: Add printf attribute to ksefltest prints Maciej Wieczor-Retman
2023-09-29 22:31 ` Reinette Chatre
2023-09-22 9:06 ` [PATCH v3 2/8] selftests/cachestat: Fix print_cachestat format Maciej Wieczor-Retman
2023-09-22 9:06 ` [PATCH v3 3/8] selftests/openat2: Fix wrong format specifier Maciej Wieczor-Retman
2023-09-22 9:06 ` [PATCH v3 4/8] selftests/pidfd: Fix ksft print formats Maciej Wieczor-Retman
2023-09-22 9:06 ` [PATCH v3 5/8] selftests/sigaltstack: Fix wrong format specifier Maciej Wieczor-Retman
2023-09-22 9:06 ` [PATCH v3 6/8] selftests/kvm: Replace attribute with macro Maciej Wieczor-Retman
2023-09-22 9:06 ` [PATCH v3 7/8] selftests/mm: Substitute attribute with a macro Maciej Wieczor-Retman
2023-09-22 9:06 ` [PATCH v3 8/8] selftests/resctrl: Fix wrong format specifier Maciej Wieczor-Retman
2023-09-29 22:32 ` Reinette Chatre
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=cover.1695373131.git.maciej.wieczor-retman@intel.com \
--to=maciej.wieczor-retman@intel.com \
--cc=akpm@linux-foundation.org \
--cc=brauner@kernel.org \
--cc=christian@kellner.me \
--cc=coltonlewis@google.com \
--cc=dmatlack@google.com \
--cc=fenghua.yu@intel.com \
--cc=hannes@cmpxchg.org \
--cc=ilpo.jarvinen@linux.intel.com \
--cc=keescook@chromium.org \
--cc=kvm@vger.kernel.org \
--cc=linux-kernel@vger.kernel.org \
--cc=linux-kselftest@vger.kernel.org \
--cc=linux-mm@kvack.org \
--cc=ndesaulniers@google.com \
--cc=nphamcs@gmail.com \
--cc=pbonzini@redhat.com \
--cc=reinette.chatre@intel.com \
--cc=seanjc@google.com \
--cc=shuah@kernel.org \
--cc=vipinsh@google.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.