2021-05-10 06:30:52

by Jiabing Wan

[permalink] [raw]
Subject: [PATCH] RDMA: Remove unnecessary struct declaration

The declaration of struct ib_grh is uncessary here,
because it is defined at line 766.

Signed-off-by: Wan Jiabing <[email protected]>
---
include/rdma/ib_verbs.h | 1 -
1 file changed, 1 deletion(-)

diff --git a/include/rdma/ib_verbs.h b/include/rdma/ib_verbs.h
index 7e2f3699b898..18a7a786711f 100644
--- a/include/rdma/ib_verbs.h
+++ b/include/rdma/ib_verbs.h
@@ -2139,7 +2139,6 @@ struct ib_flow_action {
};

struct ib_mad;
-struct ib_grh;

enum ib_process_mad_flags {
IB_MAD_IGNORE_MKEY = 1,
--
2.20.1


2021-05-11 04:57:38

by Leon Romanovsky

[permalink] [raw]
Subject: Re: [PATCH] RDMA: Remove unnecessary struct declaration

On Mon, May 10, 2021 at 02:28:42PM +0800, Wan Jiabing wrote:
> The declaration of struct ib_grh is uncessary here,
> because it is defined at line 766.
>
> Signed-off-by: Wan Jiabing <[email protected]>
> ---
> include/rdma/ib_verbs.h | 1 -
> 1 file changed, 1 deletion(-)
>

I'm pretty sure that my ROB is not needed, but anyway.

Reviewed-by: Leon Romanovsky <[email protected]>

Thanks

2021-05-11 19:54:26

by Jason Gunthorpe

[permalink] [raw]
Subject: Re: [PATCH] RDMA: Remove unnecessary struct declaration

On Mon, May 10, 2021 at 02:28:42PM +0800, Wan Jiabing wrote:
> The declaration of struct ib_grh is uncessary here,
> because it is defined at line 766.
>
> Signed-off-by: Wan Jiabing <[email protected]>
> Reviewed-by: Leon Romanovsky <[email protected]>
> ---
> include/rdma/ib_verbs.h | 1 -
> 1 file changed, 1 deletion(-)

Applied to for-next, thanks

Jason