linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* linux-next: build failure after merge of the ptrace tree
@ 2011-08-25  4:12 Stephen Rothwell
  2011-08-25 14:37 ` Oleg Nesterov
  2011-08-25 16:35 ` [PATCH] add EXPORT_SYMBOL(set_current_blocked) Oleg Nesterov
  0 siblings, 2 replies; 3+ messages in thread
From: Stephen Rothwell @ 2011-08-25  4:12 UTC (permalink / raw)
  To: Tejun Heo, Oleg Nesterov; +Cc: linux-next, linux-kernel, Matt Fleming

[-- Attachment #1: Type: text/plain, Size: 407 bytes --]

Hi all,

After merging the ptrace tree, today's linux-next build (powerpc
ppc64_defconfig) failed like this:

ERROR: ".set_current_blocked" [fs/autofs4/autofs4.ko] undefined!

Presumably set_current_blocked() needs to be EXPORTed.

I have used the ptrace tree from next-20110824 for today.
-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au
http://www.canb.auug.org.au/~sfr/

[-- Attachment #2: Type: application/pgp-signature, Size: 490 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: linux-next: build failure after merge of the ptrace tree
  2011-08-25  4:12 linux-next: build failure after merge of the ptrace tree Stephen Rothwell
@ 2011-08-25 14:37 ` Oleg Nesterov
  2011-08-25 16:35 ` [PATCH] add EXPORT_SYMBOL(set_current_blocked) Oleg Nesterov
  1 sibling, 0 replies; 3+ messages in thread
From: Oleg Nesterov @ 2011-08-25 14:37 UTC (permalink / raw)
  To: Stephen Rothwell; +Cc: Tejun Heo, linux-next, linux-kernel, Matt Fleming

Hi Stephen,

On 08/25, Stephen Rothwell wrote:
>
> Hi all,
>
> After merging the ptrace tree, today's linux-next build (powerpc
> ppc64_defconfig) failed like this:
>
> ERROR: ".set_current_blocked" [fs/autofs4/autofs4.ko] undefined!

In case you didn't notice, we are doing our best to complicate
your life :/

Sorry.

> Presumably set_current_blocked() needs to be EXPORTed.

Yes. I've reset the last 4 commits which touch the code in fs/.
Will add them back after the patch which exports set_current_blocked.

Thanks a lot!

Oleg.


^ permalink raw reply	[flat|nested] 3+ messages in thread

* [PATCH] add EXPORT_SYMBOL(set_current_blocked)
  2011-08-25  4:12 linux-next: build failure after merge of the ptrace tree Stephen Rothwell
  2011-08-25 14:37 ` Oleg Nesterov
@ 2011-08-25 16:35 ` Oleg Nesterov
  1 sibling, 0 replies; 3+ messages in thread
From: Oleg Nesterov @ 2011-08-25 16:35 UTC (permalink / raw)
  To: Stephen Rothwell, Tejun Heo, Matt Fleming; +Cc: linux-kernel

Export set_current_blocked() for modules. We have the code which
wrongly plays with ->blocked, and sigprocmask() should be avoided.

Reported-by: Stephen Rothwell <sfr@canb.auug.org.au>
Signed-off-by: Oleg Nesterov <oleg@redhat.com>
---

 kernel/signal.c |    1 +
 1 file changed, 1 insertion(+)

--- 3.1/kernel/signal.c~export_set_current_blocked	2011-08-25 18:23:42.000000000 +0200
+++ 3.1/kernel/signal.c	2011-08-25 18:24:02.000000000 +0200
@@ -2463,6 +2463,7 @@ void set_current_blocked(const sigset_t 
 	__set_task_blocked(tsk, newset);
 	spin_unlock_irq(&tsk->sighand->siglock);
 }
+EXPORT_SYMBOL(set_current_blocked);
 
 /*
  * This is also useful for kernel threads that want to temporarily


^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2011-08-25 16:38 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2011-08-25  4:12 linux-next: build failure after merge of the ptrace tree Stephen Rothwell
2011-08-25 14:37 ` Oleg Nesterov
2011-08-25 16:35 ` [PATCH] add EXPORT_SYMBOL(set_current_blocked) Oleg Nesterov

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).