lkml.org 
[lkml]   [2013]   [Jun]   [3]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [RFC PATCH 2/2] SELinux: cache inode checks inside struct inode
From
Date
On Tue, 2013-06-04 at 06:31 +0900, Linus Torvalds wrote:
>
>
> On Mon, 3 Jun 2013, Eric Paris wrote:
> >
> > #ifdef CONFIG_SECURITY
> > + seqcount_t i_security_seqcount;
> > + u32 i_last_task_sid;
> > + u32 i_last_granting;
> > + u32 i_last_perms;
> > + u32 i_audit_allow;
> > void *i_security;
> > #endif
>
> This is much too big. I was really hoping for "another word that the
> security layer can use" or similar.

Not sure how that can work :-(

> Something this big would be acceptable if it would be a *generic* security
> cache, and others could use it too, and would avoid ever actually calling
> into any security layer at all (ie we could do the checks entirely at the
> VFS layer). Then it would be fine. But for just the fact that SELinux is
> too slow? No.

There is nothing about it that can't be VFS-erized. The fields are:

readlockless way to get the data
which task was allowed
which perms were allowed
what generation of security policy allowed it
what perms should be forced to call security hook anyway

defining "perms" from a VFS PoV is hard.

doing any of this with 'stacking' is hard. Then again, I'm only so so
on the value of stacking. I've waffled a few times...

I can do it entirely inside selinux, but we are still going to have the
cache hit you were originally seeing as we dereference isec to get the
info....




\
 
 \ /
  Last update: 2013-06-04 01:41    [W:0.087 / U:0.200 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site