Subject: Re: [patch 1/2] powerpc/eeh: remove eeh_event_handler()->daemonize()

From: Benjamin Herrenschmidt <[email-masked]>
Date: Thu, 29 Mar 2012 10:22:48 +1100

On Wed, 2012-03-28 at 15:20 -0700, [email-masked] wrote:
> From: Oleg Nesterov <[email-masked]>
> 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.

> Signed-off-by: Oleg Nesterov <[email-masked]>
> Acked-by: Linas Vepstas <[email-masked]>
> Acked-by: Tejun Heo <[email-masked]>
> Acked-by: Matt Fleming <[email-masked]>
> Acked-by: Benjamin Herrenschmidt <[email-masked]>
> Signed-off-by: Andrew Morton <[email-masked]>
> ---
> 
>  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);
> _