vhost: revert avoid concurrency when logging dirty pages
authorMaxime Coquelin <maxime.coquelin@redhat.com>
Fri, 20 Apr 2018 08:39:21 +0000 (10:39 +0200)
committerFerruh Yigit <ferruh.yigit@intel.com>
Fri, 27 Apr 2018 17:01:00 +0000 (18:01 +0100)
commitbb77d555d4c422d987b492f6ee6c0533a6486d4e
tree2449205fddad21d8a60d117f45253c1094671509
parenta427b42adb4933ea54be6c2c26c5db34b0d561a4
vhost: revert avoid concurrency when logging dirty pages

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@redhat.com>
lib/librte_vhost/vhost.h