On Fri, Jun 22, 2007 at 01:03:05PM -0700, Dave Hansen wrote: > > ext3_change_inode_journal_flag() is only called from one > location: ext3_ioctl(EXT3_IOC_SETFLAGS). That ioctl > case already has a IS_RDONLY() call in it so this one > is superfluous. Ok, and stands nicely on it's own. - To unsubscribe from this list: send the line "unsubscribe linux-fsdevel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html