lkml.org 
[lkml]   [2021]   [Mar]   [11]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
SubjectRe: [PATCH] kernel/futex: Change pi_state_update_owner() to an inline function
Date
On Tue, Mar 09 2021 at 10:40, Xiangyang Yu wrote:
> In our performance tests, we find that the performance of
> sysbench is descend. Function call consumes too many instructions,
> change pi_state_update_owner() to an inline function.

Serioulsy? sysbench does not use PI futexes which means that it does not
ever invoke this function. So it's completely irrelevant for that test
whether the function is marked inline or not.

Aside of that the compiler can ignore 'inline' or inline a static
function. But even if there would be an extra call, then

> Test case: https://github.com/akopytov/sysbench
> Test cmd: ./sysbench --test=threads --num-threads=$thr
> --thread-yields=100 --thread-locks=2 run
>
> $thr stock non-inline inline
> 256 0.8511 0.9254 0.8588
> 512 0.8489 0.9232 0.8503
> 1024 0.8347 0.9248 0.8495
> 2048 0.8338 0.9575 0.8402

a performance change of this magnitude is certainly not the result. I
also have no idea what these numbers actually mean. Nothing in a
sysbench output with the same parameters looks even close.

But whatever the above is, the 'inline' is neither the cause nor the
cure.

Thanks,

tglx

\
 
 \ /
  Last update: 2021-03-11 17:08    [W:0.084 / U:0.180 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site