Re: [PATCH v5 10/13] SIW receive path

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

 




---
Bernard Metzler, PhD
Tech. Leader High Performance I/O, Principal Research Staff
IBM Zurich Research Laboratory
Saeumerstrasse 4
CH-8803 Rueschlikon, Switzerland
+41 44 724 8605

-----"Leon Romanovsky" <leon@xxxxxxxxxx> wrote: -----

>To: "Bernard Metzler" <BMT@xxxxxxxxxxxxxx>
>From: "Leon Romanovsky" <leon@xxxxxxxxxx>
>Date: 02/28/2019 08:31PM
>Cc: "Gal Pressman" <galpress@xxxxxxxxxx>, linux-rdma@xxxxxxxxxxxxxxx
>Subject: Re: [PATCH v5 10/13] SIW receive path
>
>On Thu, Feb 28, 2019 at 02:47:39PM +0000, Bernard Metzler wrote:
>>
>>
>
><...>
>
>> >> +int siw_proc_send(struct siw_qp *qp, struct siw_iwarp_rx *rctx)
>> >> +{
>> >> +	struct siw_wqe		*wqe;
>> >> +	struct siw_sge		*sge;
>> >> +	u32			data_bytes,	/* all data bytes available */
>> >> +				rcvd_bytes;	/* sum of data bytes rcvd */
>> >
>> >Split declarations please, this applies to more occurrences in
>this
>> >file.
>> >
>> Please help me, I don't see what needs to be changed.
>
>He suggests to change the line above from "u32 data_bytes,
>rcvd_bytes;"
>to be "u32 data_bytes; u32 rcvd_bytes;"
>
>And please remove vertical space alignment.
>
yeah, this is what clang-format does for me ;)

Best,
Bernard.




[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