From nobody Tue Apr 23 14:31:35 2024 Delivered-To: importer@patchew.org Authentication-Results: mx.zohomail.com; spf=none (zoho.com: 198.145.21.10 is neither permitted nor denied by domain of lists.01.org) smtp.mailfrom=edk2-devel-bounces@lists.01.org; dmarc=fail(p=none dis=none) header.from=intel.com Return-Path: Received: from ml01.01.org (ml01.01.org [198.145.21.10]) by mx.zohomail.com with SMTPS id 1534128349898776.1012405140418; Sun, 12 Aug 2018 19:45:49 -0700 (PDT) Received: from [127.0.0.1] (localhost [IPv6:::1]) by ml01.01.org (Postfix) with ESMTP id 48488210ED78F; Sun, 12 Aug 2018 19:45:48 -0700 (PDT) Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by ml01.01.org (Postfix) with ESMTPS id BCCEA210D97A4 for ; Sun, 12 Aug 2018 19:45:46 -0700 (PDT) Received: from fmsmga006.fm.intel.com ([10.253.24.20]) by orsmga102.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384; 12 Aug 2018 19:45:45 -0700 Received: from songpeng.ccr.corp.intel.com ([10.239.9.136]) by fmsmga006.fm.intel.com with ESMTP; 12 Aug 2018 19:45:44 -0700 X-Original-To: edk2-devel@lists.01.org Received-SPF: none (zoho.com: 198.145.21.10 is neither permitted nor denied by domain of lists.01.org) client-ip=198.145.21.10; envelope-from=edk2-devel-bounces@lists.01.org; helo=ml01.01.org; Received-SPF: Pass (sender SPF authorized) identity=mailfrom; client-ip=134.134.136.24; helo=mga09.intel.com; envelope-from=songpeng.li@intel.com; receiver=edk2-devel@lists.01.org X-Amp-Result: SKIPPED(no attachment in message) X-Amp-File-Uploaded: False X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.53,231,1531810800"; d="scan'208";a="254244490" From: Songpeng Li To: edk2-devel@lists.01.org Date: Mon, 13 Aug 2018 10:45:19 +0800 Message-Id: <20180813024519.5704-1-songpeng.li@intel.com> X-Mailer: git-send-email 2.18.0.windows.1 Subject: [edk2] [PATCH] NetworkPkg: Remove the redundant code and definition. X-BeenThere: edk2-devel@lists.01.org X-Mailman-Version: 2.1.27 Precedence: list List-Id: EDK II Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: Siyuan Fu , Jiaxin Wu MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Errors-To: edk2-devel-bounces@lists.01.org Sender: "edk2-devel" X-ZohoMail: RDMRC_1 RSF_4 Z_629925259 SPT_0 Content-Type: text/plain; charset="utf-8" Cc: Jiaxin Wu Cc: Siyuan Fu Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=3D1064 Contributed-under: TianoCore Contribution Agreement 1.1 Signed-off-by: Songpeng Li Reviewed-by: Fu Siyuan Reviewed-by: Jiaxin Wu --- 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/IScsiDx= e.inf index 2d96611b44..60737c96ab 100644 --- a/NetworkPkg/IScsiDxe/IScsiDxe.inf +++ b/NetworkPkg/IScsiDxe/IScsiDxe.inf @@ -124,7 +124,6 @@ gEfiIfrTianoGuid ## SOMETIMES_PRODUCES ## U= NDEFINED gEfiAcpiTableGuid ## SOMETIMES_CONSUMES ## S= ystemTable gEfiAcpi10TableGuid ## SOMETIMES_CONSUMES ## S= ystemTable - gEfiAcpi20TableGuid ## SOMETIMES_CONSUMES ## S= ystemTable gEfiAdapterInfoNetworkBootGuid ## SOMETIMES_CONSUMES ## U= NDEFINED gEfiAdapterInfoUndiIpv6SupportGuid ## SOMETIMES_CONSUMES ## G= UID =20 diff --git a/NetworkPkg/IScsiDxe/IScsiProto.c b/NetworkPkg/IScsiDxe/IScsiPr= oto.c index 7619360568..f4a49c677a 100644 --- a/NetworkPkg/IScsiDxe/IScsiProto.c +++ b/NetworkPkg/IScsiDxe/IScsiProto.c @@ -2096,39 +2096,6 @@ IScsiDelTcb ( } =20 =20 -/** - 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 =3D NULL; - - NET_LIST_FOR_EACH (Entry, TcbList) { - Tcb =3D NET_LIST_USER_STRUCT (Entry, ISCSI_TCB, Link); - - if (Tcb->InitiatorTaskTag =3D=3D InitiatorTaskTag) { - break; - } - } - - return Tcb; -} - - /** Create a data segment, pad it, and calculate the CRC if needed. =20 diff --git a/NetworkPkg/IpSecDxe/Ikev2/Payload.c b/NetworkPkg/IpSecDxe/Ikev= 2/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; } =20 -/** - Save some useful payloads after accepting the Packet. - - @param[in] SessionCommon Pointer to IKEV2_SESSION_COMMON related to th= e operation. - @param[in] IkePacket Pointer to received IkePacet. - @param[in] IkeType The type used to indicate it is in IkeSa or C= hildSa or Info - exchange. - -**/ -VOID -Ikev2OnPacketAccepted ( - IN IKEV2_SESSION_COMMON *SessionCommon, - IN IKE_PACKET *IkePacket, - IN UINT8 IkeType - ) -{ - return; -} =20 /** =20 diff --git a/NetworkPkg/IpSecDxe/Ikev2/Utility.c b/NetworkPkg/IpSecDxe/Ikev= 2/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; } =20 -/** - 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; -} =20 /** Free specified Seession Common. The session common would belong to a IKE= SA or @@ -659,33 +644,6 @@ Ikev2ChildSaSessionReg ( return ; } =20 -/** - 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 =3D IKEV2_CHILD_SA_SESSION_BY_IKE_SA (Entry); - - if (ChildSaSession->MessageId =3D=3D Mid) { - return ChildSaSession; - } - } - return NULL; -} =20 /** This function find the Child SA by the specified SPI. @@ -776,22 +734,6 @@ Ikev2ChildSaSessionRemove ( return NULL; } =20 -/** - 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. =20 @@ -1137,24 +1079,6 @@ Ikev2ChildSaAssociateSpdEntry ( } =20 =20 -/** - 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; -} =20 /** Validate the IKE header of received IKE packet. diff --git a/NetworkPkg/IpSecDxe/Ikev2/Utility.h b/NetworkPkg/IpSecDxe/Ikev= 2/Utility.h index 8383ff8dd6..79def7740d 100644 --- a/NetworkPkg/IpSecDxe/Ikev2/Utility.h +++ b/NetworkPkg/IpSecDxe/Ikev2/Utility.h @@ -104,19 +104,6 @@ Ikev2SaSessionRemove ( ); =20 =20 -/** - 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 pa= cket. =20 @@ -211,20 +198,6 @@ Ikev2ChildSaSessionLookupBySpi ( IN UINT32 Spi ); =20 -/** - 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 - ); =20 /** Insert a Child SA Session into the specified ChildSa list.. @@ -257,18 +230,6 @@ Ikev2ChildSaSessionRemove ( IN UINT8 ListType ); =20 -/** - 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 - ); =20 /** Free the memory located for the specified IKEV2_CHILD_SA_SESSION. @@ -328,19 +289,6 @@ Ikev2ChildExchangeRequestType( IN IKE_PACKET *IkePacket ); =20 -/** - 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 - ); - =20 /** Associate a SPD selector to the Child SA Session. @@ -882,21 +830,6 @@ Ikev2DecodePacket ( IN UINTN IkeType ); =20 -/** - Save some useful payloads after accepting the Packet. - - @param[in] SessionCommon Pointer to IKEV2_SESSION_COMMON related to th= e operation. - @param[in] IkePacket Pointer to received IkePacet. - @param[in] IkeType The type used to indicate it is in IkeSa or C= hildSa or Info - exchange. - -**/ -VOID -Ikev2OnPacketAccepted ( - IN IKEV2_SESSION_COMMON *SessionCommon, - IN IKE_PACKET *IkePacket, - IN UINT8 IkeType - ); =20 /** 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 ( =20 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 ); =20 -/** - 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 =20 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; } =20 - -/** - 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 =3D NET_LIST_USER_STRUCT (Entry, PXEBC_DHCP6_OPTION_NODE, Link); - RemoveEntryList (Entry); - FreePool (Node); - } -} - /** Retrieve the boot server address using the EFI_DNS6_PROTOCOL. =20 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; =20 =20 -/** - 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 - ); =20 =20 /** --=20 2.18.0.windows.1 _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel