From mboxrd@z Thu Jan 1 00:00:00 1970 From: Haggai Eran Subject: Re: [PATCH v2 01/12] IB/core: Guarantee that a local_dma_lkey is available Date: Sun, 2 Aug 2015 16:09:01 +0300 Message-ID: <55BE166D.5000700@mellanox.com> References: <1438298547-21404-1-git-send-email-jgunthorpe@obsidianresearch.com> <1438298547-21404-2-git-send-email-jgunthorpe@obsidianresearch.com> Mime-Version: 1.0 Content-Type: text/plain; charset="windows-1252" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1438298547-21404-2-git-send-email-jgunthorpe@obsidianresearch.com> Sender: target-devel-owner@vger.kernel.org To: Jason Gunthorpe , Doug Ledford , linux-rdma@vger.kernel.org Cc: Amir Vadai , Bart Van Assche , Chien Yen , Christoph Hellwig , Dominique Martinet , Eli Cohen , Eric Van Hensbergen , Ido Shamay , Latchesar Ionkov , Or Gerlitz , Roi Dayan , Ron Minnich , Sagi Grimberg , Simon Derr , Tom Tucker , rds-devel@oss.oracle.com, target-devel@vger.kernel.org, v9fs-developer@lists.sourceforge.net List-Id: linux-rdma@vger.kernel.org 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. Haggai