Re: [PATCH] powerpc pseries rtasd: Convert to kthread API.

From: Christoph Hellwig
Date: Sun Apr 22 2007 - 08:35:30 EST


On Thu, Apr 19, 2007 at 01:58:46AM -0600, Eric W. Biederman wrote:
> From: Eric W. Biederman <ebiederm@xxxxxxxxxxxx>
>
> This patch modifies the startup of rtasd to use kthread_run instaed of
> a combination of kernel_thread and daemonize. Making the code a little
> simpler and more maintainble.

Looks okay, but I have some questions about the original code.

Why does the driver only check if it really needs to run in the
thread and calls vmalloc from it? If we did all these in the
initialization function we could actually properly unwind.

-
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/