"H. Peter Anvin" <[EMAIL PROTECTED]> writes:
> This updates the FAT attributes as well as (hopefully) corrects the
> handling of VFAT ctime. The FAT attributes are implemented as a
> 32-bit ioctl, per the previous discussions.
[...]
> + /* This MUST be done before doing anything irreversible... */
> + if ( (err = notify_change(filp->f_dentry, &ia)) )
> + goto up;
> +
> + if (sbi->options.sys_immutable) {
> + if ( attr & ATTR_SYS )
> + inode->i_flags |= S_IMMUTABLE;
> + else
> + inode->i_flags &= S_IMMUTABLE;
> + }
> +
> + MSDOS_I(inode)->i_attrs = attr & ATTR_UNUSED;
Looks good to me. However, we would need to add the mark_inode_ditry()
after seting iattr. Because another write_inode() path can clear the
dirty flag before setting ->i_attr.
I'll apply the patch and add it.
Thanks.
--
OGAWA Hirofumi <[EMAIL PROTECTED]>
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to [EMAIL PROTECTED]
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
