On 05/04/2018 12:22 PM, Ram Pai wrote: > @@ -407,9 +414,6 @@ static bool pkey_access_permitted(int pkey, bool write, > bool execute) > int pkey_shift; > u64 amr; > > - if (!pkey) > - return true; > - > if (!is_pkey_enabled(pkey)) > return true;
Looks fine to me. Obviously doesn't have any impact on x86 or the generic code. One question, though. Which other check makes up for this removed !pkey check?