Re: [PATCH] Separate lock events with types

From: Hitoshi Mitake
Date: Mon Apr 05 2010 - 06:37:46 EST

On 03/27/10 08:33, Frederic Weisbecker wrote:
> On Wed, Feb 24, 2010 at 06:02:46PM +0900, Hitoshi Mitake wrote:
>> Sorry for my long silence...
>> Thanks for Frederic's great work like trace_lock_class_init(),
>> overhead of perf lock was reduced a lot.
>> But still there is overhead which cannot be disregarded.
>> So I'd like to suggest that separating lock trace events into each types of lock.
>> e.g.
>> trace_lock_acquire() -> spin_trace_lock_acquire(), rwlock_trace_lock_acquire()
>> I think that mutex and spinlock are completely different things.
>> And as I describe below, filtering at recording phase can reduce overhead of tracing.
>> This patch is the proof of concept. The way this patch employes
>> is different from one I described above. This patch adds if statement
>> before trace_lock_*(). Implementation of separating events per types will be
>> a big one, so this is an only trial edition for performance improvements.
> Instead of having one different event for each type of locks,
> I would rather suggest to add a "lock type" field in the (future)
> lock_init_class class. This requires we implement event injection
> properly before.
> So if we store the lock type in the lockdep_map, we can just dump
> the type on lock class initialization:
> - on register_lock_class
> - on event injection to catchup with lock that have already registered

Yeah, this is a smarter way than the one I suggested.

But specifying types of locks to focuse on will require special way.
I thought that ioctl() will make this type specifying possible,
but it will not match to other events...
e.g. perf record will be required dirty change for it.

Do you have any good ideas at this point?

To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at
Please read the FAQ at