[bug report] crypto: ccp - Let a v5 CCP provide the same function as v3

2016-10-12 Thread Dan Carpenter
Hello Gary R Hook, The patch 4b394a232df7: "crypto: ccp - Let a v5 CCP provide the same function as v3" from Jul 26, 2016, leads to the following static checker warning: drivers/crypto/ccp/ccp-dev-v5.c:30 ccp_lsb_alloc() warn: always true condition '(cmd_q->lsb >= 0) => (0-u32max

[PATCH] crypto: CCP - change type of struct member lsb to signed

2016-10-12 Thread Gary R Hook
The lsb field uses a value of -1 to indicate that it is unassigned. Therefore type must be a signed int. Signed-off-by: Gary R Hook --- drivers/crypto/ccp/ccp-dev.h |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/crypto/ccp/ccp-dev.h

Re: [PATCH] padata: add helper function for queue length

2016-10-12 Thread Jason A. Donenfeld
Hi Steffen, On Fri, Oct 7, 2016 at 5:15 AM, Steffen Klassert wrote: > Why you want to have this? I'm working on some bufferbloat/queue code that could benefit from knowing how many items are currently in flight. The goal is to always keep padata busy, but never

[bug report] chcr: Support for Chelsio's Crypto Hardware

2016-10-12 Thread Dan Carpenter
Hello Hariprasad Shenai, This is a semi-automatic email about new static checker warnings. The patch 324429d74127: "chcr: Support for Chelsio's Crypto Hardware" from Aug 17, 2016, leads to the following Smatch complaint: drivers/crypto/chelsio/chcr_algo.c:378 write_sg_data_page_desc()