From patchwork Thu Oct 12 12:19:15 2017 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Adrien Mazarguil X-Patchwork-Id: 30252 X-Patchwork-Delegate: ferruh.yigit@amd.com Return-Path: X-Original-To: patchwork@dpdk.org Delivered-To: patchwork@dpdk.org Received: from [92.243.14.124] (localhost [127.0.0.1]) by dpdk.org (Postfix) with ESMTP id 0B58F1B29A; Thu, 12 Oct 2017 14:20:09 +0200 (CEST) Received: from mail-wm0-f45.google.com (mail-wm0-f45.google.com [74.125.82.45]) by dpdk.org (Postfix) with ESMTP id DB8441B292 for ; Thu, 12 Oct 2017 14:20:05 +0200 (CEST) Received: by mail-wm0-f45.google.com with SMTP id m72so12688618wmc.1 for ; Thu, 12 Oct 2017 05:20:05 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=6wind-com.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=vmih+FvMSMikNbW5jTO2tpA5pTHE7hUHgxVeKIjeF0Q=; b=lzkrVtCvaoXNbxvb0706dlB2KjRv8OZ07PRuJfmA7ZSHXMek2BmV3WhNxZJGNiA4vz vBQga2ScBcuMKV6ZF95oeo5vJHVpj6ivPQSRZvHateH7kcOk9Ghp+ZAPTKj2uedOzrIS /q6Jv0ZI/04+zoJ0EUtUig19+nF49Ky1KvtxKSzzCBsFnlBN2F/u5928OG+a+ha4jwCu DCPKqPUbAcpkgPy4iUCfwCcVkii++bV815EP9F0/Oank19YWkNRzyRAyUeYHa2szEaFG X6mPCbTOK/8Dqc5tX7e5oHeGYnjfQ/xrgRoopHNRwCDhOC21RTyQGIQvvQB6C6Q3somj tDVw== 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:in-reply-to :references; bh=vmih+FvMSMikNbW5jTO2tpA5pTHE7hUHgxVeKIjeF0Q=; b=ehTI0FZccET9cpR5ZsTqezI3JLNVJQmeVyYf9TU0nbgRd92ygg6y0lcPWcKhp4zGIG al1VYhJdQVXtpbppQSDkW2MFlS58w1/Rp02Ur3fgCBo/YiaRfmBeF7BQy49zq927ndB4 gdVG3/bWGB0BXspky/b6rKB/yhiq3qGtM/M2w5aQKOtaoEpkQ0pMNJU1+o0Tf7HTib+y cdoayC+K0AingpI2NlUoF3CZX19SywFxkQWIfnBPbVe7AfpxWJo1TDkCudRrLDYkb7mJ kvo+SoqHn88DeuN7CwYNEwNDOuzojXsQ7ROABXiOEYAJWgzsP/l+DZM3ks4C7KB6OROm 6EHg== X-Gm-Message-State: AMCzsaVdFAJ4TQhLGpX3lYTBXZ2ad9LwBE2qu6K5VAdZ2iAlSBRs7Olp WvHjn5weQlggbU6vIWtFeLSzBw== X-Google-Smtp-Source: AOwi7QCL567CSd/2VOzc2GDNiCEc9Zm62bkLwoSOpNWd9u6lFfZ1pHOzh2Q/dCGJ5FeLPspMCsIGFw== X-Received: by 10.28.93.84 with SMTP id r81mr1843580wmb.149.1507810805360; Thu, 12 Oct 2017 05:20:05 -0700 (PDT) Received: from 6wind.com (host.78.145.23.62.rev.coltfrance.com. [62.23.145.78]) by smtp.gmail.com with ESMTPSA id x73sm123640wme.34.2017.10.12.05.20.04 (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Thu, 12 Oct 2017 05:20:04 -0700 (PDT) From: Adrien Mazarguil To: Ferruh Yigit Cc: Nelio Laranjeiro , dev@dpdk.org Date: Thu, 12 Oct 2017 14:19:15 +0200 Message-Id: <5692b38996390dd0ea1eab3cd92d44cc7b1eea2f.1507809961.git.adrien.mazarguil@6wind.com> X-Mailer: git-send-email 2.1.4 In-Reply-To: References: Subject: [dpdk-dev] [PATCH v2 01/29] ethdev: expose flow API error helper X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" rte_flow_error_set() is a convenient helper to initialize error objects. Since there is no fundamental reason to prevent applications from using it, expose it through the public interface after modifying its return value from positive to negative. This is done for consistency with the rest of the public interface. Documentation is updated accordingly. Signed-off-by: Adrien Mazarguil Acked-by: Nelio Laranjeiro --- doc/guides/prog_guide/rte_flow.rst | 23 +++++++++++-- drivers/net/mlx4/mlx4_flow.c | 6 ++-- drivers/net/tap/tap_flow.c | 2 +- lib/librte_ether/rte_ethdev_version.map | 1 + lib/librte_ether/rte_flow.c | 49 +++++++++++++++++++--------- lib/librte_ether/rte_flow.h | 24 ++++++++++++++ lib/librte_ether/rte_flow_driver.h | 38 --------------------- 7 files changed, 83 insertions(+), 60 deletions(-) diff --git a/doc/guides/prog_guide/rte_flow.rst b/doc/guides/prog_guide/rte_flow.rst index 73f12ee..3113881 100644 --- a/doc/guides/prog_guide/rte_flow.rst +++ b/doc/guides/prog_guide/rte_flow.rst @@ -1695,6 +1695,25 @@ freed by the application, however its pointer can be considered valid only as long as its associated DPDK port remains configured. Closing the underlying device or unloading the PMD invalidates it. +Helpers +------- + +Error initializer +~~~~~~~~~~~~~~~~~ + +.. code-block:: c + + static inline int + rte_flow_error_set(struct rte_flow_error *error, + int code, + enum rte_flow_error_type type, + const void *cause, + const char *message); + +This function initializes ``error`` (if non-NULL) with the provided +parameters and sets ``rte_errno`` to ``code``. A negative error ``code`` is +then returned. + Caveats ------- @@ -1760,13 +1779,11 @@ the legacy filtering framework, which should eventually disappear. whatsoever). They only make sure these callbacks are non-NULL or return the ``ENOSYS`` (function not supported) error. -This interface additionally defines the following helper functions: +This interface additionally defines the following helper function: - ``rte_flow_ops_get()``: get generic flow operations structure from a port. -- ``rte_flow_error_set()``: initialize generic flow error structure. - More will be added over time. Device compatibility diff --git a/drivers/net/mlx4/mlx4_flow.c b/drivers/net/mlx4/mlx4_flow.c index 0885a91..018843b 100644 --- a/drivers/net/mlx4/mlx4_flow.c +++ b/drivers/net/mlx4/mlx4_flow.c @@ -955,9 +955,9 @@ mlx4_flow_isolate(struct rte_eth_dev *dev, mlx4_mac_addr_del(priv); } else if (mlx4_mac_addr_add(priv) < 0) { priv->isolated = 1; - return -rte_flow_error_set(error, rte_errno, - RTE_FLOW_ERROR_TYPE_UNSPECIFIED, - NULL, "cannot leave isolated mode"); + return rte_flow_error_set(error, rte_errno, + RTE_FLOW_ERROR_TYPE_UNSPECIFIED, + NULL, "cannot leave isolated mode"); } return 0; } diff --git a/drivers/net/tap/tap_flow.c b/drivers/net/tap/tap_flow.c index 28d793f..ffc0b85 100644 --- a/drivers/net/tap/tap_flow.c +++ b/drivers/net/tap/tap_flow.c @@ -1462,7 +1462,7 @@ tap_flow_isolate(struct rte_eth_dev *dev, return 0; error: pmd->flow_isolate = 0; - return -rte_flow_error_set( + return rte_flow_error_set( error, ENOTSUP, RTE_FLOW_ERROR_TYPE_UNSPECIFIED, NULL, "TC rule creation failed"); } diff --git a/lib/librte_ether/rte_ethdev_version.map b/lib/librte_ether/rte_ethdev_version.map index 92c9e29..e27f596 100644 --- a/lib/librte_ether/rte_ethdev_version.map +++ b/lib/librte_ether/rte_ethdev_version.map @@ -193,5 +193,6 @@ DPDK_17.11 { rte_eth_dev_pool_ops_supported; rte_eth_dev_reset; + rte_flow_error_set; } DPDK_17.08; diff --git a/lib/librte_ether/rte_flow.c b/lib/librte_ether/rte_flow.c index e276fb2..6659063 100644 --- a/lib/librte_ether/rte_flow.c +++ b/lib/librte_ether/rte_flow.c @@ -145,9 +145,9 @@ rte_flow_validate(uint16_t port_id, return -rte_errno; if (likely(!!ops->validate)) return ops->validate(dev, attr, pattern, actions, error); - return -rte_flow_error_set(error, ENOSYS, - RTE_FLOW_ERROR_TYPE_UNSPECIFIED, - NULL, rte_strerror(ENOSYS)); + return rte_flow_error_set(error, ENOSYS, + RTE_FLOW_ERROR_TYPE_UNSPECIFIED, + NULL, rte_strerror(ENOSYS)); } /* Create a flow rule on a given port. */ @@ -183,9 +183,9 @@ rte_flow_destroy(uint16_t port_id, return -rte_errno; if (likely(!!ops->destroy)) return ops->destroy(dev, flow, error); - return -rte_flow_error_set(error, ENOSYS, - RTE_FLOW_ERROR_TYPE_UNSPECIFIED, - NULL, rte_strerror(ENOSYS)); + return rte_flow_error_set(error, ENOSYS, + RTE_FLOW_ERROR_TYPE_UNSPECIFIED, + NULL, rte_strerror(ENOSYS)); } /* Destroy all flow rules associated with a port. */ @@ -200,9 +200,9 @@ rte_flow_flush(uint16_t port_id, return -rte_errno; if (likely(!!ops->flush)) return ops->flush(dev, error); - return -rte_flow_error_set(error, ENOSYS, - RTE_FLOW_ERROR_TYPE_UNSPECIFIED, - NULL, rte_strerror(ENOSYS)); + return rte_flow_error_set(error, ENOSYS, + RTE_FLOW_ERROR_TYPE_UNSPECIFIED, + NULL, rte_strerror(ENOSYS)); } /* Query an existing flow rule. */ @@ -220,9 +220,9 @@ rte_flow_query(uint16_t port_id, return -rte_errno; if (likely(!!ops->query)) return ops->query(dev, flow, action, data, error); - return -rte_flow_error_set(error, ENOSYS, - RTE_FLOW_ERROR_TYPE_UNSPECIFIED, - NULL, rte_strerror(ENOSYS)); + return rte_flow_error_set(error, ENOSYS, + RTE_FLOW_ERROR_TYPE_UNSPECIFIED, + NULL, rte_strerror(ENOSYS)); } /* Restrict ingress traffic to the defined flow rules. */ @@ -238,9 +238,28 @@ rte_flow_isolate(uint16_t port_id, return -rte_errno; if (likely(!!ops->isolate)) return ops->isolate(dev, set, error); - return -rte_flow_error_set(error, ENOSYS, - RTE_FLOW_ERROR_TYPE_UNSPECIFIED, - NULL, rte_strerror(ENOSYS)); + return rte_flow_error_set(error, ENOSYS, + RTE_FLOW_ERROR_TYPE_UNSPECIFIED, + NULL, rte_strerror(ENOSYS)); +} + +/* Initialize flow error structure. */ +int +rte_flow_error_set(struct rte_flow_error *error, + int code, + enum rte_flow_error_type type, + const void *cause, + const char *message) +{ + if (error) { + *error = (struct rte_flow_error){ + .type = type, + .cause = cause, + .message = message, + }; + } + rte_errno = code; + return -code; } /** Compute storage space needed by item specification. */ diff --git a/lib/librte_ether/rte_flow.h b/lib/librte_ether/rte_flow.h index d37b0ad..a0ffb71 100644 --- a/lib/librte_ether/rte_flow.h +++ b/lib/librte_ether/rte_flow.h @@ -1322,6 +1322,30 @@ int rte_flow_isolate(uint16_t port_id, int set, struct rte_flow_error *error); /** + * Initialize flow error structure. + * + * @param[out] error + * Pointer to flow error structure (may be NULL). + * @param code + * Related error code (rte_errno). + * @param type + * Cause field and error types. + * @param cause + * Object responsible for the error. + * @param message + * Human-readable error message. + * + * @return + * Negative error code (errno value) and rte_errno is set. + */ +int +rte_flow_error_set(struct rte_flow_error *error, + int code, + enum rte_flow_error_type type, + const void *cause, + const char *message); + +/** * Generic flow representation. * * This form is sufficient to describe an rte_flow independently from any diff --git a/lib/librte_ether/rte_flow_driver.h b/lib/librte_ether/rte_flow_driver.h index 8573cef..254d1cb 100644 --- a/lib/librte_ether/rte_flow_driver.h +++ b/lib/librte_ether/rte_flow_driver.h @@ -45,7 +45,6 @@ #include -#include #include "rte_ethdev.h" #include "rte_flow.h" @@ -128,43 +127,6 @@ struct rte_flow_ops { }; /** - * Initialize generic flow error structure. - * - * This function also sets rte_errno to a given value. - * - * @param[out] error - * Pointer to flow error structure (may be NULL). - * @param code - * Related error code (rte_errno). - * @param type - * Cause field and error types. - * @param cause - * Object responsible for the error. - * @param message - * Human-readable error message. - * - * @return - * Error code. - */ -static inline int -rte_flow_error_set(struct rte_flow_error *error, - int code, - enum rte_flow_error_type type, - const void *cause, - const char *message) -{ - if (error) { - *error = (struct rte_flow_error){ - .type = type, - .cause = cause, - .message = message, - }; - } - rte_errno = code; - return code; -} - -/** * Get generic flow operations structure from a port. * * @param port_id