Re: [PATCH] lib/Kconfig.debug: add a comment to PROVE_LOCKING impact

2018-10-10 Thread Lukasz Luba
Hi Longman, Thanks for the patches. I have applied them and run on ARM. In context switch test, there is ~2% improvement (but still ~15us instead of 5-6us). In dhrystone: test results are the same. There is also 2% improvement in boot time. You can add me on CC list if you have some patches

Re: [PATCH] lib/Kconfig.debug: add a comment to PROVE_LOCKING impact

2018-10-10 Thread Lukasz Luba
Hi Longman, Thanks for the patches. I have applied them and run on ARM. In context switch test, there is ~2% improvement (but still ~15us instead of 5-6us). In dhrystone: test results are the same. There is also 2% improvement in boot time. You can add me on CC list if you have some patches

Re: [PATCH] lib/Kconfig.debug: add a comment to PROVE_LOCKING impact

2018-10-09 Thread Waiman Long
On 10/09/2018 11:39 AM, Lukasz Luba wrote: > This patch add some comment related to performance impact, > which can be really big (x3 times slower context switch). > > Signed-off-by: Lukasz Luba > --- > lib/Kconfig.debug | 11 +++ > 1 file changed, 11 insertions(+) > > diff --git

Re: [PATCH] lib/Kconfig.debug: add a comment to PROVE_LOCKING impact

2018-10-09 Thread Waiman Long
On 10/09/2018 11:39 AM, Lukasz Luba wrote: > This patch add some comment related to performance impact, > which can be really big (x3 times slower context switch). > > Signed-off-by: Lukasz Luba > --- > lib/Kconfig.debug | 11 +++ > 1 file changed, 11 insertions(+) > > diff --git

Re: [PATCH] lib/Kconfig.debug: add a comment to PROVE_LOCKING impact

2018-10-09 Thread Nikolay Borisov
On 9.10.2018 18:39, Lukasz Luba wrote: > This patch add some comment related to performance impact, > which can be really big (x3 times slower context switch). > > Signed-off-by: Lukasz Luba I don't think this brings any value. lockdep is a debugging aid aimed at developers. A developer

Re: [PATCH] lib/Kconfig.debug: add a comment to PROVE_LOCKING impact

2018-10-09 Thread Nikolay Borisov
On 9.10.2018 18:39, Lukasz Luba wrote: > This patch add some comment related to performance impact, > which can be really big (x3 times slower context switch). > > Signed-off-by: Lukasz Luba I don't think this brings any value. lockdep is a debugging aid aimed at developers. A developer

[PATCH] lib/Kconfig.debug: add a comment to PROVE_LOCKING impact

2018-10-09 Thread Lukasz Luba
This patch add some comment related to performance impact, which can be really big (x3 times slower context switch). Signed-off-by: Lukasz Luba --- lib/Kconfig.debug | 11 +++ 1 file changed, 11 insertions(+) diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug index 4966c4f..9e67a2a3

[PATCH] lib/Kconfig.debug: add a comment to PROVE_LOCKING impact

2018-10-09 Thread Lukasz Luba
This patch add some comment related to performance impact, which can be really big (x3 times slower context switch). Signed-off-by: Lukasz Luba --- lib/Kconfig.debug | 11 +++ 1 file changed, 11 insertions(+) diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug index 4966c4f..9e67a2a3