Re: [PATCH -mm 2/2] Introduce freezer flags

From: Sam Ravnborg
Date: Fri Apr 27 2007 - 12:19:17 EST


On Fri, Apr 27, 2007 at 05:40:16PM +0200, Rafael J. Wysocki wrote:

> --- linux-2.6.21-rc7-mm2.orig/kernel/fork.c
> +++ linux-2.6.21-rc7-mm2/kernel/fork.c
> @@ -921,11 +921,14 @@ static inline void copy_flags(unsigned l
> {
> unsigned long new_flags = p->flags;
>
> - new_flags &= ~(PF_SUPERPRIV | PF_NOFREEZE);
> + new_flags &= ~PF_SUPERPRIV;
> new_flags |= PF_FORKNOEXEC;
> if (!(clone_flags & CLONE_PTRACE))
> p->ptrace = 0;
> p->flags = new_flags;
> +#ifdef CONFIG_FREEZER
> + p->freezer_flags = 0;
> +#endif
> }
Could we have a dedicated clear_freeze_flag() so we could drop the ifdef?

> + clear_freeze_flag(p);

Or maybe we have it already?


Sam
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/