Mike, Should I also update the devel branches in edk2-platform? Or the branch owner will take care of it?
BestRegards Fu Siyuan > -----Original Message----- > From: Kinney, Michael D > Sent: Wednesday, October 31, 2018 5:15 AM > To: Fu, Siyuan <siyuan...@intel.com>; Zeng, Star <star.z...@intel.com>; > edk2-devel@lists.01.org; Kinney, Michael D <michael.d.kin...@intel.com> > Cc: Leif Lindholm <leif.lindh...@linaro.org>; Andrew Fish (af...@apple.com) > <af...@apple.com>; Laszlo Ersek (ler...@redhat.com) <ler...@redhat.com>; > Gao, Liming <liming....@intel.com> > Subject: RE: [edk2] [PATCH v1 0/7] Delete TCP, PXE, iSCSI driver in > MdeModulePkg. > > 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 <star.z...@intel.com>; edk2- > > de...@lists.01.org > > Cc: Kinney, Michael D <michael.d.kin...@intel.com>; Leif > > Lindholm <leif.lindh...@linaro.org>; Andrew Fish > > (af...@apple.com) <af...@apple.com>; Laszlo Ersek > > (ler...@redhat.com) <ler...@redhat.com>; Gao, Liming > > <liming....@intel.com> > > 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 <siyuan...@intel.com>; edk2- > > de...@lists.01.org > > > Cc: Kinney, Michael D <michael.d.kin...@intel.com>; > > Leif Lindholm > > > <leif.lindh...@linaro.org>; Andrew Fish > > (af...@apple.com) > > > <af...@apple.com>; Laszlo Ersek (ler...@redhat.com) > > <ler...@redhat.com>; > > > Gao, Liming <liming....@intel.com>; Zeng, Star > > <star.z...@intel.com> > > > 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- > > boun...@lists.01.org] On Behalf Of Fu > > > Siyuan > > > Sent: Tuesday, October 30, 2018 3:33 PM > > > To: edk2-devel@lists.01.org > > > 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 > > > edk2-devel@lists.01.org > > > https://lists.01.org/mailman/listinfo/edk2-devel _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel