On Tue, Jan 22, 2008 at 03:13:58PM -0800, Dave Hansen wrote: > The emergency remount code forcibly removes FMODE_WRITE from > filps. The r/o bind mount code notices that this was done > without a proper mnt_drop_write() and properly gives a > warning. > > This patch does a mnt_drop_write() and also notes in the > filp that this was done to suppress any warning that would > have otherwise been triggered. > > I also wonder if inode->i_writecount is made inconsistent > by the emergency remount code. I guess it is, but the > damage is limited to a single inode instead of being > visible more globally like the mnt write count. Probably > not really worth fixing.
The right fix is to not simply remove FMODE_WRITE, but just remove this whole function. Until we have a proper revoke it will cause more harm than good. - To unsubscribe from this list: send the line "unsubscribe sparclinux" in the body of a message to [EMAIL PROTECTED] More majordomo info at http://vger.kernel.org/majordomo-info.html