1. Patchew
  2. EDK2
  3. [edk2] [PATCH 00/10] Formalize the reset system core design
  4. View patch

[edk2] [PATCH 08/10] MdePkg/UefiRuntimeLib: Support more module types.

Ruiyu Ni posted 10 patches 6 years, 9 months ago
There is a newer version of this series
[edk2] [PATCH 08/10] MdePkg/UefiRuntimeLib: Support more module types.
Posted by Ruiyu Ni 6 years, 9 months ago 
Because DxeResetSystemLib links to this library to provide
reset system services, change UefiRuntimeLib to support
the same set of module types as what DxeResetSystemLib does.

Cc: Liming Gao <liming.gao@intel.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Cc: Star Zeng <star.zeng@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com>
---
 MdePkg/Library/UefiRuntimeLib/UefiRuntimeLib.inf | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/MdePkg/Library/UefiRuntimeLib/UefiRuntimeLib.inf b/MdePkg/Library/UefiRuntimeLib/UefiRuntimeLib.inf
index 8f46495fc5..d053da545a 100644
--- a/MdePkg/Library/UefiRuntimeLib/UefiRuntimeLib.inf
+++ b/MdePkg/Library/UefiRuntimeLib/UefiRuntimeLib.inf
@@ -5,7 +5,7 @@
 #  EVT_SIGNAL_EXIT_BOOT_SERVICES event, to provide runtime services.
 # This instance also supports SAL drivers for better performance.
 #
-# Copyright (c) 2006 - 2014, Intel Corporation. All rights reserved.<BR>
+# Copyright (c) 2006 - 2017, Intel Corporation. All rights reserved.<BR>
 #
 #  This program and the accompanying materials
 #  are licensed and made available under the terms and conditions of the BSD License
@@ -24,7 +24,7 @@ [Defines]
   FILE_GUID                      = b1ee6c28-54aa-4d17-b705-3e28ccb27b2e
   MODULE_TYPE                    = DXE_RUNTIME_DRIVER
   VERSION_STRING                 = 1.0
-  LIBRARY_CLASS                  = UefiRuntimeLib|DXE_RUNTIME_DRIVER DXE_SAL_DRIVER
+  LIBRARY_CLASS                  = UefiRuntimeLib|DXE_RUNTIME_DRIVER DXE_SAL_DRIVER DXE_CORE DXE_DRIVER DXE_SMM_DRIVER 
 
   CONSTRUCTOR                    = RuntimeDriverLibConstruct
   DESTRUCTOR                     = RuntimeDriverLibDeconstruct
-- 
2.15.1.windows.2

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [PATCH 08/10] MdePkg/UefiRuntimeLib: Support more module types.
Posted by Zeng, Star 6 years, 8 months ago 
On 2018/2/2 14:45, Ruiyu Ni wrote:
> Because DxeResetSystemLib links to this library to provide
> reset system services, change UefiRuntimeLib to support
> the same set of module types as what DxeResetSystemLib does.
> 
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Michael D Kinney <michael.d.kinney@intel.com>
> Cc: Star Zeng <star.zeng@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com>

Do you think it is ok or not to let DxeResetSystemLib consume 
UefiRuntimeServicesTableLib and use gRT->ResetSystem? Then this patch 
will be not needed.

If we still prefer to let DxeResetSystemLib consume UefiRuntimeLib, I am 
also ok. Reviewed-by: Star Zeng <star.zeng@intel.com>

Thanks,
Star

> ---
>   MdePkg/Library/UefiRuntimeLib/UefiRuntimeLib.inf | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
> 
> diff --git a/MdePkg/Library/UefiRuntimeLib/UefiRuntimeLib.inf b/MdePkg/Library/UefiRuntimeLib/UefiRuntimeLib.inf
> index 8f46495fc5..d053da545a 100644
> --- a/MdePkg/Library/UefiRuntimeLib/UefiRuntimeLib.inf
> +++ b/MdePkg/Library/UefiRuntimeLib/UefiRuntimeLib.inf
> @@ -5,7 +5,7 @@
>   #  EVT_SIGNAL_EXIT_BOOT_SERVICES event, to provide runtime services.
>   # This instance also supports SAL drivers for better performance.
>   #
> -# Copyright (c) 2006 - 2014, Intel Corporation. All rights reserved.<BR>
> +# Copyright (c) 2006 - 2017, Intel Corporation. All rights reserved.<BR>
>   #
>   #  This program and the accompanying materials
>   #  are licensed and made available under the terms and conditions of the BSD License
> @@ -24,7 +24,7 @@ [Defines]
>     FILE_GUID                      = b1ee6c28-54aa-4d17-b705-3e28ccb27b2e
>     MODULE_TYPE                    = DXE_RUNTIME_DRIVER
>     VERSION_STRING                 = 1.0
> -  LIBRARY_CLASS                  = UefiRuntimeLib|DXE_RUNTIME_DRIVER DXE_SAL_DRIVER
> +  LIBRARY_CLASS                  = UefiRuntimeLib|DXE_RUNTIME_DRIVER DXE_SAL_DRIVER DXE_CORE DXE_DRIVER DXE_SMM_DRIVER
>   
>     CONSTRUCTOR                    = RuntimeDriverLibConstruct
>     DESTRUCTOR                     = RuntimeDriverLibDeconstruct
> 

