From: "Wu, Jiaxin" <jiaxin.wu@intel.com>
To: "Li, Songpeng" <songpeng.li@intel.com>,
"edk2-devel@lists.01.org" <edk2-devel@lists.01.org>
Cc: "Fu, Siyuan" <siyuan.fu@intel.com>
Subject: Re: [PATCH] NetworkPkg: Remove the redundant code and definition.
Date: Wed, 15 Aug 2018 00:45:20 +0000 [thread overview]
Message-ID: <895558F6EA4E3B41AC93A00D163B72741648B1B7@SHSMSX103.ccr.corp.intel.com> (raw)
In-Reply-To: <20180813024519.5704-1-songpeng.li@intel.com>
Looks good to me.
Reviewed-by: Jiaxin Wu <jiaxin.wu@intel.com>
> -----Original Message-----
> From: Li, Songpeng
> Sent: Monday, August 13, 2018 10:45 AM
> To: edk2-devel@lists.01.org
> Cc: Wu, Jiaxin <jiaxin.wu@intel.com>; Fu, Siyuan <siyuan.fu@intel.com>
> Subject: [PATCH] NetworkPkg: Remove the redundant code and definition.
>
> Cc: Jiaxin Wu <jiaxin.wu@intel.com>
> Cc: Siyuan Fu <siyuan.fu@intel.com>
> Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1064
> Contributed-under: TianoCore Contribution Agreement 1.1
> Signed-off-by: Songpeng Li <songpeng.li@intel.com>
> ---
> NetworkPkg/IScsiDxe/IScsiDxe.inf | 1 -
> NetworkPkg/IScsiDxe/IScsiProto.c | 33 ------------
> NetworkPkg/IpSecDxe/Ikev2/Payload.c | 18 -------
> NetworkPkg/IpSecDxe/Ikev2/Utility.c | 76 ----------------------------
> NetworkPkg/IpSecDxe/Ikev2/Utility.h | 67 ------------------------
> NetworkPkg/TcpDxe/TcpOption.c | 30 -----------
> NetworkPkg/TcpDxe/TcpOption.h | 16 ------
> NetworkPkg/TlsDxe/TlsDxe.inf | 1 -
> NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c | 23 ---------
> NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.h | 10 ----
> 10 files changed, 275 deletions(-)
>
> diff --git a/NetworkPkg/IScsiDxe/IScsiDxe.inf
> b/NetworkPkg/IScsiDxe/IScsiDxe.inf
> index 2d96611b44..60737c96ab 100644
> --- a/NetworkPkg/IScsiDxe/IScsiDxe.inf
> +++ b/NetworkPkg/IScsiDxe/IScsiDxe.inf
> @@ -124,7 +124,6 @@
> gEfiIfrTianoGuid ## SOMETIMES_PRODUCES ## UNDEFINED
> gEfiAcpiTableGuid ## SOMETIMES_CONSUMES ##
> SystemTable
> gEfiAcpi10TableGuid ## SOMETIMES_CONSUMES ##
> SystemTable
> - gEfiAcpi20TableGuid ## SOMETIMES_CONSUMES ##
> SystemTable
> gEfiAdapterInfoNetworkBootGuid ## SOMETIMES_CONSUMES ##
> UNDEFINED
> gEfiAdapterInfoUndiIpv6SupportGuid ## SOMETIMES_CONSUMES ##
> GUID
>
> diff --git a/NetworkPkg/IScsiDxe/IScsiProto.c
> b/NetworkPkg/IScsiDxe/IScsiProto.c
> index 7619360568..f4a49c677a 100644
> --- a/NetworkPkg/IScsiDxe/IScsiProto.c
> +++ b/NetworkPkg/IScsiDxe/IScsiProto.c
> @@ -2096,39 +2096,6 @@ IScsiDelTcb (
> }
>
>
> -/**
> - Find the task control block by the initator task tag.
> -
> - @param[in] TcbList The tcb list.
> - @param[in] InitiatorTaskTag The initiator task tag.
> -
> - @return The task control block found.
> - @retval NULL The task control block cannot be found.
> -
> -**/
> -ISCSI_TCB *
> -IScsiFindTcbByITT (
> - IN LIST_ENTRY *TcbList,
> - IN UINT32 InitiatorTaskTag
> - )
> -{
> - ISCSI_TCB *Tcb;
> - LIST_ENTRY *Entry;
> -
> - Tcb = NULL;
> -
> - NET_LIST_FOR_EACH (Entry, TcbList) {
> - Tcb = NET_LIST_USER_STRUCT (Entry, ISCSI_TCB, Link);
> -
> - if (Tcb->InitiatorTaskTag == InitiatorTaskTag) {
> - break;
> - }
> - }
> -
> - return Tcb;
> -}
> -
> -
> /**
> Create a data segment, pad it, and calculate the CRC if needed.
>
> diff --git a/NetworkPkg/IpSecDxe/Ikev2/Payload.c
> b/NetworkPkg/IpSecDxe/Ikev2/Payload.c
> index 218c26f934..1bb5e2e5e5 100644
> --- a/NetworkPkg/IpSecDxe/Ikev2/Payload.c
> +++ b/NetworkPkg/IpSecDxe/Ikev2/Payload.c
> @@ -3104,24 +3104,6 @@ ON_EXIT:
> return Status;
> }
>
> -/**
> - Save some useful payloads after accepting the Packet.
> -
> - @param[in] SessionCommon Pointer to IKEV2_SESSION_COMMON
> related to the operation.
> - @param[in] IkePacket Pointer to received IkePacet.
> - @param[in] IkeType The type used to indicate it is in IkeSa or ChildSa or
> Info
> - exchange.
> -
> -**/
> -VOID
> -Ikev2OnPacketAccepted (
> - IN IKEV2_SESSION_COMMON *SessionCommon,
> - IN IKE_PACKET *IkePacket,
> - IN UINT8 IkeType
> - )
> -{
> - return;
> -}
>
> /**
>
> diff --git a/NetworkPkg/IpSecDxe/Ikev2/Utility.c
> b/NetworkPkg/IpSecDxe/Ikev2/Utility.c
> index 698aba1327..0c9c929705 100644
> --- a/NetworkPkg/IpSecDxe/Ikev2/Utility.c
> +++ b/NetworkPkg/IpSecDxe/Ikev2/Utility.c
> @@ -290,21 +290,6 @@ Ikev2SaSessionRemove (
> return NULL;
> }
>
> -/**
> - Marking a SA session as on deleting.
> -
> - @param[in] IkeSaSession Pointer to IKEV2_SA_SESSION.
> -
> - @retval EFI_SUCCESS Find the related SA session and marked it.
> -
> -**/
> -EFI_STATUS
> -Ikev2SaSessionOnDeleting (
> - IN IKEV2_SA_SESSION *IkeSaSession
> - )
> -{
> - return EFI_SUCCESS;
> -}
>
> /**
> Free specified Seession Common. The session common would belong to a
> IKE SA or
> @@ -659,33 +644,6 @@ Ikev2ChildSaSessionReg (
> return ;
> }
>
> -/**
> - Find the ChildSaSession by it's MessagId.
> -
> - @param[in] SaSessionList Pointer to a ChildSaSession List.
> - @param[in] Mid The messageId used to search ChildSaSession.
> -
> - @return Pointer to IKEV2_CHILD_SA_SESSION or NULL.
> -
> -**/
> -IKEV2_CHILD_SA_SESSION *
> -Ikev2ChildSaSessionLookupByMid (
> - IN LIST_ENTRY *SaSessionList,
> - IN UINT32 Mid
> - )
> -{
> - LIST_ENTRY *Entry;
> - IKEV2_CHILD_SA_SESSION *ChildSaSession;
> -
> - NET_LIST_FOR_EACH (Entry, SaSessionList) {
> - ChildSaSession = IKEV2_CHILD_SA_SESSION_BY_IKE_SA (Entry);
> -
> - if (ChildSaSession->MessageId == Mid) {
> - return ChildSaSession;
> - }
> - }
> - return NULL;
> -}
>
> /**
> This function find the Child SA by the specified SPI.
> @@ -776,22 +734,6 @@ Ikev2ChildSaSessionRemove (
> return NULL;
> }
>
> -/**
> - Mark a specified Child SA Session as on deleting.
> -
> - @param[in] ChildSaSession Pointer to IKEV2_CHILD_SA_SESSION.
> -
> - @retval EFI_SUCCESS Operation is successful.
> -
> -**/
> -EFI_STATUS
> -Ikev2ChildSaSessionOnDeleting (
> - IN IKEV2_CHILD_SA_SESSION *ChildSaSession
> - )
> -{
> - return EFI_SUCCESS;
> -}
> -
> /**
> Free the memory located for the specified IKEV2_CHILD_SA_SESSION.
>
> @@ -1137,24 +1079,6 @@ Ikev2ChildSaAssociateSpdEntry (
> }
>
>
> -/**
> - This function finds the SPI from Create Child SA Exchange Packet.
> -
> - @param[in] IkePacket Pointer to IKE_PACKET to be searched.
> -
> - @retval SPI number or 0 if it is not supported.
> -
> -**/
> -UINT32
> -Ikev2ChildExchangeRekeySpi (
> - IN IKE_PACKET *IkePacket
> - )
> -{
> - //
> - // Not support yet.
> - //
> - return 0;
> -}
>
> /**
> Validate the IKE header of received IKE packet.
> diff --git a/NetworkPkg/IpSecDxe/Ikev2/Utility.h
> b/NetworkPkg/IpSecDxe/Ikev2/Utility.h
> index 8383ff8dd6..79def7740d 100644
> --- a/NetworkPkg/IpSecDxe/Ikev2/Utility.h
> +++ b/NetworkPkg/IpSecDxe/Ikev2/Utility.h
> @@ -104,19 +104,6 @@ Ikev2SaSessionRemove (
> );
>
>
> -/**
> - Marking a SA session as on deleting.
> -
> - @param[in] IkeSaSession Pointer to IKEV2_SA_SESSION.
> -
> - @retval EFI_SUCCESS Find the related SA session and marked it.
> -
> -**/
> -EFI_STATUS
> -Ikev2SaSessionOnDeleting (
> - IN IKEV2_SA_SESSION *IkeSaSession
> - );
> -
> /**
> After IKE/Child SA is estiblished, close the time event and free sent packet.
>
> @@ -211,20 +198,6 @@ Ikev2ChildSaSessionLookupBySpi (
> IN UINT32 Spi
> );
>
> -/**
> - Find the ChildSaSession by it's MessagId.
> -
> - @param[in] SaSessionList Pointer to a ChildSaSession List.
> - @param[in] Mid The messageId used to search ChildSaSession.
> -
> - @return Pointer to IKEV2_CHILD_SA_SESSION.
> -
> -**/
> -IKEV2_CHILD_SA_SESSION *
> -Ikev2ChildSaSessionLookupByMid (
> - IN LIST_ENTRY *SaSessionList,
> - IN UINT32 Mid
> - );
>
> /**
> Insert a Child SA Session into the specified ChildSa list..
> @@ -257,18 +230,6 @@ Ikev2ChildSaSessionRemove (
> IN UINT8 ListType
> );
>
> -/**
> - Mark a specified Child SA Session as on deleting.
> -
> - @param[in] ChildSaSession Pointer to IKEV2_CHILD_SA_SESSION.
> -
> - @retval EFI_SUCCESS Operation is successful.
> -
> -**/
> -EFI_STATUS
> -Ikev2ChildSaSessionOnDeleting (
> - IN IKEV2_CHILD_SA_SESSION *ChildSaSession
> - );
>
> /**
> Free the memory located for the specified IKEV2_CHILD_SA_SESSION.
> @@ -328,19 +289,6 @@ Ikev2ChildExchangeRequestType(
> IN IKE_PACKET *IkePacket
> );
>
> -/**
> - This function finds the SPI from Create Child Sa Exchange Packet.
> -
> - @param[in] IkePacket Pointer to IKE_PACKET to be searched.
> -
> - @retval SPI number.
> -
> -**/
> -UINT32
> -Ikev2ChildExchangeRekeySpi(
> - IN IKE_PACKET *IkePacket
> - );
> -
>
> /**
> Associate a SPD selector to the Child SA Session.
> @@ -882,21 +830,6 @@ Ikev2DecodePacket (
> IN UINTN IkeType
> );
>
> -/**
> - Save some useful payloads after accepting the Packet.
> -
> - @param[in] SessionCommon Pointer to IKEV2_SESSION_COMMON
> related to the operation.
> - @param[in] IkePacket Pointer to received IkePacet.
> - @param[in] IkeType The type used to indicate it is in IkeSa or ChildSa or
> Info
> - exchange.
> -
> -**/
> -VOID
> -Ikev2OnPacketAccepted (
> - IN IKEV2_SESSION_COMMON *SessionCommon,
> - IN IKE_PACKET *IkePacket,
> - IN UINT8 IkeType
> - );
>
> /**
> Send out IKEV2 packet.
> diff --git a/NetworkPkg/TcpDxe/TcpOption.c
> b/NetworkPkg/TcpDxe/TcpOption.c
> index bacce1070d..73d781729f 100644
> --- a/NetworkPkg/TcpDxe/TcpOption.c
> +++ b/NetworkPkg/TcpDxe/TcpOption.c
> @@ -342,33 +342,3 @@ TcpParseOption (
>
> return 0;
> }
> -
> -/**
> - Check the segment against PAWS.
> -
> - @param[in] Tcb Pointer to the TCP_CB of this TCP instance.
> - @param[in] TSVal The timestamp value.
> -
> - @retval 1 The segment passed the PAWS check.
> - @retval 0 The segment failed to pass the PAWS check.
> -
> -**/
> -UINT32
> -TcpPawsOK (
> - IN TCP_CB *Tcb,
> - IN UINT32 TSVal
> - )
> -{
> - //
> - // PAWS as defined in RFC1323, buggy...
> - //
> - if (TCP_TIME_LT (TSVal, Tcb->TsRecent) &&
> - TCP_TIME_LT (Tcb->TsRecentAge + TCP_PAWS_24DAY, mTcpTick)
> - ) {
> -
> - return 0;
> -
> - }
> -
> - return 1;
> -}
> diff --git a/NetworkPkg/TcpDxe/TcpOption.h
> b/NetworkPkg/TcpDxe/TcpOption.h
> index 0ccadb9536..9d8581b669 100644
> --- a/NetworkPkg/TcpDxe/TcpOption.h
> +++ b/NetworkPkg/TcpDxe/TcpOption.h
> @@ -126,20 +126,4 @@ TcpParseOption (
> IN OUT TCP_OPTION *Option
> );
>
> -/**
> - Check the segment against PAWS.
> -
> - @param[in] Tcb Pointer to the TCP_CB of this TCP instance.
> - @param[in] TSVal The timestamp value.
> -
> - @retval 1 The segment passed the PAWS check.
> - @retval 0 The segment failed to pass the PAWS check.
> -
> -**/
> -UINT32
> -TcpPawsOK (
> - IN TCP_CB *Tcb,
> - IN UINT32 TSVal
> - );
> -
> #endif
> diff --git a/NetworkPkg/TlsDxe/TlsDxe.inf b/NetworkPkg/TlsDxe/TlsDxe.inf
> index 907feb735b..bbe8c554a2 100644
> --- a/NetworkPkg/TlsDxe/TlsDxe.inf
> +++ b/NetworkPkg/TlsDxe/TlsDxe.inf
> @@ -52,7 +52,6 @@
> BaseLib
> UefiLib
> DebugLib
> - NetLib
> BaseCryptLib
> TlsLib
>
> diff --git a/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c
> b/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c
> index c06b0242bb..5bbc1922a6 100644
> --- a/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c
> +++ b/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.c
> @@ -202,29 +202,6 @@ PxeBcCacheDhcp6Packet (
> return EFI_SUCCESS;
> }
>
> -
> -/**
> - Free all the nodes in the list for boot file.
> -
> - @param[in] Head The pointer to the head of list.
> -
> -**/
> -VOID
> -PxeBcFreeBootFileOption (
> - IN LIST_ENTRY *Head
> - )
> -{
> - LIST_ENTRY *Entry;
> - LIST_ENTRY *NextEntry;
> - PXEBC_DHCP6_OPTION_NODE *Node;
> -
> - NET_LIST_FOR_EACH_SAFE (Entry, NextEntry, Head) {
> - Node = NET_LIST_USER_STRUCT (Entry, PXEBC_DHCP6_OPTION_NODE,
> Link);
> - RemoveEntryList (Entry);
> - FreePool (Node);
> - }
> -}
> -
> /**
> Retrieve the boot server address using the EFI_DNS6_PROTOCOL.
>
> diff --git a/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.h
> b/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.h
> index f3dcd89743..d6e6745f0d 100644
> --- a/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.h
> +++ b/NetworkPkg/UefiPxeBcDxe/PxeBcDhcp6.h
> @@ -116,16 +116,6 @@ typedef struct {
> } PXEBC_DHCP6_PACKET_CACHE;
>
>
> -/**
> - Free all the nodes in the boot file list.
> -
> - @param[in] Head The pointer to the head of the list.
> -
> -**/
> -VOID
> -PxeBcFreeBootFileOption (
> - IN LIST_ENTRY *Head
> - );
>
>
> /**
> --
> 2.18.0.windows.1
next prev parent reply other threads:[~2018-08-15 0:45 UTC|newest]
Thread overview: 5+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-08-13 2:45 [PATCH] NetworkPkg: Remove the redundant code and definition Songpeng Li
2018-08-13 4:42 ` Fu, Siyuan
2018-08-14 15:14 ` Laszlo Ersek
2018-08-15 0:45 ` Wu, Jiaxin [this message]
[not found] <20180809083322.7888-1-songpeng.li@intel.com>
2018-08-13 4:43 ` Fu, Siyuan
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=895558F6EA4E3B41AC93A00D163B72741648B1B7@SHSMSX103.ccr.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