The Linux Kernel Mailing List
 help / color / mirror / Atom feed
From: Shuah Khan <shuahkh@osg.samsung.com>
To: john.stultz@linaro.org, shuah@kernel.org
Cc: Shuah Khan <shuahkh@osg.samsung.com>,
	tglx@linutronix.de, sboyd@codeaurora.org,
	linux-kernel@vger.kernel.org, linux-kselftest@vger.kernel.org
Subject: [PATCH 0/3] selfttests: timers ksft_ stubs handling changes
Date: Thu, 10 Aug 2017 17:53:17 -0600	[thread overview]
Message-ID: <cover.1502404902.git.shuahkh@osg.samsung.com> (raw)

This patch series consists of changes to:

Move ksft_ stubs from individual tests into kselftest_stubs.h and change
tests to include it.

Fix posix_timers and freq-step tests to run without ksft_ framework.

This is in preparation to convert timers tests to ksft TAP 13 format.

Question for John Stultz:

The conversion work will be easier without the requirement to be able to
build and run these tests without ksft_ framework. So far the stubs are
simpler. It is might be necessary to ifdef some code paths to have sane
output for both KTEST and !KTEST cases.

Would it be easier to pull in kselftest.h into timers external repo
(if one still exists). This is based on the observation that newer
timer tests don't support !KTEST case e.g: posix_timers and freq-step.

Please review and let me know how you would like me to proceed with the
conversion. I am looking for answer to how important is it to continue to
support !KTEST case.

Shuah Khan (3):
  selftests: timers: move ksft_ stubs from tests into kselftest_stubs.h
  selftests: timers: posix_timers: fix to work without ksft framework
  selftests: timers: freq-step: to work without ksft framework

 tools/testing/selftests/timers/adjtick.c           |  9 +--------
 .../testing/selftests/timers/alarmtimer-suspend.c  |  9 +--------
 tools/testing/selftests/timers/change_skew.c       |  9 +--------
 .../testing/selftests/timers/clocksource-switch.c  |  9 +--------
 tools/testing/selftests/timers/freq-step.c         |  4 ++++
 .../testing/selftests/timers/inconsistency-check.c |  9 +--------
 tools/testing/selftests/timers/kselftest_stubs.h   | 23 ++++++++++++++++++++++
 tools/testing/selftests/timers/leap-a-day.c        |  9 +--------
 tools/testing/selftests/timers/leapcrash.c         |  9 +--------
 tools/testing/selftests/timers/mqueue-lat.c        |  9 +--------
 tools/testing/selftests/timers/nanosleep.c         |  9 +--------
 tools/testing/selftests/timers/nsleep-lat.c        |  9 +--------
 tools/testing/selftests/timers/posix_timers.c      |  4 ++++
 tools/testing/selftests/timers/raw_skew.c          |  9 +--------
 tools/testing/selftests/timers/set-2038.c          |  9 +--------
 tools/testing/selftests/timers/set-tai.c           |  9 +--------
 tools/testing/selftests/timers/set-timer-lat.c     |  9 +--------
 tools/testing/selftests/timers/set-tz.c            |  9 +--------
 tools/testing/selftests/timers/skew_consistency.c  |  9 +--------
 tools/testing/selftests/timers/threadtest.c        |  9 +--------
 tools/testing/selftests/timers/valid-adjtimex.c    |  9 +--------
 21 files changed, 49 insertions(+), 144 deletions(-)
 create mode 100644 tools/testing/selftests/timers/kselftest_stubs.h

-- 
2.11.0

             reply	other threads:[~2017-08-10 23:53 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-08-10 23:53 Shuah Khan [this message]
2017-08-10 23:53 ` [PATCH 1/3] selftests: timers: move ksft_ stubs from tests into kselftest_stubs.h Shuah Khan
2017-08-10 23:53 ` [PATCH 2/3] selftests: timers: posix_timers: fix to work without ksft framework Shuah Khan
2017-08-10 23:53 ` [PATCH 3/3] selftests: timers: freq-step: " Shuah Khan
2017-08-11  0:10 ` [PATCH 0/3] selfttests: timers ksft_ stubs handling changes John Stultz
2017-08-11  0:51   ` Shuah Khan
2017-08-11  0:57     ` John Stultz

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.1502404902.git.shuahkh@osg.samsung.com \
    --to=shuahkh@osg.samsung.com \
    --cc=john.stultz@linaro.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-kselftest@vger.kernel.org \
    --cc=sboyd@codeaurora.org \
    --cc=shuah@kernel.org \
    --cc=tglx@linutronix.de \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox