| Message ID | 1593011328-10258-1-git-send-email-laoar.shao@gmail.com (mailing list archive) |
|---|---|
| State | Accepted, archived |
| Headers | show |
| Series | xfs: remove useless definitions in xfs_linux.h | expand |
On Wed, Jun 24, 2020 at 11:08:48AM -0400, Yafang Shao wrote: > Remove current_pid(), current_test_flags() and > current_clear_flags_nested(), because they are useless. > > Signed-off-by: Yafang Shao <laoar.shao@gmail.com> Looks good, Reviewed-by: Christoph Hellwig <hch@lst.de>
On Wed, Jun 24, 2020 at 11:08:48AM -0400, Yafang Shao wrote: > Remove current_pid(), current_test_flags() and > current_clear_flags_nested(), because they are useless. > > Signed-off-by: Yafang Shao <laoar.shao@gmail.com> Looks ok, will put in tree... Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com> --D > --- > fs/xfs/xfs_linux.h | 4 ---- > 1 file changed, 4 deletions(-) > > diff --git a/fs/xfs/xfs_linux.h b/fs/xfs/xfs_linux.h > index 9f70d2f..ab737fe 100644 > --- a/fs/xfs/xfs_linux.h > +++ b/fs/xfs/xfs_linux.h > @@ -102,12 +102,8 @@ > #define xfs_cowb_secs xfs_params.cowb_timer.val > > #define current_cpu() (raw_smp_processor_id()) > -#define current_pid() (current->pid) > -#define current_test_flags(f) (current->flags & (f)) > #define current_set_flags_nested(sp, f) \ > (*(sp) = current->flags, current->flags |= (f)) > -#define current_clear_flags_nested(sp, f) \ > - (*(sp) = current->flags, current->flags &= ~(f)) > #define current_restore_flags_nested(sp, f) \ > (current->flags = ((current->flags & ~(f)) | (*(sp) & (f)))) > > -- > 1.8.3.1 >
diff --git a/fs/xfs/xfs_linux.h b/fs/xfs/xfs_linux.h index 9f70d2f..ab737fe 100644 --- a/fs/xfs/xfs_linux.h +++ b/fs/xfs/xfs_linux.h @@ -102,12 +102,8 @@ #define xfs_cowb_secs xfs_params.cowb_timer.val #define current_cpu() (raw_smp_processor_id()) -#define current_pid() (current->pid) -#define current_test_flags(f) (current->flags & (f)) #define current_set_flags_nested(sp, f) \ (*(sp) = current->flags, current->flags |= (f)) -#define current_clear_flags_nested(sp, f) \ - (*(sp) = current->flags, current->flags &= ~(f)) #define current_restore_flags_nested(sp, f) \ (current->flags = ((current->flags & ~(f)) | (*(sp) & (f))))
Remove current_pid(), current_test_flags() and current_clear_flags_nested(), because they are useless. Signed-off-by: Yafang Shao <laoar.shao@gmail.com> --- fs/xfs/xfs_linux.h | 4 ---- 1 file changed, 4 deletions(-)