Re: [PATCH] crypto: caam - fix dbg_dump_sg() style issues

[Horia Geanta Neag]

On 9/22/2016 2:42 PM, Horia Geanta Neag wrote:
> On 9/22/2016 1:58 PM, Catalin Vasile wrote:
>> Signed-off-by: Catalin Vasile 
> Reviewed-by: Horia Geantă 
Actually please fix also the warning reported by kbuild test robot in
dbg_dump_sg: s/min(x, y)/min_t(size_t, x, y)

Horia

--
To unsubscribe from this list: send the line "unsubscribe linux-crypto" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Re: [PATCH] crypto: caam - fix dbg_dump_sg() style issues

[Horia Geanta Neag]

On 9/22/2016 1:58 PM, Catalin Vasile wrote:
> Signed-off-by: Catalin Vasile 
Reviewed-by: Horia Geantă 

--
To unsubscribe from this list: send the line "unsubscribe linux-crypto" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[PATCH] crypto: caam - fix dbg_dump_sg() style issues

[Catalin Vasile]

Signed-off-by: Catalin Vasile 
---
 drivers/crypto/caam/caamalg.c | 43 ---
 1 file changed, 20 insertions(+), 23 deletions(-)

diff --git a/drivers/crypto/caam/caamalg.c b/drivers/crypto/caam/caamalg.c
index eb97562..c8ebddb 100644
--- a/drivers/crypto/caam/caamalg.c
+++ b/drivers/crypto/caam/caamalg.c
@@ -117,8 +117,7 @@
 
 static void dbg_dump_sg(const char *level, const char *prefix_str,
int prefix_type, int rowsize, int groupsize,
-   struct scatterlist *sg, size_t tlen, bool ascii,
-   bool may_sleep)
+   struct scatterlist *sg, size_t tlen, bool ascii)
 {
struct scatterlist *it;
void *it_page;
@@ -132,7 +131,7 @@ static void dbg_dump_sg(const char *level, const char 
*prefix_str,
 */
it_page = kmap_atomic(sg_page(it));
if (unlikely(!it_page)) {
-   printk(KERN_ERR "dbg_dump_sg: kmap failed\n");
+   pr_err("dbg_dump_sg: kmap failed\n");
return;
}
 
@@ -145,6 +144,12 @@ static void dbg_dump_sg(const char *level, const char 
*prefix_str,
kunmap_atomic(it_page);
}
 }
+#else
+static void dbg_dump_sg(const char *level, const char *prefix_str,
+   int prefix_type, int rowsize, int groupsize,
+   struct scatterlist *sg, size_t tlen, bool ascii)
+{
+}
 #endif
 
 static struct list_head alg_list;
@@ -2018,10 +2023,11 @@ static void ablkcipher_encrypt_done(struct device 
*jrdev, u32 *desc, u32 err,
print_hex_dump(KERN_ERR, "dstiv  @"__stringify(__LINE__)": ",
   DUMP_PREFIX_ADDRESS, 16, 4, req->info,
   edesc->src_nents > 1 ? 100 : ivsize, 1);
+#endif
+
dbg_dump_sg(KERN_ERR, "dst@"__stringify(__LINE__)": ",
DUMP_PREFIX_ADDRESS, 16, 4, req->dst,
-   edesc->dst_nents > 1 ? 100 : req->nbytes, 1, true);
-#endif
+   edesc->dst_nents > 1 ? 100 : req->nbytes, 1);
 
ablkcipher_unmap(jrdev, edesc, req);
kfree(edesc);
@@ -2050,10 +2056,10 @@ static void ablkcipher_decrypt_done(struct device 
*jrdev, u32 *desc, u32 err,
print_hex_dump(KERN_ERR, "dstiv  @"__stringify(__LINE__)": ",
   DUMP_PREFIX_ADDRESS, 16, 4, req->info,
   ivsize, 1);
+#endif
dbg_dump_sg(KERN_ERR, "dst@"__stringify(__LINE__)": ",
DUMP_PREFIX_ADDRESS, 16, 4, req->dst,
-   edesc->dst_nents > 1 ? 100 : req->nbytes, 1, true);
-#endif
+   edesc->dst_nents > 1 ? 100 : req->nbytes, 1);
 
ablkcipher_unmap(jrdev, edesc, req);
kfree(edesc);
@@ -2207,16 +2213,14 @@ static void init_ablkcipher_job(u32 *sh_desc, 
dma_addr_t ptr,
int len, sec4_sg_index = 0;
 
 #ifdef DEBUG
-   bool may_sleep = ((req->base.flags & (CRYPTO_TFM_REQ_MAY_BACKLOG |
- CRYPTO_TFM_REQ_MAY_SLEEP)) != 0);
print_hex_dump(KERN_ERR, "presciv@"__stringify(__LINE__)": ",
   DUMP_PREFIX_ADDRESS, 16, 4, req->info,
   ivsize, 1);
printk(KERN_ERR "asked=%d, nbytes%d\n", (int)edesc->src_nents ? 100 : 
req->nbytes, req->nbytes);
+#endif
dbg_dump_sg(KERN_ERR, "src@"__stringify(__LINE__)": ",
DUMP_PREFIX_ADDRESS, 16, 4, req->src,
-   edesc->src_nents ? 100 : req->nbytes, 1, may_sleep);
-#endif
+   edesc->src_nents ? 100 : req->nbytes, 1);
 
len = desc_len(sh_desc);
init_job_desc_shared(desc, ptr, len, HDR_SHARE_DEFER | HDR_REVERSE);
@@ -2267,15 +2271,13 @@ static void init_ablkcipher_giv_job(u32 *sh_desc, 
dma_addr_t ptr,
int len, sec4_sg_index = 0;
 
 #ifdef DEBUG
-   bool may_sleep = ((req->base.flags & (CRYPTO_TFM_REQ_MAY_BACKLOG |
- CRYPTO_TFM_REQ_MAY_SLEEP)) != 0);
print_hex_dump(KERN_ERR, "presciv@" __stringify(__LINE__) ": ",
   DUMP_PREFIX_ADDRESS, 16, 4, req->info,
   ivsize, 1);
+#endif
dbg_dump_sg(KERN_ERR, "src@" __stringify(__LINE__) ": ",
DUMP_PREFIX_ADDRESS, 16, 4, req->src,
-   edesc->src_nents ? 100 : req->nbytes, 1, may_sleep);
-#endif
+   edesc->src_nents ? 100 : req->nbytes, 1);
 
len = desc_len(sh_desc);
init_job_desc_shared(desc, ptr, len, HDR_SHARE_DEFER | HDR_REVERSE);
@@ -2544,14 +2546,6 @@ static int aead_decrypt(struct aead_request *req)
u32 *desc;
int ret = 0;
 
-#ifdef DEBUG
-   bool may_sleep = ((req->base.flags & (CRYPTO_TFM_REQ_MAY_BACKLOG |
- CRYPTO_TFM_REQ_MAY_SLEEP)) != 0);
-   dbg_dump_sg(KERN_ERR, "dec src@"__stringify(__LINE__)": ",
-   DUMP_