[dpdk-dev] [PATCH 31/41] ethdev: use contiguous allocation for DMA memory
Burakov, Anatoly
anatoly.burakov at intel.com
Mon Mar 5 10:08:43 CET 2018
On 03-Mar-18 2:05 PM, Andrew Rybchenko wrote:
> On 03/03/2018 04:46 PM, Anatoly Burakov wrote:
>> This fixes the following drivers in one go:
>
> Does it mean that these drivers are broken in the middle of patch set
> and fixed now?
> If so, it would be good to avoid it. It breaks bisect.
>
Depends on the definition of "broken". Legacy memory mode will still
work for all drivers throughout the patchset. As for new memory mode,
yes, it will be "broken in the middle of the patchset", but due to the
fact that there's enormous amount of code to review between fbarray
changes, malloc changes, contiguous allocation changes and adding new
rte_memzone API's, i favored ease of code review over bisect.
I can of course reorder and roll up several different patchset and all
driver updates into one giant patch, but do you really want to be the
one reviewing such a patch?
--
Thanks,
Anatoly
More information about the dev
mailing list