patches for rt-tests

patches for rt-tests

[Uwe Kleine-König]

Hi Clark,

I fixed some minor issues of rt-tests that I noticed while packaging it
for Debian.  I would be glad if you took them.  (And if you think they
are worth a 0.39, consider pushing the tag, too (hint hint).)

I will send the patches as a reply to this mail for easier review.

The following changes since commit 1b81d40771d977a3eb36dd4f17b7e87e69e5777e:
  Clark Williams (1):
        version bump to 0.38

are available in the git repository at:

  git://git.pengutronix.de/git/ukl/rt-tests.git for-clark

Uwe Kleine-König (3):
      escape minus signs in manpages
      signaltest: fix typo in copyright header
      Makefile: don't override DESTDIR, prefix, bindir and mandir

 Makefile                    |    8 ++++----
 src/cyclictest/cyclictest.8 |   14 +++++++-------
 src/pi_tests/pi_stress.8    |   16 ++++++++--------
 src/signaltest/signaltest.c |    2 +-
 4 files changed, 20 insertions(+), 20 deletions(-)
--
To unsubscribe from this list: send the line "unsubscribe linux-rt-users" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 1/3] escape minus signs in manpages

[Uwe Kleine-König]

'-' chars are interpreted as hyphens by groff, not as minus signs.  This
means that if you're using an UTF-8 locale searching for '-' to find
options doesn't work.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
 src/cyclictest/cyclictest.8 |   14 +++++++-------
 src/pi_tests/pi_stress.8    |   16 ++++++++--------
 2 files changed, 15 insertions(+), 15 deletions(-)

diff --git a/src/cyclictest/cyclictest.8 b/src/cyclictest/cyclictest.8
index b54e213..48b2c5e 100644
--- a/src/cyclictest/cyclictest.8
+++ b/src/cyclictest/cyclictest.8
@@ -79,13 +79,13 @@ CLOCK_REALTIME can be set by settimeofday, while CLOCK_MONOTONIC can not be modi
 This option has no influence when the \-s option is given.
 .TP
 .B \-C, \-\-context
-context switch tracing (used with -b)
+context switch tracing (used with \-b)
 .TP
 .B \-d, \-\-distance=DIST
-Set the distance of thread intervals in microseconds (default is 500us). When cylictest is called with the \-t option and more than one thread is created, then this distance value is added to the interval of the threads: Interval(thread N) = Interval(thread N-1) + DIST
+Set the distance of thread intervals in microseconds (default is 500us). When cylictest is called with the \-t option and more than one thread is created, then this distance value is added to the interval of the threads: Interval(thread N) = Interval(thread N\-1) + DIST
 .TP
 .B \-E, \-\-event
-event tracing (used with -b)
+event tracing (used with \-b)
 .TP
 .B \-f, \-\-ftrace
 Enable function tracing using ftrace as tracer. This option is available only with \-b.
@@ -114,7 +114,7 @@ times for multiple trace options. For example trace options look at /sys/kernel/
 .TP
 .B \-p, \-\-prio=PRIO
 Set the priority of the first thread. The given priority is set to the first test thread. Each further thread gets a lower priority:
-Priority(Thread N) = max(Priority(Thread N-1) - 1, 0)
+Priority(Thread N) = max(Priority(Thread N\-1) \- 1, 0)
 .TP
 .B \-q, \-\-quiet
 Run the tests quiet and print only a summary on exit. Useful for automated tests, where only the summary output needs to be captured.
@@ -126,7 +126,7 @@ Use relative timers instead of absolute. The default behaviour of the tests is t
 Use sys_nanosleep and sys_setitimer instead of posix timers. Note, that \-s can only be used with one thread because itimers are per process and not per thread. \-s in combination with \-n uses the nanosleep syscall and is not restricted to one thread.
 .TP
 .B \-T, \-\-tracer=TRACEFUNC
-set the ftrace tracer function. Used with the -b option. Must be one
+set the ftrace tracer function. Used with the \-b option. Must be one
 of the trace functions available from <debugfs-mountpoint>/kernel/debug/tracing/available_tracers
 .TP
 .B \-t, \-\-threads[=NUM]
@@ -147,10 +147,10 @@ where n=task number c=count v=latency value in us. Use this option in combinatio
 Run the test for the specified time, which defaults to seconds. Append 'm', 'h', or 'd' to specify minutes, hours or days
 .TP
 .B \\-w, \-\-wakeup 
