Re: [2.6 patch] kernel/ns_cgroup.c should #include<linux/nsproxy.h>

From: Serge E. Hallyn
Date: Fri Feb 22 2008 - 19:02:21 EST


Quoting Adrian Bunk (bunk@xxxxxxxxxx):
> Every file should include the headers containing the externs its global
> functions (in this case for ns_cgroup_clone()).
>
> Signed-off-by: Adrian Bunk <bunk@xxxxxxxxxx>

Seems fine... and a good way to ensure that the prototype and
definition match.

Acked-by: Serge Hallyn <serue@xxxxxxxxxx>

thanks,
-serge

>
> ---
> d3446a02068091d59425c51bde1daea777398e44 diff --git a/kernel/ns_cgroup.c b/kernel/ns_cgroup.c
> index aead4d6..659f321 100644
> --- a/kernel/ns_cgroup.c
> +++ b/kernel/ns_cgroup.c
> @@ -7,6 +7,7 @@
> #include <linux/module.h>
> #include <linux/cgroup.h>
> #include <linux/fs.h>
> +#include <linux/nsproxy.h>
>
> struct ns_cgroup {
> struct cgroup_subsys_state css;
--
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/