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.
_______________________________________________
openib-general mailing list
[email protected]
http://openib.org/mailman/listinfo/openib-general
To unsubscribe, please visit http://openib.org/mailman/listinfo/openib-general
