Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

criu does not restore process start time #2504

Open
hanwen-flow opened this issue Oct 28, 2024 · 4 comments
Open

criu does not restore process start time #2504

hanwen-flow opened this issue Oct 28, 2024 · 4 comments

Comments

@hanwen-flow
Copy link

Description

The process start time (entry 22 in /proc/$PID/stat) is not restored faithfully.

This is a problem, b/c the software I'm trying to checkpoint/restore has a client that uses the pid + start time to check if it the server wasn't changed from under it.

$ docker run -d --name looper ubuntu:latest /bin/bash -c 'i=0; p=$BASHPID; while true; do echo -n  "$i "; cat /proc/$p/stat| awk "{print \$22;}"; i=$(expr $i + 1); sleep 1; done'
562b85dcf5cf086495b8f39ee1c18e88083f4c48327d568fad88d09d3f059040


$ docker logs looper
0 138623
1 138623
2 138623
3 138623
4 138623
5 138623

$ docker checkpoint create looper cp1
cp1
$ docker container start --checkpoint cp1  looper
$ docker logs looper
...
14 138623
15 138623
16 138623
17 138623
18 140372
19 140372
20 140372
21 140372
22 140372

version info:

$ docker version
Client: Docker Engine - Community
 Version:           27.3.1
 API version:       1.47
 Go version:        go1.22.7
 Git commit:        ce12230
 Built:             Fri Sep 20 11:41:00 2024
 OS/Arch:           linux/amd64
 Context:           default

Server: Docker Engine - Community
 Engine:
  Version:          27.3.1
  API version:      1.47 (minimum version 1.24)
  Go version:       go1.22.7
  Git commit:       41ca978
  Built:            Fri Sep 20 11:41:00 2024
  OS/Arch:          linux/amd64
  Experimental:     true
 containerd:
  Version:          1.7.22
  GitCommit:        7f7fdf5fed64eb6a7caf99b3e12efcf9d60e311c
 runc:
  Version:          1.1.14
  GitCommit:        v1.1.14-0-g2c9f560
 docker-init:
  Version:          0.19.0
  GitCommit:        de40ad0

$ criu --version
Version: 3.18
GitID: v3.18-320-gdfb56eed6

$ uname -a 
Linux hanwen-flow 6.8.0-47-generic #47~22.04.1-Ubuntu SMP PREEMPT_DYNAMIC Wed Oct  2 16:16:55 UTC 2 x86_64 x86_64 x86_64 GNU/Linux

$ sudo criu check --all
sudo: mon_handle_sigchld: waitpid: No child processes
Looks good.
$ 
@avagin
Copy link
Member

avagin commented Oct 31, 2024

It is a known issue. Someone has to introduce a kernel interface for that.

@avagin
Copy link
Member

avagin commented Oct 31, 2024

@Snorch
Copy link
Member

Snorch commented Nov 4, 2024

We can try doing same as it is done in OpenVZ

OpenVZ/vzkernel@2602bde
OpenVZ@df8fec2

Sadly it relies heavily on "ve" container object which is OpenVZ specific, but probably it would be not so hard to rework it to time-namespace based approach instead.

  • Add new field to task_struct (or some other per-task place if on task struct there is no place), e.g. start_boottime_timens, to show it in /proc/$pid/stat inside timens, outside we should probably show regular start_boottime
  • Make it adjustable, e.g. prctl_set_task_ct_fields via prctl (maybe restrict it to CAP_SYS_ADMIN, so that container user can't change it)
  • Read start time from proper context on dump and restore it on restore

Copy link

github-actions bot commented Dec 5, 2024

A friendly reminder that this issue had no activity for 30 days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants