diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2017-03-08 11:06:05 -0800 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2017-03-08 11:06:05 -0800 |
commit | 8557b8e43a1de99b12b0d61384742e0d5d0adf84 (patch) | |
tree | 133775755b99d1f44713d822879b377b2bc62849 /lib | |
parent | 04bb94b13c02e9dbc92d622cddf88937127bd7ed (diff) | |
parent | f7c6401ff84ab8ffffc281a29aa0a787f7eb346e (diff) | |
download | linux-8557b8e43a1de99b12b0d61384742e0d5d0adf84.tar.gz linux-8557b8e43a1de99b12b0d61384742e0d5d0adf84.tar.xz |
Merge tag 'ktest-v4.11-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-ktest
Pull ktest fixes from Steven Rostedt:
"Greg Kroah-Hartman reported to me that the ktest of v4.11-rc1 locked
up in an infinite loop while doing the make mrproper.
Looking into the cause I noticed that a recent update to the function
run_command (used for running all shell commands, including "make
mrproper") changed the internal loop to use the function
wait_for_input.
The wait_for_input function uses select to look at two file
descriptors. One is the file descriptor of the command it is running,
the other is STDIN. The STDIN check was not checking the return status
of the sysread call, and was also just writing a lot of data into
syswrite without regard to the size of the data read.
Changing the code to check the return status of sysread, and also to
still process the passed in descriptor data without looping back to
the select fixed Greg's problem.
While looking at this code I also realized that the loop did not honor
the timeout if STDIN always had input (or for some reason return
error). this could prevent wait_for_input to timeout on the file
descriptor it is suppose to be waiting for. That is fixed too"
* tag 'ktest-v4.11-rc1' of git://git.kernel.org/pub/scm/linux/kernel/git/rostedt/linux-ktest:
ktest: Make sure wait_for_input does honor the timeout
ktest: Fix while loop in wait_for_input
Diffstat (limited to 'lib')
0 files changed, 0 insertions, 0 deletions