Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   "Xu, Min M" <min.m.xu@intel.com>
To:     "Kirill A. Shutemov" <kirill.shutemov@linux.intel.com>,
        Borislav Petkov <bp@suse.de>,
        "Gao, Jiaqi" <jiaqi.gao@intel.com>
CC:     Dionna Amalie Glaze <dionnaglaze@google.com>,
        Michael Roth <michael.roth@amd.com>,
        Borislav Petkov <bp@alien8.de>,
        "Kirill A. Shutemov" <kirill@shutemov.name>,
        "Lutomirski, Andy" <luto@kernel.org>,
        "Christopherson,, Sean" <seanjc@google.com>,
        Andrew Morton <akpm@linux-foundation.org>,
        "Rodel, Jorg" <jroedel@suse.de>, Ard Biesheuvel <ardb@kernel.org>,
        Andi Kleen <ak@linux.intel.com>,
        "Kuppuswamy Sathyanarayanan" 
        <sathyanarayanan.kuppuswamy@linux.intel.com>,
        David Rientjes <rientjes@google.com>,
        Vlastimil Babka <vbabka@suse.cz>,
        Tom Lendacky <thomas.lendacky@amd.com>,
        Thomas Gleixner <tglx@linutronix.de>,
        "Peter Zijlstra" <peterz@infradead.org>,
        Paolo Bonzini <pbonzini@redhat.com>,
        "Ingo Molnar" <mingo@redhat.com>,
        Varad Gautam <varad.gautam@suse.com>,
        "Dario Faggioli" <dfaggioli@suse.com>,
        "Hansen, Dave" <dave.hansen@intel.com>,
        "Mike Rapoport" <rppt@kernel.org>,
        David Hildenbrand <david@redhat.com>,
        "x86@kernel.org" <x86@kernel.org>,
        "linux-mm@kvack.org" <linux-mm@kvack.org>,
        "linux-coco@lists.linux.dev" <linux-coco@lists.linux.dev>,
        "linux-efi@vger.kernel.org" <linux-efi@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>
Subject: RE: [PATCHv5 06/12] x86/boot/compressed: Handle unaccepted memory
Thread-Topic: [PATCHv5 06/12] x86/boot/compressed: Handle unaccepted memory
Thread-Index: AQHYZtgcM5CpDUl/gkuqqiQv4xAy4a0hDHpA
Date:   Mon, 16 May 2022 06:46:53 +0000
Message-ID: <PH0PR11MB5064B561086BE6350CC1DCDCC5CF9@PH0PR11MB5064.namprd11.prod.outlook.com>
References: <20220425033934.68551-1-kirill.shutemov@linux.intel.com>
 <20220425033934.68551-7-kirill.shutemov@linux.intel.com>
 <YnE4ZzzVrxUnr3Uv@zn.tnic>
 <20220506153013.e6v4q2qhuhqumfiu@box.shutemov.name>
 <YnpGnMoviGoK4Ucq@zn.tnic>
 <CAAH4kHYRxgUNnGRUO473q02q3akLzgiTvbA2qKEP5jq6jFV-uA@mail.gmail.com>
 <Yn4ed1gupKmNz2jn@zn.tnic>
 <20220513144515.fx2cvo3rjued3vy5@black.fi.intel.com>
