Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933738AbbDXT3z (ORCPT ); Fri, 24 Apr 2015 15:29:55 -0400 Received: from mail-ob0-f169.google.com ([209.85.214.169]:36773 "EHLO mail-ob0-f169.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932667AbbDXT3x (ORCPT ); Fri, 24 Apr 2015 15:29:53 -0400 MIME-Version: 1.0 In-Reply-To: References: <20150423054738.GA3722@thinkpad> Date: Fri, 24 Apr 2015 12:29:53 -0700 Message-ID: Subject: Re: [PATCH] perf/x86/intel/uncore: fix IMC missing box initialization From: Stephane Eranian To: Bjorn Helgaas Cc: Vince Weaver , LKML , Peter Zijlstra , "ak@linux.intel.com" , "Liang, Kan" , "mingo@elte.hu" , Sonny Rao Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 2453 Lines: 59 On Fri, Apr 24, 2015 at 12:22 PM, Bjorn Helgaas wrote: > On Fri, Apr 24, 2015 at 9:38 AM, Stephane Eranian wrote: >> On Fri, Apr 24, 2015 at 7:03 AM, Vince Weaver wrote: >>> On Fri, 24 Apr 2015, Vince Weaver wrote: >>> >>>> [ 79.473121] kernel BUG at mm/vmalloc.c:1335! >>>> [ 79.477705] invalid opcode: 0000 [#1] SMP >>> ... >>>> [ 79.694391] Call Trace: >>>> [ 79.696997] >>>> [ 79.699090] [] get_vm_area_caller+0x40/0x50 >>>> [ 79.705505] [] ? snb_uncore_imc_init_box+0x6d/0x90 >>> >>> This maps to: >>> >>> static void snb_uncore_imc_init_box(struct intel_uncore_box *box) { >>> ... >>> box->io_addr = ioremap(addr, SNB_UNCORE_PCI_IMC_MAP_SIZE); >>> >>> The machine I am seeing this on is a Haswell desktop, Intel 6/60/3. >>> >> I also got that on one Lenovo IvyBridge laptop but I had just updated the BIOS. >> It was working before with older kernels. I am not sure what is going on here. >> It will be easier to debug once I get my desktop Haswell back and setup with >> serial console. Need to verify that the address of the BAR is the same compared >> with older kernels. >> >> Maybe Bjorn can shed some lights. Why would ioremap() die like this in the >> latest 4.0 tree. > > 1328 static struct vm_struct *__get_vm_area_node(unsigned long size, > 1329 unsigned long align, unsigned long flags, > unsigned long start, > 1330 unsigned long end, int node, gfp_t gfp_mask, > const void *caller) > 1331 { > 1332 struct vmap_area *va; > 1333 struct vm_struct *area; > 1334 > 1335 BUG_ON(in_interrupt()); <----------------- > > Is there some perfmon initialization happening in interrupt context? May be in IPI context if I read Vince's stack trace. This leads me to believe that this patch: commit c05199e5a57a579fea1e8fa65e2b511ceb524ffc Author: Kan Liang Date: Tue Jan 20 04:54:25 2015 +0000 perf/x86/intel/uncore: Move uncore_box_init() out of driver initialization If I revert it, I bet things will work again. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/