crypto: caam - only query h/w in job ring dequeue path
authorKim Phillips <kim.phillips@freescale.com>
Sat, 23 Jun 2012 00:48:55 +0000 (19:48 -0500)
committerHerbert Xu <herbert@gondor.apana.org.au>
Wed, 27 Jun 2012 06:42:07 +0000 (14:42 +0800)
Code was needlessly checking the s/w job ring when there
would be nothing to process if the h/w's output completion
ring were empty anyway.

Signed-off-by: Kim Phillips <kim.phillips@freescale.com>
Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au>
drivers/crypto/caam/jr.c

index 11d93f2437057184da0f935e3d3b9aa4d623c694..7ae5e51a0597812060774aaeaa34bb6617098a6f 100644 (file)
@@ -59,15 +59,15 @@ static void caam_jr_dequeue(unsigned long devarg)
        u32 *userdesc, userstatus;
        void *userarg;
 
-       spin_lock_bh(&jrp->outlock);
+       while (rd_reg32(&jrp->rregs->outring_used)) {
 
-       head = ACCESS_ONCE(jrp->head);
-       sw_idx = tail = jrp->tail;
+               head = ACCESS_ONCE(jrp->head);
 
-       while (CIRC_CNT(head, tail, JOBR_DEPTH) >= 1 &&
-              rd_reg32(&jrp->rregs->outring_used)) {
+               spin_lock_bh(&jrp->outlock);
 
+               sw_idx = tail = jrp->tail;
                hw_idx = jrp->out_ring_read_index;
+
                for (i = 0; CIRC_CNT(head, tail + i, JOBR_DEPTH) >= 1; i++) {
                        sw_idx = (tail + i) & (JOBR_DEPTH - 1);
 
@@ -121,15 +121,8 @@ static void caam_jr_dequeue(unsigned long devarg)
 
                /* Finally, execute user's callback */
                usercall(dev, userdesc, userstatus, userarg);
-
-               spin_lock_bh(&jrp->outlock);
-
-               head = ACCESS_ONCE(jrp->head);
-               sw_idx = tail = jrp->tail;
        }
 
-       spin_unlock_bh(&jrp->outlock);
-
        /* reenable / unmask IRQs */
        clrbits32(&jrp->rregs->rconfig_lo, JRCFG_IMSK);
 }