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 [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
