On 06/01/2016 09:49 AM, Jonathan Corbet wrote: >> > + sys_pkey_alloc(0, PKEY_DISABLE_WRITE | PKEY_DISABLE_READ); >> > + sys_pkey_mprotect(ptr, size, PROT_READ|PROT_WRITE); >> > + something(ptr); > That should, IMO, be something like: > > key = pkey_alloc(...); > pkey_mprotect(ptr, size, PROT_READ|PROT_WRITE, key);
That is true as well. I'll fix that up as well. Thanks for taking a look!