Re: [PATCH rdma-core 2/3] srp_daemon/srp_daemon.c: Eliminate some unneeded code in get_shared_pkeys

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Apr 13, 2017 at 10:48:25AM -0400, Hal Rosenstock wrote:
> 
> PathRecords are obtained using SubAdmGet and not SubAdmGetTable
> so RMPP is not involved. Eliminate unneeded initialization of MAD
> RMPP fields.
> 
> Signed-off-by: Hal Rosenstock <hal@xxxxxxxxxxxx>
> ---
>  srp_daemon/srp_daemon.c | 2 --
>  1 file changed, 2 deletions(-)
> 
> diff --git a/srp_daemon/srp_daemon.c b/srp_daemon/srp_daemon.c
> index 9fb2ecc..59a6137 100644
> --- a/srp_daemon/srp_daemon.c
> +++ b/srp_daemon/srp_daemon.c
> @@ -1134,8 +1134,6 @@ static int get_shared_pkeys(struct resources *res,
>  
>  		/* Mark components: DLID, SLID, PKEY */
>  		out_sa_mad->comp_mask = htobe64(1 << 4 | 1 << 5 | 1 << 13);
> -		out_sa_mad->rmpp_hdr.rmpp_version = UMAD_RMPP_VERSION;
> -		out_sa_mad->rmpp_hdr.rmpp_type = 1;
>  		path_rec = (struct ib_path_rec *)out_sa_mad->data;
>  		path_rec->slid = htobe16(local_port_lid);
>  		path_rec->dlid = htobe16(dest_port_lid);
> -- 
> 1.8.2.3
> 

Reviewed-by: Honggang Li <honli@xxxxxxxxxx>

--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux