From: "Carsey, Jaben" <jaben.carsey@intel.com>
To: Leif Lindholm <leif.lindholm@linaro.org>,
"Zhang, Shenglei" <shenglei.zhang@intel.com>
Cc: "edk2-devel@lists.01.org" <edk2-devel@lists.01.org>,
"Dong, Eric" <eric.dong@intel.com>,
"Zeng, Star" <star.zeng@intel.com>
Subject: Re: [PATCH 00/26] MdeModulePkg Remove redundant functions
Date: Wed, 8 Aug 2018 16:05:43 +0000 [thread overview]
Message-ID: <CB6E33457884FA40993F35157061515CA4110D34@FMSMSX103.amr.corp.intel.com> (raw)
In-Reply-To: <20180808145500.sh4dt3rwwdgcern6@bivouac.eciton.net>
I have a question:
How do we know that no platforms use these functions given that not all of everyone's platforms are stored in the same place?
I agree with Leif on timing; I really like cleanup.
-Jaben
> -----Original Message-----
> From: edk2-devel [mailto:edk2-devel-bounces@lists.01.org] On Behalf Of
> Leif Lindholm
> Sent: Wednesday, August 08, 2018 7:55 AM
> To: Zhang, Shenglei <shenglei.zhang@intel.com>
> Cc: edk2-devel@lists.01.org; Dong, Eric <eric.dong@intel.com>; Zeng, Star
> <star.zeng@intel.com>
> Subject: Re: [edk2] [PATCH 00/26] MdeModulePkg Remove redundant
> functions
>
> Eric, Star,
>
> On Wed, Aug 08, 2018 at 04:46:46PM +0800, shenglei wrote:
> > A lot of redundant functions which are never calld have been removed.I
> manually
> > search these fuctions in source files to make sure that they are not used
> elsewhere.
> > Also the MdeModulePkg was built to ensure that.
>
> I have no objection to this set, but it is quite invasive.
> So can we please ensure it does not get pushed until after the 201808
> stable tag has been made? (Mike has just announced the quiet period.)
>
> Best Regards,
> Leif
>
>
> > shenglei (26):
> > MdeModulePkg CapsuleApp: Remove a redundant function
> > MdeModulePkg UiApp: Remove redundant functions
> > MdeModulePkg AtaAtapiPassThru: Remove redundant functions
> > MdeModulePkg EhciDxe: Remove redundant functions
> > MdeModulePkg NvmExpressDxe: Remove redundant functions
> > MdeModulePkg PciBusDxe: Remove redundant functions
> > MdeModulePkg SdMmcPciHcDxe: Remove redundant functions
> > MdeModulePkg UhciPei: Remove redundant functions
> > MdeModulePkg XhciDxe: Remove redundant functions
> > MdeModulePkg XhciPei: Remove redundant functions
> > MdeModulePkg UfsBlockIoPei: Remove redundant functions
> > MdeModulePkg UfsPassThruDxe: Remove redundant functions
> > MdeModulePkg UsbBotPei: Remove redundant functions
> > MdeModulePkg UsbBusDxe: Remove redundant functions
> > MdeModulePkg UsbBusPei: Remove redundant functions
> > MdeModulePkg Core/Dxe: Remove redundant functions
> > MdeModulePkg PiSmmCore: Remove redundant functions
> > MdeModulePkg Ps2KeyboardDxe: Remove a redundant function
> > MdeModulePkg EmmcBlockIoPei: Remove a redundant function
> > MdeModulePkg SdBlockIoPei: Remove a redundant function
> > MdeModulePkg TerminalDxe: Remove a redundant function
> > MdeModulePkg HiiDatabaseDxe: Remove a redundant function
> > MdeModulePkg Ip4Dxe: Remove a redundant function
> > MdeModulePkg IScsiDxe: Remove a redundant function
> > MdeModulePkg Tcp4Dxe: Remove redundant functions
> > MdeModulePkg SetupBrowserDxe: Remove a redundant function
> >
> > .../Application/CapsuleApp/CapsuleDump.c | 31 ---
> > MdeModulePkg/Application/UiApp/FrontPage.c | 40 ---
> > MdeModulePkg/Application/UiApp/Ui.h | 30 --
> > .../Bus/Ata/AtaAtapiPassThru/AhciMode.c | 104 -------
> > .../Bus/Ata/AtaAtapiPassThru/IdeMode.c | 257 ------------------
> > .../Bus/Isa/Ps2KeyboardDxe/Ps2KbdCtrller.c | 26 --
> > .../Bus/Isa/Ps2KeyboardDxe/Ps2Keyboard.h | 12 -
> > MdeModulePkg/Bus/Pci/EhciDxe/EhciDebug.c | 27 --
> > MdeModulePkg/Bus/Pci/EhciDxe/EhciDebug.h | 11 -
> > MdeModulePkg/Bus/Pci/EhciDxe/EhciReg.c | 44 ---
> > .../Bus/Pci/NvmExpressDxe/NvmExpressHci.c | 110 --------
> > .../Bus/Pci/PciBusDxe/PciDeviceSupport.c | 80 ------
> > .../Bus/Pci/PciBusDxe/PciDeviceSupport.h | 17 --
> > MdeModulePkg/Bus/Pci/PciBusDxe/PciIo.c | 41 ---
> > MdeModulePkg/Bus/Pci/PciBusDxe/PciIo.h | 21 --
> > MdeModulePkg/Bus/Pci/SdMmcPciHcDxe/SdDevice.c | 107 --------
> > MdeModulePkg/Bus/Pci/UhciPei/DmaMem.c | 22 --
> > MdeModulePkg/Bus/Pci/UhciPei/UhcPeim.c | 125 ---------
> > MdeModulePkg/Bus/Pci/UhciPei/UhcPeim.h | 78 ------
> > MdeModulePkg/Bus/Pci/XhciDxe/XhciReg.c | 66 -----
> > MdeModulePkg/Bus/Pci/XhciDxe/XhciReg.h | 28 --
> > MdeModulePkg/Bus/Pci/XhciPei/UsbHcMem.c | 24 --
> > MdeModulePkg/Bus/Pci/XhciPei/XhcPeim.c | 22 --
> > MdeModulePkg/Bus/Pci/XhciPei/XhciReg.h | 14 -
> > .../Bus/Sd/EmmcBlockIoPei/EmmcHcMem.c | 24 --
> > MdeModulePkg/Bus/Sd/SdBlockIoPei/SdHcMem.c | 24 --
> > .../Bus/Ufs/UfsBlockIoPei/UfsBlockIoPei.c | 101 -------
> > MdeModulePkg/Bus/Ufs/UfsBlockIoPei/UfsHcMem.c | 24 --
> > MdeModulePkg/Bus/Ufs/UfsBlockIoPei/UfsHci.c | 180 ------------
> > .../Bus/Ufs/UfsPassThruDxe/UfsPassThruHci.c | 49 ----
> > MdeModulePkg/Bus/Usb/UsbBotPei/PeiUsbLib.c | 190 -------------
> > MdeModulePkg/Bus/Usb/UsbBotPei/PeiUsbLib.h | 98 -------
> > MdeModulePkg/Bus/Usb/UsbBusDxe/UsbHub.c | 68 -----
> > MdeModulePkg/Bus/Usb/UsbBusDxe/UsbUtility.c | 146 ----------
> > MdeModulePkg/Bus/Usb/UsbBusDxe/UsbUtility.h | 114 --------
> > MdeModulePkg/Bus/Usb/UsbBusPei/HubPeim.c | 39 ---
> > MdeModulePkg/Bus/Usb/UsbBusPei/HubPeim.h | 18 --
> > MdeModulePkg/Bus/Usb/UsbBusPei/PeiUsbLib.c | 77 ------
> > MdeModulePkg/Bus/Usb/UsbBusPei/PeiUsbLib.h | 35 ---
> > MdeModulePkg/Core/Dxe/DxeMain.h | 13 -
> > MdeModulePkg/Core/Dxe/DxeMain/DxeMain.c | 22 --
> > MdeModulePkg/Core/Dxe/Mem/HeapGuard.c | 78 ------
> > MdeModulePkg/Core/PiSmmCore/HeapGuard.c | 166 -----------
> > .../Core/PiSmmCore/MemoryAttributesTable.c | 131 ---------
> > MdeModulePkg/Core/PiSmmCore/Page.c | 121 ---------
> > .../Universal/Console/TerminalDxe/Terminal.h | 12 -
> > .../Console/TerminalDxe/TerminalConIn.c | 25 --
> > .../Universal/HiiDatabaseDxe/ConfigRouting.c | 47 ----
> > .../Universal/Network/IScsiDxe/IScsiProto.c | 31 ---
> > .../Universal/Network/Ip4Dxe/Ip4Config2Impl.c | 16 --
> > .../Universal/Network/Tcp4Dxe/SockImpl.c | 35 ---
> > .../Universal/Network/Tcp4Dxe/SockInterface.c | 41 ---
> > .../Universal/Network/Tcp4Dxe/Socket.h | 32 ---
> > .../Universal/Network/Tcp4Dxe/Tcp4Option.c | 28 --
> > .../Universal/Network/Tcp4Dxe/Tcp4Option.h | 15 -
> > .../Universal/SetupBrowserDxe/IfrParse.c | 33 ---
> > 56 files changed, 3370 deletions(-)
> >
> > --
> > 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
next prev parent reply other threads:[~2018-08-08 16:05 UTC|newest]
Thread overview: 94+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-08-08 8:46 [PATCH 00/26] MdeModulePkg Remove redundant functions shenglei
2018-08-08 8:46 ` [PATCH 01/26] MdeModulePkg CapsuleApp: Remove a redundant function shenglei
2018-08-09 10:10 ` Zeng, Star
2018-08-10 9:13 ` Zeng, Star
2018-08-08 8:46 ` [PATCH 02/26] MdeModulePkg UiApp: Remove redundant functions shenglei
2018-08-08 18:06 ` Laszlo Ersek
2018-08-09 3:25 ` Zeng, Star
2018-08-08 8:46 ` [PATCH 03/26] MdeModulePkg AtaAtapiPassThru: " shenglei
2018-08-08 18:11 ` Laszlo Ersek
2018-08-09 10:15 ` Zeng, Star
2018-08-08 8:46 ` [PATCH 04/26] MdeModulePkg EhciDxe: " shenglei
2018-08-08 18:08 ` Laszlo Ersek
2018-08-09 3:14 ` Ni, Ruiyu
2018-08-09 10:18 ` Zeng, Star
2018-08-08 8:46 ` [PATCH 05/26] MdeModulePkg NvmExpressDxe: " shenglei
2018-08-09 10:19 ` Zeng, Star
2018-08-09 13:33 ` Laszlo Ersek
2018-08-10 2:45 ` Wu, Hao A
2018-08-08 8:46 ` [PATCH 06/26] MdeModulePkg PciBusDxe: " shenglei
2018-08-09 3:09 ` Ni, Ruiyu
2018-08-09 13:35 ` Laszlo Ersek
2018-08-10 8:54 ` Zeng, Star
2018-08-08 8:46 ` [PATCH 07/26] MdeModulePkg SdMmcPciHcDxe: " shenglei
2018-08-08 8:46 ` [PATCH 08/26] MdeModulePkg UhciPei: " shenglei
2018-08-09 3:10 ` Ni, Ruiyu
2018-08-10 8:57 ` Zeng, Star
2018-08-08 8:46 ` [PATCH 09/26] MdeModulePkg XhciDxe: " shenglei
2018-08-09 3:09 ` Ni, Ruiyu
2018-08-09 13:47 ` Laszlo Ersek
2018-08-10 9:01 ` Zeng, Star
2018-08-08 8:46 ` [PATCH 10/26] MdeModulePkg XhciPei: " shenglei
2018-08-09 3:12 ` Ni, Ruiyu
2018-08-10 9:01 ` Zeng, Star
2018-08-08 8:46 ` [PATCH 11/26] MdeModulePkg UfsBlockIoPei: " shenglei
2018-08-10 2:57 ` Wu, Hao A
2018-08-10 9:04 ` Zeng, Star
2018-08-08 8:46 ` [PATCH 12/26] MdeModulePkg UfsPassThruDxe: " shenglei
2018-08-10 2:45 ` Wu, Hao A
2018-08-10 9:05 ` Zeng, Star
2018-08-08 8:46 ` [PATCH 13/26] MdeModulePkg UsbBotPei: " shenglei
2018-08-10 9:08 ` Zeng, Star
2018-08-08 8:47 ` [PATCH 14/26] MdeModulePkg UsbBusDxe: " shenglei
2018-08-09 3:12 ` Ni, Ruiyu
2018-08-09 13:52 ` Laszlo Ersek
2018-08-10 9:09 ` Zeng, Star
2018-08-08 8:47 ` [PATCH 15/26] MdeModulePkg UsbBusPei: " shenglei
2018-08-09 3:12 ` Ni, Ruiyu
2018-08-10 9:08 ` Zeng, Star
2018-08-08 8:47 ` [PATCH 16/26] MdeModulePkg Core/Dxe: " shenglei
2018-08-09 3:10 ` Ni, Ruiyu
2018-08-09 3:43 ` Wang, Jian J
2018-08-09 13:56 ` Laszlo Ersek
2018-08-10 9:10 ` Zeng, Star
2018-08-08 8:47 ` [PATCH 17/26] MdeModulePkg PiSmmCore: " shenglei
2018-08-09 3:12 ` Ni, Ruiyu
2018-08-09 3:44 ` Wang, Jian J
2018-08-09 14:07 ` Laszlo Ersek
2018-08-10 9:12 ` Zeng, Star
2018-08-08 8:47 ` [PATCH 18/26] MdeModulePkg Ps2KeyboardDxe: Remove a redundant function shenglei
2018-08-09 3:12 ` Ni, Ruiyu
2018-08-09 14:10 ` Laszlo Ersek
2018-08-10 9:15 ` Zeng, Star
2018-08-08 8:47 ` [PATCH 19/26] MdeModulePkg EmmcBlockIoPei: " shenglei
2018-08-10 2:57 ` Wu, Hao A
2018-08-10 9:16 ` Zeng, Star
2018-08-08 8:47 ` [PATCH 20/26] MdeModulePkg SdBlockIoPei: " shenglei
2018-08-10 2:57 ` Wu, Hao A
2018-08-10 9:16 ` Zeng, Star
2018-08-08 8:47 ` [PATCH 21/26] MdeModulePkg TerminalDxe: " shenglei
2018-08-09 3:13 ` Ni, Ruiyu
2018-08-09 10:25 ` Zeng, Star
2018-08-10 9:14 ` Zeng, Star
2018-08-09 14:12 ` Laszlo Ersek
2018-08-08 8:47 ` [PATCH 22/26] MdeModulePkg HiiDatabaseDxe: " shenglei
2018-08-09 10:23 ` Zeng, Star
2018-08-10 9:14 ` Zeng, Star
2018-08-09 14:15 ` Laszlo Ersek
2018-08-08 8:47 ` [PATCH 23/26] MdeModulePkg Ip4Dxe: " shenglei
2018-08-09 10:21 ` Zeng, Star
2018-08-09 14:16 ` Laszlo Ersek
2018-08-08 8:47 ` [PATCH 24/26] MdeModulePkg IScsiDxe: " shenglei
2018-08-09 10:21 ` Zeng, Star
2018-08-09 14:18 ` Laszlo Ersek
2018-08-08 8:47 ` [PATCH 25/26] MdeModulePkg Tcp4Dxe: Remove redundant functions shenglei
2018-08-09 10:21 ` Zeng, Star
2018-08-09 14:21 ` Laszlo Ersek
2018-08-08 8:47 ` [PATCH 26/26] MdeModulePkg SetupBrowserDxe: Remove a redundant function shenglei
2018-08-09 10:22 ` Zeng, Star
2018-08-09 14:26 ` Laszlo Ersek
2018-08-08 14:55 ` [PATCH 00/26] MdeModulePkg Remove redundant functions Leif Lindholm
2018-08-08 16:05 ` Carsey, Jaben [this message]
2018-08-08 17:21 ` Laszlo Ersek
2018-08-09 3:44 ` Zeng, Star
2018-08-09 3:33 ` Zeng, Star
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-list from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CB6E33457884FA40993F35157061515CA4110D34@FMSMSX103.amr.corp.intel.com \
--to=devel@edk2.groups.io \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox