[dpdk-dev] [PATCH v2] lpm: remove redundant check when adding lpm rule

Bruce Richardson bruce.richardson at intel.com
Wed Aug 3 11:16:05 CEST 2016


On Tue, Aug 02, 2016 at 11:36:41PM +0200, Thomas Monjalon wrote:
> 2016-08-02 17:04, Bruce Richardson:
> > Having to make this change twice shows up the fact that we are still carrying
> > around some version changes for older releases. Given that we are now past the
> > 16.07 release, the old code can probably be removed. Any volunteers to maybe
> > do up a patch for that.
> 
> The first step would be to announce an ABI breakage.
> Do you plan to do other breaking changes? We may try to group them.
> 
Why does an ABI breakage need to be announced? The code has been in place for
some time, and was called out as an API change in the release notes for 16.04.
Any app compiled against either 16.04 or 16.07 release will work fine once the
code is removed. Any app compiled against an earlier version:
a) is not guaranteed to work, because we only guarantee 1-version
compatibility right now
b) in practice almost certainly won't work with 16.11 anyway, due to 
ABI changes in other areas.

Therefore, I would view an ABI announcement as rather pointless.

/Bruce


More information about the dev mailing list