From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from walk.intel-email.com (walk.intel-email.com [101.227.64.242]) by mx.groups.io with SMTP id smtpd.web11.35100.1674090534391040119 for ; Wed, 18 Jan 2023 17:08:54 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@byosoft.com.cn header.s=cloud-union header.b=OLvxwnlB; spf=pass (domain: byosoft.com.cn, ip: 101.227.64.242, mailfrom: gaoliming@byosoft.com.cn) Received: from walk.intel-email.com (localhost [127.0.0.1]) by walk.intel-email.com (Postfix) with ESMTP id 54995CD1F6A3 for ; Thu, 19 Jan 2023 09:08:51 +0800 (CST) DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=byosoft.com.cn; s=cloud-union; t=1674090531; bh=nmXk0P08x3MX507ESw9KSMLcOMxtQQ8xFm6U4AuU9/w=; h=From:To:Cc:References:In-Reply-To:Subject:Date; b=OLvxwnlB/WnJz6zEyzk98W7Jncrh7XAyDqqI9rM85181maphkrTDqb4fCWaOmU3/S eC6BLur9at8CS59S84LfZb4FRyOjRcEurnhlYgpAP65cCgGJnRfdBkLn5h0Pzjg1Ac AnHkEKJoJLXLtml4UgWnA4r5zU1u1AKfUO7JkZ8U= Received: from localhost (localhost [127.0.0.1]) by walk.intel-email.com (Postfix) with ESMTP id 4FB81CD1F6FF for ; Thu, 19 Jan 2023 09:08:51 +0800 (CST) Received: from walk.intel-email.com (localhost [127.0.0.1]) by walk.intel-email.com (Postfix) with ESMTP id 242F4CD1F6BA for ; Thu, 19 Jan 2023 09:08:51 +0800 (CST) Authentication-Results: walk.intel-email.com; none Received: from mail.byosoft.com.cn (mail.byosoft.com.cn [58.240.74.242]) by walk.intel-email.com (Postfix) with SMTP id A4C95CD1F696 for ; Thu, 19 Jan 2023 09:08:48 +0800 (CST) Received: from DESKTOPS6D0PVI ([58.246.60.130]) (envelope-sender ) by 192.168.6.13 with ESMTP for ; Thu, 19 Jan 2023 09:08:42 +0800 X-WM-Sender: gaoliming@byosoft.com.cn X-Originating-IP: 58.246.60.130 X-WM-AuthFlag: YES X-WM-AuthUser: gaoliming@byosoft.com.cn From: "gaoliming" To: , , "'Xu, Min M'" Cc: "'Bi, Dandan'" , "'Aktas, Erdem'" , "'James Bottomley'" , "'Gerd Hoffmann'" , "'Tom Lendacky'" , "'Michael Roth'" References: <20230118084158.1412-1-min.m.xu@intel.com> In-Reply-To: Subject: =?UTF-8?B?5Zue5aSNOiBbZWRrMi1kZXZlbF0gW1BBVENIIFYxIDEvMV0gTWRlTW9kdWxlUGtnOiBFZmlVbmFjY2VwdGVkTWVtb3J5VHlwZSBpcyBub3QgYWxsb3dlZCBpbiBBbGxvY2F0ZVBvb2w=?= Date: Thu, 19 Jan 2023 09:08:45 +0800 Message-ID: <000101d92ba2$946566b0$bd303410$@byosoft.com.cn> MIME-Version: 1.0 X-Mailer: Microsoft Outlook 16.0 Thread-Index: AQNT5Yylu+NPMvNZ7jm4WiKvYu+l+gGRAviRq6Lx9HA= Sender: "gaoliming" Content-Type: text/plain; charset="gb2312" Content-Transfer-Encoding: quoted-printable Content-Language: zh-cn Reviewed-by: Liming Gao > -----=D3=CA=BC=FE=D4=AD=BC=FE----- > =B7=A2=BC=FE=C8=CB: devel@edk2.groups.io =B4=FA=B1= =ED Yao, Jiewen > =B7=A2=CB=CD=CA=B1=BC=E4: 2023=C4=EA1=D4=C218=C8=D5 18:04 > =CA=D5=BC=FE=C8=CB: Xu, Min M ; devel@edk2.groups.io > =B3=AD=CB=CD: Gao, Liming ; Bi, Dandan > ; Aktas, Erdem ; James > Bottomley ; Gerd Hoffmann ; Tom > Lendacky ; Michael Roth > > =D6=F7=CC=E2: Re: [edk2-devel] [PATCH V1 1/1] MdeModulePkg: > EfiUnacceptedMemoryType is not allowed in AllocatePool >=20 > Reviewed-by: Jiewen Yao >=20 > > -----Original Message----- > > From: Xu, Min M > > Sent: Wednesday, January 18, 2023 4:42 PM > > To: devel@edk2.groups.io > > Cc: Xu, Min M ; Gao, Liming > > ; Bi, Dandan ; Aktas, > > Erdem ; James Bottomley > ; > > Yao, Jiewen ; Gerd Hoffmann > ; > > Tom Lendacky ; Michael Roth > > > > Subject: [PATCH V1 1/1] MdeModulePkg: EfiUnacceptedMemoryType is not > > allowed in AllocatePool > > > > From: Min M Xu > > > > BZ: https://bugzilla.tianocore.org/show_bug.cgi?id=3D4315 > > > > MemoryType of EfiUnacceptedMemoryType should not be allocated in > > AllocatePool. Instead it should return EFI_INVALID_PARAMETER. > > > > Cc: Liming Gao > > Cc: Dandan Bi > > Cc: Erdem Aktas > > Cc: James Bottomley > > Cc: Jiewen Yao > > Cc: Gerd Hoffmann > > Cc: Tom Lendacky > > Cc: Michael Roth > > Reported-by: Liming Gao > > Signed-off-by: Min Xu > > --- > > MdeModulePkg/Core/Dxe/Mem/Pool.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > diff --git a/MdeModulePkg/Core/Dxe/Mem/Pool.c > > b/MdeModulePkg/Core/Dxe/Mem/Pool.c > > index 7aaf501600cf..b20cbfdedbab 100644 > > --- a/MdeModulePkg/Core/Dxe/Mem/Pool.c > > +++ b/MdeModulePkg/Core/Dxe/Mem/Pool.c > > @@ -213,7 +213,7 @@ CoreInternalAllocatePool ( > > // If it's not a valid type, fail it > > // > > if (((PoolType >=3D EfiMaxMemoryType) && (PoolType < > > MEMORY_TYPE_OEM_RESERVED_MIN)) || > > - (PoolType =3D=3D EfiConventionalMemory) || (PoolType =3D=3D > > EfiPersistentMemory)) > > + (PoolType =3D=3D EfiConventionalMemory) || (PoolType =3D=3D > > EfiPersistentMemory) || (PoolType =3D=3D EfiUnacceptedMemoryType)) > > { > > return EFI_INVALID_PARAMETER; > > } > > -- > > 2.29.2.windows.2 >=20 >=20 >=20 >=20 >=20