From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S1757688AbcJXVjE (ORCPT <rfc822;w@1wt.eu>);
        Mon, 24 Oct 2016 17:39:04 -0400
Received: from Galois.linutronix.de ([146.0.238.70]:59445 "EHLO
        Galois.linutronix.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S1757032AbcJXVjB (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Mon, 24 Oct 2016 17:39:01 -0400
Date: Mon, 24 Oct 2016 23:36:10 +0200 (CEST)
From: Thomas Gleixner <tglx@linutronix.de>
To: Linus Torvalds <torvalds@linux-foundation.org>
cc: Sergey Senozhatsky <sergey.senozhatsky@gmail.com>,
        LKML <linux-kernel@vger.kernel.org>,
        Andrew Morton <akpm@linux-foundation.org>,
        Ingo Molnar <mingo@kernel.org>, "H. Peter Anvin" <hpa@zytor.com>
Subject: Re: [GIT pull] timer updates for 4.9
In-Reply-To: <CA+55aFxLdf3brn_wJK7cPiDCLoSg2g-C3Sm0B6zZbcYkCO0hFw@mail.gmail.com>
Message-ID: <alpine.DEB.2.20.1610242324250.4815@nanos>
References: <alpine.DEB.2.20.1610221356280.4885@nanos> <CA+55aFw3caJpp36sciVin6=GQF_eHpAHMd2J0bxwrcKoWiuiog@mail.gmail.com> <CA+55aFyDAWBNfyYM_BG2D5-g1skj=iCkhfW9ZNA_h2zZDdfy_g@mail.gmail.com> <alpine.DEB.2.20.1610240947160.4872@nanos>
 <alpine.DEB.2.20.1610241638170.4983@nanos> <CA+55aFysd6G2Se9DJXmfJdWieksHVHBzoxkK5982iuu3MTArkQ@mail.gmail.com> <alpine.DEB.2.20.1610242048570.4815@nanos> <CA+55aFxLdf3brn_wJK7cPiDCLoSg2g-C3Sm0B6zZbcYkCO0hFw@mail.gmail.com>
User-Agent: Alpine 2.20 (DEB 67 2015-01-07)
MIME-Version: 1.0
Content-Type: text/plain; charset=US-ASCII
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On Mon, 24 Oct 2016, Linus Torvalds wrote:
> The one I'm *currently* testing seems to work for me, but that's
> because I moved the timer into klogd and that just avoids the issue.
> 
> But I'll attach my old patch just for your testing pleasure.

Of course it does not trigger... I probably need to test it on a desktop
machine to get all the goodies of that gnome shell ignoring signals.

> The fact that it actually mostly worked - and then had some very odd
> behavior much later on - makes me wonder if we might have other users
> that could have timers that get started early, and nobody notices
> because most of the time it doesn't cause any real issues.

I added a bunch of timers before init_timers(). There is no fallout aside
of the timers being queued into a random bucket and the spinlock bad magic
bug, which actually happens at spin_lock and not at unlock as I assumed.

While queueing timers before init_timers() is not correct, I think that the
early queueing is a red herring. We should really try to isolate the root
cause of this wreckage.

Can you retest that old patch with my combo patch applied and see if it
still triggers? If yes, which I expect, debugobjects might give us the hint
what to look for.

Thanks,

	tglx