Received: by 2002:ac0:a5a7:0:0:0:0:0 with SMTP id m36-v6csp1175936imm; Wed, 15 Aug 2018 12:46:53 -0700 (PDT) X-Google-Smtp-Source: AA+uWPxEfZqQMcChfiDTRLk5ZULL2vMv6wZ8aqbXHkGUFcMJWWSzQcoBEcfBSvGrhLrUiR4PO6bT X-Received: by 2002:a63:be4a:: with SMTP id g10-v6mr23405053pgo.378.1534362413894; Wed, 15 Aug 2018 12:46:53 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1534362413; cv=none; d=google.com; s=arc-20160816; b=UCj3FA5q2FgFbeOqUnMNc/djcwxwAligAE8piHPZ+080DBI8jtSj3yoYxaFl+1azq2 gm8DvfTBDfJcCvj8+sj+KO+gwlq48J3dkOV5LuiN8XmMy0JPxRYyiKCxk4GMYEzcT9rn jC9FIHI44gSTwpjQ5RIkZdcf6PNhrQ7Q4RL9AC3JgrFSXN1RS+rDA6NmJeq9Aw3hBeeC UASDN6DEFE9xpOdm0JTMvGWmBG6wHMiwk5awLntWJnfucYrz/6fbevb27R0vov838kxD eHXtrQLGpPU9SdKIRiNtTpSXEHhQOBSYvdhb6Nzv2cphQuAtKzMgBViYiKKChJQHt2R8 iIFg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:cc:to:subject:message-id:date:from :in-reply-to:references:mime-version:dkim-signature :arc-authentication-results; bh=YYh9GDDso6xdaM96rzYq8m0zOUEfU+iT8h6UTGVAEZ0=; b=qvy3MiBuNC1inOSSdUrqjPJ7XhC7K614NQAVu0gfOyEdRMRYoUpC6hxK9EDE5ZLtX+ 92Q+UT3mxDOiquBDDRyoKuR0a89CG+pjxzpbjQ+FWM4Lut+M/b+IP/8eZ66EP04g3TTl FRl1jtP/sTH5J8H3oFUU3KvfMd796mT3HHmKpIz+FIXy7lsQ0EKUpKPA0tnho5ngigWK PbErdu+gOuQMeiH1XREi8azTSo2q8vskbBmtC33+9bjSjQK3PbAH78OpJAUXi/yZS0Um p31B0OwyeUyHhCFZdhb9NLpK+xSREFeQOwMRzc9upkbtseKiadnmZZxQOGQvGjw3q482 Oo/Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@linux-foundation.org header.s=google header.b=O7FXuvht; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id l63-v6si24801940pfg.326.2018.08.15.12.46.15; Wed, 15 Aug 2018 12:46:53 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; dkim=pass header.i=@linux-foundation.org header.s=google header.b=O7FXuvht; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727864AbeHOWgL (ORCPT + 99 others); Wed, 15 Aug 2018 18:36:11 -0400 Received: from mail-it0-f67.google.com ([209.85.214.67]:40547 "EHLO mail-it0-f67.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727660AbeHOWgK (ORCPT ); Wed, 15 Aug 2018 18:36:10 -0400 Received: by mail-it0-f67.google.com with SMTP id h23-v6so3498729ita.5; Wed, 15 Aug 2018 12:42:39 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linux-foundation.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=YYh9GDDso6xdaM96rzYq8m0zOUEfU+iT8h6UTGVAEZ0=; b=O7FXuvht0Q1rgnUf/cT+GbusP3M+jqCmIhNTiLmNTiFVjqH0vDIhqyLzKnhNGBn2BV X2YagQVcywD0bj4Fgvs954QMAdw8g1OfCMM5i6b6Oi3lX6L2RwSjDkjz2SpNh8CMGZLl wpDR1EDdA2QysdU+gQI+nBrgDcDMaqExE1aYM= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=YYh9GDDso6xdaM96rzYq8m0zOUEfU+iT8h6UTGVAEZ0=; b=MLm0/eyJt3lwNa8CasFkM6Fk0qlT/BU3kAqqMSZ8AmhLmMDMpqd1twNZHOacKTLvS5 GuvaOOyGbYXBGBZ6DBB72X+WE4O4XL6MMvmFFvKHbt6THLB6PqnWvu4BPhTwxMwNfKud Ajy51rWuHmyQLzTQsC5NgkaaA1eKbi6gl866WbiwY1PsDS9yunzedsRwVF++/g5g+xrT BOi896b4J9GhkjxRxSPrbZtRS2dACUbmveKmnl1RI+OfUW3ECM5pN/Z8uA5YMR2go1YY KjPe0t8bpXpn2ZKm+5H/IgG29XE7wBQXWzazQy9ylHiMFR6W9rJrJrL7l4SNs4eXrWY8 a2FQ== X-Gm-Message-State: AOUpUlGtmKUNZiu63QKVpZZTyB3DyM5+EJzUjF+eNR/HzLQiwGuO/OFi 7qF4IEJLplCuj0ug/uYhOfHFJ5N+LrgnUUiLVrE= X-Received: by 2002:a24:5002:: with SMTP id m2-v6mr18800922itb.16.1534362158467; Wed, 15 Aug 2018 12:42:38 -0700 (PDT) MIME-Version: 1.0 References: <20180814171520.883143803@linuxfoundation.org> <6272e062-6cfc-0669-bebf-ebb039da051a@dd-wrt.com> <87k1orms9i.fsf@turtle.gmx.de> <55df6754-ce8c-520a-13f9-c610ac498173@dd-wrt.com> <20180815185553.GA18219@roeck-us.net> <69700876-f31a-ea0a-bdea-2e82b7a5d308@dd-wrt.com> <20180815192630.GA20575@roeck-us.net> In-Reply-To: <20180815192630.GA20575@roeck-us.net> From: Linus Torvalds Date: Wed, 15 Aug 2018 12:42:27 -0700 Message-ID: Subject: Re: [PATCH 4.9 000/107] 4.9.120-stable review To: Guenter Roeck Cc: s.gottschall@dd-wrt.com, Sven Joachim , Greg Kroah-Hartman , Linux Kernel Mailing List , Andrew Morton , Shuah Khan , patches@kernelci.org, Ben Hutchings , lkft-triage@lists.linaro.org, stable Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Wed, Aug 15, 2018 at 12:26 PM Guenter Roeck wrote: > > Also seen in mainline. Meaning my non-SMP test builds are broken. Hmm. Grr. I think it's due mainly due to commit 447ae3166702 ("x86: Don't include linux/irq.h from asm/hardirq.h"), which exposes a number of files that had some dodgy include file dependenencies, and just randomly happened to get it right because of that odd include that caused problems. That commit itself is definitely the right thing to do, but yes, we clearly had a lot of cases of things getting core header files included purely by luck. And because this was all done under embargo, we didn't get the kind of test robot coverage we usually get. Part of it can also be due to subtle merge issues - even if the original branch got good coverage, later changes sometimes ended up adding things like that. For example, my merge of the L1TF code found that in the meantime, arch/x86/kernel/kvmclock.c had added a call to kzalloc(), which used to work just fine, but with the header cleanup it turned out that kvmclock.c had never included , so now it didn't build right. And that was just the one I noticed because of my limited build tests. And yes, every single developer has CONFIG_SMP in their config, but perhaps equally importantly, I suspect CONFIG_SMP ends up getting more header files included almost by mistake, so it can _continue_ to hide these kinds of incomplete header file includes that just happen to work. > Anyway, care to send a proper patch ? I am sure Linus will be more > than happy to apply it. I think "happy" is too strong a word for my state of mind with all this, but yes, I'll apply it, and you'll get the glory of fixing some configuration that clearly didn't get properly tested. In the meantime, maybe I should just do a "make allmodconfig" and then disable SMP and see if that shows anything for me. Linus