public inbox for igt-dev@lists.freedesktop.org
 help / color / mirror / Atom feed
* [igt-dev] [PATCH i-g-t v2 0/1] tests/prime_vgem: Fix broken handling of interprocess pipes
@ 2019-07-11  8:39 Janusz Krzysztofik
  2019-07-11  8:39 ` [igt-dev] [PATCH i-g-t v2 1/1] " Janusz Krzysztofik
                   ` (3 more replies)
  0 siblings, 4 replies; 7+ messages in thread
From: Janusz Krzysztofik @ 2019-07-11  8:39 UTC (permalink / raw)
  To: Petri Latvala, Arkadiusz Hiler; +Cc: igt-dev

If a child process dies for any reason while basic-fence-read or
basic-fence-mmap subtest is run, the subtest may hang indefinitely on
read() from a pipe which is supposed to be written to by that child (can
be interrupted though).  Fix it by first closing unused pipe ends on both
parent and child side before using the pipe, as recommended by pipe(2)
manual page.

Please note that due to the IGT library way of handling signals, now the
subtests may fail with error code 141 (128 + SIGPIPE) and no usual DEBUG
message block on stderr nor FAIL message line on stdout, however some
meaningful error messages will still be printed on stderr by the failing
child processes.

v2: resend with Reviewed-by: Chris Wilson <chris@chris-wilson.co.uk>
    added

Janusz Krzysztofik (1):
  tests/prime_vgem: Fix broken handling of interprocess pipes

 tests/prime_vgem.c | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

-- 
2.21.0

_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev

^ permalink raw reply	[flat|nested] 7+ messages in thread

end of thread, other threads:[~2019-07-12  5:47 UTC | newest]

Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2019-07-11  8:39 [igt-dev] [PATCH i-g-t v2 0/1] tests/prime_vgem: Fix broken handling of interprocess pipes Janusz Krzysztofik
2019-07-11  8:39 ` [igt-dev] [PATCH i-g-t v2 1/1] " Janusz Krzysztofik
2019-07-11 10:39 ` [igt-dev] [PATCH i-g-t v2 0/1] " Ser, Simon
2019-07-11 10:49   ` Janusz Krzysztofik
2019-07-11 11:17     ` Ser, Simon
2019-07-11 14:17 ` [igt-dev] ✓ Fi.CI.BAT: success for tests/prime_vgem: Fix broken handling of interprocess pipes (rev2) Patchwork
2019-07-12  5:47 ` [igt-dev] ✓ Fi.CI.IGT: " Patchwork

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox