From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1165698AbeE1O1Q (ORCPT ); Mon, 28 May 2018 10:27:16 -0400 Received: from Galois.linutronix.de ([146.0.238.70]:38218 "EHLO Galois.linutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1161787AbeE1O1J (ORCPT ); Mon, 28 May 2018 10:27:09 -0400 Date: Mon, 28 May 2018 16:27:00 +0200 (CEST) From: Thomas Gleixner To: Tariq Toukan cc: Song Liu , Dmitry Safonov <0x7f454c46@gmail.com>, open list , Maor Gottlieb , kernel-team@fb.com Subject: Re: WARNING and PANIC in irq_matrix_free In-Reply-To: <1b4ded38-f217-fafa-d920-cddf980a685c@mellanox.com> Message-ID: References: <16f47fa4-1555-cddb-3dfb-7d56fb992ea1@mellanox.com> <1b4ded38-f217-fafa-d920-cddf980a685c@mellanox.com> User-Agent: Alpine 2.21 (DEB 202 2017-01-01) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII X-Linutronix-Spam-Score: -1.0 X-Linutronix-Spam-Level: - X-Linutronix-Spam-Status: No , -1.0 points, 5.0 required, ALL_TRUSTED=-1,SHORTCIRCUIT=-0.0001 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, 28 May 2018, Tariq Toukan wrote: > On 28/05/2018 1:53 PM, Thomas Gleixner wrote: > > On Fri, 25 May 2018, Song Liu wrote: > > > On Wed, May 23, 2018 at 1:49 AM, Thomas Gleixner > > > wrote: > > > > On Wed, 23 May 2018, Tariq Toukan wrote: > > > > > I have your patch merged into my internal branch, it prints the > > > > > following: > > > > > > > > > > [ 4898.226258] Trying to clear prev_vector: 0 > > > > > [ 4898.226439] Trying to clear prev_vector: 0 > > > > > > > > > > i.e. vector(0) is lower than FIRST_EXTERNAL_VECTOR. > > > > > > > > Could you please enable the vector and irq matrix trace points and > > > > capture > > > > the trace when this happens? > > > > Does the patch below fix it? > > > > Thanks, > > > > tglx > > > > 8<------------------- > > diff --git a/arch/x86/kernel/apic/vector.c b/arch/x86/kernel/apic/vector.c > > index bb6f7a2148d7..54af3d4884b1 100644 > > --- a/arch/x86/kernel/apic/vector.c > > +++ b/arch/x86/kernel/apic/vector.c > > @@ -148,6 +148,7 @@ static void apic_update_vector(struct irq_data *irqd, > > unsigned int newvec, > > * prev_vector for this and the offlined target case. > > */ > > apicd->prev_vector = 0; > > + apicd->move_in_progress = false; > > if (!apicd->vector || apicd->vector == MANAGED_IRQ_SHUTDOWN_VECTOR) > > goto setnew; > > /* > > > > I took it into my internal branch. Will let you know. Note, I'd still like to see a trace w/o the patch which shows which order of events leads to this. Even if the patch cures it, it might just paper over it and not fixing the root cause. Thanks, tglx