From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: mx.groups.io; dkim=missing; spf=pass (domain: intel.com, ip: 134.134.136.126, mailfrom: chao.b.zhang@intel.com) Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by groups.io with SMTP; Tue, 28 May 2019 17:34:42 -0700 X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False Received: from fmsmga002.fm.intel.com ([10.253.24.26]) by orsmga106.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 28 May 2019 17:34:41 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.60,525,1549958400"; d="scan'208,217";a="179396377" Received: from fmsmsx105.amr.corp.intel.com ([10.18.124.203]) by fmsmga002.fm.intel.com with ESMTP; 28 May 2019 17:34:41 -0700 Received: from shsmsx101.ccr.corp.intel.com (10.239.4.153) by FMSMSX105.amr.corp.intel.com (10.18.124.203) with Microsoft SMTP Server (TLS) id 14.3.408.0; Tue, 28 May 2019 17:34:40 -0700 Received: from shsmsx102.ccr.corp.intel.com ([169.254.2.249]) by SHSMSX101.ccr.corp.intel.com ([169.254.1.129]) with mapi id 14.03.0415.000; Wed, 29 May 2019 08:34:38 +0800 From: "Zhang, Chao B" To: "Gao, Liming" , "Xu, Wei6" , "devel@edk2.groups.io" CC: "Wang, Jian J" , "Wu, Hao A" , "Kinney, Michael D" Subject: Re: [edk2-devel][Patch 0/7] Implement Capsule On Disk. Thread-Topic: [edk2-devel][Patch 0/7] Implement Capsule On Disk. Thread-Index: AQHVFXeiVx/DiFk22U+NelUmmGI6+KaAu1GAgACGN4A= Date: Wed, 29 May 2019 00:34:38 +0000 Message-ID: References: <20190528170546.15876-1-wei6.xu@intel.com> <4A89E2EF3DFEDB4C8BFDE51014F606A14E456757@SHSMSX104.ccr.corp.intel.com> In-Reply-To: <4A89E2EF3DFEDB4C8BFDE51014F606A14E456757@SHSMSX104.ccr.corp.intel.com> Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ctpclassification: CTP_NT x-titus-metadata-40: eyJDYXRlZ29yeUxhYmVscyI6IiIsIk1ldGFkYXRhIjp7Im5zIjoiaHR0cDpcL1wvd3d3LnRpdHVzLmNvbVwvbnNcL0ludGVsMyIsImlkIjoiMmYyN2IyZWUtNTk5MC00NmYzLTk4NzMtZmMwZjJiOTBjNThhIiwicHJvcHMiOlt7Im4iOiJDVFBDbGFzc2lmaWNhdGlvbiIsInZhbHMiOlt7InZhbHVlIjoiQ1RQX05UIn1dfV19LCJTdWJqZWN0TGFiZWxzIjpbXSwiVE1DVmVyc2lvbiI6IjE3LjEwLjE4MDQuNDkiLCJUcnVzdGVkTGFiZWxIYXNoIjoic3R3a2ZTWHkrZ2ZaTlJZMlgwYktqVkRPS3JiMk9DNlJid0htTnBcL2U0QXBrZWM2WVgrd09RWVppNTNIUlIrUVcifQ== dlp-product: dlpe-windows dlp-version: 11.0.600.7 dlp-reaction: no-action x-originating-ip: [10.239.127.40] MIME-Version: 1.0 Return-Path: chao.b.zhang@intel.com Content-Language: en-US Content-Type: multipart/alternative; boundary="_000_FF72C7E4248F3C4E9BDF19D4918E90F24DEBAF6Eshsmsx102ccrcor_" --_000_FF72C7E4248F3C4E9BDF19D4918E90F24DEBAF6Eshsmsx102ccrcor_ Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable Yes, will check in after Q2 tag From: Gao, Liming Sent: Wednesday, May 29, 2019 8:34 AM To: Xu, Wei6 ; devel@edk2.groups.io Cc: Wang, Jian J ; Wu, Hao A ; K= inney, Michael D ; Zhang, Chao B Subject: RE: [edk2-devel][Patch 0/7] Implement Capsule On Disk. So, this feature is for next Q3 stable tag. Right? > -----Original Message----- > From: Xu, Wei6 > Sent: Wednesday, May 29, 2019 1:06 AM > To: devel@edk2.groups.io > Cc: Wang, Jian J >; W= u, Hao A >; Kinney, Michael D= >; Gao, > Liming >; Zhang, Chao B= > > Subject: [edk2-devel][Patch 0/7] Implement Capsule On Disk. > > BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3D1852 > > This patch set implements Capsule On Disk. > Depends on whether platform supports Capsule-In-Ram, Capsule On Disk feat= ure is composed of 2 solutions: > Solution A): Load capsules out of TCB, rely on UpdateCapsule() runtime se= rvice to deliver Capsule-On-Disk. > Solution B): Relocate capsules into a temp file which will be stored in r= oot directory on a platform specific storage device. > Leverage existing storage stack in PEI to load all capsule on disk images= and create capsule hobs for the capsules. > This solution has bigger TCB, but can work without Capsule-In-RAM support= . > > > Cc: Jian J Wang > > Cc: Hao A Wu > > Cc: Michael D Kinney > > Cc: Liming Gao > > Cc: Chao B Zhang > > > xuwei6 (7): > MdePkg: Add Pei Boot In CapsuleOnDisk Mode Ppi definition. > MdeModulePkg: Add Capsule On Disk related definition. > MdeModulePkg: Add CapsuleOnDiskLoadPei PEIM. > MdeModulePkg/BdsDxe: Support Capsule On Disk. > MdeModulePkg/CapsuleRuntimeDxe: Introduce PCD to control this feature. > MdeModulePkg/DxeIpl: Support Capsule On Disk. > MdeModulePkg: Add Capsule On Disk APIs into CapsuleLib. > > MdeModulePkg/Core/DxeIplPeim/DxeIpl.h | 3 +- > MdeModulePkg/Core/DxeIplPeim/DxeIpl.inf | 20 +- > MdeModulePkg/Core/DxeIplPeim/DxeLoad.c | 37 +- > MdeModulePkg/Include/Library/CapsuleLib.h | 94 +- > MdeModulePkg/Include/Ppi/CapsuleOnDisk.h | 48 + > .../Library/DxeCapsuleLibFmp/CapsuleOnDisk.c | 1984 ++++++++++++++= ++++++ > .../Library/DxeCapsuleLibFmp/CapsuleOnDisk.h | 63 + > .../Library/DxeCapsuleLibFmp/DxeCapsuleLib.c | 51 +- > .../Library/DxeCapsuleLibFmp/DxeCapsuleLib.inf | 20 +- > .../DxeCapsuleLibFmp/DxeCapsuleProcessLib.c | 121 +- > .../Library/DxeCapsuleLibFmp/DxeCapsuleReportLib.c | 66 +- > .../DxeCapsuleLibFmp/DxeRuntimeCapsuleLib.inf | 3 +- > .../Library/DxeCapsuleLibNull/DxeCapsuleLibNull.c | 85 +- > MdeModulePkg/MdeModulePkg.dec | 43 + > MdeModulePkg/MdeModulePkg.dsc | 4 + > MdeModulePkg/MdeModulePkg.uni | 32 + > MdeModulePkg/Universal/BdsDxe/BdsDxe.inf | 3 +- > MdeModulePkg/Universal/BdsDxe/BdsEntry.c | 6 +- > .../CapsuleOnDiskLoadPei/CapsuleOnDiskLoadPei.c | 442 +++++ > .../CapsuleOnDiskLoadPei/CapsuleOnDiskLoadPei.inf | 64 + > .../CapsuleOnDiskLoadPei/CapsuleOnDiskLoadPei.uni | 15 + > .../CapsuleOnDiskLoadPeiExtra.uni | 14 + > .../CapsuleRuntimeDxe/CapsuleRuntimeDxe.inf | 1 + > .../Universal/CapsuleRuntimeDxe/CapsuleService.c | 10 +- > MdePkg/Include/Ppi/BootInRecoveryMode.h | 9 +- > MdePkg/MdePkg.dec | 3 + > 26 files changed, 3200 insertions(+), 41 deletions(-) > create mode 100644 MdeModulePkg/Include/Ppi/CapsuleOnDisk.h > create mode 100644 MdeModulePkg/Library/DxeCapsuleLibFmp/CapsuleOnDisk.c > create mode 100644 MdeModulePkg/Library/DxeCapsuleLibFmp/CapsuleOnDisk.h > create mode 100644 MdeModulePkg/Universal/CapsuleOnDiskLoadPei/CapsuleOn= DiskLoadPei.c > create mode 100644 MdeModulePkg/Universal/CapsuleOnDiskLoadPei/CapsuleOn= DiskLoadPei.inf > create mode 100644 MdeModulePkg/Universal/CapsuleOnDiskLoadPei/CapsuleOn= DiskLoadPei.uni > create mode 100644 MdeModulePkg/Universal/CapsuleOnDiskLoadPei/CapsuleOn= DiskLoadPeiExtra.uni > > -- > 2.16.2.windows.1 --_000_FF72C7E4248F3C4E9BDF19D4918E90F24DEBAF6Eshsmsx102ccrcor_ Content-Type: text/html; charset="us-ascii" Content-Transfer-Encoding: quoted-printable

