Am Samstag, 10. Juni 2017, 10:59:40 CEST schrieb Herbert Xu:
Hi Herbert,
>
> Indeed, we should just kill the wait or perhaps convert it to
> a WARN_ON.
As the inflight variable is only used for refcounting and the related wait, I
would propose to remove that variable entirely.
Ciao
Stephan
On Sat, Jun 10, 2017 at 10:33:28AM +0200, Stephan Müller wrote:
>
> Right. Shouldn't we drop the ctx->inflight completely?
>
> The code in the current patch set contains:
>
> when an async operation is queued:
>
> sock_hold(sk);
> ctx->inflight++;
>
> upon
Am Samstag, 10. Juni 2017, 05:13:16 CEST schrieb Herbert Xu:
Hi Herbert,
> On Tue, May 23, 2017 at 04:31:59PM +0200, Stephan Müller wrote:
> > static void skcipher_sock_destruct(struct sock *sk)
> > {
> >
> > struct alg_sock *ask = alg_sk(sk);
> > struct skcipher_ctx *ctx =
On Tue, May 23, 2017 at 04:31:59PM +0200, Stephan Müller wrote:
>
> static void skcipher_sock_destruct(struct sock *sk)
> {
> struct alg_sock *ask = alg_sk(sk);
> struct skcipher_ctx *ctx = ask->private;
> - struct crypto_skcipher *tfm = crypto_skcipher_reqtfm(>req);
> +