Re: [PATCH 1/1] Trivial: devtmpfsd: Setting taskrunning/interruptible states
From: Greg KH
Date: Wed Sep 21 2011 - 12:01:38 EST
On Wed, Sep 21, 2011 at 09:09:33PM +0530, Kautuk Consul wrote:
> This trivial patch makes the following changes in devtmpfsd() :
This is not the definition of "trivial" in that you are changing the
logic of the code, not just doing spelling changes.
> - Set the state to TASK_INTERRUPTIBLE using __set_current_state
> instead of set_current_state as the spin_unlock is an implicit
> memory barrier.
Why? What is this hurting with the original code?
> - After return from schedule(), there is no need to set the current
> state to TASK_RUNNING as the wake_up_process() function call will
> do this for us.
Are you sure?
Have you tested this patch and everything works properly?
greg k-h
--
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/