> -----Original Message----- > From: Thomas Monjalon <tho...@monjalon.net> > Sent: Wednesday, October 21, 2020 3:59 PM > To: Ruifeng Wang <ruifeng.w...@arm.com> > Cc: Bruce Richardson <bruce.richard...@intel.com>; Vladimir Medvedkin > <vladimir.medved...@intel.com>; dev@dpdk.org; Honnappa Nagarahalli > <honnappa.nagaraha...@arm.com>; nd <n...@arm.com>; David Marchand > <david.march...@redhat.com>; Kevin Traynor <ktray...@redhat.com> > Subject: Re: [PATCH v2 2/2] lpm: hide internal data > > 21/10/2020 05:02, Ruifeng Wang: > > --- a/doc/guides/rel_notes/release_20_11.rst > > +++ b/doc/guides/rel_notes/release_20_11.rst > > @@ -602,6 +602,8 @@ ABI Changes > > > > * sched: Added new fields to ``struct rte_sched_subport_port_params``. > > > > +* lpm: Removed fields other than ``tbl24`` and ``tbl8`` from the struct > ``rte_lpm``. > > + The removed fields were made internal. > > > > Known Issues > > ------------ > > Can be changed on apply, but please note when adding a new paragraph, > that you should add a new blank line, keeping 2 blank lines before the next > section as it was before your patch. > > PS: having this kind of minor comment means you are a regular contributor, > so we expect perfect patches :) > Sorry for the extra burden added to merge. Will pay more attention on format and other details.
Thanks. > Thanks >