tests/pm_pc8: remove sleep() call when setting up the environment
authorPaulo Zanoni <paulo.r.zanoni@intel.com>
Mon, 17 Feb 2014 19:29:36 +0000 (16:29 -0300)
committerPaulo Zanoni <paulo.r.zanoni@intel.com>
Fri, 21 Feb 2014 13:07:30 +0000 (10:07 -0300)
This sleep was added because sometimes we didn't reach PC8+
residencies, but it was still not enough to prevent the problem every
time, and it is really not needed most of the times. I have
investigated more and it seems that we only have to wait until after
some minutes have past since the machine booted. So just remove the
sleep for now since when you run each subtest in a separate process,
you end up having to sleep at every subtest.

Signed-off-by: Paulo Zanoni <paulo.r.zanoni@intel.com>
tests/pm_pc8.c

index f34a113..982b82e 100644 (file)
@@ -741,11 +741,6 @@ static void setup_non_graphics_runtime_pm(void)
                write(fd, "auto\n", 5);
                close(fd);
        }
-
-       /* For some yet unknown reason, it takes some time for the machine to
-        * reach PC8+ residencies after we do this. I don't really know how much
-        * we should wait, but this value seems to be working for me. */
-       sleep(10);
 }
 
 static void setup_environment(void)