On Tue, Jul 19, 2016 at 02:11:13PM -0600, Jason Gunthorpe wrote: > On Tue, Jul 19, 2016 at 11:02:32PM +0300, Yuval Shaia wrote: > > > ok, i see the point but as Leon Romanovsky mentioned - this is not an issue > > with the patch itself, right? i.e. the original code relays on the fact > > that HW supports UD-CSUM and did not checked QP's sg limit (we even do not > > have any QP at this stage yet). > > ud-csum was only supported by one IB vendor and that vendor supports > SG. > > You are changing things to enable SG on hardware that has never had SG > enabled, so you need to make sure it works right.. My thoughts exactly. Ira -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html