From mboxrd@z Thu Jan 1 00:00:00 1970 From: Leon Romanovsky Subject: Re: [PATCH] i40iw: Set 128B as the only supported RQ WQE size Date: Wed, 21 Dec 2016 07:48:05 +0200 Message-ID: <20161221054805.GB1074@mtr-leonro.local> References: <20161219203227.86392-1-henry.orosco@intel.com> <20161220112417.GV1074@mtr-leonro.local> <748B799B6A00724488C603FD7E5E7EB94CBAEEA0@FMSMSX114.amr.corp.intel.com> <20161220134203.GW1074@mtr-leonro.local> <748B799B6A00724488C603FD7E5E7EB94CBAFBBC@FMSMSX114.amr.corp.intel.com> <20161220200239.GA1074@mtr-leonro.local> <748B799B6A00724488C603FD7E5E7EB94CBB12AF@FMSMSX114.amr.corp.intel.com> Mime-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="NxDiCysn+RCvdoZt" Return-path: Content-Disposition: inline In-Reply-To: <748B799B6A00724488C603FD7E5E7EB94CBB12AF-XfjTATA9Em864kNsxIetb7fspsVTdybXVpNB7YpNyf8@public.gmane.org> Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: "Tung, Chien Tin" Cc: "Orosco, Henry" , "dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org" , "linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org" , "e1000-rdma-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org" List-Id: linux-rdma@vger.kernel.org --NxDiCysn+RCvdoZt Content-Type: text/plain; charset=us-ascii Content-Disposition: inline On Wed, Dec 21, 2016 at 12:04:55AM +0000, Tung, Chien Tin wrote: > > > > -----Original Message----- > > From: Leon Romanovsky [mailto:leon-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org] > > Sent: Tuesday, December 20, 2016 2:03 PM > > To: Tung, Chien Tin > > Cc: Orosco, Henry ; dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org; linux- > > rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; e1000-rdma-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org > > Subject: Re: [PATCH] i40iw: Set 128B as the only supported RQ WQE size > > > > On Tue, Dec 20, 2016 at 02:46:30PM +0000, Tung, Chien Tin wrote: > > > > > > > > > > -----Original Message----- > > > > From: Leon Romanovsky [mailto:leon-DgEjT+Ai2ygdnm+yROfE0A@public.gmane.org] > > > > Sent: Tuesday, December 20, 2016 7:42 AM > > > > To: Tung, Chien Tin > > > > Cc: Orosco, Henry ; dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org; > > > > linux- rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; e1000-rdma-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org > > > > Subject: Re: [PATCH] i40iw: Set 128B as the only supported RQ WQE > > > > size > > > > > > > > On Tue, Dec 20, 2016 at 12:44:04PM +0000, Tung, Chien Tin wrote: > > > > > > > > > > > > > > > > -----Original Message----- > > > > > > From: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org [mailto:linux-rdma- > > > > > > owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org] On Behalf Of Leon Romanovsky > > > > > > Sent: Tuesday, December 20, 2016 5:24 AM > > > > > > To: Orosco, Henry > > > > > > Cc: dledford-H+wXaHxf7aLQT0dZR+AlfA@public.gmane.org; linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org; e1000- > > > > > > rdma-5NWGOfrQmneRv+LV9MX5uipxlwaOVQ5f@public.gmane.org; Tung, Chien Tin > > > > > > > > > > > > Subject: Re: [PATCH] i40iw: Set 128B as the only supported RQ > > > > > > WQE size > > > > > > > > > > > > > diff --git a/drivers/infiniband/hw/i40iw/i40iw_ucontext.h > > > > > > > b/drivers/infiniband/hw/i40iw/i40iw_ucontext.h > > > > > > > index 12acd68..57d3f1d 100644 > > > > > > > --- a/drivers/infiniband/hw/i40iw/i40iw_ucontext.h > > > > > > > +++ b/drivers/infiniband/hw/i40iw/i40iw_ucontext.h > > > > > > > @@ -39,8 +39,8 @@ > > > > > > > > > > > > > > #include > > > > > > > > > > > > > > -#define I40IW_ABI_USERSPACE_VER 4 > > > > > > > -#define I40IW_ABI_KERNEL_VER 4 > > > > > > > +#define I40IW_ABI_VER 5 > > > > > > > + > > > > > > > > > > > > Why did you remove defines and move to use constants "4" and "5" > > > > instead? > > > > > [Chien Tin Tung] This is the ABI version change, did you read the > > > > > commit > > > > message? Two defines were not necessary. > > > > > > > > > > > > > Thank you Chian Tin Tung for your informative answer and yes, I read > > > > commit message. > > > > > > > > If you think that these defines are not needed, so can you please > > > > remove the code which uses hardcoded "case 4" and "case 5"? > > > [Chien Tin Tung] Those are explicit checks against a particular ABI > > > version and it is clearer to use the numbers rather than some define. > > > > Can you please stop to add [..] in front of your response? This Outlook style is > > annoying and not needed. > > > > Regarding you response, you added the same check in two places and from > > grep/ctags/cscope/lxr perspective it is easier to spot them with a define. > > You have yet to provide a good argument for changing the 4 and 5 to defines, > so I don't know why you are telling me how to find them. Very puzzling. I'm not telling to you, but asking from you to provide maintainable code which will be read more than once. Right now, you blindly copy-pasted hardcoded values in three different places (2 kernel and 1 rdma-core). Feel free to ask your colleagues (Faisal L. and Shiraz S.) if they interested in maintainable driver code or they prefer your prone to errors copy-paste technique. > I already made my case on this. Enough said. > > Chien > > > > [Chien] > > > > > > > > > > > Or maybe they still needed and should be renamed? > > > [Chien Tin Tung] They are not needed. Before we were checking against > > > one version and one version only. Now we are making the provider > > > library and driver backward compatible. All we need is one define to > > > specify current ABI version. > > > > > > Chien > -- > 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 --NxDiCysn+RCvdoZt Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEEkhr/r4Op1/04yqaB5GN7iDZyWKcFAlhaF5UACgkQ5GN7iDZy WKefOxAAoYksZVM2qtMI1Vl7ZESxbbkVQXN8toqV5o6EFi7ZGIQJHRdPtOUznuUq RAk94umsSMC1LbHz+qM17EOAWC8lhpFIZwzLsjv+/MrcfqTiUX4UAV9BVgcrwEs2 yazEf+bhMpn1iqcgGVIu2cp4sc4WblA2bD3blAylA1fcUPvtYKI+2BgjTishmgtA rodQu6G4l/M56b6Zu/6gECmRlwPNoj80zpLbAp79D/d8LHktrEHYa4HmVkjIslNj tbY0t6wvnD/BR4NW8m52XJcbnbkGJUIJAtQIsnAng05QLBrqsjj3IHsFIMyMHRjS oVcOXE+u41dhK7OrOdg/94JeSTB86vrR2tLLa3xgnAdEXcWDPN6fkv2jLv9XjOmk Eq6XBpau2x6YgsuZfjg6kTIAhbazyv6vzVbgbu6J389ArNwkDNZ5U1mikk4zqj/y RMSH5ixLgOFE+4ilApS1uIIx5LYvM71zYG94rIxm5zUs9O35e1ngUiHLRldskWDk ZzSX9cuHaDMY8H2OZAnZiAJ3O2rPQ2OokOGEkcLpBmqTxU/Y3BTK3uyAF31nr8RF WI4gaMT2ZSakz3VxsWE1K/pYTgXh7RiPNlVhBNV/Dp3KYo5EWrDDRS0mvwtFnnKb U/juszoPewSkRIzv4C8I1TPiOLXSnneYGpWr9fLv9MbzUiqbdT0= =o3U8 -----END PGP SIGNATURE----- --NxDiCysn+RCvdoZt-- -- 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