From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from us-smtp-delivery-1.mimecast.com (us-smtp-delivery-1.mimecast.com [207.211.31.81]) by mx.groups.io with SMTP id smtpd.web10.8261.1592577344302933739 for ; Fri, 19 Jun 2020 07:35:44 -0700 Authentication-Results: mx.groups.io; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b=DmgpiCAU; spf=pass (domain: redhat.com, ip: 207.211.31.81, mailfrom: lersek@redhat.com) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1592577343; 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:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=m+zeloT7CtIQQFGUzODBDPfZxrhYeE8AysvleJ95IKU=; b=DmgpiCAUxR+OUKhmSzGgNWsx1H7idLx93FQ51WUr3BOpbAj1Sb7eC7H41yn9tNfaHQqwWf v+svxscubfKVhaz19a4LVgHlLTKkgVp/+SRybBFMbobhhOWrfMcY3rrQXVm/toVr+ZygCk e9HHGeFTuVEyviG1u6OV6tAzEtXzIdk= Received: from mimecast-mx01.redhat.com (mimecast-mx01.redhat.com [209.132.183.4]) (Using TLS) by relay.mimecast.com with ESMTP id us-mta-291-56p7oJz_NXKgwkICxAf-ag-1; Fri, 19 Jun 2020 10:35:38 -0400 X-MC-Unique: 56p7oJz_NXKgwkICxAf-ag-1 Received: from smtp.corp.redhat.com (int-mx05.intmail.prod.int.phx2.redhat.com [10.5.11.15]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mimecast-mx01.redhat.com (Postfix) with ESMTPS id E6438801503; Fri, 19 Jun 2020 14:35:36 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-115-60.ams2.redhat.com [10.36.115.60]) by smtp.corp.redhat.com (Postfix) with ESMTP id ECC055BAC7; Fri, 19 Jun 2020 14:35:35 +0000 (UTC) Subject: Re: [edk2-devel] [PATCH v2 0/2] AMD procesor MSR_IA32_MISC_ENABLE To: devel@edk2.groups.io, eric.dong@intel.com, Garrett Kirkendall Cc: "Ni, Ray" References: <20200618152245.6483-1-Garrett.Kirkendall@amd.com> From: "Laszlo Ersek" Message-ID: <6da19b79-e862-b782-a3df-9933e386b4a7@redhat.com> Date: Fri, 19 Jun 2020 16:35:34 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Firefox/52.0 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: X-Scanned-By: MIMEDefang 2.79 on 10.5.11.15 X-Mimecast-Spam-Score: 0 X-Mimecast-Originator: redhat.com Content-Language: en-US Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Hi Garrett, On 06/19/20 03:00, Dong, Eric wrote: > Hi Garrett, > > I create a pull request to verify your changes and it reports some > errors for your changes. https://github.com/tianocore/edk2/pull/710 > > please help to resolve these errors before sending your new version > changes, also you can create your PR to verify your new changes. It seems like the standalone "PcAtChipsetPkg.dsc" build failed. The reason seems to be that PcAtChipsetPkg.dsc contains a lib class resolution like LocalApicLib|UefiCpuPkg/Library/BaseXApicLib/BaseXApicLib.inf but does not contain a UefiCpuLib class resolution. Speaking more generally, the following DSC files reference at least one of "BaseXApicLib.inf" and "BaseXApicX2ApicLib.inf": - edk2: IntelFsp2WrapperPkg/IntelFsp2WrapperPkg.dsc OvmfPkg/OvmfPkgIa32.dsc OvmfPkg/OvmfPkgIa32X64.dsc OvmfPkg/OvmfPkgX64.dsc OvmfPkg/OvmfXen.dsc PcAtChipsetPkg/PcAtChipsetPkg.dsc SourceLevelDebugPkg/SourceLevelDebugPkg.dsc UefiCpuPkg/UefiCpuPkg.dsc UefiPayloadPkg/UefiPayloadPkgIa32.dsc UefiPayloadPkg/UefiPayloadPkgIa32X64.dsc - edk2-platforms: Platform/Intel/MinPlatformPkg/Include/Dsc/CoreCommonLib.dsc Platform/Intel/QuarkPlatformPkg/Quark.dsc Platform/Intel/QuarkPlatformPkg/QuarkMin.dsc Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgIA32.dsc Platform/Intel/Vlv2TbltDevicePkg/PlatformPkgX64.dsc Among these files, the following ones do *not* already have the UefiCpuLib|UefiCpuPkg/Library/BaseUefiCpuLib/BaseUefiCpuLib.inf lib class resolution: - edk2: PcAtChipsetPkg/PcAtChipsetPkg.dsc [found by CI] SourceLevelDebugPkg/SourceLevelDebugPkg.dsc - edk2-platforms: [none] Therefore, this patch set is sound, it is just incomplete. We need two patches *prepended*: one patch that adds the UefiCpuLib resolution to "PcAtChipsetPkg/PcAtChipsetPkg.dsc", and another that adds the same resolution to "SourceLevelDebugPkg/SourceLevelDebugPkg.dsc". Then the current (v2) patches can be included verbatim as patch v3 #3, and patch v3 #4. Garrett: you can also invoke a personal CI run -- just submit a github.com pull request with your v3 patches, before posting them to the list. You will not be able to set the "push" label on your PR, so the CI system will auto-close the PR (without merging it). But, it will tell you the build result (pass or failure). Thanks! Laszlo