]> git.baikalelectronics.ru Git - kernel.git/commit
IB/rxe: remove unnecessary skb_clone
authorZhu Yanjun <yanjun.zhu@oracle.com>
Tue, 27 Feb 2018 11:04:32 +0000 (06:04 -0500)
committerJason Gunthorpe <jgg@mellanox.com>
Wed, 7 Mar 2018 22:56:14 +0000 (15:56 -0700)
commite35c753f1bd34a3ae0a59aff42872b568c219f2b
tree996c99efb1b426b73d0e0f2f2f6437fa1755848f
parentc12b7949b38b5b0e258f2ebe6125b54b0997debb
IB/rxe: remove unnecessary skb_clone

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@oracle.com>
CC: Junxiao Bi <junxiao.bi@oracle.com>
Signed-off-by: Zhu Yanjun <yanjun.zhu@oracle.com>
Signed-off-by: Doug Ledford <dledford@redhat.com>
drivers/infiniband/sw/rxe/rxe_resp.c