From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <ard.biesheuvel@linaro.org>
Received: from mail-it0-x22c.google.com (mail-it0-x22c.google.com
 [IPv6:2607:f8b0:4001:c0b::22c])
 (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits))
 (No client certificate requested)
 by ml01.01.org (Postfix) with ESMTPS id ED3E281F27
 for <edk2-devel@lists.01.org>; Thu,  9 Feb 2017 07:28:37 -0800 (PST)
Received: by mail-it0-x22c.google.com with SMTP id r185so18508093ita.0
 for <edk2-devel@lists.01.org>; Thu, 09 Feb 2017 07:28:37 -0800 (PST)
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google;
 h=mime-version:in-reply-to:references:from:date:message-id:subject:to
 :cc; bh=Aq69oejE6MVOVWIeE2Auc2YaIKKvH3wfwBRYJ1qwaec=;
 b=LObzOqbjQQaJEzXsEur/cQZWCLNJXx1QFDYbAmLoiOvHEXDS6NVETclFkdv4IsuD+S
 w/gAGKMFt6tsl2xoBYgEVLTv/2fYPTTqbVBlpyNV0I7hQ8dP33vkoUaQ/ku/c6zeVJw/
 3FrF8zFTi6qyTqbHd2A1Fd10C/xBhPMpBG+gQ=
X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed;
 d=1e100.net; s=20161025;
 h=x-gm-message-state:mime-version:in-reply-to:references:from:date
 :message-id:subject:to:cc;
 bh=Aq69oejE6MVOVWIeE2Auc2YaIKKvH3wfwBRYJ1qwaec=;
 b=RSzGc+7sv14ZoF2LGTfj2qV7sSjmt1PSH+hhMJvphXLjloklzAoSJcccxDlXv2Oc9S
 Wp32gPiRFhAwDz8r9piUcFnyjmeqqXPlf9pfTLbWZ0krshjOIvXJzqAFplrrMGVegMcc
 EIKlhFDOarvRH99JYL1jJAXeu/ZKSHtUWsXOKMwfduifc40WA5X8ZVD2FsBOyHnSzF77
 RJ5ZwZtouBBNt13A6lRCt/jr+fUXeUxd4N2dRGU+m3h/Mgr0V1f68Ae1h8RjsBwu2rCc
 ecQIejDeJgqk4vxpGReYuB+c+HMvk2KigqqP1bzW3+OWQojPqKmQ3jX+EkR5l0cX30/Y
 y7bQ==
X-Gm-Message-State: AIkVDXLk8tvypfLqRwhng3yEdOrxai4bf+S3m5N2KjyzA3emhmpAFS9Z04pwD6b99JvT6z16f2UWhcQq/LlVtgfx
X-Received: by 10.36.107.194 with SMTP id v185mr21743625itc.59.1486654117369; 
 Thu, 09 Feb 2017 07:28:37 -0800 (PST)
MIME-Version: 1.0
Received: by 10.107.144.139 with HTTP; Thu, 9 Feb 2017 07:28:36 -0800 (PST)
In-Reply-To: <74D8A39837DF1E4DA445A8C0B3885C503A8EC023@shsmsx102.ccr.corp.intel.com>
References: <1486624832-15736-1-git-send-email-jiewen.yao@intel.com>
 <74D8A39837DF1E4DA445A8C0B3885C503A8EBD52@shsmsx102.ccr.corp.intel.com>
 <CAKv+Gu8yDQSXadM5fUyw1yqNUKiaaosZ1E=CZ3ugiZqzBALwJw@mail.gmail.com>
 <CAKv+Gu88s2zZeHOb-sPMbd_usgkZfEePR4kpNYZk7-VvRsRuvw@mail.gmail.com>
 <CAKv+Gu8tiGxcXO_AZh98CFR0FeSz00HpkN6Ea-wszy9TFfRD-w@mail.gmail.com>
 <74D8A39837DF1E4DA445A8C0B3885C503A8EBEC3@shsmsx102.ccr.corp.intel.com>
 <CAKv+Gu_djRnM6XMMufgHSgk885MbP7KHWjiyqtr08qkbQURFjQ@mail.gmail.com>
 <74D8A39837DF1E4DA445A8C0B3885C503A8EBF20@shsmsx102.ccr.corp.intel.com>
 <CAKv+Gu8je9nimevNp2oAte1-sXx+kJiMTV5pKH_-cwCqgOCumA@mail.gmail.com>
 <74D8A39837DF1E4DA445A8C0B3885C503A8EC023@shsmsx102.ccr.corp.intel.com>
From: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Date: Thu, 9 Feb 2017 15:28:36 +0000
Message-ID: <CAKv+Gu96DE53HU=Odgwvn1HhfyrQRr_yYfK_CnHMVeWpOeXpbQ@mail.gmail.com>
To: "Yao, Jiewen" <jiewen.yao@intel.com>
Cc: "Tian, Feng" <feng.tian@intel.com>, 
 "edk2-devel@lists.01.org" <edk2-devel@lists.01.org>,
 Leif Lindholm <leif.lindholm@linaro.org>, 
 "Kinney, Michael D" <michael.d.kinney@intel.com>, "Fan,
 Jeff" <jeff.fan@intel.com>, "Zeng, Star" <star.zeng@intel.com>
Subject: Re: [PATCH V3 0/4] DXE Memory Protection
X-BeenThere: edk2-devel@lists.01.org
X-Mailman-Version: 2.1.21
Precedence: list
List-Id: EDK II Development  <edk2-devel.lists.01.org>
List-Unsubscribe: <https://lists.01.org/mailman/options/edk2-devel>,
 <mailto:edk2-devel-request@lists.01.org?subject=unsubscribe>
List-Archive: <http://lists.01.org/pipermail/edk2-devel/>
List-Post: <mailto:edk2-devel@lists.01.org>
List-Help: <mailto:edk2-devel-request@lists.01.org?subject=help>
List-Subscribe: <https://lists.01.org/mailman/listinfo/edk2-devel>,
 <mailto:edk2-devel-request@lists.01.org?subject=subscribe>
X-List-Received-Date: Thu, 09 Feb 2017 15:28:38 -0000
Content-Type: text/plain; charset=UTF-8

On 9 February 2017 at 15:27, Yao, Jiewen <jiewen.yao@intel.com> wrote:
> 1)       That is great. I appreciate your quick response and help.
>
> I will drop my patch for ARM 2/4, and wait for yours.
>

OK

>
>
> 2)       For ImageEnd alignment issue, I agree with you.
>
> I plan to round up with:
>
> ImageRecord->ImageSize = ALIGN_VALUE(LoadedImage->ImageSize,
> SectionAlignment);
>
> before SetUefiImageProtectionAttributes (ImageRecord, Protect);
>

Great, that should fix my issue!

Thanks,
Ard.