]> www.infradead.org Git - users/dedekind/aiaiai.git/commit
aiaiai-test-patchset: remove leftover 'wait for bisect test' code
authorBrian Norris <computersforpeace@gmail.com>
Wed, 5 Aug 2015 15:52:25 +0000 (08:52 -0700)
committerJacob Keller <jacob.e.keller@intel.com>
Tue, 11 Aug 2015 22:12:29 +0000 (15:12 -0700)
commit81bcdb7bdaec2b6b43d15eaf0509445c96aef17c
treee5feb037db7d5d79b3469526e932f32ed49f563a
parentb7f7ede9aadd4b77397bf012f957ec0002a22446
aiaiai-test-patchset: remove leftover 'wait for bisect test' code

I didn't notice this code in my first (quick) pass at fixing a race
condition here. Apparently we *were* trying to wait for the bisection
test to complete, but we just weren't doing it at the right time.

Anyway, with the current code, multi-config test runs will fail, since
the second config will have a valid $pid_bisect from the previous
config, meaning we will try to 'wait' again for the same expired
process. The shell doesn't like that, so we fail.

Let's move the bisect process's PID into a function-local context, and
ensure that each config is completely finished before moving to the next
one. So there will be a single, obvious place to wait for the process
completion.

Signed-off-by: Brian Norris <computersforpeace@gmail.com>
Signed-off-by: Jacob Keller <jacob.e.keller@intel.com>
aiaiai-test-patchset