From mboxrd@z Thu Jan 1 00:00:00 1970 From: Yann Droneaud Subject: Re: [PATCH v2 0/4] Series short description Date: Fri, 28 Mar 2014 19:10:55 +0100 Message-ID: <1396030255.3297.82.camel@localhost.localdomain> References: <20140328172606.23646.35980.stgit@phlsvslse11.ph.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-Reply-To: <20140328172606.23646.35980.stgit-K+u1se/DcYrLESAwzcoQNrvm/XP+8Wra@public.gmane.org> Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Mike Marciniszyn Cc: roland-BHEL68pLQRGGvPXPguhicg@public.gmane.org, linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, Christoph Raisch , Hoang-Nam Nguyen , Bart Van Assche List-Id: linux-rdma@vger.kernel.org Le vendredi 28 mars 2014 =C3=A0 13:26 -0400, Mike Marciniszyn a =C3=A9c= rit : > This patch series modified the ib_sg_dma API to > eliminate the .dma_len and .dma_address methods. >=20 > In all present cases that overload these methods > (ipath, qib, ehca), the lack of these methods are > compensated for by code changes to the driver .map_sg > to insure that the vanilla sg_dma_address() and > sg_dma_len() will do the same thing as the equivalent > former ib_sg_dma_address() and ib_sg_dma_len() calls > into the drivers. >=20 > This patch series is a followup to this recent submission > http://marc.info/?l=3Dlinux-rdma&m=3D139602422108727&w=3D2 > and Bart's similar comment in > http://marc.info/?l=3Dlinux-netdev&m=3D135643746610259&w=3D2. >=20 > Roland, I'm not sure of the history of these methods and > what rationale caused them to be added? >=20 > There is obviously an interim step that could be done > here to preseve the overload by not changing verbs, yet > change qib/ipath/echa to not overload them. This would > allow the drivers to work even when the "correct" API's > had not been used by the ULP. >=20 > This version reorders the patch series order based > on comments from Yann Droneaud > so that the overload removal is last. >=20 > The commit messages are enhanced a bit as well. >=20 Thanks, that makes more sense now. Reviewed-by: Yann Droneaud Regards --=20 Yann Droneaud OPTEYA -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" i= n the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html