On 1/9/2018 4:16 PM, Ferruh Yigit wrote:
On 12/13/2017 12:05 PM, Hemant Agrawal wrote:
Fixes: 05ba55bc2b1a ("net/dpaa: add packet dump for debugging")
Fixes: 37f9b54bd3cf ("net/dpaa: support Tx and Rx queue setup")
Cc: sta...@dpdk.org>
Signed-off-by: Hemant Agrawal <hemant.agra...@nxp.com>

Hi Hemant,

fix coverity issues is not very helpful as commit title, can you please document
what really fixed.

And there is a special format for coverity fixes:

"

    Coverity issue: ......
    Fixes: ............ ("...")
    Cc: sta...@dpdk.org [if required]

    Signed-off-by: ....
"

There are samples in git history. It seems this format is not documented and
Marko will help to document it.

Hi Ferruh,
        thanks for the review. Please ignore v2, I will fix it in v3.

Regards,
Hemant

Reply via email to