[dpdk-dev] [PATCH] Revert "vhost: avoid concurrency when logging dirty pages"

Ferruh Yigit ferruh.yigit at intel.com
Fri Apr 27 16:34:03 CEST 2018


On 4/20/2018 9:39 AM, Maxime Coquelin wrote:
> This reverts commit 394313fff39d0f994325c47f7eab39daf5dc9e11.
> 
> While the patch did solve concurrency issue, it induces more
> pages copies as some clean pages are marked as dirty for
> performance reasons. Moreover, as there is no more contention
> doing the logging, the rate of packets than can be processed is
> higher, leading to even more pages to be dirtied.
> 
> It has been reported that with more than one queue pair, and
> with a relatively low packet rate (1Mpps), the live migration
> never converges until the flow is stopped.
> 
> While a better solution is found, it is better to reset to the
> old behaviour, i.e. using atomic operation for dirty pages
> logging.
> 
> Signed-off-by: Maxime Coquelin <maxime.coquelin at redhat.com>

Applied to dpdk-next-net/master, thanks.


More information about the dev mailing list