vfs: micro-optimize acl_permission_check()
authorLinus Torvalds <torvalds@linux-foundation.org>
Fri, 13 May 2011 18:51:01 +0000 (11:51 -0700)
committerLinus Torvalds <torvalds@linux-foundation.org>
Fri, 13 May 2011 18:51:01 +0000 (11:51 -0700)
It's a hot function, and we're better off not mixing types in the mask
calculations.  The compiler just ends up mixing 16-bit and 32-bit
operations, for no good reason.

So do everything in 'unsigned int' rather than mixing 'unsigned int'
masking with a 'umode_t' (16-bit) mode variable.

This, together with the parent commit (47a150edc2ae: "Cache user_ns in
struct cred") makes acl_permission_check() much nicer.

Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
fs/namei.c

index 54fc993e3027d4dff98dcf4f14ebbb4d817cdb0f..e3c4f112ebf754525de7d5634c298e9d4ba4fba8 100644 (file)
@@ -179,7 +179,7 @@ EXPORT_SYMBOL(putname);
 static int acl_permission_check(struct inode *inode, int mask, unsigned int flags,
                int (*check_acl)(struct inode *inode, int mask, unsigned int flags))
 {
-       umode_t                 mode = inode->i_mode;
+       unsigned int mode = inode->i_mode;
 
        mask &= MAY_READ | MAY_WRITE | MAY_EXEC;