Thread (104 messages) 104 messages, 19 authors, 2020-03-17

Re: [dpdk-dev] [PATCH v2 4/4] add ABI checks

From: Ananyev, Konstantin <hidden>
Date: 2020-01-30 23:49:24

-----Original Message-----
From: dev <redacted> On Behalf Of Thomas Monjalon
Sent: Thursday, January 30, 2020 4:00 PM
To: Anoob Joseph <redacted>; akhil.goyal@nxp.com; Trahe, Fiona <redacted>
Cc: dev@dpdk.org; David Marchand <redacted>; Richardson, Bruce <redacted>;
nhorman@tuxdriver.com; Mcnamara, John [off-list ref]; Trahe, Fiona [off-list ref]; Kusztal, ArkadiuszX
[off-list ref]; Yigit, Ferruh [off-list ref]
Subject: Re: [dpdk-dev] [PATCH v2 4/4] add ABI checks

30/01/2020 14:06, Trahe, Fiona:
quoted
We were unaware the LIST_END change could constitute an ABI breakage, but can see how it affects the array size when picked up.
We're exploring options.

I agree with Anoob's point that if we don't allow the LIST_END to be modified, then it means no feature can be implemented without ABI
breakage.
quoted
Anyone  object to removing those LIST_END elements - or have a better suggestion? Would have to be in 20.11 I suppose.
Yes, having max value right after the last value is ridiculous,
it prevents adding any value.
In 20.11, we should remove all these *_END and *_MAX from API enums
and replace them with a separate #define with reasonnable maximums.
I think we'd better avoid public structs that have array of _MAX elems in them.

Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help