[CRIU] [PATCH 00/11] lazy-pages: handle multiple processes

[Pavel Emelyanov]

On 04/13/2016 11:05 AM, Adrian Reber wrote:
> On Mon, Apr 11, 2016 at 09:19:43AM +0300, Mike Rapoport wrote:
>> I took a liberty to implement handling of multiple processes in the
>> lazy-pages daemon before we've finished the design discussion about
>> communication protocol and page transfer related stuff.
> 
> I am not sure I completely understand the intentions behind this.

AFAIU we all agreed that we'll have single uffd daemon to handle
#PF-s from the whole restored tree. So this set makes a step towards
it and make the daemon handle #PF-s from multiple uffds.

> So it sounds like the uffd daemon running on the same side as the
> restore process, because right now we only have local lazy-restore,
> does not terminate after handling all page-requests from one restore.
> Which sounds good. But right now the part started with 'criu lazy-pages'
> still needs a checkpoint directory where it can read the checkpoint
> files. I haven't seen the point where the 'criu lazy-pages' part gets
> the new directory FD.
> 
> I don't have anything against this patch series, but this somehow is a
> decision in favor of a 'dumb' 'criu lazy-pages' which only forwards the
> requests from the network to the UFFD. As this is also my preferred
> solution I am happy to go this way, but from my point of view this
> somehow shortcuts our other discussion thread. If this is the way we
> want to continue then these patches are a good step in the right
> direction.
> 
> 		Adrian
> .
>