10/12/2018 11:49, Burakov, Anatoly:
> On 07-Dec-18 8:10 PM, Seth Howell wrote:
> > 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")
> >
> > Signed-off-by: Seth Howell <seth.how...@intel.com>
> > Signed-off-by: Darek Stojaczyk <dariusz.stojac...@intel.com>
> > ---
>
> Lost my review tag...
>
> Reviewed-by: Anatoly Burakov <anatoly.bura...@intel.com>
+Cc: sta...@dpdk.org
Applied, thanks
