[Date Prev] | [Thread Prev] | [Thread Next] | [Date Next] -- [Date Index] | [Thread Index] | [List Home]
Subject: Re: [PATCH v10 5/7] migration/ram.c: add a notifier chain for precopy
On 12/03/2018 01:20 PM, Peter Xu wrote:
On Mon, Dec 03, 2018 at 10:18:29AM +0800, Wei Wang wrote:This patch adds a notifier chain for the memory precopy. This enables various precopy optimizations to be invoked at specific places. Signed-off-by: Wei Wang <firstname.lastname@example.org> CC: Dr. David Alan Gilbert <email@example.com> CC: Juan Quintela <firstname.lastname@example.org> CC: Michael S. Tsirkin <email@example.com> CC: Peter Xu <firstname.lastname@example.org> ---[...]+int precopy_notify(PrecopyNotifyReason reason, Error **errp);This function could be hidden from include/migration/misc.h, but I don't know whether that is a reason for a repost. And what I meant was that we fail precopy if notifier failed the hooks, but warning is fine too anyway no real use case there.
Yes, I was also thinking about this. Failing precopy needs to change some upper layer functions which are defined with "void". There is no use case that needs to fail precopy currently, so I chose to keep the change minimal. But we can make more changes if people think it's necessary to do. Best, Wei