diff options
author | Eryu Guan <guaneryu@gmail.com> | 2016-04-05 19:47:21 -0400 |
---|---|---|
committer | Dave Chinner <david@fromorbit.com> | 2016-04-05 19:47:21 -0400 |
commit | 6e3e6d55e51774ec7cfc24975749bbddb28a9051 (patch) | |
tree | 87c089ce4be95cc249ee81612ff595269b8971bd | |
parent | 664b60f6babc98ee03c2ff15b9482cc8c5e15a83 (diff) |
xfs: mute some sparse warnings
These three warnings are fixed:
fs/xfs/xfs_inode.c:1033:44: warning: Using plain integer as NULL pointer
fs/xfs/xfs_inode_item.c:525:20: warning: context imbalance in 'xfs_inode_item_push' - unexpected unlock
fs/xfs/xfs_dquot.c:696:1: warning: symbol 'xfs_dq_get_next_id' was not declared. Should it be static?
Signed-off-by: Eryu Guan <guaneryu@gmail.com>
Reviewed-by: Christoph Hellwig <hch@lst.de>
Signed-off-by: Dave Chinner <david@fromorbit.com>
-rw-r--r-- | fs/xfs/xfs_dquot.c | 2 | ||||
-rw-r--r-- | fs/xfs/xfs_inode.c | 2 | ||||
-rw-r--r-- | fs/xfs/xfs_inode_item.c | 2 |
3 files changed, 4 insertions, 2 deletions
diff --git a/fs/xfs/xfs_dquot.c b/fs/xfs/xfs_dquot.c index 316b2a1bdba5..8f51370c95c4 100644 --- a/fs/xfs/xfs_dquot.c +++ b/fs/xfs/xfs_dquot.c | |||
@@ -692,7 +692,7 @@ error0: | |||
692 | * end of the chunk, skip ahead to first id in next allocated chunk | 692 | * end of the chunk, skip ahead to first id in next allocated chunk |
693 | * using the SEEK_DATA interface. | 693 | * using the SEEK_DATA interface. |
694 | */ | 694 | */ |
695 | int | 695 | static int |
696 | xfs_dq_get_next_id( | 696 | xfs_dq_get_next_id( |
697 | xfs_mount_t *mp, | 697 | xfs_mount_t *mp, |
698 | uint type, | 698 | uint type, |
diff --git a/fs/xfs/xfs_inode.c b/fs/xfs/xfs_inode.c index 96f606deee31..1445a99a6868 100644 --- a/fs/xfs/xfs_inode.c +++ b/fs/xfs/xfs_inode.c | |||
@@ -1030,7 +1030,7 @@ xfs_dir_ialloc( | |||
1030 | tp->t_flags &= ~(XFS_TRANS_DQ_DIRTY); | 1030 | tp->t_flags &= ~(XFS_TRANS_DQ_DIRTY); |
1031 | } | 1031 | } |
1032 | 1032 | ||
1033 | code = xfs_trans_roll(&tp, 0); | 1033 | code = xfs_trans_roll(&tp, NULL); |
1034 | if (committed != NULL) | 1034 | if (committed != NULL) |
1035 | *committed = 1; | 1035 | *committed = 1; |
1036 | 1036 | ||
diff --git a/fs/xfs/xfs_inode_item.c b/fs/xfs/xfs_inode_item.c index c48b5b18d771..d02cbab18f45 100644 --- a/fs/xfs/xfs_inode_item.c +++ b/fs/xfs/xfs_inode_item.c | |||
@@ -479,6 +479,8 @@ STATIC uint | |||
479 | xfs_inode_item_push( | 479 | xfs_inode_item_push( |
480 | struct xfs_log_item *lip, | 480 | struct xfs_log_item *lip, |
481 | struct list_head *buffer_list) | 481 | struct list_head *buffer_list) |
482 | __releases(&lip->li_ailp->xa_lock) | ||
483 | __acquires(&lip->li_ailp->xa_lock) | ||
482 | { | 484 | { |
483 | struct xfs_inode_log_item *iip = INODE_ITEM(lip); | 485 | struct xfs_inode_log_item *iip = INODE_ITEM(lip); |
484 | struct xfs_inode *ip = iip->ili_inode; | 486 | struct xfs_inode *ip = iip->ili_inode; |