diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2012-04-23 22:52:00 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2012-04-23 22:52:00 -0400 |
commit | 95f714727436836bb46236ce2bcd8ee8f9274aed (patch) | |
tree | 200ea790bf3299c9120b65b890ac85f56337d848 /fs | |
parent | ce1ee9f5b99fa4ba332692194aa57b604112762f (diff) | |
parent | db7e5c668e1b16061fe2d94d3cba022dd360c5d4 (diff) |
Merge tag 'ext4_for_linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tytso/ext4
Pull ext4 bug fixes from Ted Ts'o:
"These are two low-risk bug fixes for ext4, fixing a compile warning
and a potential deadlock."
* tag 'ext4_for_linus' of git://git.kernel.org/pub/scm/linux/kernel/git/tytso/ext4:
super.c: unused variable warning without CONFIG_QUOTA
jbd2: use GFP_NOFS for blkdev_issue_flush
Diffstat (limited to 'fs')
-rw-r--r-- | fs/ext4/super.c | 2 | ||||
-rw-r--r-- | fs/jbd2/commit.c | 4 |
2 files changed, 4 insertions, 2 deletions
diff --git a/fs/ext4/super.c b/fs/ext4/super.c index 6da193564e43..e1fb1d5de58e 100644 --- a/fs/ext4/super.c +++ b/fs/ext4/super.c | |||
@@ -1597,7 +1597,9 @@ static int parse_options(char *options, struct super_block *sb, | |||
1597 | unsigned int *journal_ioprio, | 1597 | unsigned int *journal_ioprio, |
1598 | int is_remount) | 1598 | int is_remount) |
1599 | { | 1599 | { |
1600 | #ifdef CONFIG_QUOTA | ||
1600 | struct ext4_sb_info *sbi = EXT4_SB(sb); | 1601 | struct ext4_sb_info *sbi = EXT4_SB(sb); |
1602 | #endif | ||
1601 | char *p; | 1603 | char *p; |
1602 | substring_t args[MAX_OPT_ARGS]; | 1604 | substring_t args[MAX_OPT_ARGS]; |
1603 | int token; | 1605 | int token; |
diff --git a/fs/jbd2/commit.c b/fs/jbd2/commit.c index 806525a7269c..840f70f50792 100644 --- a/fs/jbd2/commit.c +++ b/fs/jbd2/commit.c | |||
@@ -723,7 +723,7 @@ start_journal_io: | |||
723 | if (commit_transaction->t_need_data_flush && | 723 | if (commit_transaction->t_need_data_flush && |
724 | (journal->j_fs_dev != journal->j_dev) && | 724 | (journal->j_fs_dev != journal->j_dev) && |
725 | (journal->j_flags & JBD2_BARRIER)) | 725 | (journal->j_flags & JBD2_BARRIER)) |
726 | blkdev_issue_flush(journal->j_fs_dev, GFP_KERNEL, NULL); | 726 | blkdev_issue_flush(journal->j_fs_dev, GFP_NOFS, NULL); |
727 | 727 | ||
728 | /* Done it all: now write the commit record asynchronously. */ | 728 | /* Done it all: now write the commit record asynchronously. */ |
729 | if (JBD2_HAS_INCOMPAT_FEATURE(journal, | 729 | if (JBD2_HAS_INCOMPAT_FEATURE(journal, |
@@ -859,7 +859,7 @@ wait_for_iobuf: | |||
859 | if (JBD2_HAS_INCOMPAT_FEATURE(journal, | 859 | if (JBD2_HAS_INCOMPAT_FEATURE(journal, |
860 | JBD2_FEATURE_INCOMPAT_ASYNC_COMMIT) && | 860 | JBD2_FEATURE_INCOMPAT_ASYNC_COMMIT) && |
861 | journal->j_flags & JBD2_BARRIER) { | 861 | journal->j_flags & JBD2_BARRIER) { |
862 | blkdev_issue_flush(journal->j_dev, GFP_KERNEL, NULL); | 862 | blkdev_issue_flush(journal->j_dev, GFP_NOFS, NULL); |
863 | } | 863 | } |
864 | 864 | ||
865 | if (err) | 865 | if (err) |