Commit 356690d0 authored by Nagadheeraj Rottela's avatar Nagadheeraj Rottela Committed by Herbert Xu
Browse files

crypto: cavium/nitrox - Invoke callback after DMA unmap



In process_response_list() invoke the callback handler after unmapping
the DMA buffers. It ensures DMA data is synced form device to cpu
before the client code access the data from callback handler.

Fixes: c9613335 ("crypto: cavium/nitrox - Added AEAD cipher support")
Signed-off-by: default avatarNagadheeraj Rottela <rnagadheeraj@marvell.com>
Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
parent d45a90cb
Loading
Loading
Loading
Loading
+6 −4
Original line number Diff line number Diff line
@@ -537,6 +537,8 @@ static void process_response_list(struct nitrox_cmdq *cmdq)
	struct nitrox_device *ndev = cmdq->ndev;
	struct nitrox_softreq *sr;
	int req_completed = 0, err = 0, budget;
	completion_t callback;
	void *cb_arg;

	/* check all pending requests */
	budget = atomic_read(&cmdq->pending_count);
@@ -564,13 +566,13 @@ static void process_response_list(struct nitrox_cmdq *cmdq)
		smp_mb__after_atomic();
		/* remove from response list */
		response_list_del(sr, cmdq);

		/* ORH error code */
		err = READ_ONCE(*sr->resp.orh) & 0xff;

		if (sr->callback)
			sr->callback(sr->cb_arg, err);
		callback = sr->callback;
		cb_arg = sr->cb_arg;
		softreq_destroy(sr);
		if (callback)
			callback(cb_arg, err);

		req_completed++;
	}