From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from SMTP02.CITRIX.COM (smtp02.citrix.com [66.165.176.63]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id 0099021E1B75A for ; Thu, 21 Sep 2017 08:02:04 -0700 (PDT) X-IronPort-AV: E=Sophos;i="5.42,425,1500940800"; d="scan'208";a="448715279" Date: Thu, 21 Sep 2017 16:05:02 +0100 From: Anthony PERARD To: "Wang, Jian J" CC: "Yao, Jiewen" , "edk2-devel@lists.01.org" , "Kinney, Michael D" , Laszlo Ersek , "Dong, Eric" , "Zeng, Star" Message-ID: <20170921150502.GC1859@perard.uk.xensource.com> References: <20170919061013.15976-1-jian.j.wang@intel.com> <74D8A39837DF1E4DA445A8C0B3885C503A9BF448@shsmsx102.ccr.corp.intel.com> MIME-Version: 1.0 In-Reply-To: User-Agent: Mutt/1.9.0 (2017-09-02) Subject: Re: [PATCH 0/2] Fixe out-of-sync issue between GCD and CPU driver X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.22 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 21 Sep 2017 15:02:05 -0000 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline On Wed, Sep 20, 2017 at 07:53:36AM +0000, Wang, Jian J wrote: > Jiewen, > > Yes. The patch has passed the validation of the UEFI booting Windows 10, Windows 7 and Fedora 25. > > If no comments from you and others, I'll push the patch to upstream soon. > Hi, I think this patch series does not build, while building OVMF we have those errors: GNUmakefile:527: recipe for target '/home/osstest/build.113661.build-amd64/xen/tools/firmware/ovmf-dir-remote/Build/OvmfX64/DEBUG_GCC49/X64/UefiCpuPkg/CpuDxe/CpuDxe/OUTPUT/Cp uPageTable.obj' failed make[8]: Leaving directory '/home/osstest/build.113661.build-amd64/xen/tools/firmware/ovmf-dir-remote/Build/OvmfX64/DEBUG_GCC49/X64/UefiCpuPkg/CpuDxe/CpuDxe' /home/osstest/build.113661.build-amd64/xen/tools/firmware/ovmf-dir-remote/UefiCpuPkg/CpuDxe/CpuPageTable.c: In function 'RefreshGcdMemoryAttributesFromPaging': /home/osstest/build.113661.build-amd64/xen/tools/firmware/ovmf-dir-remote/UefiCpuPkg/CpuDxe/CpuPageTable.c:852:10: error: 'DoUpdate' may be used uninitialized in this functio n [-Werror=maybe-uninitialized] if (DoUpdate) { ^ /home/osstest/build.113661.build-amd64/xen/tools/firmware/ovmf-dir-remote/UefiCpuPkg/CpuDxe/CpuPageTable.c:853:9: error: 'Capabilities' may be used uninitialized in this func tion [-Werror=maybe-uninitialized] gDS->SetMemorySpaceCapabilities (BaseAddress, Length, Capabilities); ^ /home/osstest/build.113661.build-amd64/xen/tools/firmware/ovmf-dir-remote/UefiCpuPkg/CpuDxe/CpuPageTable.c:855:90: error: 'Attributes' may be used uninitialized in this funct ion [-Werror=maybe-uninitialized] DEBUG ((DEBUG_INFO, "Update memory space attribute: [%02d] %016lx - %016lx (%08lx -> %08lx)\r\n", ^ cc1: all warnings being treated as errors Full log can be found at: http://logs.test-lab.xenproject.org/osstest/logs/113661/build-amd64/6.ts-xen-build.log Regards, -- Anthony PERARD