From patchwork Mon Dec 26 23:23:36 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Bill Fischofer X-Patchwork-Id: 89041 Delivered-To: patch@linaro.org Received: by 10.140.20.101 with SMTP id 92csp4738436qgi; Mon, 26 Dec 2016 15:23:59 -0800 (PST) X-Received: by 10.233.221.130 with SMTP id r124mr28206119qkf.183.1482794639210; Mon, 26 Dec 2016 15:23:59 -0800 (PST) Return-Path: Received: from lists.linaro.org (lists.linaro.org. [54.225.227.206]) by mx.google.com with ESMTP id z6si12631358qtz.170.2016.12.26.15.23.58; Mon, 26 Dec 2016 15:23:59 -0800 (PST) Received-SPF: pass (google.com: domain of lng-odp-bounces@lists.linaro.org designates 54.225.227.206 as permitted sender) client-ip=54.225.227.206; Authentication-Results: mx.google.com; spf=pass (google.com: domain of lng-odp-bounces@lists.linaro.org designates 54.225.227.206 as permitted sender) smtp.mailfrom=lng-odp-bounces@lists.linaro.org; dmarc=pass (p=NONE dis=NONE) header.from=linaro.org Received: by lists.linaro.org (Postfix, from userid 109) id 692C760D69; Mon, 26 Dec 2016 23:23:58 +0000 (UTC) X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on ip-10-142-244-252 X-Spam-Level: X-Spam-Status: No, score=-2.6 required=5.0 tests=BAYES_00, RCVD_IN_DNSWL_LOW, RCVD_IN_MSPIKE_H3, RCVD_IN_MSPIKE_WL, URIBL_BLOCKED autolearn=disabled version=3.4.0 Received: from [127.0.0.1] (localhost [127.0.0.1]) by lists.linaro.org (Postfix) with ESMTP id 00D8A60D16; Mon, 26 Dec 2016 23:23:51 +0000 (UTC) X-Original-To: lng-odp@lists.linaro.org Delivered-To: lng-odp@lists.linaro.org Received: by lists.linaro.org (Postfix, from userid 109) id 8CBAC60CC6; Mon, 26 Dec 2016 23:23:46 +0000 (UTC) Received: from mail-oi0-f46.google.com (mail-oi0-f46.google.com [209.85.218.46]) by lists.linaro.org (Postfix) with ESMTPS id 4E32960CC6 for ; Mon, 26 Dec 2016 23:23:45 +0000 (UTC) Received: by mail-oi0-f46.google.com with SMTP id v84so333301155oie.3 for ; Mon, 26 Dec 2016 15:23:45 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=n0loKTb1QvzpL/5FrTTLIa/uZgd9Q5VGw91DZbb+Mzg=; b=pfKiU8c71MW5fZhYyFOv4NWtVwQtz6T+LWCBpZDExzWyRVOAS2DNnCgjc1HS19BTR8 0YcofFWP4lID/R5Y/awCyCfnmHrZH0Le4Lk2UxgYgAah114vkyRxXlrqPROqdkmtCCAY ZMO2YKeuogNWqZUx9cTj+KPxilRMPfFX+0qFWAt8VU7rZEBeWCKSWb61E1ophWLFifIe crzqKpOFzHHMSY/ha5/KUT7ty3IuBhKcwfXy8+UeJThl5OO5tbOQ53P7YhvICH8t/Ei9 CY8/cixOH1yTlYTgx478WNU5QT9z3VuaMGkBPdHx46sQOlXnPqEofPiks51k3l4lHLUR XAXA== X-Gm-Message-State: AIkVDXITn6hbOYfPfq7nvaQyr6dG4gZDM9Vpomwp2tVyCFJ22/SFc9QEdlax/F86jbtsCOERamQ= X-Received: by 10.157.63.38 with SMTP id m35mr15638550otc.156.1482794624615; Mon, 26 Dec 2016 15:23:44 -0800 (PST) Received: from localhost.localdomain (cpe-70-121-83-241.austin.res.rr.com. [70.121.83.241]) by smtp.gmail.com with ESMTPSA id 62sm21086728ots.41.2016.12.26.15.23.43 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Mon, 26 Dec 2016 15:23:43 -0800 (PST) From: Bill Fischofer To: lng-odp@lists.linaro.org Date: Mon, 26 Dec 2016 17:23:36 -0600 Message-Id: <1482794620-22778-1-git-send-email-bill.fischofer@linaro.org> X-Mailer: git-send-email 2.7.4 Subject: [lng-odp] [API-NEXT PATCHv3 1/5] api: packet: add support for packet references X-BeenThere: lng-odp@lists.linaro.org X-Mailman-Version: 2.1.16 Precedence: list List-Id: "The OpenDataPlane \(ODP\) List" List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: lng-odp-bounces@lists.linaro.org Sender: "lng-odp" Introduce three new APIs that support efficient sharing of portions of packets. odp_packet_ref_static() creates an alias for a base packet odp_packet_ref() creates a reference to a base packet odp_packet_ref_pkt() creates a reference to a base packet from a supplied header packet In addition, three other APIs simplify working with references odp_packet_is_ref() says whether a packet is a reference odp_packet_has_ref() says whether a packet has had a reference made to it odp_packet_unshared_len() gives the length of the prefix bytes that are unique to this reference. These are the only bytes of the packet that may be modified safely. Signed-off-by: Bill Fischofer --- include/odp/api/spec/packet.h | 190 +++++++++++++++++++++++++++++++++++++++++- 1 file changed, 189 insertions(+), 1 deletion(-) -- 2.7.4 diff --git a/include/odp/api/spec/packet.h b/include/odp/api/spec/packet.h index 4a86eba..0791496 100644 --- a/include/odp/api/spec/packet.h +++ b/include/odp/api/spec/packet.h @@ -256,6 +256,20 @@ uint32_t odp_packet_seg_len(odp_packet_t pkt); uint32_t odp_packet_len(odp_packet_t pkt); /** + * Packet unshared data len + * + * Returns the sum of data lengths over all unshared packet segments. These + * are the only bytes that should be modified by the caller. The rest of the + * packet should be treated as read only. odp_packet_unshared_len() will be + * equal to odp_packet_len() if odp_packet_has_ref() is 0. + * + * @param pkt Packet handle + * + * @return Packet unshared data length + */ +uint32_t odp_packet_unshared_len(odp_packet_t pkt); + +/** * Packet headroom length * * Returns the current packet level headroom length. @@ -795,7 +809,7 @@ uint32_t odp_packet_seg_data_len(odp_packet_t pkt, odp_packet_seg_t seg); * data pointers. Otherwise, all old pointers remain valid. * * The resulting packet is always allocated from the same pool as - * the destination packet. The source packet may have been allocate from + * the destination packet. The source packet may have been allocated from * any pool. * * On failure, both handles remain valid and packets are not modified. @@ -848,6 +862,180 @@ int odp_packet_split(odp_packet_t *pkt, uint32_t len, odp_packet_t *tail); /* * + * References + * ******************************************************** + * + */ + +/** + * Create a static reference to a packet + * + * A static reference is used to obtain an additional handle for referring to + * a packet so that the storage behind it is not freed until all references to + * the packet are freed. This can be used, for example, to support efficient + * retransmission processing. + * + * The intent of a static reference is that both the packet and the returned + * reference to it will be treated as read only after this call. Results are + * undefined if this restriction is not observed. + * + * Static references have restrictions but may have performance advantages on + * some platforms if the caller does not intend to modify the reference + * packet. If modification is needed (e.g., to prefix a unique header onto the + * packet) then odp_packet_ref() or odp_packet_ref_pkt() should be used. + * + * @param pkt Handle of the packet for which a static reference is + * to be created. + * + * @return Handle of the static reference packet + * @retval ODP_PACKET_INVALID Operation failed. pkt remains unchanged. + */ +odp_packet_t odp_packet_ref_static(odp_packet_t pkt); + +/** + * Create a reference to a packet + * + * Create a dynamic reference to a packet starting at a specified byte + * offset. This reference may be used as an argument to header manipulation + * APIs to prefix a unique header onto the shared base. The storage associated + * with the referenced packet is not freed until all references to it are + * freed, which permits easy multicasting or retransmission processing to be + * performed. Following a successful call, the shared portions of the + * referenced packet packet should be treated as read only. Results are + * undefined if this restriction is not observed. + * + * This operation logically prepends a zero-length header onto the referenced + * packet beginning at the specified offset. This header is always drawn from + * the same pool as the referenced packet. + * + * Because references are unique packets, any bytes pushed onto the head of a + * reference via odp_packet_push_head() or odp_packet_extend_head() are unique + * to this reference and are not seen by the referenced packet or by any other + * reference to the same packet. Similarly, any bytes pushed onto the + * head of the referenced packet are not visible to any reference created + * prior to the push operation. + * + * Note that the existence of references does not in any way change the + * requirement that a single packet handle may only be manipulated by one + * thread at a time. The odp_packet_unshared_len() API may be used to + * determine the number of bytes starting at offset zero that are unique to + * this packet handle. Only these bytes should be considered modifiable. All + * other bytes accessible from this handle should be treated as read only. + * + * The packet used as input to this routine may itself by a reference to + * some other packet. Implementations MAY restrict the ability to create + * such compound references. Attempts to exceed any implementation limits in + * this regard will result in this call failing and returning + * ODP_PACKET_INVALID. + * + * If the caller does not indend to push a header onto either the returned + * reference or the packet being referenced, or otherwise modify either + * packet, the odp_packet_ref_static() API may be used. This may be a more + * efficient means of obtaining another reference to a packet if both will be + * treated as read only. + * + * @param pkt Handle of the packet for which a reference is to be + * created. + * + * @param offset Byte offset in pkt at which the shared reference is to + * begin. This must be in the range 0..odp_packet_len(pkt)-1. + * + * @return Handle of the reference packet + * @retval ODP_PACKET_INVALID Operation failed. pkt remains unchanged. + * + + */ +odp_packet_t odp_packet_ref(odp_packet_t pkt, uint32_t offset); + +/** + * Create a reference to another packet from a header packet + * + * Create a dynamic reference to a packet starting at a specified byte offset + * by prepending a supplied header packet. The resulting reference consists of + * the unshared header followed by the shared referenced packet starting at + * the specified offset. This shared portion should be regarded as read + * only. The storage associated with the shared portion of the reference is + * not freed until all references to it are freed, which permits easy + * multicasting or retransmission processing to be performed. + * + * Because references are unique packets, any bytes pushed onto the head of a + * reference via odp_packet_push_head() or odp_packet_extend_head() are unique + * to this reference and are not seen by the referenced packet or by any other + * reference to the same packet. Similarly, any bytes pushed onto the + * head of the referenced packet are not visible to any reference created + * prior to the push operation. + * + * Note that the existence of references does not in any way change the + * requirement that a single packet handle may only be manipulated by one + * thread at a time. The odp_packet_unshared_len() API may be used to + * determine the number of bytes starting at offset zero that are unique to + * this packet handle. Only these bytes should be considered modifiable. All + * other bytes accessible from this handle should be treated as read only. + * + * Either packet input to this routine may itself be a reference, however + * individual implementations MAY impose limits on how deeply references may be + * nested and fail the call if those limits are exceeded. + * + * The packets used as input to this routine may reside in different pools, + * however individual implementations MAY require that both reside in the same + * pool and fail the call if this restriction is not observed. For best + * portability, both should reside in the same pool. + * + * odp_packet_pool() on the returned reference returns the pool id of the + * header packet. + * + * @param pkt Handle of the packet for which a reference is to be + * created. + * + * @param offset Byte offset in pkt at which the shared reference is to + * begin. Must be in the range 0..odp_packet_len(pkt)-1. + * + * @param hdr Handle of the header packet to be prefixed onto pkt to create + * the reference. If this is specified as ODP_PACKET_INVALID, + * this call is equivalent to odp_packet_ref(). The supplied hdr + * must be distinct from pkt and results are undefined if an + * attempt is made to create circular references. + * + * @return Handle of the reference packet. This may or may not be the + * same as the input hdr packet. The caller should only use this + * handle since the original hdr packet no longer has any + * independent existence. + * + * @retval ODP_PACKET_INVALID Operation failed. Both pkt and hdr are unchanged. + */ +odp_packet_t odp_packet_ref_pkt(odp_packet_t pkt, uint32_t offset, + odp_packet_t hdr); + +/** + * Test if a packet is a reference + * + * A packet is a reference if it was created by odp_packet_ref() or + * odp_packet_ref_pkt(). Note that a reference created from another + * reference is considered a compound reference. Calls on such packets will + * result in return values greater than 1. + * + * @param pkt Packet handle + * + * @retval 0 Packet is not a reference + * @retval >0 Packet is a reference consisting of N individual packets. + */ +int odp_packet_is_ref(odp_packet_t pkt); + +/** + * Test if a packet has a reference + * + * A packet has a reference if a reference was created to it by + * odp_packet_ref_static(), odp_packet_ref(), or odp_packet_ref_pkt(). + * + * @param pkt Packet handle + * + * @retval 0 Packet does not have any references + * @retval >0 Packet has N references to it + */ +int odp_packet_has_ref(odp_packet_t pkt); + +/* + * * Copy * ******************************************************** *