On 10/20/2017 12:50 PM, De Lara Guarch, Pablo wrote:

quoted

-----Original Message-----
From: aviadye@dev.mellanox.co.il [mailto:aviadye@dev.mellanox.co.il]
Sent: Thursday, October 19, 2017 7:53 PM
To: dev@dpdk.org; Gonzalez Monroy, Sergio
[off-list ref]; De Lara Guarch, Pablo
[off-list ref]; aviadye@mellanox.com
Cc: borisp@mellanox.com; akhil.goyal@nxp.com;
hemant.agrawal@nxp.com; Nicolau, Radu [off-list ref];
Doherty, Declan [off-list ref]; aviadye@dev.mellanox.co.il;
liranl@mellanox.com; nelio.laranjeiro@6wind.com; thomas@monjalon.net
Subject: [dpdk-dev][PATCH v2 1/6] examples/ipsec-secgw: fix initialization
of aead crypto devices

From: Aviad Yehezkel <redacted>

Fixes: 501e9c226adf ("examples/ipsec-secgw: add AEAD parameters")
Cc: pablo.de.lara.guarch@intel.com

You should CC stable here.

You referred me to http://dpdk.org/doc/guides/contributing/patches.html
I followed the documentation:
"
When fixing a regression, it is required to reference the id of the 
commit which introduced the bug, and put the original author of that 
commit on CC. Yiu can generate the required lines using the following 
git alias, which print the commit SHA and the author of the original code:
git config alias.fixline "log -1 --abbrev=12 --format='Fixes: %h 
(\"%s\")%nCc: %ae'"
"
Please explain when to use above alias and when to put stable or 
something else.

Thanks!

quoted

Issue: None

What does this mean? This is a fix to an existing issue,
so this doesn't make sense.

This is leftovers from my internal git continuous integration system. It 
refers to internal bug in my local bugzilla.
Will remove that and automate next patches not to include it.

Thanks!

Apart from this:

Acked-by: Pablo de Lara <redacted>