Re: [PATCH] lockstat: report avg wait and hold times

2013-10-08 Thread Davidlohr Bueso
On Thu, 2013-10-03 at 14:15 +0200, Ingo Molnar wrote: > * Davidlohr Bueso wrote: [...] > > --- a/kernel/lockdep_proc.c > > +++ b/kernel/lockdep_proc.c > > @@ -421,6 +421,7 @@ static void seq_lock_time(struct seq_file *m, struct > > lock_time *lt) > > seq_time(m, lt->min); > > seq_time(m,

Re: [PATCH] lockstat: report avg wait and hold times

2013-10-03 Thread Ingo Molnar
btw., please use the patch below to go forward - I fixed a few small details in it which we don't want to lose. Thanks, Ingo > >From 7af86a44ae480aecdb7c5b74922a17cbe8a747ea Mon Sep 17 00:00:00 2001 From: Davidlohr Bueso Date: Wed, 2 Oct 2013 13:48:48 -0700 Subject

Re: [PATCH] lockstat: report avg wait and hold times

2013-10-03 Thread Ingo Molnar
* Davidlohr Bueso wrote: > While both the nr and total times are showed, having the avg lock hold > and wait times show in the report is quite useful when working on > performance related issues. Furthermore, I find myself constantly > doing the calculations manually. > > In addition, some of t