[PATCH] RDMA/rxe: Fix redundant break statement in switch-case.

Rohit Chavan posted 1 patch 2 years, 3 months ago
drivers/infiniband/sw/rxe/rxe_verbs.c | 1 -
1 file changed, 1 deletion(-)
[PATCH] RDMA/rxe: Fix redundant break statement in switch-case.
Posted by Rohit Chavan 2 years, 3 months ago
Removed unreachable break statement after return.

Signed-off-by: Rohit Chavan <roheetchavan@gmail.com>
---
 drivers/infiniband/sw/rxe/rxe_verbs.c | 1 -
 1 file changed, 1 deletion(-)

diff --git a/drivers/infiniband/sw/rxe/rxe_verbs.c b/drivers/infiniband/sw/rxe/rxe_verbs.c
index 903f0b71447e..48f86839d36a 100644
--- a/drivers/infiniband/sw/rxe/rxe_verbs.c
+++ b/drivers/infiniband/sw/rxe/rxe_verbs.c
@@ -798,7 +798,6 @@ static int init_send_wr(struct rxe_qp *qp, struct rxe_send_wr *wr,
 			rxe_err_qp(qp, "unsupported wr opcode %d",
 					wr->opcode);
 			return -EINVAL;
-			break;
 		}
 	}

--
2.30.2
Re: [PATCH] RDMA/rxe: Fix redundant break statement in switch-case.
Posted by Leon Romanovsky 2 years, 3 months ago
On Tue, 22 Aug 2023 14:43:04 +0530, Rohit Chavan wrote:
> Removed unreachable break statement after return.
> 
> 

Applied, thanks!

[1/1] RDMA/rxe: Fix redundant break statement in switch-case.
      https://git.kernel.org/rdma/rdma/c/6812e069990547

Best regards,
-- 
Leon Romanovsky <leon@kernel.org>
Re: [PATCH] RDMA/rxe: Fix redundant break statement in switch-case.
Posted by Zhu Yanjun 2 years, 3 months ago
On Tue, Aug 22, 2023 at 5:13 PM Rohit Chavan <roheetchavan@gmail.com> wrote:
>
> Removed unreachable break statement after return.
>
> Signed-off-by: Rohit Chavan <roheetchavan@gmail.com>

Thanks a lot.
Acked-by: Zhu Yanjun <zyjzyj2000@gmail.com>

Zhu Yanjun

> ---
>  drivers/infiniband/sw/rxe/rxe_verbs.c | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/drivers/infiniband/sw/rxe/rxe_verbs.c b/drivers/infiniband/sw/rxe/rxe_verbs.c
> index 903f0b71447e..48f86839d36a 100644
> --- a/drivers/infiniband/sw/rxe/rxe_verbs.c
> +++ b/drivers/infiniband/sw/rxe/rxe_verbs.c
> @@ -798,7 +798,6 @@ static int init_send_wr(struct rxe_qp *qp, struct rxe_send_wr *wr,
>                         rxe_err_qp(qp, "unsupported wr opcode %d",
>                                         wr->opcode);
>                         return -EINVAL;
> -                       break;
>                 }
>         }
>
> --
> 2.30.2
>