[CRIU] [PATCH 0/4] proc: split proc_parse.h on a few parts
Andrei Vagin
avagin at virtuozzo.com
Tue Aug 16 10:22:29 PDT 2016
On Tue, Aug 16, 2016 at 06:39:16PM +0300, Mike Rapoport wrote:
> 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?
Never ever again I will not write patches before the morning bath :-D
>
> > 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