[dpdk-dev] [PATCHv8 19/46] pool/dpaa2: add DPAA2 hardware offloaded mempool

Hemant Agrawal hemant.agrawal at nxp.com
Thu Mar 9 06:57:26 CET 2017

On 3/8/2017 9:09 PM, Thomas Monjalon wrote:
> 2017-03-08 18:22, Hemant Agrawal:
>>> On Fri, 3 Mar 2017 18:16:36 +0530, Hemant Agrawal
>>> <hemant.agrawal at nxp.com> wrote:
>>> I think the current mempool handlers should be moved first in a
>>> separate patch.
> Yes it should have been done earlier.
>> Are you seeing any benefit by making it a separate patch series?
> A separate patchset for moving mempool handlers will be easy to review
> and accept.
> If integrated in this series, it is kind of hidden and prevent the
> visibility and review it deserves.
> By the way the mempool move should be directly committed in the main
> repository, while this series targets next-net.

hw mempool has dependency on mc-bus. So, we will break it into *5* series,

1. mc-bus - which can go on main repo - Rebased over main repo
2. dpaa2-pool - it can also go on main rep (depends on mc-bus) - Rebased 
over 'main repo + 1'.
3. dpaa2-pmd - this depends on mc-bus and dpaa2-pool
       A. mc-bus -another series rebased over net-next
       B. dpaa2-pool - another series rebased over 'net-next + A'
       C. the pmd itself rebased over 'net-next + A + B'

Are you sure, you will like us to do the above and flood the mailing 
list :)

I think, this will make the review difficult.

Bruce, Ferruh, Olivier - does the above help you? If so, we can do it.

My preference is to keep it as it is. It can be applied on 'net-next' or 
'main' as a whole. But I am open for suggestions.

>> it will be difficult and tricky for us. The dpaa2_pool has a dependency
>> on mc bus patches. dpaa2_pmd has dependency on dpaa2_pool and mc buses.
> You will just have to rebase this series on top of the new one.

More information about the dev mailing list