From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wm0-f69.google.com (mail-wm0-f69.google.com [74.125.82.69]) by kanga.kvack.org (Postfix) with ESMTP id 1D88F6B0009 for ; Mon, 26 Mar 2018 09:19:17 -0400 (EDT) Received: by mail-wm0-f69.google.com with SMTP id b76so4382244wmg.9 for ; Mon, 26 Mar 2018 06:19:17 -0700 (PDT) Received: from mx2.suse.de (mx2.suse.de. [195.135.220.15]) by mx.google.com with ESMTPS id a91si11790733wrc.50.2018.03.26.06.19.15 for (version=TLS1 cipher=AES128-SHA bits=128/128); Mon, 26 Mar 2018 06:19:15 -0700 (PDT) Date: Mon, 26 Mar 2018 15:19:11 +0200 From: Michal Hocko Subject: Re: [PATCH] lockdep: Show address of "struct lockdep_map" at print_lock(). Message-ID: <20180326131911.GI5652@dhcp22.suse.cz> References: <1522059513-5461-1-git-send-email-penguin-kernel@I-love.SAKURA.ne.jp> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1522059513-5461-1-git-send-email-penguin-kernel@I-love.SAKURA.ne.jp> Sender: owner-linux-mm@kvack.org List-ID: To: Tetsuo Handa Cc: peterz@infradead.org, mingo@redhat.com, linux-kernel@vger.kernel.org, linux-mm@kvack.org, Borislav Petkov , David Rientjes , Thomas Gleixner On Mon 26-03-18 19:18:33, Tetsuo Handa wrote: > Currently, print_lock() is printing hlock->acquire_ip field in both > "[<%px>]" and "%pS" format. But "[<%px>]" is little useful nowadays, for > we use scripts/faddr2line which receives "%pS" for finding the location > in the source code. > > Since "struct lockdep_map" is embedded into lock objects, we can know > which instance of a lock object is acquired using hlock->instance field. > This will help finding which threads are causing a lock contention when > e.g. the OOM reaper failed to acquire an OOM victim's mmap_sem for read. How? All I can see is that we can match which instances are the same. This would be an interesting thing to know AFAICS because you can tell different instances of lock apart. So the patch makes some sense to me, I am just not sure about changelog. -- Michal Hocko SUSE Labs