[dpdk-dev] [PATCH] doc: announce ABI change for librte_port
> -Original Message- > From: Thomas Monjalon [mailto:thomas.monjalon at 6wind.com] > Sent: Thursday, July 16, 2015 1:26 PM > To: Dumitrescu, Cristian > Cc: dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH] doc: announce ABI change for librte_port > > 2015-07-16 13:19, Cristian Dumitrescu: > > +* librte_port (rte_port.h): Macros to access the packet meta-data stored > within > > + the packet buffer will be adjusted to cover the packet mbuf structure as > well, > > + as currently they are able to access any packet buffer location except > > the > > + packet mbuf structure. The consequence is that applications currently > using > > + these macros will have to adjust the value of the offset parameter of > these > > + macros by increasing it with sizeof(struc rte_mbuf). The affected macros > are: > > + RTE_MBUF_METADATA_UINT<8, 16, 32, 64>_PTR and > > + RTE_MBUF_METADATA_UINT<8, 16, 32, 64>. In terms of code changes, > most likely > > + only the definition of RTE_MBUF_METADATA_UINT8_PTR macro will be > changed from > > + ``(&((uint8_t *) &(mbuf)[1])[offset])`` to > > + ``(&((uint8_t *) (mbuf))[offset])``. > > Cristian, > General comment: you are too verbose :) > Specifically on this patch: same comment ;) No problem, will simplify and resend. Thanks, Thomas.
[dpdk-dev] [PATCH] doc: announce ABI change for librte_port
2015-07-16 13:19, Cristian Dumitrescu: > +* librte_port (rte_port.h): Macros to access the packet meta-data stored > within > + the packet buffer will be adjusted to cover the packet mbuf structure as > well, > + as currently they are able to access any packet buffer location except the > + packet mbuf structure. The consequence is that applications currently using > + these macros will have to adjust the value of the offset parameter of these > + macros by increasing it with sizeof(struc rte_mbuf). The affected macros > are: > + RTE_MBUF_METADATA_UINT<8, 16, 32, 64>_PTR and > + RTE_MBUF_METADATA_UINT<8, 16, 32, 64>. In terms of code changes, most > likely > + only the definition of RTE_MBUF_METADATA_UINT8_PTR macro will be changed > from > + ``(&((uint8_t *) &(mbuf)[1])[offset])`` to > + ``(&((uint8_t *) (mbuf))[offset])``. Cristian, General comment: you are too verbose :) Specifically on this patch: same comment ;)
[dpdk-dev] [PATCH] doc: announce ABI change for librte_port
Signed-off-by: Cristian Dumitrescu --- doc/guides/rel_notes/abi.rst | 12 1 files changed, 12 insertions(+), 0 deletions(-) diff --git a/doc/guides/rel_notes/abi.rst b/doc/guides/rel_notes/abi.rst index 9e98d62..271e08e 100644 --- a/doc/guides/rel_notes/abi.rst +++ b/doc/guides/rel_notes/abi.rst @@ -34,3 +34,15 @@ Deprecation Notices creates a dummy/empty malloc library to fulfill binaries with dynamic linking dependencies on librte_malloc.so. Such dummy library will not be created from release 2.2 so binaries will need to be rebuilt. + +* librte_port (rte_port.h): Macros to access the packet meta-data stored within + the packet buffer will be adjusted to cover the packet mbuf structure as well, + as currently they are able to access any packet buffer location except the + packet mbuf structure. The consequence is that applications currently using + these macros will have to adjust the value of the offset parameter of these + macros by increasing it with sizeof(struc rte_mbuf). The affected macros are: + RTE_MBUF_METADATA_UINT<8, 16, 32, 64>_PTR and + RTE_MBUF_METADATA_UINT<8, 16, 32, 64>. In terms of code changes, most likely + only the definition of RTE_MBUF_METADATA_UINT8_PTR macro will be changed from + ``(&((uint8_t *) &(mbuf)[1])[offset])`` to + ``(&((uint8_t *) (mbuf))[offset])``. -- 1.7.4.1
[dpdk-dev] [PATCH] doc: announce ABI change for librte_port
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Cristian Dumitrescu > Sent: Thursday, July 16, 2015 1:20 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH] doc: announce ABI change for librte_port > > > Signed-off-by: Cristian Dumitrescu > --- Acked-by: Jasvinder Singh
[dpdk-dev] [PATCH] doc: announce ABI change for librte_port
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Cristian Dumitrescu > Sent: Thursday, July 16, 2015 2:20 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH] doc: announce ABI change for librte_port > > > Signed-off-by: Cristian Dumitrescu > --- > doc/guides/rel_notes/abi.rst | 12 > 1 files changed, 12 insertions(+), 0 deletions(-) > > diff --git a/doc/guides/rel_notes/abi.rst b/doc/guides/rel_notes/abi.rst > index 9e98d62..271e08e 100644 > --- a/doc/guides/rel_notes/abi.rst > +++ b/doc/guides/rel_notes/abi.rst > @@ -34,3 +34,15 @@ Deprecation Notices >creates a dummy/empty malloc library to fulfill binaries with dynamic > linking >dependencies on librte_malloc.so. Such dummy library will not be created > from >release 2.2 so binaries will need to be rebuilt. > + > +* librte_port (rte_port.h): Macros to access the packet meta-data stored > within > + the packet buffer will be adjusted to cover the packet mbuf structure as > well, > + as currently they are able to access any packet buffer location except the > + packet mbuf structure. The consequence is that applications currently using > + these macros will have to adjust the value of the offset parameter of these > + macros by increasing it with sizeof(struc rte_mbuf). The affected macros > are: > + RTE_MBUF_METADATA_UINT<8, 16, 32, 64>_PTR and > + RTE_MBUF_METADATA_UINT<8, 16, 32, 64>. In terms of code changes, most > likely > + only the definition of RTE_MBUF_METADATA_UINT8_PTR macro will be > changed from > + ``(&((uint8_t *) &(mbuf)[1])[offset])`` to > + ``(&((uint8_t *) (mbuf))[offset])``. > -- > 1.7.4.1 > 1.7.4.1 Acked-by: Daniel Mrzyglod
[dpdk-dev] [PATCH] doc: announce ABI change for librte_port
> -Original Message- > From: dev [mailto:dev-bounces at dpdk.org] On Behalf Of Cristian Dumitrescu > Sent: Thursday, July 16, 2015 2:20 PM > To: dev at dpdk.org > Subject: [dpdk-dev] [PATCH] doc: announce ABI change for librte_port > > > Signed-off-by: Cristian Dumitrescu Acked-by: Maciej Gajdzica