Re: [PATCH] crypto : sha1-mb : Add missing args_digest offset

[Tim Chen <tim.c.chen@linux.intel.com>]

On Mon, 2016-01-25 at 16:46 -0800, Megha Dey wrote:
> From: Megha Dey <megha.dey@linux.intel.com>
> 
> The _args_digest is defined as _args+_digest, both of which are the first
> members of 2 separate structures, effectively yielding _args_digest to have
> a value of zero. Thus, no errors have spawned yet due to this. To ensure
> sanity, adding the missing _args_digest offset to the sha1_mb_mgr_submit.S.
> 

Thanks for cleaning this up.

Acked-by: Tim Chen <tim.c.chen@linux.intel.com>

> Signed-off-by: Megha Dey <megha.dey@linux.intel.com>
> ---
>  arch/x86/crypto/sha-mb/sha1_mb_mgr_submit_avx2.S | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/arch/x86/crypto/sha-mb/sha1_mb_mgr_submit_avx2.S b/arch/x86/crypto/sha-mb/sha1_mb_mgr_submit_avx2.S
> index 2ab9560..c420d89 100644
> --- a/arch/x86/crypto/sha-mb/sha1_mb_mgr_submit_avx2.S
> +++ b/arch/x86/crypto/sha-mb/sha1_mb_mgr_submit_avx2.S
> @@ -197,7 +197,7 @@ len_is_0:
>  	vpinsrd  $1, _args_digest+1*32(state , idx, 4), %xmm0, %xmm0
>  	vpinsrd  $2, _args_digest+2*32(state , idx, 4), %xmm0, %xmm0
>  	vpinsrd  $3, _args_digest+3*32(state , idx, 4), %xmm0, %xmm0
> -	movl    4*32(state, idx, 4), DWORD_tmp
> +	movl     _args_digest+4*32(state, idx, 4), DWORD_tmp
>  
>  	vmovdqu  %xmm0, _result_digest(job_rax)
>  	movl    DWORD_tmp, _result_digest+1*16(job_rax)