lkml.org 
[lkml]   [2005]   [Feb]   [4]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH] Updated FAT attributes patch
From
Date
"H. Peter Anvin" <hpa@zytor.com> 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 <hirofumi@mail.parknet.co.jp>
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/

\
 
 \ /
  Last update: 2005-03-22 14:10    [W:0.033 / U:0.172 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site