diff mbox series

[1/2] xfs: Remove duplicate xfs_da_format.h header

Message ID 20240430034728.86811-1-jiapeng.chong@linux.alibaba.com (mailing list archive)
State Accepted, archived
Headers show
Series [1/2] xfs: Remove duplicate xfs_da_format.h header | expand

Commit Message

Jiapeng Chong April 30, 2024, 3:47 a.m. UTC
./fs/xfs/libxfs/xfs_trans_resv.c: xfs_da_format.h is included more than once.

Reported-by: Abaci Robot <abaci@linux.alibaba.com>
Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=8931
Signed-off-by: Jiapeng Chong <jiapeng.chong@linux.alibaba.com>
---
 fs/xfs/libxfs/xfs_trans_resv.c | 1 -
 1 file changed, 1 deletion(-)

Comments

Christoph Hellwig April 30, 2024, 4:04 a.m. UTC | #1
On Tue, Apr 30, 2024 at 11:47:27AM +0800, Jiapeng Chong wrote:
> ./fs/xfs/libxfs/xfs_trans_resv.c: xfs_da_format.h is included more than once.
> 
> Reported-by: Abaci Robot <abaci@linux.alibaba.com>
> Closes: https://bugzilla.openanolis.cn/show_bug.cgi?id=8931

I don't think we do Closes for random bugzilla, never mind for trivial
cleanups.

The changes itself looks fine, though:

Reviewed-by: Christoph Hellwig <hch@lst.de>
diff mbox series

Patch

diff --git a/fs/xfs/libxfs/xfs_trans_resv.c b/fs/xfs/libxfs/xfs_trans_resv.c
index 6dbe6e7251e7..3dc8f785bf29 100644
--- a/fs/xfs/libxfs/xfs_trans_resv.c
+++ b/fs/xfs/libxfs/xfs_trans_resv.c
@@ -22,7 +22,6 @@ 
 #include "xfs_rtbitmap.h"
 #include "xfs_attr_item.h"
 #include "xfs_log.h"
-#include "xfs_da_format.h"
 
 #define _ALLOC	true
 #define _FREE	false