diff options
author | Christoph Hellwig <hch@lst.de> | 2008-12-03 06:20:32 -0500 |
---|---|---|
committer | Niv Sardi <xaiki@sgi.com> | 2008-12-03 23:39:22 -0500 |
commit | 6bd16ff27060819d16b3e7abe59b6644b349aea3 (patch) | |
tree | 6a7250063b8d727f35887bfe188a4dc7802fa181 /fs/xfs | |
parent | 5efcbb853bc2f051d720a191268f8dd901fea9c2 (diff) |
kill dead inode flags
There are a few inode flags around that aren't used anywhere, so remove
them. Also update xfsidbg to display all used inode flags correctly.
Signed-off-by: Christoph Hellwig <hch@lst.de>
Reviewed-by: Dave Chinner <david@fromorbit.com>
Signed-off-by: Niv Sardi <xaiki@sgi.com>
Diffstat (limited to 'fs/xfs')
-rw-r--r-- | fs/xfs/linux-2.6/xfs_aops.c | 11 | ||||
-rw-r--r-- | fs/xfs/linux-2.6/xfs_file.c | 10 | ||||
-rw-r--r-- | fs/xfs/linux-2.6/xfs_iops.c | 5 | ||||
-rw-r--r-- | fs/xfs/linux-2.6/xfs_super.c | 1 | ||||
-rw-r--r-- | fs/xfs/xfs_iget.c | 1 | ||||
-rw-r--r-- | fs/xfs/xfs_inode.h | 17 |
6 files changed, 11 insertions, 34 deletions
diff --git a/fs/xfs/linux-2.6/xfs_aops.c b/fs/xfs/linux-2.6/xfs_aops.c index 8fbc97df3609..bb224d07e1ef 100644 --- a/fs/xfs/linux-2.6/xfs_aops.c +++ b/fs/xfs/linux-2.6/xfs_aops.c | |||
@@ -317,14 +317,9 @@ xfs_map_blocks( | |||
317 | xfs_iomap_t *mapp, | 317 | xfs_iomap_t *mapp, |
318 | int flags) | 318 | int flags) |
319 | { | 319 | { |
320 | xfs_inode_t *ip = XFS_I(inode); | 320 | int nmaps = 1; |
321 | int error, nmaps = 1; | 321 | |
322 | 322 | return -xfs_iomap(XFS_I(inode), offset, count, flags, mapp, &nmaps); | |
323 | error = xfs_iomap(ip, offset, count, | ||
324 | flags, mapp, &nmaps); | ||
325 | if (!error && (flags & (BMAPI_WRITE|BMAPI_ALLOCATE))) | ||
326 | xfs_iflags_set(ip, XFS_IMODIFIED); | ||
327 | return -error; | ||
328 | } | 323 | } |
329 | 324 | ||
330 | STATIC_INLINE int | 325 | STATIC_INLINE int |
diff --git a/fs/xfs/linux-2.6/xfs_file.c b/fs/xfs/linux-2.6/xfs_file.c index d377db05d806..f999d20a429c 100644 --- a/fs/xfs/linux-2.6/xfs_file.c +++ b/fs/xfs/linux-2.6/xfs_file.c | |||
@@ -281,11 +281,8 @@ xfs_file_ioctl( | |||
281 | unsigned int cmd, | 281 | unsigned int cmd, |
282 | unsigned long p) | 282 | unsigned long p) |
283 | { | 283 | { |
284 | int error; | ||
285 | struct inode *inode = filp->f_path.dentry->d_inode; | 284 | struct inode *inode = filp->f_path.dentry->d_inode; |
286 | 285 | ||
287 | error = xfs_ioctl(XFS_I(inode), filp, 0, cmd, (void __user *)p); | ||
288 | xfs_iflags_set(XFS_I(inode), XFS_IMODIFIED); | ||
289 | 286 | ||
290 | /* NOTE: some of the ioctl's return positive #'s as a | 287 | /* NOTE: some of the ioctl's return positive #'s as a |
291 | * byte count indicating success, such as | 288 | * byte count indicating success, such as |
@@ -293,7 +290,7 @@ xfs_file_ioctl( | |||
293 | * like most other routines. This means true | 290 | * like most other routines. This means true |
294 | * errors need to be returned as a negative value. | 291 | * errors need to be returned as a negative value. |
295 | */ | 292 | */ |
296 | return error; | 293 | return xfs_ioctl(XFS_I(inode), filp, 0, cmd, (void __user *)p); |
297 | } | 294 | } |
298 | 295 | ||
299 | STATIC long | 296 | STATIC long |
@@ -302,11 +299,8 @@ xfs_file_ioctl_invis( | |||
302 | unsigned int cmd, | 299 | unsigned int cmd, |
303 | unsigned long p) | 300 | unsigned long p) |
304 | { | 301 | { |
305 | int error; | ||
306 | struct inode *inode = filp->f_path.dentry->d_inode; | 302 | struct inode *inode = filp->f_path.dentry->d_inode; |
307 | 303 | ||
308 | error = xfs_ioctl(XFS_I(inode), filp, IO_INVIS, cmd, (void __user *)p); | ||
309 | xfs_iflags_set(XFS_I(inode), XFS_IMODIFIED); | ||
310 | 304 | ||
311 | /* NOTE: some of the ioctl's return positive #'s as a | 305 | /* NOTE: some of the ioctl's return positive #'s as a |
312 | * byte count indicating success, such as | 306 | * byte count indicating success, such as |
@@ -314,7 +308,7 @@ xfs_file_ioctl_invis( | |||
314 | * like most other routines. This means true | 308 | * like most other routines. This means true |
315 | * errors need to be returned as a negative value. | 309 | * errors need to be returned as a negative value. |
316 | */ | 310 | */ |
317 | return error; | 311 | return xfs_ioctl(XFS_I(inode), filp, IO_INVIS, cmd, (void __user *)p); |
318 | } | 312 | } |
319 | 313 | ||
320 | /* | 314 | /* |
diff --git a/fs/xfs/linux-2.6/xfs_iops.c b/fs/xfs/linux-2.6/xfs_iops.c index 76b570dd1ab2..7aa53fefc67f 100644 --- a/fs/xfs/linux-2.6/xfs_iops.c +++ b/fs/xfs/linux-2.6/xfs_iops.c | |||
@@ -159,8 +159,6 @@ xfs_init_security( | |||
159 | } | 159 | } |
160 | 160 | ||
161 | error = xfs_attr_set(ip, name, value, length, ATTR_SECURE); | 161 | error = xfs_attr_set(ip, name, value, length, ATTR_SECURE); |
162 | if (!error) | ||
163 | xfs_iflags_set(ip, XFS_IMODIFIED); | ||
164 | 162 | ||
165 | kfree(name); | 163 | kfree(name); |
166 | kfree(value); | 164 | kfree(value); |
@@ -261,7 +259,6 @@ xfs_vn_mknod( | |||
261 | error = _ACL_INHERIT(inode, mode, default_acl); | 259 | error = _ACL_INHERIT(inode, mode, default_acl); |
262 | if (unlikely(error)) | 260 | if (unlikely(error)) |
263 | goto out_cleanup_inode; | 261 | goto out_cleanup_inode; |
264 | xfs_iflags_set(ip, XFS_IMODIFIED); | ||
265 | _ACL_FREE(default_acl); | 262 | _ACL_FREE(default_acl); |
266 | } | 263 | } |
267 | 264 | ||
@@ -377,7 +374,6 @@ xfs_vn_link( | |||
377 | if (unlikely(error)) | 374 | if (unlikely(error)) |
378 | return -error; | 375 | return -error; |
379 | 376 | ||
380 | xfs_iflags_set(XFS_I(dir), XFS_IMODIFIED); | ||
381 | atomic_inc(&inode->i_count); | 377 | atomic_inc(&inode->i_count); |
382 | d_instantiate(dentry, inode); | 378 | d_instantiate(dentry, inode); |
383 | return 0; | 379 | return 0; |
@@ -888,7 +884,6 @@ xfs_setup_inode( | |||
888 | inode->i_ctime.tv_sec = ip->i_d.di_ctime.t_sec; | 884 | inode->i_ctime.tv_sec = ip->i_d.di_ctime.t_sec; |
889 | inode->i_ctime.tv_nsec = ip->i_d.di_ctime.t_nsec; | 885 | inode->i_ctime.tv_nsec = ip->i_d.di_ctime.t_nsec; |
890 | xfs_diflags_to_iflags(inode, ip); | 886 | xfs_diflags_to_iflags(inode, ip); |
891 | xfs_iflags_clear(ip, XFS_IMODIFIED); | ||
892 | 887 | ||
893 | switch (inode->i_mode & S_IFMT) { | 888 | switch (inode->i_mode & S_IFMT) { |
894 | case S_IFREG: | 889 | case S_IFREG: |
diff --git a/fs/xfs/linux-2.6/xfs_super.c b/fs/xfs/linux-2.6/xfs_super.c index 5389f077874b..37f2d11be4ac 100644 --- a/fs/xfs/linux-2.6/xfs_super.c +++ b/fs/xfs/linux-2.6/xfs_super.c | |||
@@ -1025,7 +1025,6 @@ xfs_fs_clear_inode( | |||
1025 | XFS_STATS_DEC(vn_active); | 1025 | XFS_STATS_DEC(vn_active); |
1026 | 1026 | ||
1027 | xfs_inactive(ip); | 1027 | xfs_inactive(ip); |
1028 | xfs_iflags_clear(ip, XFS_IMODIFIED); | ||
1029 | } | 1028 | } |
1030 | 1029 | ||
1031 | STATIC void | 1030 | STATIC void |
diff --git a/fs/xfs/xfs_iget.c b/fs/xfs/xfs_iget.c index f58e5e000fce..82eb8ed56338 100644 --- a/fs/xfs/xfs_iget.c +++ b/fs/xfs/xfs_iget.c | |||
@@ -362,7 +362,6 @@ again: | |||
362 | } | 362 | } |
363 | xfs_put_perag(mp, pag); | 363 | xfs_put_perag(mp, pag); |
364 | 364 | ||
365 | xfs_iflags_set(ip, XFS_IMODIFIED); | ||
366 | *ipp = ip; | 365 | *ipp = ip; |
367 | 366 | ||
368 | ASSERT(ip->i_df.if_ext_max == | 367 | ASSERT(ip->i_df.if_ext_max == |
diff --git a/fs/xfs/xfs_inode.h b/fs/xfs/xfs_inode.h index 03ae96bdae7e..ddac1b806c0b 100644 --- a/fs/xfs/xfs_inode.h +++ b/fs/xfs/xfs_inode.h | |||
@@ -403,17 +403,12 @@ static inline void xfs_ifunlock(xfs_inode_t *ip) | |||
403 | /* | 403 | /* |
404 | * In-core inode flags. | 404 | * In-core inode flags. |
405 | */ | 405 | */ |
406 | #define XFS_IGRIO 0x0001 /* inode used for guaranteed rate i/o */ | 406 | #define XFS_IRECLAIM 0x0001 /* we have started reclaiming this inode */ |
407 | #define XFS_IUIOSZ 0x0002 /* inode i/o sizes have been explicitly set */ | 407 | #define XFS_ISTALE 0x0002 /* inode has been staled */ |
408 | #define XFS_IQUIESCE 0x0004 /* we have started quiescing for this inode */ | 408 | #define XFS_IRECLAIMABLE 0x0004 /* inode can be reclaimed */ |
409 | #define XFS_IRECLAIM 0x0008 /* we have started reclaiming this inode */ | 409 | #define XFS_INEW 0x0008 /* inode has just been allocated */ |
410 | #define XFS_ISTALE 0x0010 /* inode has been staled */ | 410 | #define XFS_IFILESTREAM 0x0010 /* inode is in a filestream directory */ |
411 | #define XFS_IRECLAIMABLE 0x0020 /* inode can be reclaimed */ | 411 | #define XFS_ITRUNCATED 0x0020 /* truncated down so flush-on-close */ |
412 | #define XFS_INEW 0x0040 | ||
413 | #define XFS_IFILESTREAM 0x0080 /* inode is in a filestream directory */ | ||
414 | #define XFS_IMODIFIED 0x0100 /* XFS inode state possibly differs */ | ||
415 | /* to the Linux inode state. */ | ||
416 | #define XFS_ITRUNCATED 0x0200 /* truncated down so flush-on-close */ | ||
417 | 412 | ||
418 | /* | 413 | /* |
419 | * Flags for inode locking. | 414 | * Flags for inode locking. |