Hi! > + if (!type) > + ret = tst_checkpoint_wait(id, timeout); > + else > + ret = tst_checkpoint_wake(id, nr_wake, timeout); I find it silly that you do negate the flag here. You could have set it the other way, or swap the wait and wake in the if and else branch... Anyway the rest looks good to me, ackend. -- Cyril Hrubis chrubis@suse.cz