From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753121AbcBOMaw (ORCPT ); Mon, 15 Feb 2016 07:30:52 -0500 Received: from mga04.intel.com ([192.55.52.120]:59683 "EHLO mga04.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752986AbcBOMau (ORCPT ); Mon, 15 Feb 2016 07:30:50 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.22,450,1449561600"; d="scan'208";a="885224206" Message-ID: <1455539446.5499.2.camel@linux.intel.com> Subject: Re: [PATCH] kernel/cpu: Distinctive name for cpu_hotplug.dep_map From: Joonas Lahtinen To: Ingo Molnar , Peter Zijlstra , ego@linux.vnet.ibm.com Cc: Linux kernel development , "Rafael J. Wysocki" , Intel graphics driver community testing & development Date: Mon, 15 Feb 2016 14:30:46 +0200 In-Reply-To: <20160203171244.GA8159@in.ibm.com> References: <1454509468-10627-1-git-send-email-joonas.lahtinen@linux.intel.com> <20160203171244.GA8159@in.ibm.com> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo Content-Type: text/plain; charset="UTF-8" X-Mailer: Evolution 3.18.4 (3.18.4-1.fc23) Mime-Version: 1.0 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, According to scripts/get_maintainer.pl Ingo or Peter would be more appropriate to merge. Added them as To: On ke, 2016-02-03 at 22:42 +0530, Gautham R Shenoy wrote: > 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 > x@lists.freedesktop.org> > > 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 > Thanks! Regards, Joonas > -- > Thanks and Regards > gautham. > -- Joonas Lahtinen Open Source Technology Center Intel Corporation