Quoting Oren Laadan (orenl@xxxxxxxxxxxxxxx): > > > Serge E. Hallyn wrote: > > Quoting Oren Laadan (orenl@xxxxxxxxxxxxxxx): > > What Dave is suggesting (as I understand it) is just changing the way > > the data is shipped between kernel and user-space. But to continue with > > sys_checkpoint() and sys_restart(). So I think it's a less fundamental > > change than you are thinking. > > Probably true, if you ignore the tree he used to illustrate the idea :o > If we agree on the 'blob' (or nearly 'blob') approach, he should suggest > to export a single file (or one file per task, but that's _it_). Well no. I'm saying that the problem with cryo was that you had to use tons of different APIs - and introduce a few new ones - to grab info about various resources using their own API. Yes Dave looks to be making you grab all the info in fine-grained pieces through individual files, but each bit of info is consumed using the exact same API. > Can you comment on point 3, that is -- > > 3. Your approach doesn't play well with what I call "checkpoint that > involves self". This term refers to a process that checkpoints itself > (and only itself), or to a process that attempts to checkpoint its own > container. In both cases, there is no other entity that will read the > data from the file system while the caller is blocked. This is where I seem to recall Dave mentioning some crazy scheme where the task would clone itself and have its clone do the pulling. Don't get me wrong - I'm not sure what Dave's intentions are, but I agree with you that we should keep working on pushing your patchset. If we get a nack based on the set_fs() stuff then we know to go with Dave's approach (or some other), otherwise Dave can keep pursuing his idea in his sandbox. But I do think his idea is cool. Not all cool ideas end up being workable, so we'll see... /me now goes to try lxc-checkpoint with Oren's patches. -serge _______________________________________________ Containers mailing list Containers@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/containers