mbox series

[0/2] remove mbuf deprecated filed

Message ID 1599809752-25256-1-git-send-email-phil.yang@arm.com (mailing list archive)
Headers
Series remove mbuf deprecated filed |

Message

Phil Yang Sept. 11, 2020, 7:35 a.m. UTC 
  Remove the deprecated[1] refcnt_atomic member in structures rte_mbuf and
rte_mbuf_ext_shared_info.

[1] http://code.dpdk.org/dpdk/latest/source/doc/guides/rel_notes/deprecation.rst#L161

Phil Yang (2):
  mbuf: remove generic mbuf atomically accessed refcnt
  mbuf: remove external buffer atomically accessed refcnt

 doc/guides/rel_notes/release_20_11.rst |  2 ++
 lib/librte_mbuf/rte_mbuf_core.h        | 13 ++-----------
 2 files changed, 4 insertions(+), 11 deletions(-)

Comments

Andrew Rybchenko Sept. 11, 2020, 7:39 a.m. UTC | #1 
On 9/11/20 10:35 AM, Phil Yang wrote:
> Remove the deprecated[1] refcnt_atomic member in structures rte_mbuf and
> rte_mbuf_ext_shared_info.
> 
> [1] http://code.dpdk.org/dpdk/latest/source/doc/guides/rel_notes/deprecation.rst#L161

LGTM, please, cleanup deprecation notice in this patches when
corresponding changes are done.
Phil Yang Sept. 11, 2020, 10:57 a.m. UTC | #2 
Andrew Rybchenko <arybchenko@solarflare.com> writes:

<snip>
> On 9/11/20 10:35 AM, Phil Yang wrote:
> > Remove the deprecated[1] refcnt_atomic member in structures rte_mbuf
> and
> > rte_mbuf_ext_shared_info.
> >
> > [1]
> http://code.dpdk.org/dpdk/latest/source/doc/guides/rel_notes/deprecatio
> n.rst#L161
> 
> LGTM, please, cleanup deprecation notice in this patches when
> corresponding changes are done.

OK. Will do.

Thanks,
-Phil
Thomas Monjalon Sept. 14, 2020, 8:47 a.m. UTC | #3 
11/09/2020 12:57, Phil Yang:
> Andrew Rybchenko <arybchenko@solarflare.com> writes:
> 
> <snip>
> > On 9/11/20 10:35 AM, Phil Yang wrote:
> > > Remove the deprecated[1] refcnt_atomic member in structures rte_mbuf
> > and
> > > rte_mbuf_ext_shared_info.
> > >
> > > [1]
> > http://code.dpdk.org/dpdk/latest/source/doc/guides/rel_notes/deprecatio
> > n.rst#L161
> > 
> > LGTM, please, cleanup deprecation notice in this patches when
> > corresponding changes are done.
> 
> OK. Will do.

I will do it while squashing and merging the patches.