_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
Re: [edk2] [PATCH 08/10] MdePkg/UefiRuntimeLib: Support more module types.
Posted by Ni, Ruiyu 6 years, 8 months ago 
On 2/7/2018 8:24 PM, Zeng, Star wrote:
> On 2018/2/2 14:45, Ruiyu Ni wrote:
>> Because DxeResetSystemLib links to this library to provide
>> reset system services, change UefiRuntimeLib to support
>> the same set of module types as what DxeResetSystemLib does.
>>
>> Cc: Liming Gao <liming.gao@intel.com>
>> Cc: Michael D Kinney <michael.d.kinney@intel.com>
>> Cc: Star Zeng <star.zeng@intel.com>
>> Contributed-under: TianoCore Contribution Agreement 1.1
>> Signed-off-by: Ruiyu Ni <ruiyu.ni@intel.com>
> 
> Do you think it is ok or not to let DxeResetSystemLib consume 
> UefiRuntimeServicesTableLib and use gRT->ResetSystem? Then this patch 
> will be not needed.

RuntimeLib handles the gRT pointer conversion when entering RT phase.
DxeResetSystemLib actually can also be used by a RT driver.
Directly calling gRT->ResetSystem from DxeResetSystemLib loses the
capability.



> 
> If we still prefer to let DxeResetSystemLib consume UefiRuntimeLib, I am 
> also ok. Reviewed-by: Star Zeng <star.zeng@intel.com>
> 
> Thanks,
> Star
> 
>> ---
>>   MdePkg/Library/UefiRuntimeLib/UefiRuntimeLib.inf | 4 ++--
>>   1 file changed, 2 insertions(+), 2 deletions(-)
>>
>> diff --git a/MdePkg/Library/UefiRuntimeLib/UefiRuntimeLib.inf 
>> b/MdePkg/Library/UefiRuntimeLib/UefiRuntimeLib.inf
>> index 8f46495fc5..d053da545a 100644
>> --- a/MdePkg/Library/UefiRuntimeLib/UefiRuntimeLib.inf
>> +++ b/MdePkg/Library/UefiRuntimeLib/UefiRuntimeLib.inf
>> @@ -5,7 +5,7 @@
>>   #  EVT_SIGNAL_EXIT_BOOT_SERVICES event, to provide runtime services.
>>   # This instance also supports SAL drivers for better performance.
>>   #
>> -# Copyright (c) 2006 - 2014, Intel Corporation. All rights reserved.<BR>
>> +# Copyright (c) 2006 - 2017, Intel Corporation. All rights reserved.<BR>
>>   #
>>   #  This program and the accompanying materials
>>   #  are licensed and made available under the terms and conditions of 
>> the BSD License
>> @@ -24,7 +24,7 @@ [Defines]
>>     FILE_GUID                      = b1ee6c28-54aa-4d17-b705-3e28ccb27b2e
>>     MODULE_TYPE                    = DXE_RUNTIME_DRIVER
>>     VERSION_STRING                 = 1.0
>> -  LIBRARY_CLASS                  = UefiRuntimeLib|DXE_RUNTIME_DRIVER 
>> DXE_SAL_DRIVER
>> +  LIBRARY_CLASS                  = UefiRuntimeLib|DXE_RUNTIME_DRIVER 
>> DXE_SAL_DRIVER DXE_CORE DXE_DRIVER DXE_SMM_DRIVER
>>     CONSTRUCTOR                    = RuntimeDriverLibConstruct
>>     DESTRUCTOR                     = RuntimeDriverLibDeconstruct
>>
> 


-- 
Thanks,
Ray
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel

Patchew 2.1-devel-b67e4c2

© 2016 - 2024 Red Hat, Inc.