Why do you repeat that inode = iget_locked(sb, cramino(cramfs_inode, offset)); if (inode && (inode->i_state & I_NEW)) {so many times?Wouldn't it be nicer to just do it once at the top? Linus