On Mon, Dec 13, 2021 at 07:18:57PM +0000, William Kucharski wrote: > I like these, but a quick question: > > Since the usercopy_abort() calls are all because the offset exceeds the page > size, is there a reason why you don't specifically state that via the detail > parameter rather than just supply a NULL pointer? Hmm ... I'd defer to Kees on this, because I'm not familiar with usercopy_abort() usage, but the only places which use the detail parameter today are slab/slub, which use it to pass the name of the slab. I think the user is supposed to infer that we overran the end of the page based on the offset & length values. > Otherwise for the patch series: > > Reviewed-by: William Kucharski <william.kucharski@xxxxxxxxxx> Thanks!
