Re: [PATCH 1/2] IB/isert: use unlikely macro in the fast path

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

 




On 8/5/2020 4:16 PM, Leon Romanovsky wrote:
On Wed, Aug 05, 2020 at 03:12:30PM +0300, Max Gurtovoy wrote:
Add performance optimization that might slightly improve small IO sizes
benchmarks.

Signed-off-by: Max Gurtovoy <maxg@xxxxxxxxxxxx>
---
  drivers/infiniband/ulp/isert/ib_isert.c | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)
I find the expectation from "unlikely/likely" keywords to be overrated.

When we introduced dissagregate post send verbs in rdma-core, we
benchmarked likely/unlikely and didn't find any significant difference
for code with and without such keywords.

Thanks

Leon,

We are using these small optimizations in all our ULPs and we saw benefit in large scale and high loads (we did the same in NVMf/RDMA).

These kind of optimizations might not be seen immediately but are accumulated.

I don't know why do you compare user-space benchmarks to storage drivers.

Can you please review the code ?

Sagi,

Can you send your comments as well ?





[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