From mboxrd@z Thu Jan 1 00:00:00 1970 From: Roland Dreier Subject: Re: About a shortcoming of the verbs API Date: Wed, 28 Jul 2010 11:05:17 -0700 Message-ID: References: <20100727182046.GT7920@obsidianresearch.com> <1280339513.31421.264.camel@chromite.mv.qlogic.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: In-Reply-To: <1280339513.31421.264.camel-/vjeY7uYZjrPXfVEPVhPGq6RkeBMCJyt@public.gmane.org> (Ralph Campbell's message of "Wed, 28 Jul 2010 10:51:53 -0700") Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Ralph Campbell Cc: Bart Van Assche , Jason Gunthorpe , Linux-RDMA List-Id: linux-rdma@vger.kernel.org > Actually, I tried to implement the completion callback > in a workqueue thread but ipoib_cm_handle_tx_wc() calls > netif_tx_lock() which isn't safe unless it is called > from an IRQ handler or netif_tx_lock_bh() is called first. Oh, sounds like a bug in IPoIB. I guess we could fix it by just changing it to netif_tx_lock_bh()? (Or is that not safe from an IRQ handler?) -- Roland Dreier || For corporate legal information go to: http://www.cisco.com/web/about/doing_business/legal/cri/index.html -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html