On Tue, Apr 07 2015 at 1:10pm -0400, Ben Collins <ben.c@xxxxxxxxxxxx> wrote: > > > On Apr 7, 2015, at 11:28 AM, Mike Snitzer <snitzer@xxxxxxxxxx> wrote: > > > > On Tue, Apr 07 2015 at 11:55P -0400, > > Mike Snitzer <snitzer@xxxxxxxxxx> wrote: > > > >> It looks like you're _always_ using the completion regardless of whether > >> crypt_convert() will be waiting (e.g. even if error is 0). > >> > >> I can see this "working" but it seems less than ideal. Would it be > >> better to record the need to use the completion in ctx and then > >> conditionally call complete()? > > > > Actually, how about using !completion_done() before calling complete()? > > If you think this would be OK, any chance you could re-test with this? > > I'll be able to test it before Friday (out of town). Thanks Hi Ben, I'm still waiting for test feedback from you on v2. Fairly certain you'll have the same results but I'd like to be certain before pushing this upstream. Thanks, Mike -- dm-devel mailing list dm-devel@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/dm-devel
