[dpdk-dev] [PATCH 07/13] mbuf: use macros only to access the mbuf metadata

2014-09-17 Thread Thomas Monjalon
2014-09-17 10:31, Richardson, Bruce: > > From: Ramia, Kannan Babu > > > > I completely agree with Cristian here, instead of leaving to applications > > where to place their meta data, we can provide a guidance by having this > > field about placement of application meta while maintaining transpare

[dpdk-dev] [PATCH 07/13] mbuf: use macros only to access the mbuf metadata

2014-09-17 Thread Richardson, Bruce
> -Original Message- > From: Ramia, Kannan Babu > Sent: Tuesday, September 16, 2014 11:06 PM > To: Dumitrescu, Cristian; Olivier MATZ; Richardson, Bruce; dev at dpdk.org > Subject: RE: [dpdk-dev] [PATCH 07/13] mbuf: use macros only to access the > mbuf metadata > >

[dpdk-dev] [PATCH 07/13] mbuf: use macros only to access the mbuf metadata

2014-09-16 Thread Ramia, Kannan Babu
...@6wind.com] Sent: Friday, September 12, 2014 10:02 PM To: Dumitrescu, Cristian; Richardson, Bruce; dev at dpdk.org Subject: Re: [dpdk-dev] [PATCH 07/13] mbuf: use macros only to access the mbuf metadata Hello Cristian, > What is the reason to remove this field? Please explain the ration

[dpdk-dev] [PATCH 07/13] mbuf: use macros only to access the mbuf metadata

2014-09-16 Thread Dumitrescu, Cristian
PM To: Dumitrescu, Cristian; Richardson, Bruce; dev at dpdk.org Subject: Re: [dpdk-dev] [PATCH 07/13] mbuf: use macros only to access the mbuf metadata Hello Cristian, > What is the reason to remove this field? Please explain the > rationale of removing this field. The rationale is explai

[dpdk-dev] [PATCH 07/13] mbuf: use macros only to access the mbuf metadata

2014-09-12 Thread Olivier MATZ
Hello Cristian, > What is the reason to remove this field? Please explain the > rationale of removing this field. The rationale is explained in http://dpdk.org/ml/archives/dev/2014-September/005232.html "The format of the metadata is up to the application". The type of data the application stor

[dpdk-dev] [PATCH 07/13] mbuf: use macros only to access the mbuf metadata

2014-09-12 Thread Dumitrescu, Cristian
[mailto:dev-boun...@dpdk.org] On Behalf Of Richardson, Bruce Sent: Tuesday, September 9, 2014 10:01 AM To: Olivier MATZ; dev at dpdk.org Subject: Re: [dpdk-dev] [PATCH 07/13] mbuf: use macros only to access the mbuf metadata > -Original Message- > From: Olivier MATZ [mailto:olivier.m

[dpdk-dev] [PATCH 07/13] mbuf: use macros only to access the mbuf metadata

2014-09-10 Thread Olivier MATZ
Hi Bruce, On 09/10/2014 05:09 PM, Bruce Richardson wrote: >> Just one question: why not removing RTE_MBUF_METADATA*() macros? >> I'd just provide one macro that gives a (void*) to the first byte >> after the mbuf structure. >> >> The format of the metadata is up to the application, that usually >>

[dpdk-dev] [PATCH 07/13] mbuf: use macros only to access the mbuf metadata

2014-09-10 Thread Bruce Richardson
On Mon, Sep 08, 2014 at 02:05:41PM +0200, Olivier MATZ wrote: > Hi Bruce, > > On 09/03/2014 05:49 PM, Bruce Richardson wrote: > > Removed the explicit zero-sized metadata definition at the end of the > > mbuf data structure. Updated the metadata macros to take account of this > > change so that al

[dpdk-dev] [PATCH 07/13] mbuf: use macros only to access the mbuf metadata

2014-09-09 Thread Richardson, Bruce
> -Original Message- > From: Olivier MATZ [mailto:olivier.matz at 6wind.com] > Sent: Monday, September 08, 2014 1:06 PM > To: Richardson, Bruce; dev at dpdk.org > Subject: Re: [dpdk-dev] [PATCH 07/13] mbuf: use macros only to access the > mbuf metadata > > Hi Bruc

[dpdk-dev] [PATCH 07/13] mbuf: use macros only to access the mbuf metadata

2014-09-08 Thread Olivier MATZ
Hi Bruce, On 09/03/2014 05:49 PM, Bruce Richardson wrote: > Removed the explicit zero-sized metadata definition at the end of the > mbuf data structure. Updated the metadata macros to take account of this > change so that all existing code which uses those macros still works. > > Signed-off-by: B

[dpdk-dev] [PATCH 07/13] mbuf: use macros only to access the mbuf metadata

2014-09-03 Thread Bruce Richardson
Removed the explicit zero-sized metadata definition at the end of the mbuf data structure. Updated the metadata macros to take account of this change so that all existing code which uses those macros still works. Signed-off-by: Bruce Richardson --- lib/librte_mbuf/rte_mbuf.h | 22 ---