In-Reply-To: <20220513144515.fx2cvo3rjued3vy5@black.fi.intel.com>
Accept-Language: en-US
Content-Language: en-US
dlp-product: dlpe-windows
dlp-reaction: no-action
dlp-version: 11.6.401.20
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?jLJDRAwoHRjM+79WQL+pmjuaNy7brbnRj3ztQYegWElXh9Bc4bK/G7wjCdrM?=
 =?us-ascii?Q?eipC29QB9NxExNSAdHIlpdNT09HFxrDMLWxSpEA8+C3oIVu+elB47goYQ3jq?=
 =?us-ascii?Q?T8s7shRdPc62yx5WYnveZLRBQM2BRdgZLNT1xzyc2aoUPAW8T6lcJ3joh0Op?=
 =?us-ascii?Q?kCISV1jR/iG87SIZueNM3QZ5LCwLzIOlt2pcDRWvQH8fRNR/V4cI185Jtv1g?=
 =?us-ascii?Q?ntRozuSs0r9/gq6laMe8E7NIgN8YQWmWADgqiaEirvsMwilz3SvQlXmSTMS6?=
 =?us-ascii?Q?QP8/Dx0Nj4Vk3ZuNLyOJiMPgxCkuYzw3cybW90R6EXb0cU58hMT+qxU6R8rP?=
 =?us-ascii?Q?DI7SCQRWTEr1JLuki4kXsLRUFXSfYTNAPmvIuqxs+ATZhttGvYXHWwWCqP1Q?=
 =?us-ascii?Q?ks8jKh00u+fcU97KQb5rqn3pxzm37UNE2O5D8FayJK/ypZFtWy6PAFIbhRcK?=
 =?us-ascii?Q?S/bXEeWijPfad9824P+cwHY+pnKUzOEPD1efSBTbcmeVBSfOj9gRr8Zwa9Mn?=
 =?us-ascii?Q?+pq8JtgXySMWYYTSfQ2OMZfrsWB7WcljP01TBKsQhuCNjHc8P/Cz+tlk+PRA?=
 =?us-ascii?Q?upV5j8PbTICuDYFe2dRunFNL7KxEASmUxp6r/ZuUWTusesmKyROGUkeXsboY?=
 =?us-ascii?Q?7BNK8H/2yCYcV/WVsH73K+hZw5AKiyokSjMtFMRhNyZGUg4ACyzKE+OjMhJl?=
 =?us-ascii?Q?6qiZx3BK6A2tFu5PxFzpycXisoJ6yLSkrqz4vnD9gwyJ8/+enOVLIsckCAFm?=
 =?us-ascii?Q?V5GPUZ2iB1J3G1vtIg2WpXV0LDcY7WI/M8pqLMXab81YlpmCO0DbODZrlgoq?=
 =?us-ascii?Q?c6LMnxW6aKwVOww8dJ4jRbCyuSNcidbV8cjQ946UqRoH2T7BliWkasVl8QP3?=
 =?us-ascii?Q?+4CMRmG4FLZTkkKdy2AuTA2hP0ruzU3AYVDkt9G2qqYEROd/Ca43+tfnjyVq?=
 =?us-ascii?Q?K5QmJRjZhNjYbAFNB9VasDN4GrF+0EKIfF9615e/jXWmj1bkcaQOtWj/u170?=
 =?us-ascii?Q?o7klL1F5NGJRBRx09/cfN434Obk7K8GMfiVanVWxC+PkS/kk/CuD3V5lhYz+?=
 =?us-ascii?Q?UmwIvFy4BlqBG04PT8aTE3db9GUoGzlX+jJm2uxYK3MhV9SEEe+rnVuT64iQ?=
 =?us-ascii?Q?vLVIYKso+jh/ryqXkeKBDRtLPDRPqcXpIy3x0oKAcoQGIdAXl5LLVqVXdJrC?=
 =?us-ascii?Q?MWHUeAjX333r2FN5lXQDd8PRALOJPczsiDGYOPE1yheS7PDtxWngA9s0zF5z?=
 =?us-ascii?Q?1Rc1105t3Uhyk0cOO7vlEaFyurXr69V4MpUzB7+WVPS3BpDt2GtXrBx2IXPx?=
 =?us-ascii?Q?sszXJMym2KRPTTo4LeD/0eUDOLMHz/Wj0P0S6Yn12SgzMP8G15d1X+5A/+zO?=
 =?us-ascii?Q?0NjgjF8mQ6ZfMOnIU2sg6JamyemTXxy/wFamNnltQCQW1I5XAJxpDQ0qQZuD?=
 =?us-ascii?Q?oRIMEzg6y1/8qjG6JNp6xshy29v96xeOB2TA3iGadJu6PCp1rGshS7dwofHe?=
 =?us-ascii?Q?rcKc16wmZ876ILMhGIt9KYQlifOSOQbseidoWxtMA6PjMtg1z0KM5XEvu8eN?=
 =?us-ascii?Q?s5zd5knxkpAJMCcfGpS16Zi81vLpwaYA95Zj6qMmMD7vVGKa2niPKahPuAqs?=
 =?us-ascii?Q?QUcXM5V9xATok2oLeYMRyhV28NpDA1Z9Sd2knDA2qqdidnGw7APDGMP3ly2T?=
 =?us-ascii?Q?YETGFPysCFaJ+ADmWlt8UShwUI7xHB47OdBVghKtUj2L9MpmFBWlsynEOqa0?=
 =?us-ascii?Q?d0vOsftdHg=3D=3D?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: PH0PR11MB5064.namprd11.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: e6c3d69c-db6f-4fdb-ff3e-08da3707dcbd
