[dpdk-dev] [PATCH v3 1/6] ethdev: fix port id storage

[Thomas Monjalon]

18/01/2018 11:15, Ferruh Yigit:
> On 1/17/2018 10:19 PM, Thomas Monjalon wrote:
> > 17/01/2018 23:09, Thomas Monjalon:
> >> 17/01/2018 22:57, Ferruh Yigit:
> >>> port_id is now 16bits, update function parameter according.
> >>>
> >>> Fixes: 4c270218aa26 ("ethdev: support security APIs")
> >>> Cc: stable at dpdk.org
> >>> Cc: declan.doherty at intel.com
> >>>
> >>> Signed-off-by: Ferruh Yigit <ferruh.yigit at intel.com>
> >>> Acked-by: Hemant Agrawal <hemant.agrawal at nxp.com>
> >>
> >> Obviously,
> >> Acked-by: Thomas Monjalon <thomas at monjalon.net>
> > 
> > Not so obvious actually.
> > It is a good fix, but an API change.
> 
> It is more like API fix.
> While whole library is using 16bits for port_id, I think we shouldn't deliver
> release with this specific API uses 8bits.
> 
> I am for getting this because what it does is wrong.
> 
> > This function was not declared experimental.
> > It must wait 18.05.

I really want to keep API stable in 18.02.
We have to keep this mistake for one more release.