* [PATCH v1] xfs: Fix the return value of xfs_rtcopy_summary()
@ 2025-12-17 5:37 Nirjhar Roy (IBM)
2025-12-17 5:57 ` Darrick J. Wong
2025-12-17 10:25 ` Christoph Hellwig
0 siblings, 2 replies; 4+ messages in thread
From: Nirjhar Roy (IBM) @ 2025-12-17 5:37 UTC (permalink / raw)
To: linux-xfs; +Cc: ritesh.list, ojaswin, djwong, hch, nirjhar.roy.lists
xfs_rtcopy_summary() should return the appropriate error code
instead of always returning 0. The caller of this function which is
xfs_growfs_rt_bmblock() is already handling the error.
Signed-off-by: Nirjhar Roy (IBM) <nirjhar.roy.lists@gmail.com>
---
fs/xfs/xfs_rtalloc.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/fs/xfs/xfs_rtalloc.c b/fs/xfs/xfs_rtalloc.c
index 6907e871fa15..bc88b965e909 100644
--- a/fs/xfs/xfs_rtalloc.c
+++ b/fs/xfs/xfs_rtalloc.c
@@ -126,7 +126,7 @@ xfs_rtcopy_summary(
error = 0;
out:
xfs_rtbuf_cache_relse(oargs);
- return 0;
+ return error;
}
/*
* Mark an extent specified by start and len allocated.
--
2.43.5
^ permalink raw reply related [flat|nested] 4+ messages in thread
* Re: [PATCH v1] xfs: Fix the return value of xfs_rtcopy_summary()
2025-12-17 5:37 [PATCH v1] xfs: Fix the return value of xfs_rtcopy_summary() Nirjhar Roy (IBM)
@ 2025-12-17 5:57 ` Darrick J. Wong
2025-12-17 10:25 ` Christoph Hellwig
1 sibling, 0 replies; 4+ messages in thread
From: Darrick J. Wong @ 2025-12-17 5:57 UTC (permalink / raw)
To: Nirjhar Roy (IBM); +Cc: linux-xfs, ritesh.list, ojaswin, hch
On Wed, Dec 17, 2025 at 11:07:42AM +0530, Nirjhar Roy (IBM) wrote:
> xfs_rtcopy_summary() should return the appropriate error code
> instead of always returning 0. The caller of this function which is
> xfs_growfs_rt_bmblock() is already handling the error.
>
> Signed-off-by: Nirjhar Roy (IBM) <nirjhar.roy.lists@gmail.com>
Looks pretty broken to me...
Reviewed-by: "Darrick J. Wong" <djwong@kernel.org>
--D
> ---
> fs/xfs/xfs_rtalloc.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/fs/xfs/xfs_rtalloc.c b/fs/xfs/xfs_rtalloc.c
> index 6907e871fa15..bc88b965e909 100644
> --- a/fs/xfs/xfs_rtalloc.c
> +++ b/fs/xfs/xfs_rtalloc.c
> @@ -126,7 +126,7 @@ xfs_rtcopy_summary(
> error = 0;
> out:
> xfs_rtbuf_cache_relse(oargs);
> - return 0;
> + return error;
> }
> /*
> * Mark an extent specified by start and len allocated.
> --
> 2.43.5
>
>
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v1] xfs: Fix the return value of xfs_rtcopy_summary()
2025-12-17 5:37 [PATCH v1] xfs: Fix the return value of xfs_rtcopy_summary() Nirjhar Roy (IBM)
2025-12-17 5:57 ` Darrick J. Wong
@ 2025-12-17 10:25 ` Christoph Hellwig
2025-12-17 10:49 ` Nirjhar Roy (IBM)
1 sibling, 1 reply; 4+ messages in thread
From: Christoph Hellwig @ 2025-12-17 10:25 UTC (permalink / raw)
To: Nirjhar Roy (IBM); +Cc: linux-xfs, ritesh.list, ojaswin, djwong, hch
On Wed, Dec 17, 2025 at 11:07:42AM +0530, Nirjhar Roy (IBM) wrote:
> xfs_rtcopy_summary() should return the appropriate error code
> instead of always returning 0. The caller of this function which is
> xfs_growfs_rt_bmblock() is already handling the error.
>
> Signed-off-by: Nirjhar Roy (IBM) <nirjhar.roy.lists@gmail.com>
Fixes tag?
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [PATCH v1] xfs: Fix the return value of xfs_rtcopy_summary()
2025-12-17 10:25 ` Christoph Hellwig
@ 2025-12-17 10:49 ` Nirjhar Roy (IBM)
0 siblings, 0 replies; 4+ messages in thread
From: Nirjhar Roy (IBM) @ 2025-12-17 10:49 UTC (permalink / raw)
To: Christoph Hellwig; +Cc: linux-xfs, ritesh.list, ojaswin, djwong
On 12/17/25 15:55, Christoph Hellwig wrote:
> On Wed, Dec 17, 2025 at 11:07:42AM +0530, Nirjhar Roy (IBM) wrote:
>> xfs_rtcopy_summary() should return the appropriate error code
>> instead of always returning 0. The caller of this function which is
>> xfs_growfs_rt_bmblock() is already handling the error.
>>
>> Signed-off-by: Nirjhar Roy (IBM) <nirjhar.roy.lists@gmail.com>
> Fixes tag?
Fixes e94b53ff699c ("xfs: cache last bitmap block in realtime allocator")
I will send a v2 with the fixes tag and Darrick's RB. Thank you for
pointing this out.
--NR
>
--
Nirjhar Roy
Linux Kernel Developer
IBM, Bangalore
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2025-12-17 10:49 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-12-17 5:37 [PATCH v1] xfs: Fix the return value of xfs_rtcopy_summary() Nirjhar Roy (IBM)
2025-12-17 5:57 ` Darrick J. Wong
2025-12-17 10:25 ` Christoph Hellwig
2025-12-17 10:49 ` Nirjhar Roy (IBM)
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox