[v2] test: add variable delta for poll.c test

Submitted by Vitaly Ostrosablin on Jan. 9, 2017, 12:26 p.m.

Details

Message ID 20170109122646.3094-1-vostrosablin@virtuozzo.com
State Superseded
Series "test: add variable delta for poll.c test"
Headers show

Commit Message

Vitaly Ostrosablin Jan. 9, 2017, 12:26 p.m.
For ZDTM purposes, poll.c uses delta of 5 seconds, which is fine, since
tests don't take a very long time to pass. However, in vzt-cpt, we start
a batch with multitude of tests and starting them all could take over an
hour. Hence, the test is bound to fail. An easy solution would be to
introduce an environment variable, which would allow to specify the
delta - ZDTM_DELTA. If it's unset (which is default behaviour in ZDTM) -
it would default to 5 seconds. However, if it's set (what we can do from
vzt-cpt) - then it will use delta which is set by our test.

Signed-off-by: Vitaly Ostrosablin <vostrosablin@virtuozzo.com>
---
 test/zdtm/static/poll.c | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

Patch hide | download patch | download mbox

diff --git a/test/zdtm/static/poll.c b/test/zdtm/static/poll.c
index c0f75745..33bc0bd4 100644
--- a/test/zdtm/static/poll.c
+++ b/test/zdtm/static/poll.c
@@ -43,6 +43,7 @@  int main(int argc, char *argv[])
 	int delta, status;
 	task_waiter_t t;
 	pid_t pid;
+	char *deltaenv;
 
 	test_init(argc, argv);
 	task_waiter_init(&t);
@@ -80,7 +81,11 @@  int main(int argc, char *argv[])
 		show_timestamp("Start", time1.tv_sec, time1.tv_usec);
 
 		task_waiter_complete(&t, 1);
-		delta = 5;
+		deltaenv = getenv("ZDTM_DELTA");
+		if (deltaenv)
+			delta = atoi(deltaenv);
+		else
+			delta = 5;
 		while (test_go()) {
 			ret = poll(ufds, 2, delta * 1000);
 			show_pollfd(ufds, 2);

Comments

Cyrill Gorcunov Jan. 9, 2017, 12:34 p.m.
On Mon, Jan 09, 2017 at 03:26:46PM +0300, Vitaly Ostrosablin wrote:
> For ZDTM purposes, poll.c uses delta of 5 seconds, which is fine, since
> tests don't take a very long time to pass. However, in vzt-cpt, we start
> a batch with multitude of tests and starting them all could take over an
> hour. Hence, the test is bound to fail. An easy solution would be to
> introduce an environment variable, which would allow to specify the
> delta - ZDTM_DELTA. If it's unset (which is default behaviour in ZDTM) -
> it would default to 5 seconds. However, if it's set (what we can do from
> vzt-cpt) - then it will use delta which is set by our test.
> 
> Signed-off-by: Vitaly Ostrosablin <vostrosablin@virtuozzo.com>
Reviewed-by: Cyrill Gorcunov <gorcunov@openvz.org>
Pavel Emelianov Jan. 11, 2017, 10:38 a.m.
Applied