X-MS-Exchange-CrossTenant-originalarrivaltime: 16 May 2022 06:46:53.3459
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 46c98d88-e344-4ed4-8496-4ed7712e255d
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: d8fM6fLcdH/clpha4lDmZ57pHTUFZmPZwf127skd/1SX69CuX9Zml/zN+3CoAvPnm9p1UIN8fPWgDAc1M1NbDQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM5PR11MB1418
X-OriginatorOrg: intel.com
X-Spam-Status: No, score=-2.4 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,SPF_HELO_NONE,
        SPF_NONE,T_SCC_BODY_TEXT_LINE autolearn=ham autolearn_force=no
        version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On May 13, 2022 10:45 PM, Kirill A. Shutemov wrote:
> On Fri, May 13, 2022 at 11:01:43AM +0200, Borislav Petkov wrote:
> > + mroth
> > - brijesh
> >
> > On Thu, May 12, 2022 at 10:34:02PM -0700, Dionna Amalie Glaze wrote:
> > > Kirill, I've been tracking these changes to see if we can handle the
> > > unaccepted memory type for SEV-SNP, but testing has been an issue.
> > > The proposed patch in Ovmf to introduce unaccepted memory seems to
> > > have stalled out last September
> > > (https://www.mail-archive.com/devel@edk2.groups.io/msg35842.html)
> > > and is particularly difficult to adapt to SEV-SNP since it doesn't
> > > follow the TDVF way of initializing all memory. Is there a different
> > > development I might have missed so that we might test these cases?
> > > Without the UEFI introducing EFI_UNACCEPTED_MEMORY type, any
> kernel
> > > uses are essentially dead code.
>=20
> + Min, Jiaqi.
>=20
> I don't follow firmware development. Min, Jiaqi, could you comment?
>=20
We have prepared the patch for unaccepted memory and it is now working in o=
ur internal release.
But there is an obstacle to upstream it to edk2 master branch.=20
The patch-set depends on the definition of UEFI_RESOURCE_MEMORY_UNACCEPTED =
in PI spec. This is proposed in https://github.com/microsoft/mu_basecore/pu=
ll/66/files#diff-b20a11152d1ce9249c691be5690b4baf52069efadf2e2546cdd2eb663d=
80c9e4R237, according to UEFI-Code-First. The proposal was approved in 2021=
 in UEFI Mantis, and will be added to the new PI.next specification. (Till =
now it has not been added in the latest PI spec.)
So UEFI_RESOURCE_MEMORY_UNACCEPTED cannot be added in MdePkg which make it =
difficult to submit the patch to edk2 community for review. See this link: =
https://edk2.groups.io/g/devel/message/87558

Please be noted: UEFI_RESOURCE_MEMORY_UNACCEPTED (defined in PI spec) is di=
fferent from EFI_UNACCEPTED_MEMORY (defined in UEFI spec)

I will submit the patch-set once the new definition is added in the new PI.=
next spec.

Thanks
Min