On Mon, Jun 29, 2020 at 11:36 AM Christoph Hellwig <hch@xxxxxx> wrote: > > Having resurrect my work there really are tons of int cases. Which > makes me thing that splitting out a setsockopt_int method which gets > passed value instead of a pointer, then converting all the simple cases > to that first and then doing the real shit later sounds like a promіsing > idea. Try my hacky patch first, and just change the code that does if (get_user(val, (int __user *)optval)) { err = -EFAULT; to do val = *(int *)optval; In fact, that pattern seems to be so common that you can probably almost do it with a sed-script or something. Linus