2007-01-26 17:30:07

by Michal Piotrowski

[permalink] [raw]
Subject: Re: mm snapshot broken-out-2007-01-26-00-36.tar.gz uploaded

[email protected] napisa?(a):
> The mm snapshot broken-out-2007-01-26-00-36.tar.gz has been uploaded to
>
> ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/mm/broken-out-2007-01-26-00-36.tar.gz

kernel/exit.c: In function 'do_exit':
kernel/exit.c:941: warning: implicit declaration of function 'preexit_task_namespaces'
kernel/exit.c:941: warning: assignment makes pointer from integer without a cast
kernel/exit.c:943: error: too many arguments to function 'exit_task_namespaces'
make[1]: *** [kernel/exit.o] Error 1
make: *** [kernel] Error 2

Regards,
Michal

--
Michal K. K. Piotrowski
LTG - Linux Testers Group
(http://www.stardust.webpages.pl/ltg/)

Signed-off-by: Michal Piotrowski <[email protected]>

--- linux-work-clean/include/linux/nsproxy.h 2007-01-26 17:40:29.000000000 +0100
+++ linux-work/include/linux/nsproxy.h 2007-01-26 18:23:19.000000000 +0100
@@ -51,6 +51,6 @@ static inline void put_and_finalize_nspr
finalize_put_nsproxy(put_nsproxy(ns));
}

-extern void exit_task_namespaces(struct task_struct *p);
+extern void exit_task_namespaces(struct task_struct *p, struct nsproxy *ns);

#endif



2007-01-26 18:14:46

by Serge E. Hallyn

[permalink] [raw]
Subject: Re: mm snapshot broken-out-2007-01-26-00-36.tar.gz uploaded

Quoting Michal Piotrowski ([email protected]):
> [email protected] napisa?(a):
> > The mm snapshot broken-out-2007-01-26-00-36.tar.gz has been uploaded to
> >
> > ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/mm/broken-out-2007-01-26-00-36.tar.gz
>
> kernel/exit.c: In function 'do_exit':
> kernel/exit.c:941: warning: implicit declaration of function 'preexit_task_namespaces'
> kernel/exit.c:941: warning: assignment makes pointer from integer without a cast
> kernel/exit.c:943: error: too many arguments to function 'exit_task_namespaces'
> make[1]: *** [kernel/exit.o] Error 1
> make: *** [kernel] Error 2
>
> Regards,
> Michal
>
> --
> Michal K. K. Piotrowski
> LTG - Linux Testers Group
> (http://www.stardust.webpages.pl/ltg/)
>
> Signed-off-by: Michal Piotrowski <[email protected]>
>
> --- linux-work-clean/include/linux/nsproxy.h 2007-01-26 17:40:29.000000000 +0100
> +++ linux-work/include/linux/nsproxy.h 2007-01-26 18:23:19.000000000 +0100
> @@ -51,6 +51,6 @@ static inline void put_and_finalize_nspr
> finalize_put_nsproxy(put_nsproxy(ns));
> }
>
> -extern void exit_task_namespaces(struct task_struct *p);
> +extern void exit_task_namespaces(struct task_struct *p, struct nsproxy *ns);
>
> #endif

Can you send me your series file for broken-out-2007-01-26-00-36.tar.gz,
and I'll try to iron these out? This is due to an older patch of
Cedric's being applied at the same time as the preexit_task_namespaces
one, and I just need to properly integrate them.

thanks,
-serge

2007-01-26 18:22:22

by Michal Piotrowski

[permalink] [raw]
Subject: Re: mm snapshot broken-out-2007-01-26-00-36.tar.gz uploaded

On 26/01/07, Serge E. Hallyn <[email protected]> wrote:
> Can you send me your series file for broken-out-2007-01-26-00-36.tar.gz,
> and I'll try to iron these out? This is due to an older patch of
> Cedric's being applied at the same time as the preexit_task_namespaces
> one, and I just need to properly integrate them.

ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/mm/broken-out-2007-01-26-00-36.tar.gz

Series file is in the archive, but not in broken-out dir as usually.

$ tar zxvf broken-out-2007-01-26-00-36.tar.gz
$ ls
broken-out broken-out-2007-01-26-00-36.tar.gz series

http://www.stardust.webpages.pl/files/tbf/euridica/mm-snapshot-2007-01-26/series

>
> thanks,
> -serge
>

Regards,
Michal

--
Michal K. K. Piotrowski
LTG - Linux Testers Group
(http://www.stardust.webpages.pl/ltg/)

2007-01-26 18:58:32

by Serge E. Hallyn

[permalink] [raw]
Subject: Re: mm snapshot broken-out-2007-01-26-00-36.tar.gz uploaded

Quoting Michal Piotrowski ([email protected]):
> On 26/01/07, Serge E. Hallyn <[email protected]> wrote:
> >Can you send me your series file for broken-out-2007-01-26-00-36.tar.gz,
> >and I'll try to iron these out? This is due to an older patch of
> >Cedric's being applied at the same time as the preexit_task_namespaces
> >one, and I just need to properly integrate them.
>
> ftp://ftp.kernel.org/pub/linux/kernel/people/akpm/mm/broken-out-2007-01-26-00-36.tar.gz
>
> Series file is in the archive, but not in broken-out dir as usually.

Ah! Sorry.

Ok, this patch should hopefully resolve the conflict between the two
patches, however I can't compile due to some noisy atomic64_xchg
problem and a few other failures.

-serge

From: "Serge E. Hallyn" <[email protected]>
Subject: [PATCH] namespaces: resolve conflicts between two patches

Two patches affecting nsproxy conflicted preventing compilation.
Hopefully this resolves all the problems.

Signed-off-by: Serge E. Hallyn <[email protected]>

---

include/linux/nsproxy.h | 8 ++------
kernel/nsproxy.c | 5 +++++
2 files changed, 7 insertions(+), 6 deletions(-)

300377d87b9889b5c2f5822bfb458f00ef3addd5
diff --git a/include/linux/nsproxy.h b/include/linux/nsproxy.h
index 45f68de..d519290 100644
--- a/include/linux/nsproxy.h
+++ b/include/linux/nsproxy.h
@@ -51,11 +51,7 @@ static inline void put_and_finalize_nspr
finalize_put_nsproxy(put_nsproxy(ns));
}

-static inline struct nsproxy *preexit_task_namespaces(struct task_struct *p)
-{
- return put_nsproxy(p->nsproxy);
-}
-
-extern void exit_task_namespaces(struct task_struct *p);
+extern struct nsproxy *preexit_task_namespaces(struct task_struct *p);
+extern void exit_task_namespaces(struct task_struct *p, struct nsproxy *ns);

#endif
diff --git a/kernel/nsproxy.c b/kernel/nsproxy.c
index afdb9fb..bbe94df 100644
--- a/kernel/nsproxy.c
+++ b/kernel/nsproxy.c
@@ -167,6 +167,11 @@ struct nsproxy *put_nsproxy(struct nspro
return NULL;
}

+struct nsproxy *preexit_task_namespaces(struct task_struct *p)
+{
+ return put_nsproxy(p->nsproxy);
+}
+
void free_nsproxy(struct nsproxy *ns)
{
if (ns->mnt_ns)
--
1.1.6