On Thu, Aug 13, 2015 at 09:04:39AM -0700, Christoph Hellwig wrote: > On Thu, Aug 13, 2015 at 09:07:14AM -0400, Doug Ledford wrote: > > > Doug: was your mail a request to fix up the two de-staged drivers? > > > I'm happy to do that if you're fine with the patch in general. amso1100 > > > should be trivial anyway, while ipath is a mess, just like the new intel > > > driver with the third copy of the soft ib stack. > > > > Correct. > > http://git.infradead.org/users/hch/rdma.git/commitdiff/efb2b0f21645b9caabcce955481ab6966e52ad90 > > contains the updates for ipath and amso1100, as well as the reviewed-by > and tested-by tags.
The uverbs change needs to drop/move the original kmalloc: next = kmalloc(ALIGN(sizeof *next, sizeof (struct ib_sge)) + user_wr->num_sge * sizeof (struct ib_sge), GFP_KERNEL); It looks like it is leaking that allocation right now. Every path replaces next with the result of alloc_mr.. Noticed a couple of trailing whitespaces too.. Jason -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html