From mboxrd@z Thu Jan 1 00:00:00 1970 From: Leon Romanovsky Subject: Re: [PATCH for kernel v4.6] IB/srpt: Revert "Convert to percpu_ida tag allocation" Date: Fri, 1 Apr 2016 06:03:22 +0300 Message-ID: <20160401030322.GH2670@leon.nu> References: <56FDBA63.7010804@sandisk.com> Reply-To: leon-2ukJVAZIZ/Y@public.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <56FDBA63.7010804-XdAiOPVOjttBDgjK7y7TUQ@public.gmane.org> Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Bart Van Assche Cc: Doug Ledford , "Nicholas A. Bellinger" , Christoph Hellwig , "linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" List-Id: linux-rdma@vger.kernel.org On Thu, Mar 31, 2016 at 05:01:39PM -0700, Bart Van Assche wrote: > That patch is wrong because it makes the ib_srpt driver use I/O > contexts allocated by transport_alloc_session_tags() but it does > not initialize these I/O contexts properly. Did you have a chance to see which initializations are missing in this case? What is needed to do if we decide to fix original patch? Except these questions, the revert is fine :) Reviewed-by: Leon Romanovsky -- 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