Thread (98 messages) 98 messages, 12 authors, 2016-03-11

Re: [PATCH 1/4] vhost: handle VHOST_USER_SET_LOG_BASE request

From: Thomas Monjalon <hidden>
Date: 2015-12-07 02:04:35

2015-12-07 10:00, Yuanhan Liu:
On Mon, Dec 07, 2015 at 12:07:28AM +0100, Thomas Monjalon wrote:
quoted
2015-12-02 15:53, Panu Matilainen:
quoted
This (and other changes in patch 2 breaks the librte_vhost ABI again, so 
you'd need to at least add a deprecation note to 2.2 to be able to do it 
in 2.3 at all according to the ABI policy.

Perhaps a better option would be adding some padding to the structs now 
for 2.2 since the vhost ABI is broken there anyway. That would at least 
give a chance to keep it compatible from 2.2 to 2.3.
Please could you point where the vhost ABI is broken in 2.2?
Thomas, here are the changes to rte_virtio_net.h:


$ git diff 381316f6a225139d22d39b5ab8d50c40607924ca..19d4d7ef2a216b5418d8edb5b004d1a58bba3cc1 \
      -- lib/librte_vhost/rte_virtio_net.h >
[...]

The problem is that the changes are not noticed in the release notes
and the LIBABIVER is still 1.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help