diff options
| author | Michael Halcrow <mhalcrow@us.ibm.com> | 2007-10-16 04:28:00 -0400 |
|---|---|---|
| committer | Linus Torvalds <torvalds@woody.linux-foundation.org> | 2007-10-16 12:43:11 -0400 |
| commit | af655dc6a9d5028d89aa35b65781631451aadc19 (patch) | |
| tree | aab30213366f9077b44fc09387a61bb140ca8240 | |
| parent | c59c2eb53f002de96f48dbc51e4f3bd946cf2458 (diff) | |
eCryptfs: collapse flag set into one statement
Andrew Morton wrote:
> > + crypt_stat->flags |= ECRYPTFS_ENCRYPTED;
> > + crypt_stat->flags |= ECRYPTFS_KEY_VALID;
>
> Maybe the compiler can optimise those two statements, but we'd
> normally provide it with some manual help.
This patch provides the compiler with some manual help for
optimizing the setting of some flags.
Signed-off-by: Michael Halcrow <mhalcrow@us.ibm.com>
Signed-off-by: Andrew Morton <akpm@linux-foundation.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
| -rw-r--r-- | fs/ecryptfs/crypto.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/fs/ecryptfs/crypto.c b/fs/ecryptfs/crypto.c index 14cc1f57e7b0..76bba7384d51 100644 --- a/fs/ecryptfs/crypto.c +++ b/fs/ecryptfs/crypto.c | |||
| @@ -1029,8 +1029,7 @@ int ecryptfs_new_file_context(struct dentry *ecryptfs_dentry) | |||
| 1029 | int rc = 0; | 1029 | int rc = 0; |
| 1030 | 1030 | ||
| 1031 | ecryptfs_set_default_crypt_stat_vals(crypt_stat, mount_crypt_stat); | 1031 | ecryptfs_set_default_crypt_stat_vals(crypt_stat, mount_crypt_stat); |
| 1032 | crypt_stat->flags |= ECRYPTFS_ENCRYPTED; | 1032 | crypt_stat->flags |= (ECRYPTFS_ENCRYPTED | ECRYPTFS_KEY_VALID); |
| 1033 | crypt_stat->flags |= ECRYPTFS_KEY_VALID; | ||
| 1034 | ecryptfs_copy_mount_wide_flags_to_inode_flags(crypt_stat, | 1033 | ecryptfs_copy_mount_wide_flags_to_inode_flags(crypt_stat, |
| 1035 | mount_crypt_stat); | 1034 | mount_crypt_stat); |
| 1036 | rc = ecryptfs_copy_mount_wide_sigs_to_inode_sigs(crypt_stat, | 1035 | rc = ecryptfs_copy_mount_wide_sigs_to_inode_sigs(crypt_stat, |