Re: [PATCH v1] RDMA/core: Cosmetic change - move member initialization to correct block

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

 



On Mon, Feb 18, 2019 at 04:23:22PM +0200, Yuval Shaia wrote:
> old_pd is used only if IB_MR_REREG_PD flags is set.
> For readability move it's initialization to where it is used.
>
> While there rewrite the whole 'if-else' block so on error jump directly
> to label and no need for 'else'
>
> Signed-off-by: Yuval Shaia <yuval.shaia@xxxxxxxxxx>
> ---
> v0 -> v1:
> 	* Accept comment from Leon and rewrite the whole if-else block
> ---
>  drivers/infiniband/core/uverbs_cmd.c | 15 +++++++--------
>  1 file changed, 7 insertions(+), 8 deletions(-)

Except that subject looks a little bit weird,

Reviewed-by: Leon Romanovsky <leonro@xxxxxxxxxxxx>

Thanks

Attachment: signature.asc
Description: PGP signature


[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