Re: [Qemu-devel] [PATCH 1/2] ide-test: fix timeouts

2015-11-24 Thread John Snow
On 11/24/2015 10:40 AM, Kevin Wolf wrote: > Am 20.11.2015 um 23:53 hat John Snow geschrieben: >> Use explicit timeouts instead of trying to fudge >> it by counting nsleep calls. >> >> Signed-off-by: John Snow > > Isn't wrapping lines at 50 characters a bit extreme? > Yes. Sometimes, I guess c

Re: [Qemu-devel] [PATCH 1/2] ide-test: fix timeouts

2015-11-24 Thread Kevin Wolf
Am 20.11.2015 um 23:53 hat John Snow geschrieben: > Use explicit timeouts instead of trying to fudge > it by counting nsleep calls. > > Signed-off-by: John Snow Isn't wrapping lines at 50 characters a bit extreme? I'm wondering why this is a fix. If anything, I would expect the new version to b

[Qemu-devel] [PATCH 1/2] ide-test: fix timeouts

2015-11-20 Thread John Snow
Use explicit timeouts instead of trying to fudge it by counting nsleep calls. Signed-off-by: John Snow --- tests/ide-test.c | 18 ++ 1 file changed, 14 insertions(+), 4 deletions(-) diff --git a/tests/ide-test.c b/tests/ide-test.c index fc1ce52..d37dc5e 100644 --- a/tests/ide-te