-task wakeup tracing (used with -b)
+task wakeup tracing (used with \-b)
 .TP
 .B \\-W, \-\-wakeuprt
-rt-task wakeup tracing (used with -b)
+rt-task wakeup tracing (used with \-b)
 .\" .SH SEE ALSO
 .\" .BR bar (1),
 .\" .BR baz (1).
diff --git a/src/pi_tests/pi_stress.8 b/src/pi_tests/pi_stress.8
index cfff3f9..4fecab5 100644
--- a/src/pi_tests/pi_stress.8
+++ b/src/pi_tests/pi_stress.8
@@ -19,13 +19,13 @@ pi_stress \- a stress test for POSIX Priority Inheritance mutexes
 .IR seconds ]
 .RB [ \-g|\-\-groups
 .IR groups
-.RB [ \-d|--debug ]
-.RB [ \-v|--verbose ]
-.RB [ \-s|--signal ]
-.RB [ \-r|--rr ]
-.RB [ \-p|--prompt ]
-.RB [ \-m|--mlockall ]
-.RB [ \-u|--uniprocessor ]
+.RB [ \-d|\-\-debug ]
+.RB [ \-v|\-\-verbose ]
+.RB [ \-s|\-\-signal ]
+.RB [ \-r|\-\-rr ]
+.RB [ \-p|\-\-prompt ]
+.RB [ \-m|\-\-mlockall ]
+.RB [ \-u|\-\-uniprocessor ]
 .br
 .\" help
 .B pi_stress
@@ -48,7 +48,7 @@ doesn't work.
 Run for
 .I n
 number of inversion conditions. This is the total number of inversions
-for all inversion groups. Default is -1 for infinite.
+for all inversion groups. Default is \-1 for infinite.
 .IP "\-t n|\-\-duration=n"
 Run the test for 
 .I n
-- 
1.6.2.4

--
To unsubscribe from this list: send the line "unsubscribe linux-rt-users" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 2/3] signaltest: fix typo in copyright header

[Uwe Kleine-König]

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Cc: Thomas Gleixner <tglx@linutronix.de>
---
 src/signaltest/signaltest.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/signaltest/signaltest.c b/src/signaltest/signaltest.c
index 02a757e..b637aae 100644
--- a/src/signaltest/signaltest.c
+++ b/src/signaltest/signaltest.c
@@ -4,7 +4,7 @@
  * (C) 2007 Thomas Gleixner <tglx@linutronix.de>
  *
  * This program is free software; you can redistribute it and/or
- * modify it under the terms of the GNU General Public License Veriosn
+ * modify it under the terms of the GNU General Public License Version
  * 2 as published by the Free Software Foundation;
  *
  */
-- 
1.6.2.4

--
To unsubscribe from this list: send the line "unsubscribe linux-rt-users" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH 3/3] Makefile: don't override DESTDIR, prefix, bindir and mandir

[Uwe Kleine-König]

This eases packaging rt-tests and shouldn't have any further impact.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
---
 Makefile |    8 ++++----
 1 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/Makefile b/Makefile
index 294d588..e139eb4 100644
--- a/Makefile
+++ b/Makefile
@@ -3,10 +3,10 @@ VERSION_STRING = "0.38"
 TARGETS	= cyclictest signaltest classic_pi pi_stress
 FLAGS	= -Wall -Wno-nonnull -O2
 LIBS 	= -lpthread -lrt
-DESTDIR	=
-prefix  = /usr/local
-bindir  = $(prefix)/bin
-mandir	= $(prefix)/share/man/man8
+DESTDIR	?=
+prefix  ?= /usr/local
+bindir  ?= $(prefix)/bin
+mandir	?= $(prefix)/share/man/man8
 
 all: $(TARGETS)
 
-- 
1.6.2.4

--
To unsubscribe from this list: send the line "unsubscribe linux-rt-users" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH 3/3] Makefile: don't override DESTDIR, prefix, bindir and mandir

[Leon Woestenberg]

Hello,

2009/5/2 Uwe Kleine-König <u.kleine-koenig@pengutronix.de>:
> This eases packaging rt-tests and shouldn't have any further impact.
>
> Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>

In addition to this, could LDFLAGS be respected?)

Acked-by: Leon Woestenberg <leon@sidebranch.com>


Regards,
-- 
Leon
--
To unsubscribe from this list: send the line "unsubscribe linux-rt-users" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html