* [PATCH 0/2] Two reiserfs fixes @ 2014-08-06 18:03 Jan Kara 2014-08-06 18:03 ` [PATCH 1/2] reiserfs: Avoid warning from unlock_new_inode() Jan Kara 2014-08-06 18:04 ` [PATCH 2/2] reiserfs: Fix use after free in journal teardown Jan Kara 0 siblings, 2 replies; 5+ messages in thread From: Jan Kara @ 2014-08-06 18:03 UTC (permalink / raw) To: reiserfs-devel; +Cc: jeffm Hello, when testing some reiserfs patches for this merge window I've come across two bugs (the first one mostly annoying, the second one was crashing my testing machine) which are fixed by the patches in this series. I plan to push these patches to Linus tomorrow but any review is certainly welcome. Thanks! Honza ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 1/2] reiserfs: Avoid warning from unlock_new_inode() 2014-08-06 18:03 [PATCH 0/2] Two reiserfs fixes Jan Kara @ 2014-08-06 18:03 ` Jan Kara 2014-08-11 19:25 ` Jeff Mahoney 2014-08-06 18:04 ` [PATCH 2/2] reiserfs: Fix use after free in journal teardown Jan Kara 1 sibling, 1 reply; 5+ messages in thread From: Jan Kara @ 2014-08-06 18:03 UTC (permalink / raw) To: reiserfs-devel; +Cc: jeffm, Jan Kara xfstest run for reiserfs produces lots of warnings like: WARNING: CPU: 4 PID: 24572 at fs/inode.c:937 unlock_new_inode+0x76/0x80() because reiserfs uses new_inode() to allocate new inodes and that doesn't set I_NEW in i_state. This seems like it could cause subtle bugs because half-initialized inodes may be visible in superblock inode list or inode hashes. So make sure inode has I_NEW set before it's visible anywhere which also gets rid of the warning. Signed-off-by: Jan Kara <jack@suse.cz> --- fs/reiserfs/namei.c | 34 ++++++++++++++++++---------------- 1 file changed, 18 insertions(+), 16 deletions(-) diff --git a/fs/reiserfs/namei.c b/fs/reiserfs/namei.c index cd11358b10c7..c4f435c4e6fc 100644 --- a/fs/reiserfs/namei.c +++ b/fs/reiserfs/namei.c @@ -580,8 +580,8 @@ static int reiserfs_add_entry(struct reiserfs_transaction_handle *th, } /* - * quota utility function, call if you've had to abort after calling - * new_inode_init, and have not called reiserfs_new_inode yet. + * Utility function to call if you've had to abort after calling + * get_new_inode, and have not called reiserfs_new_inode yet. * This should only be called on inodes that do not have stat data * inserted into the tree yet. */ @@ -590,6 +590,7 @@ static int drop_new_inode(struct inode *inode) dquot_drop(inode); make_bad_inode(inode); inode->i_flags |= S_NOQUOTA; + unlock_new_inode(inode); iput(inode); return 0; } @@ -600,8 +601,16 @@ static int drop_new_inode(struct inode *inode) * outside of a transaction, so we had to pull some bits of * reiserfs_new_inode out into this func. */ -static int new_inode_init(struct inode *inode, struct inode *dir, umode_t mode) +static struct inode *get_new_inode(struct inode *dir, umode_t mode) { + struct inode *inode = new_inode_pseudo(dir->i_sb); + + if (!inode) + return NULL; + /* Make sure inode is invisible until it's fully set up */ + inode->i_state |= I_NEW; + inode_sb_list_add(inode); + /* * Make inode invalid - just in case we are going to drop it before * the initialization happens @@ -614,7 +623,8 @@ static int new_inode_init(struct inode *inode, struct inode *dir, umode_t mode) */ inode_init_owner(inode, dir, mode); dquot_initialize(inode); - return 0; + + return inode; } static int reiserfs_create(struct inode *dir, struct dentry *dentry, umode_t mode, @@ -635,10 +645,8 @@ static int reiserfs_create(struct inode *dir, struct dentry *dentry, umode_t mod dquot_initialize(dir); - if (!(inode = new_inode(dir->i_sb))) { + if (!(inode = get_new_inode(dir, mode))) return -ENOMEM; - } - new_inode_init(inode, dir, mode); jbegin_count += reiserfs_cache_default_acl(dir); retval = reiserfs_security_init(dir, inode, &dentry->d_name, &security); @@ -712,10 +720,8 @@ static int reiserfs_mknod(struct inode *dir, struct dentry *dentry, umode_t mode dquot_initialize(dir); - if (!(inode = new_inode(dir->i_sb))) { + if (!(inode = get_new_inode(dir, mode))) return -ENOMEM; - } - new_inode_init(inode, dir, mode); jbegin_count += reiserfs_cache_default_acl(dir); retval = reiserfs_security_init(dir, inode, &dentry->d_name, &security); @@ -797,10 +803,8 @@ static int reiserfs_mkdir(struct inode *dir, struct dentry *dentry, umode_t mode REISERFS_I(dir)->new_packing_locality = 1; #endif mode = S_IFDIR | mode; - if (!(inode = new_inode(dir->i_sb))) { + if (!(inode = get_new_inode(dir, mode))) return -ENOMEM; - } - new_inode_init(inode, dir, mode); jbegin_count += reiserfs_cache_default_acl(dir); retval = reiserfs_security_init(dir, inode, &dentry->d_name, &security); @@ -1097,10 +1101,8 @@ static int reiserfs_symlink(struct inode *parent_dir, dquot_initialize(parent_dir); - if (!(inode = new_inode(parent_dir->i_sb))) { + if (!(inode = get_new_inode(parent_dir, mode))) return -ENOMEM; - } - new_inode_init(inode, parent_dir, mode); retval = reiserfs_security_init(parent_dir, inode, &dentry->d_name, &security); -- 1.8.1.4 ^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [PATCH 1/2] reiserfs: Avoid warning from unlock_new_inode() 2014-08-06 18:03 ` [PATCH 1/2] reiserfs: Avoid warning from unlock_new_inode() Jan Kara @ 2014-08-11 19:25 ` Jeff Mahoney 2014-08-12 10:52 ` Jan Kara 0 siblings, 1 reply; 5+ messages in thread From: Jeff Mahoney @ 2014-08-11 19:25 UTC (permalink / raw) To: Jan Kara, reiserfs-devel -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On 8/6/14, 2:03 PM, Jan Kara wrote: > xfstest run for reiserfs produces lots of warnings like: > > WARNING: CPU: 4 PID: 24572 at fs/inode.c:937 > unlock_new_inode+0x76/0x80() > > because reiserfs uses new_inode() to allocate new inodes and that > doesn't set I_NEW in i_state. This seems like it could cause subtle > bugs because half-initialized inodes may be visible in superblock > inode list or inode hashes. So make sure inode has I_NEW set before > it's visible anywhere which also gets rid of the warning. > > Signed-off-by: Jan Kara <jack@suse.cz> The problem is that reiserfs_new_inode drops down to the error handling case without having called insert_inode_locked4. I have a patch that just moves the quota check after the insertion so we don't need to do gross things like if (!inode_unhashed(inode)) unlock_new_inode(inode) in the error path. It's consistent with how ext4 does it. - -Jeff > --- fs/reiserfs/namei.c | 34 ++++++++++++++++++---------------- 1 > file changed, 18 insertions(+), 16 deletions(-) > > diff --git a/fs/reiserfs/namei.c b/fs/reiserfs/namei.c index > cd11358b10c7..c4f435c4e6fc 100644 --- a/fs/reiserfs/namei.c +++ > b/fs/reiserfs/namei.c @@ -580,8 +580,8 @@ static int > reiserfs_add_entry(struct reiserfs_transaction_handle *th, } > > /* - * quota utility function, call if you've had to abort after > calling - * new_inode_init, and have not called reiserfs_new_inode > yet. + * Utility function to call if you've had to abort after > calling + * get_new_inode, and have not called reiserfs_new_inode > yet. * This should only be called on inodes that do not have stat > data * inserted into the tree yet. */ @@ -590,6 +590,7 @@ static > int drop_new_inode(struct inode *inode) dquot_drop(inode); > make_bad_inode(inode); inode->i_flags |= S_NOQUOTA; + > unlock_new_inode(inode); iput(inode); return 0; } @@ -600,8 +601,16 > @@ static int drop_new_inode(struct inode *inode) * outside of a > transaction, so we had to pull some bits of * reiserfs_new_inode > out into this func. */ -static int new_inode_init(struct inode > *inode, struct inode *dir, umode_t mode) +static struct inode > *get_new_inode(struct inode *dir, umode_t mode) { + struct inode > *inode = new_inode_pseudo(dir->i_sb); + + if (!inode) + return > NULL; + /* Make sure inode is invisible until it's fully set up */ > + inode->i_state |= I_NEW; + inode_sb_list_add(inode); + /* * Make > inode invalid - just in case we are going to drop it before * the > initialization happens @@ -614,7 +623,8 @@ static int > new_inode_init(struct inode *inode, struct inode *dir, umode_t > mode) */ inode_init_owner(inode, dir, mode); > dquot_initialize(inode); - return 0; + + return inode; } > > static int reiserfs_create(struct inode *dir, struct dentry > *dentry, umode_t mode, @@ -635,10 +645,8 @@ static int > reiserfs_create(struct inode *dir, struct dentry *dentry, umode_t > mod > > dquot_initialize(dir); > > - if (!(inode = new_inode(dir->i_sb))) { + if (!(inode = > get_new_inode(dir, mode))) return -ENOMEM; - } - > new_inode_init(inode, dir, mode); > > jbegin_count += reiserfs_cache_default_acl(dir); retval = > reiserfs_security_init(dir, inode, &dentry->d_name, &security); @@ > -712,10 +720,8 @@ static int reiserfs_mknod(struct inode *dir, > struct dentry *dentry, umode_t mode > > dquot_initialize(dir); > > - if (!(inode = new_inode(dir->i_sb))) { + if (!(inode = > get_new_inode(dir, mode))) return -ENOMEM; - } - > new_inode_init(inode, dir, mode); > > jbegin_count += reiserfs_cache_default_acl(dir); retval = > reiserfs_security_init(dir, inode, &dentry->d_name, &security); @@ > -797,10 +803,8 @@ static int reiserfs_mkdir(struct inode *dir, > struct dentry *dentry, umode_t mode > REISERFS_I(dir)->new_packing_locality = 1; #endif mode = S_IFDIR | > mode; - if (!(inode = new_inode(dir->i_sb))) { + if (!(inode = > get_new_inode(dir, mode))) return -ENOMEM; - } - > new_inode_init(inode, dir, mode); > > jbegin_count += reiserfs_cache_default_acl(dir); retval = > reiserfs_security_init(dir, inode, &dentry->d_name, &security); @@ > -1097,10 +1101,8 @@ static int reiserfs_symlink(struct inode > *parent_dir, > > dquot_initialize(parent_dir); > > - if (!(inode = new_inode(parent_dir->i_sb))) { + if (!(inode = > get_new_inode(parent_dir, mode))) return -ENOMEM; - } - > new_inode_init(inode, parent_dir, mode); > > retval = reiserfs_security_init(parent_dir, inode, > &dentry->d_name, &security); > - -- Jeff Mahoney SUSE Labs -----BEGIN PGP SIGNATURE----- Version: GnuPG/MacGPG2 v2.0.19 (Darwin) iQIcBAEBAgAGBQJT6RibAAoJEB57S2MheeWyQN8P/A9BE9Xbp0tSwBi1TklqE3Hm xKK1IHgdVPODpysKKbr6V2llVOl+Fis/1kT7SKyVZ5LraVhrfQU/37HzO0535ITR n1bBiQV/PgejEtkSqKDMispk0kGaVCCgg2o6scHg3Qr8w9qF2mmrTUUOQPcMsaEa YKmd7sN+3+RbvP7mHderAM4Yjb+EXzjlgfqyOZYg+jY8Id+b5Yn5TekMQJMvvsPh tWiGaDF9cYsVZL2fdNUvOXdUMycYx2xDTzcCNBOEHh9FHS3kIVrCtrwfXQJEys5a nB1Es94tAW2M3Au/8L3kugcy8iR7gxqGFctCtfmtUpm7SiusNDIfXyNpbRtgdbF5 QFlqmNF2mLDYKNr93SbMLbF+fnl1NzkoFmrsu17bWiiQaxmqNDsnZeNUANWSrXsE QXS2Ardd24+BOW2aiSQXXwSkaHeQy3h/H9gX+TqbpUBdsRJO9bggm7IDFNn7n4ow mM0jRlu5tzyUHQRjzkwqqAqjBTB9l+rEV5zwikNE9QzWjovlKZ+NltW3mkMtYKKX P/wQs49zKT8g+InLiLFXanYrZkMpQ/HgQGw3rRMd5o5P2I0kih3Flaa2Gwlf4P7n 6xcv+STczVUg/htRUIMppQVIkriOD92QVmKAzmIsPBXIPTpJNVjrknc0e8MDDfql 8j0moTvJ8CloSg25k0W5 =xfQ5 -----END PGP SIGNATURE----- ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH 1/2] reiserfs: Avoid warning from unlock_new_inode() 2014-08-11 19:25 ` Jeff Mahoney @ 2014-08-12 10:52 ` Jan Kara 0 siblings, 0 replies; 5+ messages in thread From: Jan Kara @ 2014-08-12 10:52 UTC (permalink / raw) To: Jeff Mahoney; +Cc: Jan Kara, reiserfs-devel On Mon 11-08-14 15:25:15, Jeff Mahoney wrote: > -----BEGIN PGP SIGNED MESSAGE----- > Hash: SHA1 > > On 8/6/14, 2:03 PM, Jan Kara wrote: > > xfstest run for reiserfs produces lots of warnings like: > > > > WARNING: CPU: 4 PID: 24572 at fs/inode.c:937 > > unlock_new_inode+0x76/0x80() > > > > because reiserfs uses new_inode() to allocate new inodes and that > > doesn't set I_NEW in i_state. This seems like it could cause subtle > > bugs because half-initialized inodes may be visible in superblock > > inode list or inode hashes. So make sure inode has I_NEW set before > > it's visible anywhere which also gets rid of the warning. > > > > Signed-off-by: Jan Kara <jack@suse.cz> > > The problem is that reiserfs_new_inode drops down to the error > handling case without having called insert_inode_locked4. I have a > patch that just moves the quota check after the insertion so we don't > need to do gross things like if (!inode_unhashed(inode)) > unlock_new_inode(inode) in the error path. It's consistent with how > ext4 does it. OK, thanks. I've removed this patch from my tree, asked Linus not to pull the original request and will send him a new one after some testing. Please send me your fix when you have it ready so that I can include it in pull request for rc2/rc3. Thanks! Honza > > - -Jeff > > > > --- fs/reiserfs/namei.c | 34 ++++++++++++++++++---------------- 1 > > file changed, 18 insertions(+), 16 deletions(-) > > > > diff --git a/fs/reiserfs/namei.c b/fs/reiserfs/namei.c index > > cd11358b10c7..c4f435c4e6fc 100644 --- a/fs/reiserfs/namei.c +++ > > b/fs/reiserfs/namei.c @@ -580,8 +580,8 @@ static int > > reiserfs_add_entry(struct reiserfs_transaction_handle *th, } > > > > /* - * quota utility function, call if you've had to abort after > > calling - * new_inode_init, and have not called reiserfs_new_inode > > yet. + * Utility function to call if you've had to abort after > > calling + * get_new_inode, and have not called reiserfs_new_inode > > yet. * This should only be called on inodes that do not have stat > > data * inserted into the tree yet. */ @@ -590,6 +590,7 @@ static > > int drop_new_inode(struct inode *inode) dquot_drop(inode); > > make_bad_inode(inode); inode->i_flags |= S_NOQUOTA; + > > unlock_new_inode(inode); iput(inode); return 0; } @@ -600,8 +601,16 > > @@ static int drop_new_inode(struct inode *inode) * outside of a > > transaction, so we had to pull some bits of * reiserfs_new_inode > > out into this func. */ -static int new_inode_init(struct inode > > *inode, struct inode *dir, umode_t mode) +static struct inode > > *get_new_inode(struct inode *dir, umode_t mode) { + struct inode > > *inode = new_inode_pseudo(dir->i_sb); + + if (!inode) + return > > NULL; + /* Make sure inode is invisible until it's fully set up */ > > + inode->i_state |= I_NEW; + inode_sb_list_add(inode); + /* * Make > > inode invalid - just in case we are going to drop it before * the > > initialization happens @@ -614,7 +623,8 @@ static int > > new_inode_init(struct inode *inode, struct inode *dir, umode_t > > mode) */ inode_init_owner(inode, dir, mode); > > dquot_initialize(inode); - return 0; + + return inode; } > > > > static int reiserfs_create(struct inode *dir, struct dentry > > *dentry, umode_t mode, @@ -635,10 +645,8 @@ static int > > reiserfs_create(struct inode *dir, struct dentry *dentry, umode_t > > mod > > > > dquot_initialize(dir); > > > > - if (!(inode = new_inode(dir->i_sb))) { + if (!(inode = > > get_new_inode(dir, mode))) return -ENOMEM; - } - > > new_inode_init(inode, dir, mode); > > > > jbegin_count += reiserfs_cache_default_acl(dir); retval = > > reiserfs_security_init(dir, inode, &dentry->d_name, &security); @@ > > -712,10 +720,8 @@ static int reiserfs_mknod(struct inode *dir, > > struct dentry *dentry, umode_t mode > > > > dquot_initialize(dir); > > > > - if (!(inode = new_inode(dir->i_sb))) { + if (!(inode = > > get_new_inode(dir, mode))) return -ENOMEM; - } - > > new_inode_init(inode, dir, mode); > > > > jbegin_count += reiserfs_cache_default_acl(dir); retval = > > reiserfs_security_init(dir, inode, &dentry->d_name, &security); @@ > > -797,10 +803,8 @@ static int reiserfs_mkdir(struct inode *dir, > > struct dentry *dentry, umode_t mode > > REISERFS_I(dir)->new_packing_locality = 1; #endif mode = S_IFDIR | > > mode; - if (!(inode = new_inode(dir->i_sb))) { + if (!(inode = > > get_new_inode(dir, mode))) return -ENOMEM; - } - > > new_inode_init(inode, dir, mode); > > > > jbegin_count += reiserfs_cache_default_acl(dir); retval = > > reiserfs_security_init(dir, inode, &dentry->d_name, &security); @@ > > -1097,10 +1101,8 @@ static int reiserfs_symlink(struct inode > > *parent_dir, > > > > dquot_initialize(parent_dir); > > > > - if (!(inode = new_inode(parent_dir->i_sb))) { + if (!(inode = > > get_new_inode(parent_dir, mode))) return -ENOMEM; - } - > > new_inode_init(inode, parent_dir, mode); > > > > retval = reiserfs_security_init(parent_dir, inode, > > &dentry->d_name, &security); > > > > > - -- > Jeff Mahoney > SUSE Labs > -----BEGIN PGP SIGNATURE----- > Version: GnuPG/MacGPG2 v2.0.19 (Darwin) > > iQIcBAEBAgAGBQJT6RibAAoJEB57S2MheeWyQN8P/A9BE9Xbp0tSwBi1TklqE3Hm > xKK1IHgdVPODpysKKbr6V2llVOl+Fis/1kT7SKyVZ5LraVhrfQU/37HzO0535ITR > n1bBiQV/PgejEtkSqKDMispk0kGaVCCgg2o6scHg3Qr8w9qF2mmrTUUOQPcMsaEa > YKmd7sN+3+RbvP7mHderAM4Yjb+EXzjlgfqyOZYg+jY8Id+b5Yn5TekMQJMvvsPh > tWiGaDF9cYsVZL2fdNUvOXdUMycYx2xDTzcCNBOEHh9FHS3kIVrCtrwfXQJEys5a > nB1Es94tAW2M3Au/8L3kugcy8iR7gxqGFctCtfmtUpm7SiusNDIfXyNpbRtgdbF5 > QFlqmNF2mLDYKNr93SbMLbF+fnl1NzkoFmrsu17bWiiQaxmqNDsnZeNUANWSrXsE > QXS2Ardd24+BOW2aiSQXXwSkaHeQy3h/H9gX+TqbpUBdsRJO9bggm7IDFNn7n4ow > mM0jRlu5tzyUHQRjzkwqqAqjBTB9l+rEV5zwikNE9QzWjovlKZ+NltW3mkMtYKKX > P/wQs49zKT8g+InLiLFXanYrZkMpQ/HgQGw3rRMd5o5P2I0kih3Flaa2Gwlf4P7n > 6xcv+STczVUg/htRUIMppQVIkriOD92QVmKAzmIsPBXIPTpJNVjrknc0e8MDDfql > 8j0moTvJ8CloSg25k0W5 > =xfQ5 > -----END PGP SIGNATURE----- -- Jan Kara <jack@suse.cz> SUSE Labs, CR ^ permalink raw reply [flat|nested] 5+ messages in thread
* [PATCH 2/2] reiserfs: Fix use after free in journal teardown 2014-08-06 18:03 [PATCH 0/2] Two reiserfs fixes Jan Kara 2014-08-06 18:03 ` [PATCH 1/2] reiserfs: Avoid warning from unlock_new_inode() Jan Kara @ 2014-08-06 18:04 ` Jan Kara 1 sibling, 0 replies; 5+ messages in thread From: Jan Kara @ 2014-08-06 18:04 UTC (permalink / raw) To: reiserfs-devel; +Cc: jeffm, Jan Kara, stable If do_journal_release() races with do_journal_end() which requeues delayed works for transaction flushing, we can leave work items for flushing outstanding transactions queued while freeing them. That results in use after free and possible crash in run_timers_softirq(). Fix the problem by not requeueing works if superblock is being shut down (MS_ACTIVE not set) and using cancel_delayed_work_sync() in do_journal_release(). CC: stable@vger.kernel.org Signed-off-by: Jan Kara <jack@suse.cz> --- fs/reiserfs/journal.c | 22 ++++++++++++++++------ fs/reiserfs/super.c | 6 +++++- 2 files changed, 21 insertions(+), 7 deletions(-) diff --git a/fs/reiserfs/journal.c b/fs/reiserfs/journal.c index e8870de4627e..a88b1b3e7db3 100644 --- a/fs/reiserfs/journal.c +++ b/fs/reiserfs/journal.c @@ -1947,8 +1947,6 @@ static int do_journal_release(struct reiserfs_transaction_handle *th, } } - /* wait for all commits to finish */ - cancel_delayed_work(&SB_JOURNAL(sb)->j_work); /* * We must release the write lock here because @@ -1956,8 +1954,14 @@ static int do_journal_release(struct reiserfs_transaction_handle *th, */ reiserfs_write_unlock(sb); + /* + * Cancel flushing of old commits. Note that neither of these works + * will be requeued because superblock is being shutdown and doesn't + * have MS_ACTIVE set. + */ cancel_delayed_work_sync(&REISERFS_SB(sb)->old_work); - flush_workqueue(REISERFS_SB(sb)->commit_wq); + /* wait for all commits to finish */ + cancel_delayed_work_sync(&SB_JOURNAL(sb)->j_work); free_journal_ram(sb); @@ -4292,9 +4296,15 @@ static int do_journal_end(struct reiserfs_transaction_handle *th, int flags) if (flush) { flush_commit_list(sb, jl, 1); flush_journal_list(sb, jl, 1); - } else if (!(jl->j_state & LIST_COMMIT_PENDING)) - queue_delayed_work(REISERFS_SB(sb)->commit_wq, - &journal->j_work, HZ / 10); + } else if (!(jl->j_state & LIST_COMMIT_PENDING)) { + /* + * Avoid queueing work when sb is being shut down. Transaction + * will be flushed on journal shutdown. + */ + if (sb->s_flags & MS_ACTIVE) + queue_delayed_work(REISERFS_SB(sb)->commit_wq, + &journal->j_work, HZ / 10); + } /* * if the next transaction has any chance of wrapping, flush diff --git a/fs/reiserfs/super.c b/fs/reiserfs/super.c index a392cef6acc6..5fd8f57e07fc 100644 --- a/fs/reiserfs/super.c +++ b/fs/reiserfs/super.c @@ -100,7 +100,11 @@ void reiserfs_schedule_old_flush(struct super_block *s) struct reiserfs_sb_info *sbi = REISERFS_SB(s); unsigned long delay; - if (s->s_flags & MS_RDONLY) + /* + * Avoid scheduling flush when sb is being shut down. It can race + * with journal shutdown and free still queued delayed work. + */ + if (s->s_flags & MS_RDONLY || !(s->s_flags & MS_ACTIVE)) return; spin_lock(&sbi->old_work_lock); -- 1.8.1.4 ^ permalink raw reply related [flat|nested] 5+ messages in thread
end of thread, other threads:[~2014-08-12 10:52 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2014-08-06 18:03 [PATCH 0/2] Two reiserfs fixes Jan Kara 2014-08-06 18:03 ` [PATCH 1/2] reiserfs: Avoid warning from unlock_new_inode() Jan Kara 2014-08-11 19:25 ` Jeff Mahoney 2014-08-12 10:52 ` Jan Kara 2014-08-06 18:04 ` [PATCH 2/2] reiserfs: Fix use after free in journal teardown Jan Kara
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).