[dpdk-dev] [PATCH v6 1/2] mbuf: support attaching external buffer to mbuf
Olivier Matz
olivier.matz at 6wind.com
Thu Apr 26 22:07:37 CEST 2018
On Thu, Apr 26, 2018 at 09:58:00PM +0200, Thomas Monjalon wrote:
> 26/04/2018 21:42, Olivier Matz:
> > On Thu, Apr 26, 2018 at 06:10:36PM +0200, Thomas Monjalon wrote:
> > > 26/04/2018 18:05, Andrew Rybchenko:
> > > > On 04/26/2018 04:10 AM, Yongseok Koh wrote:
> > > > > -#define RTE_MBUF_INDIRECT(mb) ((mb)->ol_flags & IND_ATTACHED_MBUF)
> > > > > +#define RTE_MBUF_INDIRECT(mb) RTE_MBUF_CLONED(mb)
> > > >
> > > > We have discussed that it would be good to deprecate RTE_MBUF_INDIRECT()
> > > > since it is not !RTE_MBUF_DIREC(). Is it lost here or intentional (may
> > > > be I've lost
> > > > in the thread)?
> > >
> > > I think it should be a separate deprecation notice.
> >
> > Agree with Andrew that RTE_MBUF_INDIRECT should be deprecated
> > to avoid confusion with !DIRECT.
>
> What do you mean?
> We should add a comment? Or poisoining the macro? Or something else?
> Should it be removed? In which release?
Sorry if I was not clear.
Not necessarly remove the macro for this release. But I think we
should announce it and remove it, following the process.
I suggest:
- for 18.05: send the deprecation notice + add a comment in the .h
saying that the macro will be deprecated in 18.08 (or 18.11, there
is no hurry if there is the comment)
- for 18.08 (or 18.11): remove the macro (I don't think poisoining
is useful in this case).
More information about the dev
mailing list