[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