[edk2-devel] [Patch] MdeModulePkg: Update MdeModulePkg.dsc to remove Network related modules

[Liming Gao]

> -----Original Message-----
> From: Laszlo Ersek [mailto:lersek at redhat.com]
> Sent: Wednesday, May 22, 2019 5:11 PM
> To: Gao, Liming <liming.gao at intel.com>; Wu, Hao A <hao.a.wu at intel.com>; devel at edk2.groups.io
> Cc: leif.lindholm at linaro.org; Kinney, Michael D <michael.d.kinney at intel.com>; afish at apple.com
> Subject: Re: [edk2-devel] [Patch] MdeModulePkg: Update MdeModulePkg.dsc to remove Network related modules
> 
> On 05/22/19 03:50, Gao, Liming wrote:
> > This is a bug. Because NetworkPkg dsc has included all network modules, the ones in MdeModule are duplicated.
> 
> I'm not sure I'm convinced -- if this patch is not applied, what
> precisely is the functionality that breaks? What are the symptoms?
> 
The problem is that NetworkPkg.dsc and MdeModulePkg.dsc both builds those network drivers. 
I am OK to get more feedback from stewards.

> Either way, I'll let the other stewards decide about this.
> 
> Thanks
> Laszlo
> 
> > This change is to remove the duplicated one.
> >
> > Thanks
> > Liming
> >> -----Original Message-----
> >> From: Wu, Hao A
> >> Sent: Wednesday, May 22, 2019 9:44 AM
> >> To: devel at edk2.groups.io; Gao, Liming <liming.gao at intel.com>
> >> Subject: RE: [edk2-devel] [Patch] MdeModulePkg: Update MdeModulePkg.dsc to remove Network related modules
> >>
> >>> -----Original Message-----
> >>> From: devel at edk2.groups.io [mailto:devel at edk2.groups.io] On Behalf Of
> >>> Liming Gao
> >>> Sent: Tuesday, May 21, 2019 11:22 PM
> >>> To: devel at edk2.groups.io
> >>> Cc: Wu, Hao A
> >>> Subject: [edk2-devel] [Patch] MdeModulePkg: Update MdeModulePkg.dsc
> >>> to remove Network related modules
> >>>
> >>> Network related modules have been included in NetworkPkg.dsc.
> >>> They can be removed from MdeModulePkg.dsc.
> >>>
> >>> Signed-off-by: Liming Gao <liming.gao at intel.com>
> >>> Cc: Hao A Wu <hao.a.wu at intel.com>
> >>> ---
> >>>  MdeModulePkg/MdeModulePkg.dsc | 19 -------------------
> >>>  1 file changed, 19 deletions(-)
> >>>
> >>> diff --git a/MdeModulePkg/MdeModulePkg.dsc
> >>> b/MdeModulePkg/MdeModulePkg.dsc
> >>> index ac7111dea1..995fd805e1 100644
> >>> --- a/MdeModulePkg/MdeModulePkg.dsc
> >>> +++ b/MdeModulePkg/MdeModulePkg.dsc
> >>> @@ -63,11 +63,6 @@
> >>>    #
> >>>    UefiUsbLib|MdePkg/Library/UefiUsbLib/UefiUsbLib.inf
> >>>    UefiScsiLib|MdePkg/Library/UefiScsiLib/UefiScsiLib.inf
> >>> -  NetLib|MdeModulePkg/Library/DxeNetLib/DxeNetLib.inf
> >>> -  IpIoLib|MdeModulePkg/Library/DxeIpIoLib/DxeIpIoLib.inf
> >>> -  UdpIoLib|MdeModulePkg/Library/DxeUdpIoLib/DxeUdpIoLib.inf
> >>> -  TcpIoLib|MdeModulePkg/Library/DxeTcpIoLib/DxeTcpIoLib.inf
> >>> -  DpcLib|MdeModulePkg/Library/DxeDpcLib/DxeDpcLib.inf
> >>>
> >>> SecurityManagementLib|MdeModulePkg/Library/DxeSecurityManagementL
> >>> ib/DxeSecurityManagementLib.inf
> >>>
> >>> TimerLib|MdePkg/Library/BaseTimerLibNullTemplate/BaseTimerLibNullTem
> >>> plate.inf
> >>>
> >>> SerialPortLib|MdePkg/Library/BaseSerialPortLibNull/BaseSerialPortLibNull.in
> >>> f
> >>> @@ -263,12 +258,8 @@
> >>>
> >>> MdeModulePkg/Library/DxeCoreMemoryAllocationLib/DxeCoreMemoryAllo
> >>> cationProfileLib.inf
> >>>
> >>> MdeModulePkg/Library/DxeCorePerformanceLib/DxeCorePerformanceLib.i
> >>> nf
> >>>
> >>> MdeModulePkg/Library/DxeCrc32GuidedSectionExtractLib/DxeCrc32Guided
> >>> SectionExtractLib.inf
> >>> -  MdeModulePkg/Library/DxeDpcLib/DxeDpcLib.inf
> >>> -  MdeModulePkg/Library/DxeIpIoLib/DxeIpIoLib.inf
> >>> -  MdeModulePkg/Library/DxeNetLib/DxeNetLib.inf
> >>>    MdeModulePkg/Library/DxePerformanceLib/DxePerformanceLib.inf
> >>>    MdeModulePkg/Library/DxeResetSystemLib/DxeResetSystemLib.inf
> >>> -  MdeModulePkg/Library/DxeUdpIoLib/DxeUdpIoLib.inf
> >>>
> >>> MdeModulePkg/Library/DxePrintLibPrint2Protocol/DxePrintLibPrint2Protoco
> >>> l.inf
> >>>
> >>> MdeModulePkg/Library/PeiCrc32GuidedSectionExtractLib/PeiCrc32GuidedS
> >>> ectionExtractLib.inf
> >>>    MdeModulePkg/Library/PeiPerformanceLib/PeiPerformanceLib.inf
> >>> @@ -357,16 +348,6 @@
> >>>    MdeModulePkg/Universal/SmbiosDxe/SmbiosDxe.inf
> >>>
> >>> MdeModulePkg/Universal/SmbiosMeasurementDxe/SmbiosMeasurement
> >>> Dxe.inf
> >>>
> >>> -  MdeModulePkg/Universal/Network/ArpDxe/ArpDxe.inf
> >>> -  MdeModulePkg/Universal/Network/Dhcp4Dxe/Dhcp4Dxe.inf
> >>> -  MdeModulePkg/Universal/Network/DpcDxe/DpcDxe.inf
> >>> -  MdeModulePkg/Universal/Network/Ip4Dxe/Ip4Dxe.inf
> >>> -  MdeModulePkg/Universal/Network/MnpDxe/MnpDxe.inf
> >>> -  MdeModulePkg/Universal/Network/VlanConfigDxe/VlanConfigDxe.inf
> >>> -  MdeModulePkg/Universal/Network/Mtftp4Dxe/Mtftp4Dxe.inf
> >>> -  MdeModulePkg/Universal/Network/SnpDxe/SnpDxe.inf
> >>> -  MdeModulePkg/Universal/Network/Udp4Dxe/Udp4Dxe.inf
> >>> -
> >>
> >> Reviewed-by: Hao A Wu <hao.a.wu at intel.com>
> >> I will let the stewards to decide whether this can be pushed at this moment.
> >>
> >> Best Regards,
> >> Hao Wu
> >>
> >>>
> >>> MdeModulePkg/Universal/PcatSingleSegmentPciCfg2Pei/PcatSingleSegmen
> >>> tPciCfg2Pei.inf
> >>>    MdeModulePkg/Universal/PCD/Dxe/Pcd.inf
> >>>    MdeModulePkg/Universal/PCD/Pei/Pcd.inf
> >>> --
> >>> 2.13.0.windows.1
> >>>
> >>>
> >>> 
> >


-=-=-=-=-=-=-=-=-=-=-=-
Groups.io Links: You receive all messages sent to this group.

View/Reply Online (#41229): https://edk2.groups.io/g/devel/message/41229
Mute This Topic: https://groups.io/mt/31697980/1813853
Group Owner: devel+owner at edk2.groups.io
Unsubscribe: https://edk2.groups.io/g/devel/unsub  [edk2-devel-archive at redhat.com]
-=-=-=-=-=-=-=-=-=-=-=-