On Fri, Jun 22, 2007 at 01:03:06PM -0700, Dave Hansen wrote: > > ext4_change_inode_journal_flag() is only called from one > location: ext4_ioctl(EXT3_IOC_SETFLAGS). That ioctl > case already has a IS_RDONLY() call in it so this one > is superfluous. Ditto. - 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