Thanks, I will squash the commits.
> -----Original Message-----
> From: Thomas Monjalon <tho...@monjalon.net>
> Sent: Tuesday, April 13, 2021 3:24 PM
> To: Ferruh Yigit <ferruh.yi...@intel.com>; Bing Zhao
> <bi...@nvidia.com>
> Cc: Ori Kam <or...@nvidia.com>; andrew.rybche...@oktetlabs.ru; Matan
> Azrad <ma...@nvidia.com>; Slava Ovsiienko <viachesl...@nvidia.com>;
> dev@dpdk.org; ajit.khapa...@broadcom.com; Gregory Etelson
> <getel...@nvidia.com>; Andrey Vesnovaty <andr...@nvidia.com>
> Subject: Re: [dpdk-dev] [PATCH v2 0/4] Change shared action API to
> action handle API
>
> External email: Use caution opening links or attachments
>
>
> 13/04/2021 03:22, Bing Zhao:
> > From: Ferruh Yigit <ferruh.yi...@intel.com>
> > > > Bing Zhao (4):
> > > > ethdev: introduce indirect action APIs
> > > > app/test-pmd: change to indirect action command
> > > > doc: update user guide for indirect action
> > > > net/mlx5: adaption to indirect action API
> > > >
> > >
> > > Hi Bing,
> > >
> > > How the patchset constructed will cause build error in patch by
> > > patch build, since first patch removes some struct/functions
> that
> > > are in use.
> > >
> > > One way to handle this is:
> > > - first add new function/structs
> > > - switch app & pmd to new function/structs
> > > - remove old function/structs
> >
> > Sure, I will reorganize the patch set in this way. Thanks for this.
>
> This is a drop-in replacement, it should be done in one patch only.
>
>
