From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=2607:f8b0:4864:20::d44; helo=mail-io1-xd44.google.com; envelope-from=ard.biesheuvel@linaro.org; receiver=edk2-devel@lists.01.org Received: from mail-io1-xd44.google.com (mail-io1-xd44.google.com [IPv6:2607:f8b0:4864:20::d44]) (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 A3CEA21A07A80 for ; Mon, 24 Sep 2018 07:58:23 -0700 (PDT) Received: by mail-io1-xd44.google.com with SMTP id l25-v6so4724476ioj.0 for ; Mon, 24 Sep 2018 07:58:23 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=mime-version:references:in-reply-to:from:date:message-id:subject:to :cc; bh=1Jn52uruLEfI+sLaa3ODPKscKciSpOnl7oALrEcEdW8=; b=SbBq71b8uTJ/pwequrZplD+Es6AL5TY+NTELr/hbCi4gQoVsAIelA3aPEOqx96qmVk uF37W8ocFGpbuBbUx6PL7zJ2FkW8SOttnPOc9ZGo5oBt+J9Lc+O2SUFLRzA3v+WoP3ae 3SQgoGbcz0a1ak/Z5BJSauiKc4vVP2FqB4Y+A= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:mime-version:references:in-reply-to:from:date :message-id:subject:to:cc; bh=1Jn52uruLEfI+sLaa3ODPKscKciSpOnl7oALrEcEdW8=; b=IkzKulG58oRdz+7/jE8zPTY0OnLg6dXI/EDS+T0Ak84fTvye4VWxkfLF1RVgJEQir3 klVwW0VxH55t1TUomu2UzEN59vnCMg74fj5AGP5sO8ijs2EvizeHXOO4a2U+9VzMeYun oL52s9k+fXpn6hoDPJHkThbmcncZOvpQGORlG265PjbOJw+VFyKCy+eIOtzCyrHg9n7s SKhHUgmjWuUDOswjg5geZ1MGavuLVlIDQSmEOqtGyr7d/52ELrUFYAxz/ykWXtFVGo8A emOJN2cchlP/0iqfFjTz4GOLMCX3o369P2Zuxl4pc3PB94K2XJjWulgH63F9arvJWjQ/ jkbQ== X-Gm-Message-State: ABuFfogoRBAhO+5S+nL8vNBaIuEqKFxXtjmETP871S9/U0o+Ws2C4c/6 t7DqXUkyvdmqOqPB5d/bdRSIIFje9KH/r/0kvtuk6/9pvEk= X-Google-Smtp-Source: ACcGV63O02kP2yB4jof2MCZ1+DXuD1oKOH6i2ykedtV6RkSoesJ+R1Q1y4iZGwJbg8wKrnVhtxLZET1+/3XQPZl07wE= X-Received: by 2002:a6b:be83:: with SMTP id o125-v6mr8370063iof.173.1537801102196; Mon, 24 Sep 2018 07:58:22 -0700 (PDT) MIME-Version: 1.0 References: <20180907054202.24798-1-ard.biesheuvel@linaro.org> In-Reply-To: <20180907054202.24798-1-ard.biesheuvel@linaro.org> From: Ard Biesheuvel Date: Mon, 24 Sep 2018 16:58:08 +0200 Message-ID: To: "edk2-devel@lists.01.org" Cc: "Zeng, Star" , Jian J Wang , "Kinney, Michael D" , "Gao, Liming" , "Zhang, Chao B" , "Yao, Jiewen" , Laszlo Ersek , Leif Lindholm Subject: Re: [PATCH v2 0/3] remove most occurrences of ELILO on IPF PE/COFF header hack X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Mon, 24 Sep 2018 14:58:23 -0000 Content-Type: text/plain; charset="UTF-8" On Fri, 7 Sep 2018 at 07:42, Ard Biesheuvel wrote: > > Now that Itanium support has been dropped from EDK2, we can remove most > occurrences of the ELILO PE/COFF loader hack from the code base. > > Note that SecurityPkg appears to have four mostly identical implementations > of the PE/COFF measuring routine, so this may be another area for cleanup > later. > > Changes since v1: > - fix copy/paste error in patch #3 which went unnoticed due to the fact that > SecurityPkg.dsc does not cover the module in question for AARCH64 > - drop EdkCompatibilityPkg patch, it is likely to go away soon anyway > - add Reviewed-by tags > > Link: https://bugzilla.tianocore.org/show_bug.cgi?id=816 > > Cc: Star Zeng > Cc: Jian J Wang > Cc: Michael D Kinney > Cc: Liming Gao > Cc: Chao Zhang > Cc: Jiewen Yao > Cc: Laszlo Ersek > Cc: Leif Lindholm > > Ard Biesheuvel (3): > MdeModulePkg: remove PE/COFF header workaround for ELILO on IPF > MdePkg/BasePeCoffLib: remove PE/COFF header workaround for ELILO on > IPF > SecurityPkg: remove PE/COFF header workaround for ELILO on IPF > Patches 1-2 pushed as 997731e796f5..60eb6c6d2e01 (#3 was merged independently by Chao) Thanks all > .../Core/Dxe/Mem/MemoryProfileRecord.c | 31 +--------- > MdeModulePkg/Core/Dxe/Misc/MemoryProtection.c | 17 +----- > MdeModulePkg/Core/Dxe/Misc/PropertiesTable.c | 17 +----- > .../Core/PiSmmCore/MemoryAttributesTable.c | 17 +----- > .../Core/PiSmmCore/SmramProfileRecord.c | 31 +--------- > MdePkg/Library/BasePeCoffLib/BasePeCoff.c | 61 +++---------------- > .../DxeImageVerificationLib.c | 47 +++----------- > .../DxeTpmMeasureBootLib.c | 27 ++------ > SecurityPkg/Tcg/Tcg2Dxe/MeasureBootPeCoff.c | 27 ++------ > .../SecureBootConfigImpl.c | 25 ++------ > 10 files changed, 39 insertions(+), 261 deletions(-) > > -- > 2.17.1 >