Yes, will check in after Q2 tag

 =

From: Gao, Liming
Sent: Wednesday, May 29, 2019 8:34 AM
To: Xu, Wei6 <wei6.xu@intel.com>; devel@edk2.groups.io
Cc: Wang, Jian J <jian.j.wang@intel.com>; Wu, Hao A <hao.a.= wu@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>; Zha= ng, Chao B <chao.b.zhang@intel.com>
Subject: RE: [edk2-devel][Patch 0/7] Implement Capsule On Disk.=

 

So, this feature is = ;for next Q3 stable tag. Right?

> -----Original Message-----
> From: Xu, Wei6
> Sent: Wednesday, May = 29, 2019 1:06 AM
> To: devel@edk2.groups.io
> Cc: Wang, Jian J = ;<jian.j.wang@intel.com>= ; Wu, Hao A <h= ao.a.wu@intel.com>; Kinney, Michael D <michael.d.kinney@intel.com>;=  Gao,
> Liming <liming.gao@intel.com>; Zhang, Chao&nbs= p;B <chao.b.zhang@intel.c= om>
> Subject: [edk2-devel][Patch&nbs= p;0/7] Implement Capsule On Disk.

> BZ: https://bugzilla.tianocore.org/show_= bug.cgi?id=3D1852

> This patch set implem= ents Capsule On Disk.
> Depends on whether pl= atform supports Capsule-In-Ram, Capsule On Disk&nb= sp;feature is composed of 2 solutions:
> Solution A): Load cap= sules out of TCB, rely on UpdateCapsule()&nbs= p;runtime service to deliver Capsule-On-Disk. > Solution B): Relocate = ;capsules into a temp file which will be=  stored in root directory on a platform&= nbsp;specific storage device.
> Leverage existing storage&= nbsp;stack in PEI to load all capsule on=  disk images and create capsule hobs for=  the capsules.
> This solution has big= ger TCB, but can work without Capsule-In-RAM&= nbsp;support.


