From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gautham R Shenoy Subject: Re: [PATCH] kernel/cpu: Distinctive name for cpu_hotplug.dep_map Date: Wed, 3 Feb 2016 22:42:44 +0530 Message-ID: <20160203171244.GA8159@in.ibm.com> References: <1454509468-10627-1-git-send-email-joonas.lahtinen@linux.intel.com> Reply-To: ego@linux.vnet.ibm.com Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Return-path: Content-Disposition: inline In-Reply-To: <1454509468-10627-1-git-send-email-joonas.lahtinen@linux.intel.com> Sender: linux-kernel-owner@vger.kernel.org To: Joonas Lahtinen Cc: Linux kernel development , "Gautham R. Shenoy" , "Rafael J. Wysocki" , Intel graphics driver community testing & development List-Id: intel-gfx@lists.freedesktop.org Hello Joonas, On Wed, Feb 03, 2016 at 04:24:28PM +0200, Joonas Lahtinen wrote: > Use distinctive name for cpu_hotplug.dep_map to avoid the actual > cpu_hotplug.lock appearing as cpu_hotplug.lock#2 in lockdep splats. > > Cc: Gautham R. Shenoy > Cc: Rafael J. Wysocki > Cc: Intel graphics driver community testing & development > Signed-off-by: Joonas Lahtinen > --- > kernel/cpu.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/kernel/cpu.c b/kernel/cpu.c > index 5b9d396..6a13f24 100644 > --- a/kernel/cpu.c > +++ b/kernel/cpu.c > @@ -78,7 +78,7 @@ static struct { > .wq = __WAIT_QUEUE_HEAD_INITIALIZER(cpu_hotplug.wq), > .lock = __MUTEX_INITIALIZER(cpu_hotplug.lock), > #ifdef CONFIG_DEBUG_LOCK_ALLOC > - .dep_map = {.name = "cpu_hotplug.lock" }, > + .dep_map = STATIC_LOCKDEP_MAP_INIT("cpu_hotplug.dep_map", &cpu_hotplug.dep_map), Looks good to me! Acked-by: Gautham R. Shenoy