On Thu, Sep 08, 2016 at 09:16:11AM -0500, Eric Sandeen wrote: > ustat() has been deprecated for a very long time, and newer > architectures (aarch64, for one) have not implemented it. > If called, it always returns false, so we can no longer use > it to determine a device's mounted status. > > We already have another mechanism for determining the mounted > status of a device in platform_check_iswritable; it iterates > over getmntent looking for the device, and checks its mount > options. We can do the same thing to check for a simple mount, > and not caring about the "ro" mount option. > > Because the loop is essentially the same, factor it into a > helper which accepts a VERBOSE flag to print info if the device > is found in the checked-for state, and a WRITABLE flag which > only checks specifically for a mounted and /writable/ device. > > Signed-off-by: Eric Sandeen <sandeen@xxxxxxxxxx> > --- > > Dave, up to you if you want to take all this in lieu of the > fixed-up patch from Felix, or if you'd like me to rebase this > once you have merged that patch. Happy to do that if you'd like - > now that I think about it, that may be the cleaner progression > of changes. Rebase, please, as it does not apply to the current tree. 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
