From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Subject: EDAC, {skx|i10nm}_edac: Fix randconfig build error From: "Luck, Tony" Message-Id: <20190322175504.GA11213@agluck-desk> Date: Fri, 22 Mar 2019 10:55:05 -0700 To: Arnd Bergmann Cc: Borislav Petkov , Mauro Carvalho Chehab , James Morse , "Zhuo, Qiuxu" , "linux-edac@vger.kernel.org" , Linux Kernel Mailing List List-ID: T24gRnJpLCBNYXIgMjIsIDIwMTkgYXQgMDM6MDA6MjVQTSArMDEwMCwgQXJuZCBCZXJnbWFubiB3 cm90ZToKPiBTb3JyeSwgdGhpcyB3YXMgbXkgbWlzdGFrZSwgbXkgZW1haWwgd2FzIGdhcmJsZWQu IFRoZSBwYXRjaCB3YXMKPiBjb3JyZWN0IHRob3VnaDogdGhlIGlkZWEgaGVyZSBpcyBub3QgdG8g Y2hhbmdlIHRoZSBLY29uZmlnIHN5bWJvbHMKPiBidXQgdG8gY2hhbmdlIHRoZSBNYWtlZmlsZSB0 byBkbyB0aGUgcmlnaHQgdGhpbmcgZXZlbiB3aGVuIEtjb25maWcKPiBpcyBzZXQgd3JvbmcuCgpX ZWxsIHRoaXMgZG9lcyBzZWVtIGxpa2UgYSAiY2xldmVyIiB3YXkgb3V0IG9mIHRoZSByYW5kY29u ZmlnCnByb2JsZW0uICBOZXcgcGF0Y2ggYXBwbGllcyBhbmQgd2hlbiBJIHNldCAuY29uZmlnIHRv IGhhdmU6CgpDT05GSUdfRURBQ19ERUJVRz15CkNPTkZJR19FREFDX1NLWD15CkNPTkZJR19FREFD X0kxME5NPW0KQ09ORklHX0VEQUNfU0tYX0NPTU1PTj15CgpJIGRvbid0IHNlZSBhbnkgYnVpbGQg ZXJyb3JzLgoKVGhlcmUgYXJlIGxvdHMgb2YgInNreF8iIHN5bWJvbHMgaW4gU3lzdGVtLm1hcAoK QnV0IEknbSBub3QgYXQgYWxsIHN1cmUgd2hhdCBoYXBwZW5lZCB0byB0aGUgSTEwTk0gZHJpdmVy LgoKSSBkb24ndCBzZWUgaXQgbWVudGlvbmVkIGluIHRoZSBvdXRwdXQgZnJvbSAibWFrZSIuCgpJ dCBkaWRuJ3QgZ2V0IGJ1aWx0IGFzIGEgbW9kdWxlIChubyAiLmtvIiBmaWxlIGZvciBpdCkuCgpJ dCBkb2Vzbid0IHNlZW0gdG8gYmUgYnVpbHQgaW4gKG5vICIubyIgaW4gZHJpdmVycy9lZGFjL2J1 aWx0LWluLmEpCgpTbyBJIGFkZGVkIGEgI2Vycm9yIGxpbmUgdG8gdGhlIHN0YXJ0IG9mIGkxMG5t X2VkYWMuYyBhbmQKcmFuICJtYWtlIiBhZ2Fpbi4gIE5vdGhpbmcuCgpTbywgSSBkb24ndCB0aGlu ayB0aGlzIGlzIGRvaW5nIHdoYXQgeW91IHRoaW5rIGl0IHNob3VsZApkby4gIEV2ZW4gaWYgaXQg ZGlkLCBpdCB3b3VsZCBzZWVtIHZlcnkgY29uZnVzaW5nIHRvIGEgdXNlcgp0aGF0IGFza2VkIGZv ciAib25lIG1vZHVsZSwgb25lIGJ1aWx0LWluIiBpbiBLY29uZmlnLCBidXQKZm91bmQgYm90aCBi dWlsdC1pbi4KCkJvcmlzOiBJJ20gdm90aW5nIGZvciBRaXV4dSdzIG1vc3QgcmVjZW50IHNvbHV0 aW9uIChtb3ZpbmcKYWxsIHRoZSBFREFDX0RFQlVHIGJpdHMgb3V0IG9mIHNreF9jb21tb24uYyku CgotVG9ueQo= From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-2.5 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_PASS,USER_AGENT_MUTT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 32939C43381 for ; Fri, 22 Mar 2019 17:55:08 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 08AE62083D for ; Fri, 22 Mar 2019 17:55:08 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728260AbfCVRzG (ORCPT ); Fri, 22 Mar 2019 13:55:06 -0400 Received: from mga11.intel.com ([192.55.52.93]:4115 "EHLO mga11.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1727801AbfCVRzG (ORCPT ); Fri, 22 Mar 2019 13:55:06 -0400 X-Amp-Result: UNKNOWN X-Amp-Original-Verdict: FILE UNKNOWN X-Amp-File-Uploaded: False Received: from orsmga003.jf.intel.com ([10.7.209.27]) by fmsmga102.fm.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 22 Mar 2019 10:55:05 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.60,256,1549958400"; d="scan'208";a="136401075" Received: from agluck-desk.sc.intel.com (HELO agluck-desk) ([10.3.52.160]) by orsmga003.jf.intel.com with ESMTP; 22 Mar 2019 10:55:05 -0700 Date: Fri, 22 Mar 2019 10:55:05 -0700 From: "Luck, Tony" To: Arnd Bergmann Cc: Borislav Petkov , Mauro Carvalho Chehab , James Morse , "Zhuo, Qiuxu" , "linux-edac@vger.kernel.org" , Linux Kernel Mailing List Subject: Re: [PATCH] EDAC, {skx|i10nm}_edac: Fix randconfig build error Message-ID: <20190322175504.GA11213@agluck-desk> References: <20190314215952.GA303@agluck-desk> <20190315094341.GA12523@zn.tnic> <20190315155718.GA17775@agluck-desk> <20190315173720.GC12523@zn.tnic> <20190315174956.GA20831@agluck-desk> <20190315180206.GD12523@zn.tnic> <20190315181108.GA21111@agluck-desk> <3908561D78D1C84285E8C5FCA982C28F7D53BD42@ORSMSX110.amr.corp.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Mar 22, 2019 at 03:00:25PM +0100, Arnd Bergmann wrote: > Sorry, this was my mistake, my email was garbled. The patch was > correct though: the idea here is not to change the Kconfig symbols > but to change the Makefile to do the right thing even when Kconfig > is set wrong. Well this does seem like a "clever" way out of the randconfig problem. New patch applies and when I set .config to have: CONFIG_EDAC_DEBUG=y CONFIG_EDAC_SKX=y CONFIG_EDAC_I10NM=m CONFIG_EDAC_SKX_COMMON=y I don't see any build errors. There are lots of "skx_" symbols in System.map But I'm not at all sure what happened to the I10NM driver. I don't see it mentioned in the output from "make". It didn't get built as a module (no ".ko" file for it). It doesn't seem to be built in (no ".o" in drivers/edac/built-in.a) So I added a #error line to the start of i10nm_edac.c and ran "make" again. Nothing. So, I don't think this is doing what you think it should do. Even if it did, it would seem very confusing to a user that asked for "one module, one built-in" in Kconfig, but found both built-in. Boris: I'm voting for Qiuxu's most recent solution (moving all the EDAC_DEBUG bits out of skx_common.c). -Tony