Hi,
Please refer coding guidelines at
https://doc.dpdk.org/guides/contributing/patches.html
Here is the example of a fix commit. In this "abcdefgh1234" is the commit id
which introduced this issue.
You can check the commit id of the changed commit using git blame <file name>.
doc: fix some parameter description
Update the docs, fixing description of some parameter.
Fixes: abcdefgh1234 ("doc: add some parameter")
Cc: [email protected]
Signed-off-by: Alex Smith <[email protected]>
>
> Hi Akhil,
>
> Not sure about how to use fix line if I don't have bug number.
>
> If I run ./devtools/check-git-log.sh it expect bug number.
> Do you recommend to log a bug for my patches ??
>
> Please let me know.
>
> Regards,
> S.Amarnath
>
> -----Original Message-----
> From: Akhil Goyal <[email protected]>
> Sent: Tuesday, October 15, 2019 4:31 PM
> To: Somalapuram, Amaranath <[email protected]>;
> [email protected]
> Cc: [email protected]
> Subject: RE: [PATCH v1 3/6] crypto/ccp: crash when using ccp cpu
> authentication
> is used
>
> [CAUTION: External Email]
>
> Title: crypto/ccp: fix CPU authentication crash
> >
> > From: Amaranath Somalapuram <[email protected]>
> >
> > when ccp_auth_opt=1 is set and
> > if authentication error occurred ccp driver crash.
> > proble is enqueue count and dequeue count miss match if we continue.
> Please rephrase this description.
>
> Fixes line??
>
> > Cc: [email protected]
> >
> > Signed-off-by: Amaranath Somalapuram <[email protected]>
> > ---
> > drivers/crypto/ccp/ccp_crypto.c | 2 +-
> > 1 file changed, 1 insertion(+), 1 deletion(-)
> >
> > diff --git a/drivers/crypto/ccp/ccp_crypto.c
> > b/drivers/crypto/ccp/ccp_crypto.c index 19ae9153d..1837c8543 100644
> > --- a/drivers/crypto/ccp/ccp_crypto.c
> > +++ b/drivers/crypto/ccp/ccp_crypto.c
> > @@ -2738,7 +2738,7 @@ process_ops_to_enqueue(struct ccp_qp *qp,
> > session, auth_ctx);
> > if (op[i]->status !=
> > RTE_CRYPTO_OP_STATUS_SUCCESS)
> > - continue;
> > +
> > CCP_LOG_ERR("RTE_CRYPTO_OP_STATUS_AUTH_FAILED");
> > } else
> > result = ccp_crypto_auth(op[i], cmd_q,
> > b_info);
> >
> > --
> > 2.17.1
