From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: redhat.com, ip: 209.132.183.28, mailfrom: lersek@redhat.com) Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by groups.io with SMTP; Wed, 03 Jul 2019 14:07:59 -0700 Received: from smtp.corp.redhat.com (int-mx08.intmail.prod.int.phx2.redhat.com [10.5.11.23]) (using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits)) (No client certificate requested) by mx1.redhat.com (Postfix) with ESMTPS id DA1F1C057F3B; Wed, 3 Jul 2019 21:07:50 +0000 (UTC) Received: from lacos-laptop-7.usersys.redhat.com (ovpn-116-89.ams2.redhat.com [10.36.116.89]) by smtp.corp.redhat.com (Postfix) with ESMTP id D42F33DB6; Wed, 3 Jul 2019 21:07:48 +0000 (UTC) Subject: Re: [edk2-devel] [Patch v3 2/2] UefiCpuPkg/PiSmmCpuDxeSmm: Enable MM MP Protocol. To: devel@edk2.groups.io, eric.dong@intel.com Cc: Ray Ni References: <20190703024242.33572-1-eric.dong@intel.com> <20190703024242.33572-3-eric.dong@intel.com> From: "Laszlo Ersek" Message-ID: <2cb255d8-6f45-c6a7-dbb9-b533a7a17979@redhat.com> Date: Wed, 3 Jul 2019 23:07:47 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.9.1 MIME-Version: 1.0 In-Reply-To: <20190703024242.33572-3-eric.dong@intel.com> X-Scanned-By: MIMEDefang 2.84 on 10.5.11.23 X-Greylist: Sender IP whitelisted, not delayed by milter-greylist-4.5.16 (mx1.redhat.com [10.5.110.32]); Wed, 03 Jul 2019 21:07:55 +0000 (UTC) Content-Type: text/plain; charset=utf-8 Content-Language: en-US Content-Transfer-Encoding: 7bit On 07/03/19 04:42, Dong, Eric wrote: > v3 changes: > 1. Fix Token clean up too early caused CheckProcedure return error. > > v2 changes: > 1. Remove some duplicated global variables. > 2. Enhance token design to support multiple task trig for different APs at the same time. > > V1 changes: > REF: https://bugzilla.tianocore.org/show_bug.cgi?id=1937 > > Add MM Mp Protocol in PiSmmCpuDxeSmm driver. > > Cc: Ray Ni > Cc: Laszlo Ersek > Signed-off-by: Eric Dong > --- > UefiCpuPkg/PiSmmCpuDxeSmm/MpService.c | 536 ++++++++++++++++++- > UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.c | 11 + > UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.h | 160 +++++- > UefiCpuPkg/PiSmmCpuDxeSmm/PiSmmCpuDxeSmm.inf | 3 + > UefiCpuPkg/PiSmmCpuDxeSmm/SmmMp.c | 372 +++++++++++++ > UefiCpuPkg/PiSmmCpuDxeSmm/SmmMp.h | 286 ++++++++++ > 6 files changed, 1351 insertions(+), 17 deletions(-) > create mode 100644 UefiCpuPkg/PiSmmCpuDxeSmm/SmmMp.c > create mode 100644 UefiCpuPkg/PiSmmCpuDxeSmm/SmmMp.h The patch applies successfully now; it just adds a few whitespace errors: > Applying: UefiCpuPkg/PiSmmCpuDxeSmm: Enable MM MP Protocol. > .git/rebase-apply/patch:94: trailing whitespace. > if ((ApIndex != gSmmCpuPrivate->SmmCoreEntryContext.CurrentlyExecutingCpu) && > .git/rebase-apply/patch:98: trailing whitespace. > if ((ApIndex2 != gSmmCpuPrivate->SmmCoreEntryContext.CurrentlyExecutingCpu) && > .git/rebase-apply/patch:222: trailing whitespace. > if (mSmmMpSyncData->CpuData[Index].Present && > .git/rebase-apply/patch:606: trailing whitespace. > UINTN CpuIndex; > .git/rebase-apply/patch:1320: new blank line at EOF. > + > warning: 5 lines add whitespace errors. No need to resubmit the series just because of those; please clean them up on push. I'll follow up with regression test results. Thanks Laszlo