lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Message-Id: <1341922673-14147-3-git-send-email-dedekind1@gmail.com> Date: Tue, 10 Jul 2012 15:17:51 +0300 From: Artem Bityutskiy <dedekind1@...il.com> To: Theodore Tso <tytso@....edu>, Jan Kara <jack@...e.cz> Cc: Linux FS Maling List <linux-fsdevel@...r.kernel.org>, Linux Kernel Maling List <linux-kernel@...r.kernel.org>, Ext4 Mailing List <linux-ext4@...r.kernel.org> Subject: [PATCHv5 2/4] ext4: Convert last user of ext4_mark_super_dirty() to ext4_handle_dirty_super() From: Jan Kara <jack@...e.cz> The last user of ext4_mark_super_dirty() in ext4_file_open() is so rare it can well be modifying the superblock properly by journalling the change. Change it and get rid of ext4_mark_super_dirty() as it's not needed anymore. Artem: small amendments. Artem: tested using xfstests for both journalled and non-journalled ext4. Signed-off-by: Jan Kara <jack@...e.cz> Tested-by: Artem Bityutskiy <artem.bityutskiy@...ux.intel.com> Signed-off-by: Artem Bityutskiy <artem.bityutskiy@...ux.intel.com> --- fs/ext4/ext4.h | 9 --------- fs/ext4/file.c | 14 +++++++++++++- 2 files changed, 13 insertions(+), 10 deletions(-) diff --git a/fs/ext4/ext4.h b/fs/ext4/ext4.h index cfc4e01..0c4042e 100644 --- a/fs/ext4/ext4.h +++ b/fs/ext4/ext4.h @@ -2321,15 +2321,6 @@ static inline void ext4_unlock_group(struct super_block *sb, spin_unlock(ext4_group_lock_ptr(sb, group)); } -static inline void ext4_mark_super_dirty(struct super_block *sb) -{ - struct ext4_super_block *es = EXT4_SB(sb)->s_es; - - ext4_superblock_csum_set(sb, es); - if (EXT4_SB(sb)->s_journal == NULL) - sb->s_dirt =1; -} - /* * Block validity checking */ diff --git a/fs/ext4/file.c b/fs/ext4/file.c index 8c7642a..3a77494 100644 --- a/fs/ext4/file.c +++ b/fs/ext4/file.c @@ -181,9 +181,21 @@ static int ext4_file_open(struct inode * inode, struct file * filp) path.dentry = mnt->mnt_root; cp = d_path(&path, buf, sizeof(buf)); if (!IS_ERR(cp)) { + handle_t *handle; + int err; + + handle = ext4_journal_start_sb(sb, 1); + if (IS_ERR(handle)) + return PTR_ERR(handle); + err = ext4_journal_get_write_access(handle, sbi->s_sbh); + if (err) { + ext4_journal_stop(handle); + return err; + } strlcpy(sbi->s_es->s_last_mounted, cp, sizeof(sbi->s_es->s_last_mounted)); - ext4_mark_super_dirty(sb); + ext4_handle_dirty_super(handle, sb); + ext4_journal_stop(handle); } } /* -- 1.7.7.6 -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists