All of lore.kernel.org
 help / color / mirror / Atom feed
* [LTP] [PATCH] Wait for complete input provided by `seq`
@ 2021-03-03 13:24 Martin Loviska
  2021-03-03 13:30 ` Petr Vorel
  0 siblings, 1 reply; 2+ messages in thread
From: Martin Loviska @ 2021-03-03 13:24 UTC (permalink / raw)
  To: ltp

From: mloviska <mloviska@suse.com>

`splice02` may exit before `seq` therefore broken pipe occurs.

Found with SLES JeOS (https://progress.opensuse.org/issues/77260).

Suggested-by: Fabian Vogt <fvogt@suse.de>
Tested-by: Petr Vorel <pvorel@suse.cz>
Reviewed-by: Petr Vorel <pvorel@suse.cz>
Signed-off-by: Martin Loviska <mloviska@suse.com>
---
 runtest/smoketest | 2 +-
 runtest/syscalls  | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/runtest/smoketest b/runtest/smoketest
index 0c24fc1fa..e6800d283 100644
--- a/runtest/smoketest
+++ b/runtest/smoketest
@@ -11,5 +11,5 @@ symlink01 symlink01
 stat04 symlink01 -T stat04
 utime01A symlink01 -T utime01
 rename01A symlink01 -T rename01
-splice02 seq 1 20 | splice02
+splice02 (seq 1 20 || :) | splice02
 route4-change-dst route-change-dst.sh
diff --git a/runtest/syscalls b/runtest/syscalls
index fe22ae5b6..13b87e6bc 100644
--- a/runtest/syscalls
+++ b/runtest/syscalls
@@ -1446,7 +1446,7 @@ socketpair02 socketpair02
 sockioctl01 sockioctl01
 
 splice01 splice01
-splice02 seq 1 20000 | splice02
+splice02 (seq 1 20000 || :) | splice02
 splice03 splice03
 splice04 splice04
 splice05 splice05
-- 
2.30.1


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

* [LTP] [PATCH] Wait for complete input provided by `seq`
  2021-03-03 13:24 [LTP] [PATCH] Wait for complete input provided by `seq` Martin Loviska
@ 2021-03-03 13:30 ` Petr Vorel
  0 siblings, 0 replies; 2+ messages in thread
From: Petr Vorel @ 2021-03-03 13:30 UTC (permalink / raw)
  To: ltp

Hi,

I'd accept this quick fix although Cyril suggested in original PR [1] that
proper solution would be to generate it's input without relying on shell piping
data into it.

Kind regards,
Petr

[1] https://github.com/linux-test-project/ltp/pull/796#issuecomment-789710935

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

end of thread, other threads:[~2021-03-03 13:30 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-03-03 13:24 [LTP] [PATCH] Wait for complete input provided by `seq` Martin Loviska
2021-03-03 13:30 ` Petr Vorel

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.