crypto: caam - Remove unnecessary smp_read_barrier_depends()
authorPranith Kumar <bobby.prani@gmail.com>
Thu, 13 Nov 2014 19:24:07 +0000 (14:24 -0500)
committerHerbert Xu <herbert@gondor.apana.org.au>
Mon, 17 Nov 2014 14:53:47 +0000 (22:53 +0800)
Recently lockless_dereference() was added which can be used in place of
hard-coding smp_read_barrier_depends(). The following PATCH makes the change.

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

index 4d18e27ffa9e72da4a7763e87d783dec89ba4b66..9207c907a128bd5ff938a2b194ffb42c385aeb9b 100644 (file)
@@ -181,8 +181,6 @@ static void caam_jr_dequeue(unsigned long devarg)
                for (i = 0; CIRC_CNT(head, tail + i, JOBR_DEPTH) >= 1; i++) {
                        sw_idx = (tail + i) & (JOBR_DEPTH - 1);
 
-                       smp_read_barrier_depends();
-
                        if (jrp->outring[hw_idx].desc ==
                            jrp->entinfo[sw_idx].desc_addr_dma)
                                break; /* found */
@@ -218,7 +216,6 @@ static void caam_jr_dequeue(unsigned long devarg)
                if (sw_idx == tail) {
                        do {
                                tail = (tail + 1) & (JOBR_DEPTH - 1);
-                               smp_read_barrier_depends();
                        } while (CIRC_CNT(head, tail, JOBR_DEPTH) >= 1 &&
                                 jrp->entinfo[tail].desc_addr_dma == 0);