diff mbox series

[v2] io_uring: rsrc: Optimize return value variable 'ret'

Message ID 20230317182538.3027-1-zeming@nfschina.com (mailing list archive)
State New
Headers show
Series [v2] io_uring: rsrc: Optimize return value variable 'ret' | expand

Commit Message

Li zeming March 17, 2023, 6:25 p.m. UTC 
The initialization assignment of the variable ret is changed to 0, only
in 'goto fail;' Use the ret variable as the function return value.

Signed-off-by: Li zeming <zeming@nfschina.com>
---
 v2: Modify the initialization value of ret variable.

 io_uring/rsrc.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Jens Axboe March 16, 2023, 2 a.m. UTC | #1 
On 3/17/23 12:25 PM, Li zeming wrote:
> The initialization assignment of the variable ret is changed to 0, only
> in 'goto fail;' Use the ret variable as the function return value.

Applied - normally I'd just sent the usual auto-generated email,
but wanted to let you know that the previous email and this email
are all sent from the future. You need to fix your date/time on
your end.
Jens Axboe March 16, 2023, 1:17 p.m. UTC | #2 
On Sat, 18 Mar 2023 02:25:38 +0800, Li zeming wrote:
> The initialization assignment of the variable ret is changed to 0, only
> in 'goto fail;' Use the ret variable as the function return value.
> 
> 

Applied, thanks!

[1/1] io_uring: rsrc: Optimize return value variable 'ret'
      commit: 6acd352dfee558194643adbed7e849fe80fd1b93

Best regards,
diff mbox series

Patch

diff --git a/io_uring/rsrc.c b/io_uring/rsrc.c
index 056f40946ff6..09a16d709cb5 100644
--- a/io_uring/rsrc.c
+++ b/io_uring/rsrc.c
@@ -410,7 +410,7 @@  __cold static int io_rsrc_data_alloc(struct io_ring_ctx *ctx,
 				     unsigned nr, struct io_rsrc_data **pdata)
 {
 	struct io_rsrc_data *data;
-	int ret = -ENOMEM;
+	int ret = 0;
 	unsigned i;
 
 	data = kzalloc(sizeof(*data), GFP_KERNEL);