On Sun, 2011-01-02 at 18:16 +0100, Bart Van Assche wrote: > On Thu, Dec 23, 2010 at 10:31 PM, David Dillow <[email protected]> wrote: > > > > We only need locks to protect our lists and number of credits available. > > By pre-consuming the credit for the request, we can reduce our lock > > coverage to just those areas. If we don't actually send the request, > > we'll need to put the credit back into the pool. > > > > [ ... ] > > > > - iu->type = iu_type; > > + list_del_init(&iu->list); > > Hello Dave, > > As far as I can see the above call of list_del_init() can be modified > into a list_del() call. The same holds for the other list_del_init() > call introduced by this patch.
Most likely; there are several such instances that I've corrected from by misinterpretation of the list debug code. Thanks, -- Dave Dillow National Center for Computational Science Oak Ridge National Laboratory (865) 241-6602 office -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
