[RFC][PATCH 0/6][usercr]: Rename/reorg usercr code
sukadev at linux.vnet.ibm.com
Mon Apr 26 10:16:04 PDT 2010
Oren Laadan [orenl at cs.columbia.edu] wrote:
> Hi Suka,
> Thanks for the patchset. A couple of comments:
> * I prefer the header exported to users to be checkpoint.h - this
> is consistent with kernel headers, and with future name of a c/r
> library if we opt libcheckpoint.a
That is fine. Will this checkpoint.h include <linux/checkpoint.h> ?
> * I also prefer to leave checkpoint.c as a separate file, as is.
> It may gain more functionality in the future. If the goal was to
> only export a single .o file, then the solution IMHO is to instead
> export a single library: libcheckpoint.a
A single .o was not really a goal. Sure, if we are ready for libcheckpoint.a
then thats a better way.
> Unless you have a strong opinion against the above, I'll go ahead
> and pull patches 1-3, leave out 4-6, and update the Makefile to
> create a libcheckpoint.a library.
How about patch 6, renaming 'common.h' to 'cr_log.h' (file internal to
usercr) ? Let me know if you want me to resend the patches dropping 4 and 5.
More information about the Containers