Matt Helsley wrote: > On Mon, Aug 24, 2009 at 04:27:25PM -0500, Serge E. Hallyn wrote: >> Quoting Matt Helsley (matthltc@xxxxxxxxxx): >>> Save/restore epoll items during checkpoint/restart respectively. [...] >>> +/* TODO confirm that get_current_user() has been restored */ >> Not sure what you mean. At this point, the task's credentials >> are still the ones used when it called sys_restart(). We won't >> update them until the end of the restart operation. However the >> normal file restore operations should have reset the file->f_cred >> to the checkpointed value - including hierarchical user namespaces. > > I wanted to check to ensure that we're properly enforcing the epoll > watch limits for the restarted tasks. So the patch looks good because > it restore the "common" file pieces before the epoll items/watches. > See below if you're curious where/how that happens. I think I can > remove this TODO now. Good point. Me thinks it deserves a comment in the code ? Oren. _______________________________________________ Containers mailing list Containers@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/containers
