On Thu, Feb 24, 2022 at 9:29 AM Stafford Horne <shorne@xxxxxxxxx> wrote: > > - > > -#define access_ok(addr, size) \ > > -({ \ > > - __chk_user_ptr(addr); \ > > - __range_ok((unsigned long)(addr), (size)); \ > > -}) > > +#include <asm-generic/access_ok.h> > > I was going to ask why we are missing __chk_user_ptr in the generic version. > But this is basically now a no-op so I think its OK. Correct, the type checking is implied by making __access_ok() an inline function that takes a __user pointer. > Acked-by: Stafford Horne <shorne@xxxxxxxxx> [openrisc, asm-generic] Thanks! Arnd