On Wed, Sep 4, 2019 at 12:51 PM Matthew Garrett <mjg59@xxxxxxxxxx> wrote: > On Fri, Aug 30, 2019 at 9:28 AM David Howells <dhowells@xxxxxxxxxx> wrote: > > > +static int lock_kernel_down(const char *where, enum lockdown_reason level) > > > > Is the last parameter the reason or the level? You're mixing the terms. > > Fair. Actually, on re-reading, I think this correct - this is setting the lockdown level, it's just that the lockdown level is an enum lockdown_reason for the sake of convenience.