* Missing PI definitions?
@ 2018-12-04 16:00 Andrew Fish
2018-12-05 0:19 ` Gao, Liming
0 siblings, 1 reply; 3+ messages in thread
From: Andrew Fish @ 2018-12-04 16:00 UTC (permalink / raw)
To: edk2-devel
Anyone remember why these defines are not in a common header in the MdePkg?
/Volumes/Case/UDK2018(vUDK2018)>git grep MEMORY_PRESENT -- *.h
EdkCompatibilityPkg/Foundation/Include/TianoTypes.h:31:#define EFI_MEMORY_PRESENT 0x0100000000000000
MdeModulePkg/Core/Dxe/DxeMain.h:101:#define EFI_MEMORY_PRESENT 0x0100000000000000ULL
MdeModulePkg/Universal/MemoryTest/GenericMemoryTestDxe/LightMemoryTest.h:42:#define EFI_MEMORY_PRESENT 0x0100000000000000ULL
MdeModulePkg/Universal/MemoryTest/NullMemoryTestDxe/NullMemoryTest.h:33:#define EFI_MEMORY_PRESENT 0x0100000000000000ULL
/Volumes/Case/UDK2018(vUDK2018)>git grep EFI_MEMORY_INITIALIZED -- *.h
EdkCompatibilityPkg/Foundation/Include/TianoTypes.h:32:#define EFI_MEMORY_INITIALIZED 0x0200000000000000
MdeModulePkg/Core/Dxe/DxeMain.h:102:#define EFI_MEMORY_INITIALIZED 0x0200000000000000ULL
MdeModulePkg/Universal/MemoryTest/GenericMemoryTestDxe/LightMemoryTest.h:43:#define EFI_MEMORY_INITIALIZED 0x0200000000000000ULL
MdeModulePkg/Universal/MemoryTest/NullMemoryTestDxe/NullMemoryTest.h:34:#define EFI_MEMORY_INITIALIZED 0x0200000000000000ULL
/Volumes/Case/UDK2018(vUDK2018)>git grep EFI_MEMORY_TESTED -- *.h
EdkCompatibilityPkg/Foundation/Include/TianoTypes.h:33:#define EFI_MEMORY_TESTED 0x0400000000000000
MdeModulePkg/Core/Dxe/DxeMain.h:103:#define EFI_MEMORY_TESTED 0x0400000000000000ULL
MdeModulePkg/Universal/MemoryTest/GenericMemoryTestDxe/LightMemoryTest.h:44:#define EFI_MEMORY_TESTED 0x0400000000000000ULL
MdeModulePkg/Universal/MemoryTest/NullMemoryTestDxe/NullMemoryTest.h:35:#define EFI_MEMORY_TESTED 0x0400000000000000ULL
Thanks,
Andrew Fish
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: Missing PI definitions?
2018-12-04 16:00 Missing PI definitions? Andrew Fish
@ 2018-12-05 0:19 ` Gao, Liming
2018-12-05 1:04 ` Andrew Fish
0 siblings, 1 reply; 3+ messages in thread
From: Gao, Liming @ 2018-12-05 0:19 UTC (permalink / raw)
To: Andrew Fish, edk2-devel
Andrew:
UEFI spec doesn't define them. They are the implement related definitions. They are not required to be exposed to OS. We can add one header file in MdeModulePkg to share them between DxeCore and MemoryTest driver. Besides, ECP package will be retired. There is no change for it.
#define EFI_MEMORY_PRESENT 0x0100000000000000ULL
#define EFI_MEMORY_INITIALIZED 0x0200000000000000ULL
#define EFI_MEMORY_TESTED 0x0400000000000000ULL
Thanks
Liming
>-----Original Message-----
>From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
>Andrew Fish
>Sent: Wednesday, December 05, 2018 12:01 AM
>To: edk2-devel <edk2-devel@lists.01.org>
>Subject: [edk2] Missing PI definitions?
>
>Anyone remember why these defines are not in a common header in the
>MdePkg?
>
>/Volumes/Case/UDK2018(vUDK2018)>git grep MEMORY_PRESENT -- *.h
>EdkCompatibilityPkg/Foundation/Include/TianoTypes.h:31:#define
>EFI_MEMORY_PRESENT 0x0100000000000000
>MdeModulePkg/Core/Dxe/DxeMain.h:101:#define EFI_MEMORY_PRESENT
>0x0100000000000000ULL
>MdeModulePkg/Universal/MemoryTest/GenericMemoryTestDxe/LightMem
>oryTest.h:42:#define EFI_MEMORY_PRESENT 0x0100000000000000ULL
>MdeModulePkg/Universal/MemoryTest/NullMemoryTestDxe/NullMemoryT
>est.h:33:#define EFI_MEMORY_PRESENT 0x0100000000000000ULL
>/Volumes/Case/UDK2018(vUDK2018)>git grep EFI_MEMORY_INITIALIZED --
>*.h
>EdkCompatibilityPkg/Foundation/Include/TianoTypes.h:32:#define
>EFI_MEMORY_INITIALIZED 0x0200000000000000
>MdeModulePkg/Core/Dxe/DxeMain.h:102:#define
>EFI_MEMORY_INITIALIZED 0x0200000000000000ULL
>MdeModulePkg/Universal/MemoryTest/GenericMemoryTestDxe/LightMem
>oryTest.h:43:#define EFI_MEMORY_INITIALIZED 0x0200000000000000ULL
>MdeModulePkg/Universal/MemoryTest/NullMemoryTestDxe/NullMemoryT
>est.h:34:#define EFI_MEMORY_INITIALIZED 0x0200000000000000ULL
>/Volumes/Case/UDK2018(vUDK2018)>git grep EFI_MEMORY_TESTED -- *.h
>EdkCompatibilityPkg/Foundation/Include/TianoTypes.h:33:#define
>EFI_MEMORY_TESTED 0x0400000000000000
>MdeModulePkg/Core/Dxe/DxeMain.h:103:#define EFI_MEMORY_TESTED
>0x0400000000000000ULL
>MdeModulePkg/Universal/MemoryTest/GenericMemoryTestDxe/LightMem
>oryTest.h:44:#define EFI_MEMORY_TESTED 0x0400000000000000ULL
>MdeModulePkg/Universal/MemoryTest/NullMemoryTestDxe/NullMemoryT
>est.h:35:#define EFI_MEMORY_TESTED 0x0400000000000000ULL
>
>
>Thanks,
>
>Andrew Fish
>_______________________________________________
>edk2-devel mailing list
>edk2-devel@lists.01.org
>https://lists.01.org/mailman/listinfo/edk2-devel
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: Missing PI definitions?
2018-12-05 0:19 ` Gao, Liming
@ 2018-12-05 1:04 ` Andrew Fish
0 siblings, 0 replies; 3+ messages in thread
From: Andrew Fish @ 2018-12-05 1:04 UTC (permalink / raw)
To: Gao, Liming; +Cc: edk2-devel
Liming,
Sorry I guess I was confusing this with EFI_RESOURCE_ATTRIBUTE_TESTED.
I'll a little confused that it is implementation given it is passed into gDS->AddMemorySpace() the PI Spec defines the values of Capabilities to be defined in the UEFI Spec. as the GetMemoryMap() attributes. It is not clear that the implementation actually owns these bits? Almost feels like we should update the PI spec to include these #defines.
I seem to remember we have been using these bits for a long time.....
Thanks,
Andrew Fish
> On Dec 4, 2018, at 4:19 PM, Gao, Liming <liming.gao@intel.com> wrote:
>
> Andrew:
> UEFI spec doesn't define them. They are the implement related definitions. They are not required to be exposed to OS. We can add one header file in MdeModulePkg to share them between DxeCore and MemoryTest driver. Besides, ECP package will be retired. There is no change for it.
>
> #define EFI_MEMORY_PRESENT 0x0100000000000000ULL
> #define EFI_MEMORY_INITIALIZED 0x0200000000000000ULL
> #define EFI_MEMORY_TESTED 0x0400000000000000ULL
>
> Thanks
> Liming
>> -----Original Message-----
>> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org <mailto:edk2-devel-bounces@lists.01.org>] On Behalf Of
>> Andrew Fish
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2018-12-05 1:04 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2018-12-04 16:00 Missing PI definitions? Andrew Fish
2018-12-05 0:19 ` Gao, Liming
2018-12-05 1:04 ` Andrew Fish
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox