From mboxrd@z Thu Jan 1 00:00:00 1970 From: Doug Ledford Subject: Re: [PATCH 2/2] IB/uverbs: clean up INIT_UDATA() macro usage Date: Wed, 27 Sep 2017 09:10:01 -0400 Message-ID: <1506517801.82712.8.camel@redhat.com> References: <20170906213521.1845893-1-arnd@arndb.de> <20170906213521.1845893-2-arnd@arndb.de> Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20170906213521.1845893-2-arnd-r2nGTMty4D4@public.gmane.org> Sender: linux-rdma-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org To: Arnd Bergmann , Sean Hefty , Hal Rosenstock Cc: Christoph Hellwig , Matan Barak , Yishai Hadas , Leon Romanovsky , Moses Reuben , linux-rdma-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: linux-rdma@vger.kernel.org On Wed, 2017-09-06 at 23:34 +0200, Arnd Bergmann wrote: > After changing INIT_UDATA_BUF_OR_NULL() to an inline function, > this does the same change to INIT_UDATA for consistency. > I'm keeping it separate as this part is much larger and > we wouldn't want to backport this to stable kernels if we > ever want to address the gcc warnings by backporting the > first patch. > > Again, using an inline function gives us better type > safety here among other issues with macros. I'm using > u64_to_user_ptr() to convert the user pointer to simplify > the logic rather than adding lots of new type casts. > > Signed-off-by: Arnd Bergmann Thanks, applied. -- Doug Ledford GPG KeyID: B826A3330E572FDD Key fingerprint = AE6B 1BDA 122B 23B4 265B 1274 B826 A333 0E57 2FDD -- 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