On Wed, Mar 21, 2018 at 02:07:38PM -0400, Brian Foster wrote: > On Wed, Mar 21, 2018 at 10:27:28AM -0700, Darrick J. Wong wrote: > > On Wed, Mar 21, 2018 at 12:13:25PM -0400, Brian Foster wrote: > > > Most of the generic data structures embedded in xfs_mount are > > > dynamically initialized immediately after mp is allocated. A few > > > fields are left out and initialized during the xfs_mountfs() > > > sequence, after mp has been attached to the superblock. > > > > > > To clean this up and help prevent premature access of associated > > > fields, refactor xfs_mount allocation and all dependent init calls > > > into a new helper. This self-documents that all low level data > > > structures (i.e., locks, trees, etc.) should be initialized before > > > xfs_mount is attached to the superblock. > > > > > > Signed-off-by: Brian Foster <bfoster@xxxxxxxxxx> > > > --- > > > > > > I realized after the fact that this steps on the ability to instrument > > > the use-before-init variant[1] of the problematic radix tree access on > > > failed mount issue that Dave tracked down. This is because we'd > > > immediately initialize the structure and thus a subsequent memset() > > > would ultimately require another proper initialization. > > > > That's simply a matter of re-calling > > > > INIT_RADIX_TREE(&mp->m_perag_tree, GFP_ATOMIC); > > > > after the msleep, right? > > > > I suppose that may work (I'll let Dave chime in)... Well, to tell the truth, this whole "VFS accesses the fs during mount: problem is isolated to only the inode cache shrinker, and it while a mount is in progress it does not run inode cache scans (can't lock s_umount because the mount holds it locked). IOWs, this crash can be fixed entirely by modifying the VFS code as I mentioned to avoid super_cache_count() doing any work during mount and unmount. So, really, none of these XFS changes are necessary to fix the problem that was reported. I'm not opposed to it as a cleanup, but let's not conflate it with the VFS superblock shrinker bug that needs to be fixed.... Cheers, Dave. -- Dave Chinner david@xxxxxxxxxxxxx -- To unsubscribe from this list: send the line "unsubscribe linux-xfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
