[CRIU] [PATCH 0/4] proc: split proc_parse.h on a few parts

Mike Rapoport mike.rapoport at gmail.com
Tue Aug 16 08:39:16 PDT 2016


On Tue, Aug 16, 2016 at 6:22 PM, Andrei Vagin <avagin at openvz.org> wrote:
> From: Andrei Vagin <avagin at virtuozzo.com>

> proc_parse.h is included from many files and when we change
> something in it, too many files need to be recopiled.
>
> This series moves functions from proc_parse.h to less generic headers.
>
> Andrei Vagin (4):
>   vma: don't define dump_filemap_t twice
>   mount: mount functions about mounts from proc_parse.h

mount: mount functions?

>   cgroup: mount functions about cgroup out of proc_parse.h

cgroup: mount functions?

>   proc: move parts about fdinfo from proc_parse.h into fdinfo.h

Apart from those looks good to me :)

>  criu/autofs.c              |   2 +-
>  criu/cgroup.c              |   1 -
>  criu/cr-check.c            |   2 +-
>  criu/eventfd.c             |   2 +-
>  criu/eventpoll.c           |   2 +-
>  criu/files.c               |   2 +-
>  criu/fsnotify.c            |   2 +-
>  criu/include/autofs.h      |   2 +
>  criu/include/cgroup.h      |  24 +++++++++++
>  criu/include/fdinfo.h      |  58 +++++++++++++++++++++++++
>  criu/include/mount.h       |  25 +++++++++++
>  criu/include/proc_parse.h  | 105 ---------------------------------------------
>  criu/include/sysfs_parse.h |   3 ++
>  criu/include/vma.h         |   3 ++
>  criu/mount.c               |   1 -
>  criu/proc_parse.c          |   1 +
>  criu/signalfd.c            |   2 +-
>  criu/timerfd.c             |   2 +-
>  18 files changed, 124 insertions(+), 115 deletions(-)
>  create mode 100644 criu/include/fdinfo.h
>
> --
> 2.7.4
>
> _______________________________________________
> CRIU mailing list
> CRIU at openvz.org
> https://lists.openvz.org/mailman/listinfo/criu



-- 
Sincerely yours,
Mike.


More information about the CRIU mailing list