On Wed, Feb 21, 2018 at 06:10:56AM -0500, Zhu Yanjun wrote: > In send_atomic_ack function, it is not necessary to make a > skb_clone. To gain better performance (high throughput and > low latency), this skb_clone is removed. > > The following tests are made. > > server client > --------- --------- > |1.1.1.1|<----rxe-channel--->|1.1.1.2| > --------- --------- > > On server: rping -s -a 1.1.1.1 -v -C 1000 -S 512 > On client: rping -c -a 1.1.1.1 -v -C 1000 -S 512 > > The kernel config CONFIG_DEBUG_KMEMLEAK is enabled on both server > and client. > > This test runs for several hours. There is no memory leak and the whole > system can work well. > > Based on the above network, the following tests are made. > > Server: ibv_rc_pingpong -d rxe0 -g 1 > Client: ibv_rc_pingpong -d rxe0 -g 1 1.1.1.1 > > The test results on Server(10 tests are made). > Before: > Throughput is 137.07 Mbit/sec > Latency is 517.76 usec/iter > > After: > Throughput is 148.85 Mbit/sec > Latency is 476.64 usec/iter > > The throughput is enhanced and the latency is reduced. > > CC: Srinivas Eeda <srinivas.eeda@xxxxxxxxxx> > CC: Junxiao Bi <junxiao.bi@xxxxxxxxxx> > Signed-off-by: Zhu Yanjun <yanjun.zhu@xxxxxxxxxx> > --- > V3-->V4: Fix typo errors in short logs. > V2-->V3: Fix typo errors. > V1-->V2: 10 tests are made. From throughput and latency, the performance is better. > --- > drivers/infiniband/sw/rxe/rxe_resp.c | 18 +++--------------- > 1 file changed, 3 insertions(+), 15 deletions(-) > Hi Zhu, Can you please don't reply and/or place "In-Reply-To" header while sending patches? Such patches are easily missed by reviewers. Thanks
Attachment:
signature.asc
Description: PGP signature
