[dpdk-dev] [PATCH v3 00/14] Make shared memory config non-public
Burakov, Anatoly
anatoly.burakov at intel.com
Wed Jul 3 12:47:19 CEST 2019
On 03-Jul-19 10:38 AM, David Marchand wrote:
> Hello Anatoly,
>
> On Thu, Jun 27, 2019 at 1:39 PM Anatoly Burakov
> <anatoly.burakov at intel.com <mailto:anatoly.burakov at intel.com>> wrote:
>
> This patchset removes the shared memory config from public
> API, and replaces all usages of said config with new API
> calls.
>
> A lot of the patchset is a search-and-replace job and should
> be pretty easy to review. The rest are pretty trivial EAL
> changes.
>
> This patchset depends on FreeBSD fixes patchset:
>
> http://patches.dpdk.org/project/dpdk/list/?series=5196
>
> v3:
> - Rebase on top of latest master
>
> v2:
> - Collapsed all changes into fewer patches
> - Addressed review comments
> - Created a new file to store the code
> - Changed namespace to "rte_mcfg_"
> - Added some unification around config init
> - Removed "packed" attribute from mem config
> - Removed unnecessary inlining
> - Added a check to explicitly forbid running multiprocess
> applications that differ in their DPDK versions
>
>
>
> For the parts I already had a look at, I still think the changes are in
> too many patches.
> A lot of this is just search/replace we can have it with the patch that
> introduces it.
> - patch 1, 2 and 3 could be squashed as a single one (plus removing the
> unused macro from patch 8)
> - idem with patch 4 and 5
> - idem with patch 6 and 7 (plus removing the unused macro from patch 8)
>
>
> I will look at the rest tomorrow (hopefully).
>
> --
> David Marchand
Sure, i can squash it down into single patches for search/replace if
that's the preference.
--
Thanks,
Anatoly
More information about the dev
mailing list