| To: | Christoph Hellwig <hch@xxxxxxxxxxxxx> |
|---|---|
| Subject: | Re: inode_permission NULL pointer dereference in 3.13-rc1 |
| From: | Al Viro <viro@xxxxxxxxxxxxxxxxxx> |
| Date: | Tue, 26 Nov 2013 13:11:34 +0000 |
| Cc: | linux-fsdevel@xxxxxxxxxxxxxxx, xfs@xxxxxxxxxxx |
| Delivered-to: | xfs@xxxxxxxxxxx |
| In-reply-to: | <20131125160648.GA4933@xxxxxxxxxxxxx> |
| References: | <20131124140413.GA19271@xxxxxxxxxxxxx> <20131124152758.GL10323@xxxxxxxxxxxxxxxxxx> <20131125160648.GA4933@xxxxxxxxxxxxx> |
| Sender: | Al Viro <viro@xxxxxxxxxxxxxxxx> |
| User-agent: | Mutt/1.5.21 (2010-09-15) |
On Mon, Nov 25, 2013 at 08:06:48AM -0800, Christoph Hellwig wrote: > On Sun, Nov 24, 2013 at 03:27:58PM +0000, Al Viro wrote: > > which seems to leave two candidates - follow_dotdot_rcu() and path_init(), > > both setting nd->inode to nd->path.dentry->d_inode... > > > > Could you try to reproduce it with something like > > if (read_seqretry(&mount_lock, nd->m_seq)) > > goto failed; > > slapped before the success exit in follow_dotdot_rcu(), just to see if > > we are hitting some races with umount here? > > Still reproducable with the diff below. Fixed by reverting the RCU'd > vfsmounts. .config, please - all I'm seeing on mine is a bloody awful leak somewhere in VM that I'd been hunting for last week, so the damn thing gets OOMed halfway through xfstests run ;-/ |
| Previous by Date: | Re: ARC-1120 and MD very sloooow, Stan Hoeppner |
|---|---|
| Next by Date: | [PATCH v2 0/3] xfs: fix quota related problems, Jeff Liu |
| Previous by Thread: | Re: inode_permission NULL pointer dereference in 3.13-rc1, Christoph Hellwig |
| Next by Thread: | Re: inode_permission NULL pointer dereference in 3.13-rc1, Christoph Hellwig |
| Indexes: | [Date] [Thread] [Top] [All Lists] |