Subject: Re: [RFC v3 19/22] landlock: Add interrupted origin

From: Mickaël Salaün <[email-masked]>
Date: Thu, 15 Sep 2016 00:14:43 +0200

On 14/09/2016 20:29, Andy Lutomirski wrote:
> On Wed, Sep 14, 2016 at 12:24 AM, Mickaël Salaün <[email-masked]> wrote:
>> This third origin of hook call should cover all possible trigger paths
>> (e.g. page fault). Landlock eBPF programs can then take decisions
>> accordingly.
>>
>> Signed-off-by: Mickaël Salaün <[email-masked]>
>> Cc: Alexei Starovoitov <[email-masked]>
>> Cc: Andy Lutomirski <[email-masked]>
>> Cc: Daniel Borkmann <[email-masked]>
>> Cc: Kees Cook <[email-masked]>
>> ---
> 
> 
>>
>> +       if (unlikely(in_interrupt())) {
> 
> IMO security hooks have no business being called from interrupts.
> Aren't they all synchronous things done by tasks?  Interrupts are
> driver things.
> 
> Are you trying to check for page faults and such?

Yes, that was the idea you did put in my mind. Not sure how to deal with
this.