From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail05.groups.io (mail05.groups.io [45.79.224.7]) by spool.mail.gandi.net (Postfix) with ESMTPS id 65446AC0D87 for ; Thu, 8 Aug 2024 08:14:51 +0000 (UTC) DKIM-Signature: a=rsa-sha256; bh=AehS/mJMSMrb3ZHdP3y0oahpfUh7TQDkAr4boDDDGTA=; c=relaxed/simple; d=groups.io; h=From:To:CC:Subject:Thread-Topic:Thread-Index:Date:Message-ID:References:In-Reply-To:Accept-Language:msip_labels:MIME-Version:Precedence:List-Subscribe:List-Help:Sender:List-Id:Mailing-List:Delivered-To:Resent-Date:Resent-From:Reply-To:List-Unsubscribe-Post:List-Unsubscribe:Content-Language:Content-Type; s=20240206; t=1723104891; v=1; b=id+har3plO7WDQQBWtxYLA5BLY7Nr2cy1VHwsXXnPBshYe6eGKDXwrxEQkQLaYPro56nKhTl 2rp/Jcf2jB7Ru2K9fI1EFGYbZ16ph4So+W+zrgK8XS0c48sxw58hcNB/6vbHX+6HS0ujA5TDftB 6DC+k8CkW251LIPG4QQgiqtCzZVjQaPcrmZxEt2bKIMl9HvGPBphNVU/V6kvSDpQO5wD4/ZGioF /6CF2xu49zf8RBZEmcyzkx++SF7h9ugHPaxlwhc1q8F0iE4wNJ58BG5c1nqmi8JDMuBti3CDJTt J0PXAHIsjRH5/w3ZLDrsuJbJ8LSCzJyXygfObENX2SV3A== X-Received: by 127.0.0.2 with SMTP id iv94YY7687511x6ARMyK5No3; Thu, 08 Aug 2024 01:14:49 -0700 X-Received: from mgamail.intel.com (mgamail.intel.com [198.175.65.14]) by mx.groups.io with SMTP id smtpd.web11.54923.1723104888932391922 for ; Thu, 08 Aug 2024 01:14:49 -0700 X-CSE-ConnectionGUID: dQIaiBL2RpWZxwcG9PpATw== X-CSE-MsgGUID: 2cFyBdsSQYKYKIKvSDarGg== X-IronPort-AV: E=McAfee;i="6700,10204,11157"; a="25007496" X-IronPort-AV: E=Sophos;i="6.09,272,1716274800"; d="scan'208,217";a="25007496" X-Received: from fmviesa001.fm.intel.com ([10.60.135.141]) by orvoesa106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 08 Aug 2024 01:14:48 -0700 X-CSE-ConnectionGUID: p96io5kBRr6n0ulkPhGe6Q== X-CSE-MsgGUID: W1F6ahsuTcqAk63h3VIk4A== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.09,272,1716274800"; d="scan'208,217";a="88051244" X-Received: from orsmsx603.amr.corp.intel.com ([10.22.229.16]) by fmviesa001.fm.intel.com with ESMTP/TLS/AES256-GCM-SHA384; 08 Aug 2024 01:14:47 -0700 X-Received: from orsmsx611.amr.corp.intel.com (10.22.229.24) by ORSMSX603.amr.corp.intel.com (10.22.229.16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.39; Thu, 8 Aug 2024 01:14:46 -0700 X-Received: from orsmsx612.amr.corp.intel.com (10.22.229.25) by ORSMSX611.amr.corp.intel.com (10.22.229.24) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.39; Thu, 8 Aug 2024 01:14:46 -0700 X-Received: from ORSEDG601.ED.cps.intel.com (10.7.248.6) by orsmsx612.amr.corp.intel.com (10.22.229.25) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_GCM_SHA256) id 15.1.2507.39 via Frontend Transport; Thu, 8 Aug 2024 01:14:46 -0700 X-Received: from NAM12-BN8-obe.outbound.protection.outlook.com (104.47.55.171) by edgegateway.intel.com (134.134.137.102) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.1.2507.39; Thu, 8 Aug 2024 01:14:46 -0700 X-Received: from MN6PR11MB8244.namprd11.prod.outlook.com (2603:10b6:208:470::14) by IA1PR11MB7728.namprd11.prod.outlook.com (2603:10b6:208:3f0::6) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.7828.31; Thu, 8 Aug 2024 08:14:43 +0000 X-Received: from MN6PR11MB8244.namprd11.prod.outlook.com ([fe80::41a4:c775:32e6:76a8]) by MN6PR11MB8244.namprd11.prod.outlook.com ([fe80::41a4:c775:32e6:76a8%3]) with mapi id 15.20.7828.023; Thu, 8 Aug 2024 08:14:43 +0000 From: "Ni, Ray" To: "devel@edk2.groups.io" , "Kun.Qin@microsoft.com" CC: "Wu, Jiaxin" , "Tan, Dun" , "Xu, Wei6" , "Zhang, Hongbin1" , "Ni, Ray" , "Kinney, Michael D" Subject: Re: [edk2-devel] Proposing v3 of MM communicate buffer Thread-Topic: Proposing v3 of MM communicate buffer Thread-Index: Adro6QAPLf/WRMT7RNuaRkedI6TpYwAgLKmV Date: Thu, 8 Aug 2024 08:14:43 +0000 Message-ID: References: In-Reply-To: Accept-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: msip_labels: MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_ActionId=8a5f3bc2-1702-443d-84a2-a63e5af27909;MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_ContentBits=0;MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Enabled=true;MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Method=Standard;MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_Name=Internal;MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_SetDate=2024-08-07T16:33:32Z;MSIP_Label_f42aa342-8706-4288-bd11-ebb85995028c_SiteId=72f988bf-86f1-41af-91ab-2d7cd011db47; x-ms-publictraffictype: Email x-ms-traffictypediagnostic: MN6PR11MB8244:EE_|IA1PR11MB7728:EE_ x-ms-office365-filtering-correlation-id: 30c2082a-27aa-4cd4-0c5f-08dcb7822865 x-ms-exchange-senderadcheck: 1 x-ms-exchange-antispam-relay: 0 x-microsoft-antispam-message-info: =?iso-8859-1?Q?N4nnYc9ojeeG4kt0J0zUtS29UKxxuOZnRix0i8ehO/GHYztP4c/F/zsK4t?= =?iso-8859-1?Q?wN5th7S+3bX3YT3ROdvxpe72v8dmrjIlbsuqmu8RLoJnNq6uKS8GnVuazc?= =?iso-8859-1?Q?gFWkT4Aa1+df0X11FSofK+fJO5Y2xb0genP1FUSgnlDEksj0WfqM9IlFl+?= =?iso-8859-1?Q?C0v65objOZHQDtxk0K8bK3HsBYHBWWLaFcHTKF+3M2qrkngkm6yOnoQaOZ?= =?iso-8859-1?Q?LqSmMKBRvrATfW6GCQl53nUCN2JQu3bLcHxBuoaIjCvAvREEO0VToSi7kK?= =?iso-8859-1?Q?6oQDRl7Xnl50Md5sbRPnXSx0iVI8UJSqC5OHXIpYZTgeGFASAy7PhtJQKM?= =?iso-8859-1?Q?luyMuDxuIXJP42SHNDAS6hLZ2jUqA6OBm+YFoOjNBpXj3JGaKFugHp8xxP?= =?iso-8859-1?Q?KwPLiDTObANL88irD+PtPZ5/ChQajM4QmvNIq5OsanIQ/y+lKur6XgKy0N?= =?iso-8859-1?Q?uiszhMQgsQ/cIPPjODSL7ufRe1Db+L8vIPJssVEV6iyipdC5fOgqDuQUGu?= =?iso-8859-1?Q?cq6EW7oSZcAsl5dF6PK15ghTseciQkfkwwnWfFKCsweOk0Gs96/xqHHpaz?= =?iso-8859-1?Q?2caBk9APnHJDo+n3ot+R98J8/LZ5B4GL6MvrrjktFOGJEze8wvfehu66QG?= =?iso-8859-1?Q?1AgfASM9ZWuvjhnjfiIJ9mbxePFPZ8dLL1khddb+1vp4qCnvLdLr3CquOg?= =?iso-8859-1?Q?D22kuAeA5uag3hvRaCuRTrMXRK7p/yk7qAcpbDdhR7DUwHFewYU3ZA8HM3?= =?iso-8859-1?Q?3j2fNl596rM+R7/iNkp/GEmqBk9mS9Pt6zs1l2l53rZKGA4W4RoFzTPzEO?= =?iso-8859-1?Q?ugpDRmQJUu7asFYNcVySSROFqB6TZQyUL9U/y120CI+wc9zpdO6laB1cxj?= =?iso-8859-1?Q?VYvr85MuvMqYdkwn/j/IFM0HS7Zapwr2IOO/p+7QhTKmE0ak0gImFiVMrd?= =?iso-8859-1?Q?jOb5etL1Z4nzBcglere25Bc+4dUIz9iuWbZoDAxAGXO9TwC3yP8Z9mda3x?= =?iso-8859-1?Q?PJ5mVj2uRd8jP0jPlk0T31cWCNWFLyjk8mrXtDnivk3MIUyXBuk5TD5y2w?= =?iso-8859-1?Q?zzQl28+3dKsZ+EneVNqeREfDMqMlNUMHwqHH1RIX4usfIikof/aaMsXJdr?= =?iso-8859-1?Q?3dAh21xaU+bpE3IZguo8gTUviXzP/3/kuqSTV9GywvtoZfMl5hm/BgOmDP?= =?iso-8859-1?Q?zKjo3MEsl2PySjomrhfY3r1qfqju2GbiB2Z/qoE5uYl6uARQn3SMxdxdvE?= =?iso-8859-1?Q?SoyD4ZFNuorIS8pclBq5nIAk8qxDAS2+j7y2kLtIPY+p0ppL5IMMojDfIM?= =?iso-8859-1?Q?CNTusn6mi+bXvR7GqR9BANTMvQ9m56lVtMBTigfSR99+lm+mpVZ60yLO7C?= =?iso-8859-1?Q?HPeq4EsJD3UGySzGnh6MmHItO0EI+Gs/MotiZDRZyE7xEtq974gcM=3D?= x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?iso-8859-1?Q?a7gwMh0eNmS7UH1O966yNKYLIW+jtM2klgeZOdk4LXlMK+otAh1AbMWdem?= =?iso-8859-1?Q?3Uk4RG3Mzt2jVqE2nR27j5/g4ReXqxAfRU0AYYCISNiTL6uuwMBHhVPNB2?= =?iso-8859-1?Q?S7wAw93YUVoj7vHrmMtdXlJMs0WT3IUB0RTkCnjWmk2vYkxytxZD2nCs4j?= =?iso-8859-1?Q?83hZIskBvko8vJ1ahUM6qQ9NWgZnWcqbDOcKlzpedUKJqPY0XntLLKghxw?= =?iso-8859-1?Q?cOMY2ish8YwqjKFfJsdWmS+6m3PYovvdJ8ZIzw1WTjQE8qnKdLiATTrPCd?= =?iso-8859-1?Q?cGAXvwFGA/v53bOOSTJFls/MBzJNPmra+vi8rZHGq6+vbcDeX7RV+l1FqM?= =?iso-8859-1?Q?+H3oEMgWN8rsQtZzOjfCCkyLZRmoTNVthF3Cy6yGWjS0F05sIhEnrgqrTp?= =?iso-8859-1?Q?8BtCr8f52GfmSsXdEXpVUrCtIudTTizivVQCJjUimT6SelXIaheHBdvGjm?= =?iso-8859-1?Q?OG2hYdHil6U19IFUJBnG5hYzwWV3wQB+k+UXpNt9XP3BErajZ4urDosXgx?= =?iso-8859-1?Q?0YAKA3OFRG16JctNigAL+bFMl7Fl4Vhd5F8EUwNWs3Wnnw96L08EN9oxg+?= =?iso-8859-1?Q?G45dYNtzOx9CPqtRjHBvY9036msyaiIz+WmH8oZ9HXoRMwzlbJ3h5LANY6?= =?iso-8859-1?Q?730Lr6n+deXFgAU5QdveZMHQqm/NFAWX9rx0fLF0q5NuxP9f1I+x84usOZ?= =?iso-8859-1?Q?jmqYQtsgW3a425mysGrOgGZbj1NTm9z/h4E+0AMsgJrlWOOrMWZaG3WVwC?= =?iso-8859-1?Q?CMuSL/wgYK0exJ10Zr7pLIkwin4KkhdLfa3i2J8P8vWQxlmO9pbSm/D0dQ?= =?iso-8859-1?Q?MZ1cDWcQd4IqPLFTrtIWGKigBBoym2c5T2xiStwOZdukNC6YxWU4aNSLsq?= =?iso-8859-1?Q?rW7eJWOgt6R9geC7JULWRxqyw2ZB7Fk6LOdfNAkhfXcQ/obmWvmqIfraP6?= =?iso-8859-1?Q?qZE0JvmgvVGKLMUaMjm+UZzVIHP1NFDaI1myvCBHbsL74Np2gk0wACbs/s?= =?iso-8859-1?Q?CgkB7p+iTM/cSpuTXF922FXUkg6oGN9NoD2PB30JM+zOHBDXWtd6tjfMDw?= =?iso-8859-1?Q?rWyukpnIwBYgHZ/N7uflTzOSakxU2UI0Ldwd3kE0XH/RLadoL7Qe+QqQex?= =?iso-8859-1?Q?eXtSDI3ad5p4TShQFfpDgCVz5Q1XJsj6BDwEH9aE5EVwRCiRZ950s0mCf0?= =?iso-8859-1?Q?pB+RCLQcRJqjUO8FLMsgUJpqEGwaBJNviqZs2eQpLF1LuTo7LWDvM+VhcV?= =?iso-8859-1?Q?Nsw6gYoBiq6Ei/rWi6xhcvxfipNWmqphatQ8cxO6lY1+3uANEolSiJ5oIP?= =?iso-8859-1?Q?1SHgXk8lT/rf40CzbBXx9mLRF6RwyeDRgNh33IvFaQ6rMB/FNl4OB4ULfu?= =?iso-8859-1?Q?D3wnrMMMTQjdGQeL5pd/V4ucGXxEjIWxo/G+f4YNyUkoomvA1nJBwo1NOl?= =?iso-8859-1?Q?f9kDgLs1nMJCGErK1fTw3px5aSzSBGzRFxLD3RmtrSdaeo9GMbz/skBGjn?= =?iso-8859-1?Q?hTM/HRMC5MmAr6b8mROsWsfXwO020g1NsnB8/40oTEikQDF/A5y5yktLg3?= =?iso-8859-1?Q?8Ay1Hf/cgf2l32lBX4zT2PQDJ7/xd7mpJErLIllqK6BTDFJU52vrus2XG2?= =?iso-8859-1?Q?H+6qCUDAfGlgg=3D?= MIME-Version: 1.0 X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: MN6PR11MB8244.namprd11.prod.outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 30c2082a-27aa-4cd4-0c5f-08dcb7822865 X-MS-Exchange-CrossTenant-originalarrivaltime: 08 Aug 2024 08:14:43.0654 (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: tso2cDiNQiAq2VHzH4+QmR3dUhdk9Tf7ZyoGmObmvabdrQWx7lrvgVG8pNJRQ3MXqPYhm+A5OydZRohe3LdnBQ== X-MS-Exchange-Transport-CrossTenantHeadersStamped: IA1PR11MB7728 X-OriginatorOrg: intel.com Precedence: Bulk List-Subscribe: List-Help: Sender: devel@edk2.groups.io List-Id: Mailing-List: list devel@edk2.groups.io; contact devel+owner@edk2.groups.io Resent-Date: Thu, 08 Aug 2024 01:14:49 -0700 Resent-From: ray.ni@intel.com Reply-To: devel@edk2.groups.io,ray.ni@intel.com List-Unsubscribe-Post: List-Unsubscribe=One-Click List-Unsubscribe: X-Gm-Message-State: Y1oqWRgzKHC19ZEWrQj0ttkOx7686176AA= Content-Language: en-US Content-Type: multipart/alternative; boundary="_000_MN6PR11MB8244D9A826CC54507A9A433A8CB92MN6PR11MB8244namp_" X-GND-Status: LEGIT Authentication-Results: spool.mail.gandi.net; dkim=pass header.d=groups.io header.s=20240206 header.b=id+har3p; dmarc=fail reason="SPF not aligned (relaxed), DKIM not aligned (relaxed)" header.from=intel.com (policy=none); spf=pass (spool.mail.gandi.net: domain of bounce@groups.io designates 45.79.224.7 as permitted sender) smtp.mailfrom=bounce@groups.io --_000_MN6PR11MB8244D9A826CC54507A9A433A8CB92MN6PR11MB8244namp_ Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable Kun, I like your proposed solution as it is backward compatible. But, I think the new PPI/Protocol is only useful when the CPU mode where PP= I/Protocol is produced does not match the CPU mode in MM. In X86, it could be: 32bit PEI + 64bit MM, 32bit DXE + 64bit MM, or vice ve= rsa. But I doubt the value of support these combinations in X86. Because th= at means the IPL (either PEI or DXE module) needs to support invoking MM Co= re in a different CPU mode. And the latest X86 platforms are switching to 64bit PEI + 64bit DXE + 64bit= MM. Does the proposal try to solve some ARM problem? Can you explain the necess= ity? I would like to avoid the complicated interfaces which do not solve a = practical problem. Thanks, Ray ________________________________ From: devel@edk2.groups.io on behalf of Kun Qin via = groups.io Sent: Thursday, August 8, 2024 2:14 To: devel@edk2.groups.io Subject: [edk2-devel] Proposing v3 of MM communicate buffer Hi all, I am trying to propose a change into PI spec and would like to gather some = feedback in this forum. Essentially, the current communicate header contains a UINTN field in place= , which is causing programing errors when trying to communicate the message between different operation m= ode (i.e. PEI in IA32 communicate into MM in x64). There are various implementations at large to = compensate for this size discrepancy through the edk2 codebase, thus fixing the existing commun= icate buffer definition will be less feasible. Thus I think proposing a new structure and implement= the corresponding header parser will be a simpler approach, which also allows a bit more flexibility= to inject new features/checks into the communication channel. The proposed change for the spec is detailed here: https://github.com/kuqin12/edk2/blob/BZ3398-MmCommunicate-Length-v4/CodeFir= st/BZ3430-SpecChange.md And the code first change is listed here: https://github.com/kuqin12/edk2/blob/BZ3398-MmCommunicate-Length-v4/ Could you please provide me with any feedback that you think might be helpf= ul for future usage of MM communicate? Any input is appreciated. Regards, Kun -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#120291): https://edk2.groups.io/g/devel/message/120291 Mute This Topic: https://groups.io/mt/107775882/7686176 Group Owner: devel+owner@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [rebecca@openfw.io] -=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D-=3D- --_000_MN6PR11MB8244D9A826CC54507A9A433A8CB92MN6PR11MB8244namp_ Content-Type: text/html; charset="iso-8859-1" Content-Transfer-Encoding: quoted-printable
Kun,
I like your proposed solution as it is backward compatible.

