Re: [patch 1/2] powerpc/eeh: remove eeh_event_handler()->daemonize()
From: Benjamin Herrenschmidt <benh@kernel.crashing.org>
Date: 2012-03-28 23:22:52
On Wed, 2012-03-28 at 15:20 -0700, akpm@linux-foundation.org wrote:
From: Oleg Nesterov <oleg@redhat.com> Subject: powerpc/eeh: remove eeh_event_handler()->daemonize() daemonize() is only needed when a user-space task does kernel_thread(). eeh_event_handler() thread is created by the worker kthread, and thus it doesn't need the soon-to-be-deprecated daemonize().
For some reason I thought you were sending that directly to Linus, but looking at your CC list that's obviously not the case :-) I'll send them myself. Cheers, Ben.
quoted hunk ↗ jump to hunk
Signed-off-by: Oleg Nesterov <oleg@redhat.com> Acked-by: Linas Vepstas <linasvepstas@gmail.com> Acked-by: Tejun Heo <tj@kernel.org> Acked-by: Matt Fleming <redacted> Acked-by: Benjamin Herrenschmidt <benh@kernel.crashing.org> Signed-off-by: Andrew Morton <akpm@linux-foundation.org> --- arch/powerpc/platforms/pseries/eeh_event.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff -puN arch/powerpc/platforms/pseries/eeh_event.c~powerpc-eeh-remove-eeh_event_handler-daemonize arch/powerpc/platforms/pseries/eeh_event.c--- a/arch/powerpc/platforms/pseries/eeh_event.c~powerpc-eeh-remove-eeh_event_handler-daemonize +++ a/arch/powerpc/platforms/pseries/eeh_event.c@@ -59,7 +59,7 @@ static int eeh_event_handler(void * dumm struct eeh_event *event; struct eeh_dev *edev; - daemonize("eehd"); + set_task_comm(current, "eehd"); set_current_state(TASK_INTERRUPTIBLE); spin_lock_irqsave(&eeh_eventlist_lock, flags);_