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

Roland Dreier rolandd at cisco.com
Tue Jun 28 12:31:51 PDT 2005


    Michael> Right, but it seems this introduces the race for apps
    Michael> doing deregister_mr properly, too, which used to work
    Michael> fine.

True.

    Michael> Right, but at least lets not introduce the hole for the
    Michael> good case.  When I call deregister_mr I have the right to
    Michael> expect that I can lock the memory immediately after that.
    Michael> So deregister mr should always wait till it has the
    Michael> semaphore.

    Michael> In the case of close, its the buggy app that suffers, and
    Michael> I'm ok with handling it in a work queue since you think
    Michael> its worth it - but lets not try to optimise that case, it
    Michael> should be ok to always use the workqueue.

OK, this looks fine.  Applied (with slight renames of function names).

Thanks,
  Roland



More information about the general mailing list