<html> <head> <base href="https://bugs.freedesktop.org/" /> </head> <body> <div> <a class="bz_bug_link bz_status_NEW " title="NEW --- - [Bisected]igt/pc8 fails" href="https://bugs.freedesktop.org/show_bug.cgi?id=71631#c4">Comment # 4</a> on <a class="bz_bug_link bz_status_NEW " title="NEW --- - [Bisected]igt/pc8 fails" href="https://bugs.freedesktop.org/show_bug.cgi?id=71631">bug 71631</a> from <a class="email" href="mailto:huax.lu@intel.com" title="lu hua <huax.lu@intel.com>"> lu hua</a> <pre>(In reply to <a href="show_bug.cgi?id=71631#c3">comment #3</a>) > diff --git a/tests/pm_pc8.c b/tests/pm_pc8.c > index efeae21..15127d4 100644 > --- a/tests/pm_pc8.c > +++ b/tests/pm_pc8.c > @@ -602,15 +602,14 @@ static void setup_runtime_pm(void) > * suite goes faster and we have a higher probability of triggering > race > * conditions. */ > fd = open(POWER_DIR "/autosuspend_delay_ms", O_WRONLY); > - igt_assert_f(fd >= 0, > - "Can't open " POWER_DIR "/autosuspend_delay_ms\n"); > + if (fd != -1) { > + /* If we fail to write to the file, it means this system > doesn't support > + * runtime PM. */ > + size = write(fd, "0\n", 2); > + has_runtime_pm = (size == 2); > > - /* If we fail to write to the file, it means this system doesn't > support > - * runtime PM. */ > - size = write(fd, "0\n", 2); > - has_runtime_pm = (size == 2); > - > - close(fd); > + close(fd); > + } > > if (!has_runtime_pm) > return; > > should make it correctly skip the test Fixed by this patch.</pre> </div> <hr> You are receiving this mail because: <ul> <li>You are the QA Contact for the bug.</li> <li>You are on the CC list for the bug.</li> </ul> </body> </html>