malloc: notify primary process about hotplug in secondary
authorSeth Howell <seth.howell@intel.com>
Fri, 7 Dec 2018 20:10:42 +0000 (13:10 -0700)
committerThomas Monjalon <thomas@monjalon.net>
Thu, 20 Dec 2018 14:25:34 +0000 (15:25 +0100)
commitfba0ca227453ee6a60f674884531fb3817ca8a17
tree4fa3091776adc1d29da99d94aac21be45aeffb29
parent6d09256148567807b772347ec4ac363c4bcb6b5a
malloc: notify primary process about hotplug in secondary

When secondary process hotplugs memory, it sends a request
to primary, which then performs the real mmap() and sends
sync requests to all secondary processes. Upon receiving
such sync request, each secondary process will notify the
upper layers of hotplugged memory (and will call all
locally registered event callbacks).

In the end we'll end up with memory event callbacks fired
in all the processes except the primary, which is a bug.

This gets critical if memory is hotplugged while a VFIO
device is attached, as the VFIO memory registration -
which is done from a memory event callback present in the
primary process only - is never called.

After this patch, a primary process fires memory event
callbacks before secondary processes start their
synchronizations - both for hotplug and hotremove.

Fixes: 07dcbfe0101f ("malloc: support multiprocess memory hotplug")
Cc: stable@dpdk.org
Signed-off-by: Seth Howell <seth.howell@intel.com>
Signed-off-by: Darek Stojaczyk <dariusz.stojaczyk@intel.com>
Reviewed-by: Anatoly Burakov <anatoly.burakov@intel.com>
lib/librte_eal/common/malloc_mp.c