> Cc: Jian J Wang = <jian.j.wang@intel.com><= /span>
> Cc: Hao A Wu <= ;hao.a.wu@intel.com> > Cc: Michael D Kinney&= nbsp;<michael.d.kinney@int= el.com>
> Cc: Liming Gao <liming.gao@intel.com> > Cc: Chao B Zhang = ;<chao.b.zhang@intel.com&g= t;

> xuwei6 (7):
>   MdePkg: Add Pe= i Boot In CapsuleOnDisk Mode Ppi definition.<= /span>
>   MdeModulePkg: Add&n= bsp;Capsule On Disk related definition.
>   MdeModulePkg: Add&n= bsp;CapsuleOnDiskLoadPei PEIM.
>   MdeModulePkg/BdsDxe:&nbs= p;Support Capsule On Disk.
>   MdeModulePkg/CapsuleRunt= imeDxe: Introduce PCD to control this feature= .
>   MdeModulePkg/DxeIpl:&nbs= p;Support Capsule On Disk.
>   MdeModulePkg: Add&n= bsp;Capsule On Disk APIs into CapsuleLib.
>  MdeModulePkg/Core/DxeIplPeim/D= xeIpl.h           &n= bsp;  |    3 +-
>  MdeModulePkg/Core/DxeIplPeim/D= xeIpl.inf           =  |   20 +-
>  MdeModulePkg/Core/DxeIplPeim/D= xeLoad.c           &= nbsp; |   37 +-
>  MdeModulePkg/Include/Library/C= apsuleLib.h          |&nb= sp;  94 +-
>  MdeModulePkg/Include/Ppi/Capsu= leOnDisk.h           = ;|   48 +
>  .../Library/DxeCapsuleLibFmp/C= apsuleOnDisk.c       | 1984 &#= 43;++++++++++++++&#= 43;++++
>  .../Library/DxeCapsuleLibFmp/C= apsuleOnDisk.h       |   = 63 +
>  .../Library/DxeCapsuleLibFmp/D= xeCapsuleLib.c       |   = 51 +-
>  .../Library/DxeCapsuleLibFmp/D= xeCapsuleLib.inf     |   20 &#= 43;-
>  .../DxeCapsuleLibFmp/DxeCapsul= eProcessLib.c        |  1= 21 +-
>  .../Library/DxeCapsuleLibFmp/D= xeCapsuleReportLib.c |   66 +-
>  .../DxeCapsuleLibFmp/DxeRuntim= eCapsuleLib.inf      |    = ;3 +-
>  .../Library/DxeCapsuleLibNull/= DxeCapsuleLibNull.c  |   85 +-
>  MdeModulePkg/MdeModulePkg.dec&= nbsp;           &nbs= p;         |   = 43 +
>  MdeModulePkg/MdeModulePkg.dsc&= nbsp;           &nbs= p;         |   =  4 +
>  MdeModulePkg/MdeModulePkg.uni&= nbsp;           &nbs= p;         |   = 32 +
>  MdeModulePkg/Universal/BdsDxe/= BdsDxe.inf           = ;|    3 +-
>  MdeModulePkg/Universal/BdsDxe/= BdsEntry.c           = ;|    6 +-
>  .../CapsuleOnDiskLoadPei/Capsu= leOnDiskLoadPei.c    |  442 ++&#= 43;++
>  .../CapsuleOnDiskLoadPei/Capsu= leOnDiskLoadPei.inf  |   64 +
>  .../CapsuleOnDiskLoadPei/Capsu= leOnDiskLoadPei.uni  |   15 +
>  .../CapsuleOnDiskLoadPeiExtra.= uni            =       |   14 +=
>  .../CapsuleRuntimeDxe/CapsuleR= untimeDxe.inf        |  &= nbsp; 1 +
>  .../Universal/CapsuleRuntimeDx= e/CapsuleService.c   |   10 +-
>  MdePkg/Include/Ppi/BootInRecov= eryMode.h           =  |    9 +-
>  MdePkg/MdePkg.dec  &= nbsp;           &nbs= p;            &= nbsp;      |    3 &#= 43;
>  26 files changed,&nb= sp;3200 insertions(+), 41 deletions(-)
>  create mode 100644&n= bsp;MdeModulePkg/Include/Ppi/CapsuleOnDisk.h
>  create mode 100644&n= bsp;MdeModulePkg/Library/DxeCapsuleLibFmp/CapsuleOnDisk.c
>  create mode 100644&n= bsp;MdeModulePkg/Library/DxeCapsuleLibFmp/CapsuleOnDisk.h
>  create mode 100644&n= bsp;MdeModulePkg/Universal/CapsuleOnDiskLoadPei/CapsuleOnDiskLoadPei.c
>  create mode 100644&n= bsp;MdeModulePkg/Universal/CapsuleOnDiskLoadPei/CapsuleOnDiskLoadPei.inf
>  create mode 100644&n= bsp;MdeModulePkg/Universal/CapsuleOnDiskLoadPei/CapsuleOnDiskLoadPei.uni
>  create mode 100644&n= bsp;MdeModulePkg/Universal/CapsuleOnDiskLoadPei/CapsuleOnDiskLoadPeiExtra.u= ni

> --
> 2.16.2.windows.1

--_000_FF72C7E4248F3C4E9BDF19D4918E90F24DEBAF6Eshsmsx102ccrcor_--