[openib-general] [PATCH] process locked in D state.

Gleb Natapov glebn at voltaire.com
Tue Jun 28 08:33:45 PDT 2005


On Tue, Jun 28, 2005 at 07:10:13AM -0700, Roland Dreier wrote:
>     Gleb> Yes, but don't forget the first part of my patch (for
>     Gleb> libibverbs/src/device.c) I think it should be applied to.
> 
> Yes, I guess it makes sense.
> 
> > -	mm = get_task_mm(current);
> 
>     Gleb> Are you sure that ib_umem_account will run on behalf the
>     Gleb> process that schedules it?  Anyway you are correctly using
>     Gleb> work->mm in the rest of the function so this line should be
>     Gleb> dropped I think.
> 
> The '-' at the beginning of the line means it is being deleted in the
> patch!  Of course schedule_work() delegates the work to a different
> process so the get_task_mm() must be done in the original process.
> 
Oops. I should be more careful :)

--
			Gleb.



More information about the general mailing list