Received: by 2002:a05:6358:d09b:b0:dc:cd0c:909e with SMTP id jc27csp6536246rwb; Mon, 5 Dec 2022 13:54:39 -0800 (PST) X-Google-Smtp-Source: AA0mqf6bq+vwNy3nbIJ44yuHDWAN91Pq6DwIOLd0Z8U8vYn7RGch3cOJ9XvN2yIYMqtkWSsOfOAh X-Received: by 2002:a17:902:e886:b0:188:7675:763a with SMTP id w6-20020a170902e88600b001887675763amr69035274plg.133.1670277279053; Mon, 05 Dec 2022 13:54:39 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1670277279; cv=none; d=google.com; s=arc-20160816; b=iOv+RJ7v2/0D/hJtGSaPQYDUzCNNHhXJ+Y2+d6+VJy2Czl4HDbmgeS6BloaBXcpXmZ Zvj/UVoUpiqcKsPRjGwnU/itAJQCdXJNJPrqaYL07lsK3rOdpJdFKeN2frNhaF8gg48k 6K0CRG4whFIiMor1mpZi0viXxbvPXoXeqRbKHTOhjgByFehdyhSe/bu2iS5jd9IOlTZI MUqawQ0QsWYyGTBvL5maB12cdxRu0OW5UhfuDrExA73mVsPnB5sLua93zPfpzwdZeaDX 5gOajs82JGvUD+Xn7Uq0dP6HS/CVK5qUT46BnROtk9coxKLk8E5NLwgm9ttkfjeWYd8o Wfxw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=fRZy8tyBFfkUb/uE9c9XkYxCY4moAtCYP4EgqoQtF5U=; b=xlM2ZPkiGDrlQj/NcI6kF/aQY8g/ujL8kk1KwAIyQery8ah205YDYh3/2KGrBVQPdx zwhZua+VDNE++ofVCuYjyDbAG91atfgnmm5TCTWEZWGD0JTJMdc+hWP2UmHEJV9tIfwO vJgabH6jUCMiebAZe7mVy1vffJ8OL3y1t0VYb4Ih07R9RVs7DPy/dT2x4OwOfEBIyUvh DqqkcZu+SBkKq+l+Og/RUguK5BJCp0mJv1II7NVmULcKPCiC0qbctsKJRUrZ1e7ME9cE Vp322lb8uIeHi1q2XWR+McwHdZWVv+0CYI/hTrUs4FUh57/TQ8jehWwmSmyVdz0GnOZh jAZw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@alien8.de header.s=dkim header.b=MDMnkWYR; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=alien8.de Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id mw17-20020a17090b4d1100b0020d457c5083si20363879pjb.147.2022.12.05.13.54.28; Mon, 05 Dec 2022 13:54:39 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@alien8.de header.s=dkim header.b=MDMnkWYR; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=alien8.de Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S232053AbiLEV1v (ORCPT + 81 others); Mon, 5 Dec 2022 16:27:51 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57200 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S233841AbiLEV1Y (ORCPT ); Mon, 5 Dec 2022 16:27:24 -0500 Received: from mail.skyhub.de (mail.skyhub.de [IPv6:2a01:4f8:190:11c2::b:1457]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 92D2F2AE3D for ; Mon, 5 Dec 2022 13:27:22 -0800 (PST) Received: from zn.tnic (p200300ea9733e7ff329c23fffea6a903.dip0.t-ipconnect.de [IPv6:2003:ea:9733:e7ff:329c:23ff:fea6:a903]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.skyhub.de (SuperMail on ZX Spectrum 128k) with ESMTPSA id C4B831EC0523; Mon, 5 Dec 2022 22:27:20 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=alien8.de; s=dkim; t=1670275640; h=from:from:reply-to:subject:subject:date:date:message-id:message-id: to:to:cc:cc:mime-version:mime-version:content-type:content-type: content-transfer-encoding:in-reply-to:in-reply-to: references:references; bh=fRZy8tyBFfkUb/uE9c9XkYxCY4moAtCYP4EgqoQtF5U=; b=MDMnkWYRBBy7pppX9cOlpojDwwhChvBzNksgPaKZIGdI84QrtT2vm6capcbG71HI1+FiiF ecAWqky2rfc2Vt4qivwMb1hVCUhgLu/+KcWTlC01qRHGz9RvojDn/rkwKJaU1Z58M0g5a7 qHVzg3URrdT8IAG9tEsx/3lLG+2TthY= Date: Mon, 5 Dec 2022 22:27:15 +0100 From: Borislav Petkov To: Ashok Raj Cc: X86-kernel , LKML Mailing List , Dave Hansen , Tony Luck , alison.schofield@intel.com, reinette.chatre@intel.com Subject: Re: [Patch V1 3/7] x86/microcode/core: Move microcode_check() to cpu/microcode/core.c Message-ID: References: <20221129210832.107850-1-ashok.raj@intel.com> <20221129210832.107850-4-ashok.raj@intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline In-Reply-To: X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_NONE,SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Dec 05, 2022 at 09:05:54AM -0800, Ashok Raj wrote: > I was trying to move this similar to how x86_read_arch_cap_msr() > moved from x86/kernel/cpu/cpu.h -> asm/cpu.h. But that is only a function prototype - not the *actual* function. > Keeping the usage local since there is just one caller to microcode_check() > but there are other users of get_cpu_cap() like in > arch/x86/xen/enlighten_pv.c which seems to be reaching out to > ../kernel/cpu/cpu.h. Yah, that's the single use outside of kernel/cpu/. Looks like a hack to me. :) But no worries, we will clean all that up sooner or later and get_cpu_cap() will disappear someday soon hopefully. > That said, what you say also makes sense. I'm fine with what you decide how > this should look. Yes pls. Thx. -- Regards/Gruss, Boris. https://people.kernel.org/tglx/notes-about-netiquette