[PATCH v2 2/2] staging: greybus: loopback_test: Fix race preventing test completion

[Bryan O'Donoghue]

On 03/01/17 10:17, Axel Haslam wrote:
> as i see it,  a successful test means:
> 1- each interfaces should send an event upon completion.
yes

> 2- the iteration count should equal iteration_max on each of the interfaces

yes

> what  am i missing?

count == max implies the kernel thread is still working and we should
continue to sleep.

I am interested in why count != max when in user-space number_of_events
== t->poll_count - I don't see how that's logically possible, though it
makes me wonder why the check is_complete() was put in place - you must
have seen this yourself ?

As a fundamental though - unless count == max, the test is not done.