On Tue, Jul 25, 2017 at 07:09:58PM -0700, Megha Dey wrote: > > +/* notify the caller of progress ; request still stays in queue */ > + > +static void notify_callback(struct mcryptd_skcipher_request_ctx *rctx, > + struct mcryptd_alg_cstate *cstate, > + int err) > +{ > + struct skcipher_request *req = cast_mcryptd_ctx_to_req(rctx); > + > + local_bh_disable(); > + rctx->complete(&req->base, err); > + local_bh_enable(); > +} Please explain why you have this crazy construct that does async operations behind the crypto API's back while pretending to be sync by always returning zero? Why is this even needed now that you have switched the underlying implementation to be async? > + /* from mcryptd, we need to callback */ > + if (irqs_disabled()) > + rctx->complete(&req->base, err); > + else { > + local_bh_disable(); > + rctx->complete(&req->base, err); > + local_bh_enable(); > + } I complained about this the first time around and yet this crap is still there. Cheers, -- Email: Herbert Xu <herbert@xxxxxxxxxxxxxxxxxxx> Home Page: http://gondor.apana.org.au/~herbert/ PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt