On Wed, Feb 20, 2013 at 02:11:46AM +0400, Cyrill Gorcunov wrote: > On Tue, Feb 19, 2013 at 02:00:35PM -0800, Andrew Morton wrote: > > > > > > I think this syscall is usefull even without c/r stuff. > > > That's why I made it with separate config option. > > > > hm, OK. > > > > But the patch also permits CONFIG_CHECKPOINT_RESTORE=y, CONFIG_KCMP=n > > which surely isn't something which CRIU wants to support? > > Hmm, yes from one pov this feature is useful for out-of-c/r user, > from another pov -- we will have to ask users to turn on additional > CONFIG entries (which i'm sure not set by default in wide range of distros). > Thus it seems less paiful way is either make it obj-(CHECKPOINT_RESTORE) > as you proposed, or obj-y by default. The last can't be undone, so I'll > prepare the patch for obj-(CHECKPOINT_RESTORE) I think.
Here is update. Peter, does it look good for you too? --- From: Cyrill Gorcunov <gorcu...@openvz.org> Subject: kcmp: Make it to depend on CHECKPOINT_RESTORE Since kcmp syscall has been implemented (initially on x86 architecture) a number of other archs wire it up as well: xtensa, sparc, sh, s390, mips, microblaze, m68k (not taking into account those who uses <asm-generic/unistd.h> for syscall numbers definitions). But the Makefile, which turns kcmp.o generation on still depends on former config-x86. Thus get rid of this limitation and make kcmp.o depend on CHECKPOINT_RESTORE option. Signed-off-by: Cyrill Gorcunov <gorcu...@openvz.org> Cc: KOSAKI Motohiro <kosaki.motoh...@gmail.com> Cc: "Eric W. Biederman" <ebied...@xmission.com> Cc: Pavel Emelyanov <xe...@parallels.com> Cc: Andrey Vagin <ava...@openvz.org> Cc: Ingo Molnar <mi...@elte.hu> Cc: H. Peter Anvin <h...@zytor.com> Cc: Thomas Gleixner <t...@linutronix.de> Cc: Glauber Costa <glom...@parallels.com> Cc: Andi Kleen <a...@firstfloor.org> Cc: Tejun Heo <t...@kernel.org> Cc: Matt Helsley <matth...@us.ibm.com> Cc: Pekka Enberg <penb...@kernel.org> Cc: Eric Dumazet <eric.duma...@gmail.com> Cc: Vasiliy Kulikov <seg...@openwall.com> Cc: Alexey Dobriyan <adobri...@gmail.com> Cc: valdis.kletni...@vt.edu Cc: Michal Marek <mma...@suse.cz> Cc: Frederic Weisbecker <fweis...@gmail.com> Cc: Andrew Morton <a...@linux-foundation.org> --- kernel/Makefile | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) Index: linux-2.6.git/kernel/Makefile =================================================================== --- linux-2.6.git.orig/kernel/Makefile +++ linux-2.6.git/kernel/Makefile @@ -25,9 +25,7 @@ endif obj-y += sched/ obj-y += power/ -ifeq ($(CONFIG_CHECKPOINT_RESTORE),y) -obj-$(CONFIG_X86) += kcmp.o -endif +obj-$(CHECKPOINT_RESTORE) += kcmp.o obj-$(CONFIG_FREEZER) += freezer.o obj-$(CONFIG_PROFILING) += profile.o obj-$(CONFIG_STACKTRACE) += stacktrace.o -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/