Fu Siyuan,

Please review edk2-platform/master and prepare a patch
for that branch if there are DSC/FDF files that refer
to the network drivers that are being removed. 

We should never break any platforms in edk2-platform/master.
The commits should be performed to the repos in the correct
order to guarantee no build breaks.

Thanks,

Mike

> -----Original Message-----
> From: Fu, Siyuan
> Sent: Tuesday, October 30, 2018 1:23 AM
> To: Zeng, Star <[email protected]>; edk2-
> [email protected]
> Cc: Kinney, Michael D <[email protected]>; Leif
> Lindholm <[email protected]>; Andrew Fish
> ([email protected]) <[email protected]>; Laszlo Ersek
> ([email protected]) <[email protected]>; Gao, Liming
> <[email protected]>
> Subject: RE: [edk2] [PATCH v1 0/7] Delete TCP, PXE,
> iSCSI driver in MdeModulePkg.
> 
> Hi, Star
> 
> This patch only covers the platforms in
> https://github.com/tianocore/edk2
> 
> I will modify the edk2 network wiki page for an updated
> sample DSC/FDF section, if this patch could pass review
> w/o objection.
> https://github.com/tianocore/tianocore.github.io/wiki/Ne
> tworkPkg-Getting-Started-Guide
> 
> Let's wait a few days to see if there is any objection
> on deleting these driver first, and I will be happy to
> generate another patch for edk2-platforms then.
> 
> Thanks for your reminder.
> 
> 
> BestRegards
> Fu Siyuan
> 
> > -----Original Message-----
> > From: Zeng, Star
> > Sent: Tuesday, October 30, 2018 3:43 PM
> > To: Fu, Siyuan <[email protected]>; edk2-
> [email protected]
> > Cc: Kinney, Michael D <[email protected]>;
> Leif Lindholm
> > <[email protected]>; Andrew Fish
> ([email protected])
> > <[email protected]>; Laszlo Ersek ([email protected])
> <[email protected]>;
> > Gao, Liming <[email protected]>; Zeng, Star
> <[email protected]>
> > Subject: RE: [edk2] [PATCH v1 0/7] Delete TCP, PXE,
> iSCSI driver in
> > MdeModulePkg.
> >
> > Hi Siyuan,
> >
> > Have you checked the platforms in
> https://github.com/tianocore/edk2-
> > platforms to see whether they need to be updated
> accordingly or not?
> >
> > Cc more people.
> >
> > Thanks,
> > Star
> > -----Original Message-----
> > From: edk2-devel [mailto:edk2-devel-
> [email protected]] On Behalf Of Fu
> > Siyuan
> > Sent: Tuesday, October 30, 2018 3:33 PM
> > To: [email protected]
> > Subject: [edk2] [PATCH v1 0/7] Delete TCP, PXE, iSCSI
> driver in
> > MdeModulePkg.
> >
> > This patch series is to delete the Tcp4Dxe,
> UefiPxeBcDxe and IScsi4Dxe
> > drivers in MdeModulePkg. These drivers will not be
> maintained and can't
> > co-work with the dual-stack drivers in NetworkPkg.
> >
> > People should use below NetworkPkg drivers instead:
> >   NetworkPkg/IScsiDxe/IScsiDxe.inf
> >   NetworkPkg/TcpDxe/TcpDxe.inf
> >   NetworkPkg/UefiPxeBcDxe/UefiPxeBcDxe.inf
> > These drivers are actively maintained with more bug
> fixes and new feature
> > support.
> >
> > Patch 1~5 update edk2 platform DSC/FDF files to use
> NetworkPkg drivers.
> > Patch 6 deletes the TCP,PXE,iSCSI driver in
> MdeModulePkg.
> > Patch 7 removes some clarification in NetworkPkg
> drivers since the related
> > driver has been deleted in Patch 6.
> >
> > Fu Siyuan (7):
> >   Nt32Pkg: Replace obsoleted network drivers from NT32
> platform DSC/FDF.
> >   EmulatorPkg: Replace obsoleted network drivers from
> platform DSC/FDF.
> >   OvmfPkg: Replace obsoleted network drivers from
> platform DSC/FDF.
> >   Vlv2TbltDevicePkg: Replace obsoleted drivers from
> platform DSC/FDF.
> >   ArmVirtPkg: Replace obsoleted network drivers from
> platform DSC/FDF.
> >   MdeModulePkg: Delete the TCP/PXE/ISCSI drivers in
> MdeModulePkg.
> >   NetworkPkg: Remove some clarification from
> TCP/PXE/ISCSI driver INF.
> >
> >  .../Network/IScsiDxe/ComponentName.c          |  283
> --
> >  .../Universal/Network/IScsiDxe/IScsiCHAP.c    |  430
> ---
> >  .../Universal/Network/IScsiDxe/IScsiConfig.c  | 1264
> -------
> >  .../Universal/Network/IScsiDxe/IScsiDhcp.c    |  472
> ---
> >  .../Universal/Network/IScsiDxe/IScsiDriver.c  |  676
> ----
> >  .../Network/IScsiDxe/IScsiExtScsiPassThru.c   |  412
> ---
> >  .../Universal/Network/IScsiDxe/IScsiIbft.c    |  539
> ---
> >  .../Network/IScsiDxe/IScsiInitiatorName.c     |  116
> -
> >  .../Universal/Network/IScsiDxe/IScsiMisc.c    |  948
> ------
> >  .../Universal/Network/IScsiDxe/IScsiProto.c   | 2799
> ---------------
> >  .../Universal/Network/IScsiDxe/IScsiTcp4Io.c  |  487
> ---
> > MdeModulePkg/Universal/Network/IScsiDxe/Md5.c |  350 -
> -
> >   .../Universal/Network/Tcp4Dxe/ComponentName.c |  433
> ---
> >  .../Universal/Network/Tcp4Dxe/SockImpl.c      | 1201
> -------
> >  .../Universal/Network/Tcp4Dxe/SockInterface.c |  990
> ------
> >  .../Network/Tcp4Dxe/Tcp4Dispatcher.c          |  717
> ----
> >  .../Universal/Network/Tcp4Dxe/Tcp4Driver.c    |  782
> -----
> >  .../Universal/Network/Tcp4Dxe/Tcp4Input.c     | 1497
> ---------
> >  .../Universal/Network/Tcp4Dxe/Tcp4Io.c        |  112
> -
> >  .../Universal/Network/Tcp4Dxe/Tcp4Main.c      |  674
> ----
> >  .../Universal/Network/Tcp4Dxe/Tcp4Misc.c      |  940
> ------
> >  .../Universal/Network/Tcp4Dxe/Tcp4Option.c    |  352
> --
> >  .../Universal/Network/Tcp4Dxe/Tcp4Output.c    | 1238
> -------
> >  .../Universal/Network/Tcp4Dxe/Tcp4Timer.c     |  584
> ----
> >  .../Network/UefiPxeBcDxe/ComponentName.c      |  365
> --
> >  .../Network/UefiPxeBcDxe/PxeBcDhcp.c          | 1999
> -----------
> >  .../Network/UefiPxeBcDxe/PxeBcDriver.c        |  665
> ----
> >  .../Network/UefiPxeBcDxe/PxeBcImpl.c          | 2989
> -----------------
> >  .../Network/UefiPxeBcDxe/PxeBcMtftp.c         |  454
> ---
> >  .../Network/UefiPxeBcDxe/PxeBcSupport.c       |  221
> --
> >  ArmVirtPkg/ArmVirtQemu.dsc                    |   10
> +-
> >  ArmVirtPkg/ArmVirtQemuFvMain.fdf.inc          |   10
> +-
> >  ArmVirtPkg/ArmVirtQemuKernel.dsc              |   10
> +-
> >  EmulatorPkg/EmulatorPkg.dsc                   |    2
> +-
> >  EmulatorPkg/EmulatorPkg.fdf                   |    2
> +-
> >  MdeModulePkg/MdeModulePkg.dsc                 |    3
> -
> >  .../Network/IScsiDxe/ComponentName.h          |  165
> -
> >  .../Universal/Network/IScsiDxe/IScsi4Dxe.uni  |   25
> -
> >  .../Network/IScsiDxe/IScsi4DxeExtra.uni       |   20
> -
> >  .../Universal/Network/IScsiDxe/IScsiCHAP.h    |  106
> -
> >  .../Universal/Network/IScsiDxe/IScsiCommon.h  |   22
> -
> >  .../Universal/Network/IScsiDxe/IScsiConfig.h  |  166
> -
> >  .../Network/IScsiDxe/IScsiConfigDxe.vfr       |  219
> --
> >  .../IScsiDxe/IScsiConfigDxeStrings.uni        |   62
> -
> >  .../Network/IScsiDxe/IScsiConfigNVDataStruc.h |  109
> -
> >  .../Universal/Network/IScsiDxe/IScsiDhcp.h    |   55
> -
> >  .../Universal/Network/IScsiDxe/IScsiDriver.h  |  140
> -
> >  .../Universal/Network/IScsiDxe/IScsiDxe.inf   |  134
> -
> >  .../Network/IScsiDxe/IScsiExtScsiPassThru.h   |   22
> -
> >  .../Universal/Network/IScsiDxe/IScsiIbft.h    |   38
> -
> >  .../Universal/Network/IScsiDxe/IScsiImpl.h    |  168
> -
> >  .../Network/IScsiDxe/IScsiInitiatorName.h     |   74
> -
> >  .../Universal/Network/IScsiDxe/IScsiMisc.h    |  317
> --
> >  .../Universal/Network/IScsiDxe/IScsiProto.h   | 1005
> ------
> >  .../Universal/Network/IScsiDxe/IScsiTcp4Io.h  |  142
> -
> >  MdeModulePkg/Universal/Network/IScsiDxe/Md5.h |   80
> -
> >  .../Universal/Network/Tcp4Dxe/SockImpl.h      |  131
> -
> >  .../Universal/Network/Tcp4Dxe/Socket.h        |  954
> ------
> >  .../Universal/Network/Tcp4Dxe/Tcp4Driver.h    |  342
> --
> >  .../Universal/Network/Tcp4Dxe/Tcp4Dxe.inf     |   94
> -
> >  .../Universal/Network/Tcp4Dxe/Tcp4Dxe.uni     |   23
> -
> >  .../Network/Tcp4Dxe/Tcp4DxeExtra.uni          |   20
> -
> >  .../Universal/Network/Tcp4Dxe/Tcp4Func.h      |  781
> -----
> >  .../Universal/Network/Tcp4Dxe/Tcp4Main.h      |  494
> ---
> >  .../Universal/Network/Tcp4Dxe/Tcp4Option.h    |  130
> -
> >  .../Universal/Network/Tcp4Dxe/Tcp4Proto.h     |  357
> --
> >  .../Network/UefiPxeBcDxe/PxeBcDhcp.h          |  502
> ---
> >  .../Network/UefiPxeBcDxe/PxeBcDriver.h        |  102
> -
> >  .../Network/UefiPxeBcDxe/PxeBcImpl.h          |  189
> --
> >  .../Network/UefiPxeBcDxe/PxeBcMtftp.h         |  137
> -
> >  .../Network/UefiPxeBcDxe/PxeBcSupport.h       |  134
> -
> >  .../Network/UefiPxeBcDxe/UefiPxe4BcDxe.uni    |   25
> -
> >  .../UefiPxeBcDxe/UefiPxe4BcDxeExtra.uni       |   20
> -
> >  .../Network/UefiPxeBcDxe/UefiPxeBcDxe.inf     |  102
> -
> >  NetworkPkg/IScsiDxe/IScsiDxe.inf              |   10
> -
> >  NetworkPkg/TcpDxe/TcpDxe.inf                  |    6
> -
> >  NetworkPkg/UefiPxeBcDxe/UefiPxeBcDxe.inf      |    6
> -
> >  Nt32Pkg/Nt32Pkg.dsc                           |   12
> +-
> >  Nt32Pkg/Nt32Pkg.fdf                           |   12
> +-
> >  OvmfPkg/OvmfPkgIa32.dsc                       |   12
> +-
> >  OvmfPkg/OvmfPkgIa32.fdf                       |   10
> +-
> >  OvmfPkg/OvmfPkgIa32X64.dsc                    |   12
> +-
> >  OvmfPkg/OvmfPkgIa32X64.fdf                    |   10
> +-
> >  OvmfPkg/OvmfPkgX64.dsc                        |   12
> +-
> >  OvmfPkg/OvmfPkgX64.fdf                        |   10
> +-
> >  Vlv2TbltDevicePkg/PlatformPkg.fdf             |   14
> +-
> >  Vlv2TbltDevicePkg/PlatformPkgGcc.fdf          |   14
> +-
> >  Vlv2TbltDevicePkg/PlatformPkgGccX64.dsc       |   18
> +-
> >  Vlv2TbltDevicePkg/PlatformPkgIA32.dsc         |   18
> +-
> >  Vlv2TbltDevicePkg/PlatformPkgX64.dsc          |   18
> +-
> >  90 files changed, 55 insertions(+), 32771 deletions(-
> )  delete mode
> > 100644
> MdeModulePkg/Universal/Network/IScsiDxe/ComponentName.c
> >  delete mode 100644
> MdeModulePkg/Universal/Network/IScsiDxe/IScsiCHAP.c
> >  delete mode 100644
> MdeModulePkg/Universal/Network/IScsiDxe/IScsiConfig.c
> >  delete mode 100644
> MdeModulePkg/Universal/Network/IScsiDxe/IScsiDhcp.c
> >  delete mode 100644
> MdeModulePkg/Universal/Network/IScsiDxe/IScsiDriver.c
> >  delete mode 100644
> >
> MdeModulePkg/Universal/Network/IScsiDxe/IScsiExtScsiPass
> Thru.c
> >  delete mode 100644
> MdeModulePkg/Universal/Network/IScsiDxe/IScsiIbft.c
> >  delete mode 100644
> >
> MdeModulePkg/Universal/Network/IScsiDxe/IScsiInitiatorNa
> me.c
> >  delete mode 100644
> MdeModulePkg/Universal/Network/IScsiDxe/IScsiMisc.c
> >  delete mode 100644
> MdeModulePkg/Universal/Network/IScsiDxe/IScsiProto.c
> >  delete mode 100644
> MdeModulePkg/Universal/Network/IScsiDxe/IScsiTcp4Io.c
> >  delete mode 100644
> MdeModulePkg/Universal/Network/IScsiDxe/Md5.c
> >  delete mode 100644
> MdeModulePkg/Universal/Network/Tcp4Dxe/ComponentName.c
> >  delete mode 100644
> MdeModulePkg/Universal/Network/Tcp4Dxe/SockImpl.c
> >  delete mode 100644
> MdeModulePkg/Universal/Network/Tcp4Dxe/SockInterface.c
> >  delete mode 100644
> >
> MdeModulePkg/Universal/Network/Tcp4Dxe/Tcp4Dispatcher.c
> >  delete mode 100644
> MdeModulePkg/Universal/Network/Tcp4Dxe/Tcp4Driver.c
> >  delete mode 100644
> MdeModulePkg/Universal/Network/Tcp4Dxe/Tcp4Input.c
> >  delete mode 100644
> MdeModulePkg/Universal/Network/Tcp4Dxe/Tcp4Io.c
> >  delete mode 100644
> MdeModulePkg/Universal/Network/Tcp4Dxe/Tcp4Main.c
> >  delete mode 100644
> MdeModulePkg/Universal/Network/Tcp4Dxe/Tcp4Misc.c
> >  delete mode 100644
> MdeModulePkg/Universal/Network/Tcp4Dxe/Tcp4Option.c
> >  delete mode 100644
> MdeModulePkg/Universal/Network/Tcp4Dxe/Tcp4Output.c
> >  delete mode 100644
> MdeModulePkg/Universal/Network/Tcp4Dxe/Tcp4Timer.c
> >  delete mode 100644
> >
> MdeModulePkg/Universal/Network/UefiPxeBcDxe/ComponentNam
> e.c
> >  delete mode 100644
> >
> MdeModulePkg/Universal/Network/UefiPxeBcDxe/PxeBcDhcp.c
> >  delete mode 100644
> >
> MdeModulePkg/Universal/Network/UefiPxeBcDxe/PxeBcDriver.
> c
> >  delete mode 100644
> >
> MdeModulePkg/Universal/Network/UefiPxeBcDxe/PxeBcImpl.c
> >  delete mode 100644
> >
> MdeModulePkg/Universal/Network/UefiPxeBcDxe/PxeBcMtftp.c
> >  delete mode 100644
> >
> MdeModulePkg/Universal/Network/UefiPxeBcDxe/PxeBcSupport
> .c
> >  delete mode 100644
> >
> MdeModulePkg/Universal/Network/IScsiDxe/ComponentName.h
> >  delete mode 100644
> MdeModulePkg/Universal/Network/IScsiDxe/IScsi4Dxe.uni
> >  delete mode 100644
> >
> MdeModulePkg/Universal/Network/IScsiDxe/IScsi4DxeExtra.u
> ni
> >  delete mode 100644
> MdeModulePkg/Universal/Network/IScsiDxe/IScsiCHAP.h
> >  delete mode 100644
> MdeModulePkg/Universal/Network/IScsiDxe/IScsiCommon.h
> >  delete mode 100644
> MdeModulePkg/Universal/Network/IScsiDxe/IScsiConfig.h
> >  delete mode 100644
> >
> MdeModulePkg/Universal/Network/IScsiDxe/IScsiConfigDxe.v
> fr
> >  delete mode 100644
> >
> MdeModulePkg/Universal/Network/IScsiDxe/IScsiConfigDxeSt
> rings.uni
> >  delete mode 100644
> >
> MdeModulePkg/Universal/Network/IScsiDxe/IScsiConfigNVDat
> aStruc.h
> >  delete mode 100644
> MdeModulePkg/Universal/Network/IScsiDxe/IScsiDhcp.h
> >  delete mode 100644
> MdeModulePkg/Universal/Network/IScsiDxe/IScsiDriver.h
> >  delete mode 100644
> MdeModulePkg/Universal/Network/IScsiDxe/IScsiDxe.inf
> >  delete mode 100644
> >
> MdeModulePkg/Universal/Network/IScsiDxe/IScsiExtScsiPass
> Thru.h
> >  delete mode 100644
> MdeModulePkg/Universal/Network/IScsiDxe/IScsiIbft.h
> >  delete mode 100644
> MdeModulePkg/Universal/Network/IScsiDxe/IScsiImpl.h
> >  delete mode 100644
> >
> MdeModulePkg/Universal/Network/IScsiDxe/IScsiInitiatorNa
> me.h
> >  delete mode 100644
> MdeModulePkg/Universal/Network/IScsiDxe/IScsiMisc.h
> >  delete mode 100644
> MdeModulePkg/Universal/Network/IScsiDxe/IScsiProto.h
> >  delete mode 100644
> MdeModulePkg/Universal/Network/IScsiDxe/IScsiTcp4Io.h
> >  delete mode 100644
> MdeModulePkg/Universal/Network/IScsiDxe/Md5.h
> >  delete mode 100644
> MdeModulePkg/Universal/Network/Tcp4Dxe/SockImpl.h
> >  delete mode 100644
> MdeModulePkg/Universal/Network/Tcp4Dxe/Socket.h
> >  delete mode 100644
> MdeModulePkg/Universal/Network/Tcp4Dxe/Tcp4Driver.h
> >  delete mode 100644
> MdeModulePkg/Universal/Network/Tcp4Dxe/Tcp4Dxe.inf
> >  delete mode 100644
> MdeModulePkg/Universal/Network/Tcp4Dxe/Tcp4Dxe.uni
> >  delete mode 100644
> >
> MdeModulePkg/Universal/Network/Tcp4Dxe/Tcp4DxeExtra.uni
> >  delete mode 100644
> MdeModulePkg/Universal/Network/Tcp4Dxe/Tcp4Func.h
> >  delete mode 100644
> MdeModulePkg/Universal/Network/Tcp4Dxe/Tcp4Main.h
> >  delete mode 100644
> MdeModulePkg/Universal/Network/Tcp4Dxe/Tcp4Option.h
> >  delete mode 100644
> MdeModulePkg/Universal/Network/Tcp4Dxe/Tcp4Proto.h
> >  delete mode 100644
> >
> MdeModulePkg/Universal/Network/UefiPxeBcDxe/PxeBcDhcp.h
> >  delete mode 100644
> >
> MdeModulePkg/Universal/Network/UefiPxeBcDxe/PxeBcDriver.
> h
> >  delete mode 100644
> >
> MdeModulePkg/Universal/Network/UefiPxeBcDxe/PxeBcImpl.h
> >  delete mode 100644
> >
> MdeModulePkg/Universal/Network/UefiPxeBcDxe/PxeBcMtftp.h
> >  delete mode 100644
> >
> MdeModulePkg/Universal/Network/UefiPxeBcDxe/PxeBcSupport
> .h
> >  delete mode 100644
> >
> MdeModulePkg/Universal/Network/UefiPxeBcDxe/UefiPxe4BcDx
> e.uni
> >  delete mode 100644
> >
> MdeModulePkg/Universal/Network/UefiPxeBcDxe/UefiPxe4BcDx
> eExtra.uni
> >  delete mode 100644
> >
> MdeModulePkg/Universal/Network/UefiPxeBcDxe/UefiPxeBcDxe
> .inf
> >
> > --
> > 2.18.0.windows.1
> >
> > _______________________________________________
> > edk2-devel mailing list
> > [email protected]
> > https://lists.01.org/mailman/listinfo/edk2-devel
_______________________________________________
edk2-devel mailing list
[email protected]
https://lists.01.org/mailman/listinfo/edk2-devel

Reply via email to