[prev in list] [next in list] [prev in thread] [next in thread] 

List:       lxc-devel
Subject:    [lxc-devel] [lxc/lxc] 75d219: c/r: use /proc/self/tid/children instead of pidfil...
From:       GitHub <noreply () github ! com>
Date:       2016-08-26 16:48:39
Message-ID: 57c072e74fdc3_26633f8f566b914c918a6 () hookshot-fe5-cp1-prd ! iad ! github ! net ! mail
[Download RAW message or body]

Branch: refs/heads/master
  Home:   https://github.com/lxc/lxc
  Commit: 75d219f0cc6be29d192677c5a8bd1bfd9528437b
      https://github.com/lxc/lxc/commit/75d219f0cc6be29d192677c5a8bd1bfd9528437b
  Author: Tycho Andersen <tycho.andersen@canonical.com>
  Date:   2016-08-26 (Fri, 26 Aug 2016)

  Changed paths:
    M src/lxc/criu.c

  Log Message:
  -----------
  c/r: use /proc/self/tid/children instead of pidfile

All we really needed a unique temp file for was passing the pid. Since CRIU
opened this with O_EXCL | O_CREAT, this was "safe" (users could still
overwrite it afterwards, but the monitor would immediately die since the
only valid number in there was the init process).

In any case, we can just read /proc/self/tid/children, which lists the
child process.

Closes #1150

Signed-off-by: Tycho Andersen <tycho.andersen@canonical.com>


  Commit: 69dffc98e32ce2f523073e108229d7d00e6493d9
      https://github.com/lxc/lxc/commit/69dffc98e32ce2f523073e108229d7d00e6493d9
  Author: Christian Brauner <cbrauner@suse.de>
  Date:   2016-08-26 (Fri, 26 Aug 2016)

  Changed paths:
    M src/lxc/criu.c

  Log Message:
  -----------
  Merge pull request #1152 from tych0/fix-migration-2.0.4

c/r: use /proc/self/tid/children instead of pidfile


Compare: https://github.com/lxc/lxc/compare/e854c5e31579...69dffc98e32c
[Attachment #3 (text/plain)]

_______________________________________________
lxc-devel mailing list
lxc-devel@lists.linuxcontainers.org
http://lists.linuxcontainers.org/listinfo/lxc-devel


[prev in list] [next in list] [prev in thread] [next in thread]