But, I think the new PPI/Protocol is only useful when the CPU mode where PP= I/Protocol is produced does not match the CPU mode in MM.

In X86, it could be: 32bit PEI + 64bit MM, 32bit DXE + 64bit MM, or vice ve= rsa. But I doubt the value of support these combinations in X86. Because th= at means the IPL (either PEI or DXE module) needs to support invoking MM Co= re in a different CPU mode.
And the latest X86 platforms are switching to 64bit PEI + 64bit DXE + 64bit= MM.

Does the proposal try to solve some ARM problem? Can you explain the necess= ity? I would like to avoid the complicated interfaces which do not solve a = practical problem.

Thanks,
Ray

From: devel@= edk2.groups.io <devel@edk2.groups.io> on behalf of Kun Qin via groups= .io <Kun.Qin=3Dmicrosoft.com@groups.io>
Sent: Thursday, August 8, 2024 2:14
To: devel@edk2.groups.io <devel@edk2.groups.io>
Subject: [edk2-devel] Proposing v3 of MM communicate buffer
 

= Hi all,<= /span>

=  

= I am try= ing to propose a change into PI spec and would like to gather some feedback= in this forum.

=  

= Essentia= lly, the current communicate header contains a UINTN field in place, which = is causing programing

= errors w= hen trying to communicate the message between different operation mode (i.e= . PEI in IA32

= communic= ate into MM in x64). There are various implementations at large to compensa= te for this

= size dis= crepancy through the edk2 codebase, thus fixing the existing communicate bu= ffer definition

= will be = less feasible. Thus I think proposing a new structure and implement the cor= responding header

= parser w= ill be a simpler approach, which also allows a bit more flexibility to inje= ct new features/checks

= into the= communication channel.

=  

= The prop= osed change for the spec is detailed here:

= =  

= And the = code first change is listed here:

https://github.com/kuqin12/e= dk2/blob/BZ3398-MmCommunicate-Length-v4/

=  

= Could yo= u please provide me with any feedback that you think might be helpful for f= uture usage of MM

= communic= ate? Any input is appreciated.

=  

= Regards,=

= Kun

=  

_._,_._,_
Groups.io Links:

=20 You receive all messages sent to this group. =20 =20

View/Reply Online (#120291) | =20 | Mute= This Topic | New Topic
Your Subscriptio= n | Contact Group Owner | Unsubscribe [rebecca@openfw.io]

_._,_._,_
--_000_MN6PR11MB8244D9A826CC54507A9A433A8CB92MN6PR11MB8244namp_--