On Sun, Aug 02, 2015 at 04:09:01PM +0300, Haggai Eran wrote:
> On 31/07/2015 02:22, Jason Gunthorpe wrote:
> > int ib_dealloc_pd(struct ib_pd *pd)
> > {
> > + if (pd->local_mr) {
> > + if (ib_dereg_mr(pd->local_mr))
> > + return -EBUSY;
> > + pd->local_mr = NULL;
> > + }
> > +
>
> It looks like ib_uverbs_alloc_pd calls ib_device.alloc_pd() directly,
> and some drivers don't use kzalloc for allocating the pd, so the
> ib_dereg_mr call above results in a general protection fault.
Indeed it does..
I'll clean up the dealloc_pd mess too.
Jason
--
To unsubscribe from this list: send the line "unsubscribe target-devel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
