From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932557AbcBIQjT (ORCPT ); Tue, 9 Feb 2016 11:39:19 -0500 Received: from mail-io0-f174.google.com ([209.85.223.174]:35959 "EHLO mail-io0-f174.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1755146AbcBIQjQ (ORCPT ); Tue, 9 Feb 2016 11:39:16 -0500 MIME-Version: 1.0 In-Reply-To: <1455031885.3807.74.camel@gmail.com> References: <56B1C9E4.4020400@suse.cz> <20160203122855.GB6762@dhcp22.suse.cz> <20160203162441.GE14091@mtj.duckdns.org> <1454518913.6148.15.camel@gmail.com> <20160203170652.GI14091@mtj.duckdns.org> <1454551217.3677.27.camel@gmail.com> <20160205164923.GC4401@htj.duckdns.org> <1454705231.3819.151.camel@gmail.com> <20160205205456.GG4401@htj.duckdns.org> <1454705989.3819.158.camel@gmail.com> <20160205210606.GH4401@htj.duckdns.org> <1455031885.3807.74.camel@gmail.com> Date: Tue, 9 Feb 2016 08:39:15 -0800 X-Google-Sender-Auth: -jxR66pedW29gLMysTuTWLeEEQ4 Message-ID: Subject: Re: Crashes with 874bbfe600a6 in 3.18.25 From: Linus Torvalds To: Mike Galbraith Cc: Tejun Heo , Michal Hocko , Jiri Slaby , Thomas Gleixner , Petr Mladek , Jan Kara , Ben Hutchings , Sasha Levin , Shaohua Li , LKML , stable , Daniel Bilik , Greg Kroah-Hartman Content-Type: text/plain; charset=UTF-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Feb 9, 2016 at 7:31 AM, Mike Galbraith wrote: > On Fri, 2016-02-05 at 16:06 -0500, Tejun Heo wrote: >> > >> > That 874bbfe6 should die. >> >> Yeah, it's gonna be killed. The commit is there because the behavior >> change broke things. We don't want to guarantee it but have been and >> can't change it right away just because we don't like it when things >> may break from it. The plan is to implement a debug option to force >> workqueue to always execute these work items on a foreign cpu to weed >> out breakages. > > A niggling question remaining is when is it gonna be killed? It probably should be killed sooner rather than later. Just document that if you need something to run on a _particular_ cpu, you need to use "schedule_delayed_work_on()" and "add_timer_on()". The proper fix was 176bed1de5bf, and 874bbfe6 was just wrong. Linus