diff options
author | Linus Torvalds <torvalds@linux-foundation.org> | 2013-05-03 12:57:16 -0400 |
---|---|---|
committer | Linus Torvalds <torvalds@linux-foundation.org> | 2013-05-03 12:57:16 -0400 |
commit | 86652188f345edec56b0074a65f6db17f16eb359 (patch) | |
tree | cbaf3a5e62b0c05ac74043685f63abde29a80c25 /fs/jfs | |
parent | 14a9e5c09d13aa36b88ea54b8b6134b60ff2a410 (diff) | |
parent | 73aaa22d5ffb2630456bac2f9a4ed9b81d0d7271 (diff) |
Merge tag 'jfs-3.10' of git://github.com/kleikamp/linux-shaggy
Pull jfs fixes from David Kleikamp:
"A couple fixes for jfs"
(What's with the unhelpful pull request "explanations" from fs people
today?)
* tag 'jfs-3.10' of git://github.com/kleikamp/linux-shaggy:
jfs: fix a couple races
jfs: avoid undefined behavior from left-shifting by 32 bits
Diffstat (limited to 'fs/jfs')
-rw-r--r-- | fs/jfs/inode.c | 2 | ||||
-rw-r--r-- | fs/jfs/jfs_imap.c | 2 | ||||
-rw-r--r-- | fs/jfs/jfs_logmgr.c | 3 |
3 files changed, 4 insertions, 3 deletions
diff --git a/fs/jfs/inode.c b/fs/jfs/inode.c index b7dc47ba675e..77554b61d124 100644 --- a/fs/jfs/inode.c +++ b/fs/jfs/inode.c | |||
@@ -125,7 +125,7 @@ int jfs_write_inode(struct inode *inode, struct writeback_control *wbc) | |||
125 | { | 125 | { |
126 | int wait = wbc->sync_mode == WB_SYNC_ALL; | 126 | int wait = wbc->sync_mode == WB_SYNC_ALL; |
127 | 127 | ||
128 | if (test_cflag(COMMIT_Nolink, inode)) | 128 | if (inode->i_nlink == 0) |
129 | return 0; | 129 | return 0; |
130 | /* | 130 | /* |
131 | * If COMMIT_DIRTY is not set, the inode isn't really dirty. | 131 | * If COMMIT_DIRTY is not set, the inode isn't really dirty. |
diff --git a/fs/jfs/jfs_imap.c b/fs/jfs/jfs_imap.c index 6ba4006e011b..f7e042b63ddb 100644 --- a/fs/jfs/jfs_imap.c +++ b/fs/jfs/jfs_imap.c | |||
@@ -1493,7 +1493,7 @@ int diAlloc(struct inode *pip, bool dir, struct inode *ip) | |||
1493 | /* mask any prior bits for the starting words of the | 1493 | /* mask any prior bits for the starting words of the |
1494 | * summary map. | 1494 | * summary map. |
1495 | */ | 1495 | */ |
1496 | mask = ONES << (EXTSPERSUM - bitno); | 1496 | mask = (bitno == 0) ? 0 : (ONES << (EXTSPERSUM - bitno)); |
1497 | inosmap = le32_to_cpu(iagp->inosmap[sword]) | mask; | 1497 | inosmap = le32_to_cpu(iagp->inosmap[sword]) | mask; |
1498 | extsmap = le32_to_cpu(iagp->extsmap[sword]) | mask; | 1498 | extsmap = le32_to_cpu(iagp->extsmap[sword]) | mask; |
1499 | 1499 | ||
diff --git a/fs/jfs/jfs_logmgr.c b/fs/jfs/jfs_logmgr.c index 2eb952c41a69..cbe48ea9318e 100644 --- a/fs/jfs/jfs_logmgr.c +++ b/fs/jfs/jfs_logmgr.c | |||
@@ -1058,7 +1058,8 @@ static int lmLogSync(struct jfs_log * log, int hard_sync) | |||
1058 | */ | 1058 | */ |
1059 | void jfs_syncpt(struct jfs_log *log, int hard_sync) | 1059 | void jfs_syncpt(struct jfs_log *log, int hard_sync) |
1060 | { LOG_LOCK(log); | 1060 | { LOG_LOCK(log); |
1061 | lmLogSync(log, hard_sync); | 1061 | if (!test_bit(log_QUIESCE, &log->flag)) |
1062 | lmLogSync(log, hard_sync); | ||
1062 | LOG_UNLOCK(log); | 1063 | LOG_UNLOCK(log); |
1063 | } | 1064 | } |
1064 | 1065 | ||