Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/17303/?format=api
{ "id": 17303, "url": "http://patches.dpdk.org/api/patches/17303/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/1480436367-20749-34-git-send-email-arybchenko@solarflare.com/", "project": { "id": 1, "url": "http://patches.dpdk.org/api/projects/1/?format=api", "name": "DPDK", "link_name": "dpdk", "list_id": "dev.dpdk.org", "list_email": "dev@dpdk.org", "web_url": "http://core.dpdk.org", "scm_url": "git://dpdk.org/dpdk", "webscm_url": "http://git.dpdk.org/dpdk", "list_archive_url": "https://inbox.dpdk.org/dev", "list_archive_url_format": "https://inbox.dpdk.org/dev/{}", "commit_url_format": "" }, "msgid": "<1480436367-20749-34-git-send-email-arybchenko@solarflare.com>", "list_archive_url": "https://inbox.dpdk.org/dev/1480436367-20749-34-git-send-email-arybchenko@solarflare.com", "date": "2016-11-29T16:19:05", "name": "[dpdk-dev,v2,33/55] net/sfc: add device configuration checks", "commit_ref": null, "pull_url": null, "state": "accepted", "archived": true, "hash": "11acf274af3e9231c8405c2d79a55892e7843446", "submitter": { "id": 607, "url": "http://patches.dpdk.org/api/people/607/?format=api", "name": "Andrew Rybchenko", "email": "arybchenko@solarflare.com" }, "delegate": { "id": 319, "url": "http://patches.dpdk.org/api/users/319/?format=api", "username": "fyigit", "first_name": "Ferruh", "last_name": "Yigit", "email": "ferruh.yigit@amd.com" }, "mbox": "http://patches.dpdk.org/project/dpdk/patch/1480436367-20749-34-git-send-email-arybchenko@solarflare.com/mbox/", "series": [], "comments": "http://patches.dpdk.org/api/patches/17303/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/17303/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "X-Original-To": "patchwork@dpdk.org", "Delivered-To": "patchwork@dpdk.org", "Received": [ "from [92.243.14.124] (localhost [IPv6:::1])\n\tby dpdk.org (Postfix) with ESMTP id 5560EFA85;\n\tTue, 29 Nov 2016 17:22:23 +0100 (CET)", "from nbfkord-smmo01.seg.att.com (nbfkord-smmo01.seg.att.com\n\t[209.65.160.76]) by dpdk.org (Postfix) with ESMTP id 2BD574CC5\n\tfor <dev@dpdk.org>; Tue, 29 Nov 2016 17:20:52 +0100 (CET)", "from unknown [12.187.104.26] (EHLO nbfkord-smmo01.seg.att.com)\n\tby nbfkord-smmo01.seg.att.com(mxl_mta-7.2.4-7) with ESMTP id\n\t4eaad385.2b3ec6ab0940.83575.00-2495.173887.nbfkord-smmo01.seg.att.com\n\t(envelope-from <arybchenko@solarflare.com>); \n\tTue, 29 Nov 2016 16:20:52 +0000 (UTC)", "from unknown [12.187.104.26]\n\tby nbfkord-smmo01.seg.att.com(mxl_mta-7.2.4-7) with SMTP id\n\tfdaad385.0.83424.00-2380.173784.nbfkord-smmo01.seg.att.com\n\t(envelope-from <arybchenko@solarflare.com>); \n\tTue, 29 Nov 2016 16:20:48 +0000 (UTC)", "from ocex03.SolarFlarecom.com (10.20.40.36) by\n\tocex03.SolarFlarecom.com (10.20.40.36) with Microsoft SMTP Server\n\t(TLS) id 15.0.1044.25; Tue, 29 Nov 2016 08:20:25 -0800", "from opal.uk.solarflarecom.com (10.17.10.1) by\n\tocex03.SolarFlarecom.com (10.20.40.36) with Microsoft SMTP Server\n\t(TLS) id\n\t15.0.1044.25 via Frontend Transport; Tue, 29 Nov 2016 08:20:25 -0800", "from uklogin.uk.solarflarecom.com (uklogin.uk.solarflarecom.com\n\t[10.17.10.10])\n\tby opal.uk.solarflarecom.com (8.13.8/8.13.8) with ESMTP id\n\tuATGKO7H029991; Tue, 29 Nov 2016 16:20:24 GMT", "from uklogin.uk.solarflarecom.com (localhost.localdomain\n\t[127.0.0.1])\n\tby uklogin.uk.solarflarecom.com (8.13.8/8.13.8) with ESMTP id\n\tuATGKM1f021233; Tue, 29 Nov 2016 16:20:24 GMT" ], "X-MXL-Hash": [ "583daae4139b6e3b-1947a77589b5472eef26f2f234a497a9eef831a6", "583daae01bc81216-668a9e458897af71a821e9578c8a8df399ba54fb" ], "From": "Andrew Rybchenko <arybchenko@solarflare.com>", "To": "<dev@dpdk.org>", "CC": "<ferruh.yigit@intel.com>", "Date": "Tue, 29 Nov 2016 16:19:05 +0000", "Message-ID": "<1480436367-20749-34-git-send-email-arybchenko@solarflare.com>", "X-Mailer": "git-send-email 1.8.2.3", "In-Reply-To": "<1480436367-20749-1-git-send-email-arybchenko@solarflare.com>", "References": "<1479740470-6723-1-git-send-email-arybchenko@solarflare.com>\n\t<1480436367-20749-1-git-send-email-arybchenko@solarflare.com>", "MIME-Version": "1.0", "Content-Type": "text/plain", "X-AnalysisOut": [ "[v=2.1 cv=UoJlQrEB c=1 sm=1 tr=0 a=8BlWFWvVlq5taO8ncb8nKg==]", "[:17 a=L24OOQBejmoA:10 a=zRKbQ67AAAAA:8 a=XGGawuPn1W7cvlOXp]", "[WkA:9 a=PA03WX8tBzeizutn5_OT:22]" ], "X-Spam": "[F=0.3581424011; CM=0.500; S=0.358(2015072901)]", "X-MAIL-FROM": "<arybchenko@solarflare.com>", "X-SOURCE-IP": "[12.187.104.26]", "Subject": "[dpdk-dev] [PATCH v2 33/55] net/sfc: add device configuration checks", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.15", "Precedence": "list", "List-Id": "patches and discussions about DPDK <dev.dpdk.org>", "List-Unsubscribe": "<http://dpdk.org/ml/options/dev>,\n\t<mailto:dev-request@dpdk.org?subject=unsubscribe>", "List-Archive": "<http://dpdk.org/ml/archives/dev/>", "List-Post": "<mailto:dev@dpdk.org>", "List-Help": "<mailto:dev-request@dpdk.org?subject=help>", "List-Subscribe": "<http://dpdk.org/ml/listinfo/dev>,\n\t<mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org", "Sender": "\"dev\" <dev-bounces@dpdk.org>" }, "content": "Manual link speed/duplex configuration is not supported yet.\nLoopback is not supported yet.\nFlow Director is not supported.\nLink status change notification using interrupt is not supported yet.\nReceive data notification using interrupts is not supported yet.\n\nReviewed-by: Andy Moreton <amoreton@solarflare.com>\nSigned-off-by: Andrew Rybchenko <arybchenko@solarflare.com>\n---\n doc/guides/nics/sfc_efx.rst | 14 ++++++++++++\n drivers/net/sfc/sfc.c | 55 +++++++++++++++++++++++++++++++++++++++++++++\n 2 files changed, 69 insertions(+)", "diff": "diff --git a/doc/guides/nics/sfc_efx.rst b/doc/guides/nics/sfc_efx.rst\nindex 2eebcd7..31e86a7 100644\n--- a/doc/guides/nics/sfc_efx.rst\n+++ b/doc/guides/nics/sfc_efx.rst\n@@ -37,6 +37,20 @@ More information can be found at `Solarflare Communications website\n <http://solarflare.com>`_.\n \n \n+Non-supported Features\n+----------------------\n+\n+The features not yet supported include:\n+\n+- Link status change interrupt\n+\n+- Receive queue interupts\n+\n+- Priority-based flow control\n+\n+- Loopback\n+\n+\n Supported NICs\n --------------\n \ndiff --git a/drivers/net/sfc/sfc.c b/drivers/net/sfc/sfc.c\nindex cbb14d7..befe68d 100644\n--- a/drivers/net/sfc/sfc.c\n+++ b/drivers/net/sfc/sfc.c\n@@ -82,9 +82,55 @@ sfc_dma_free(const struct sfc_adapter *sa, efsys_mem_t *esmp)\n \tmemset(esmp, 0, sizeof(*esmp));\n }\n \n+/*\n+ * Check requested device level configuration.\n+ * Receive and transmit configuration is checked in corresponding\n+ * modules.\n+ */\n+static int\n+sfc_check_conf(struct sfc_adapter *sa)\n+{\n+\tconst struct rte_eth_conf *conf = &sa->eth_dev->data->dev_conf;\n+\tint rc = 0;\n+\n+\tif (conf->link_speeds != ETH_LINK_SPEED_AUTONEG) {\n+\t\tsfc_err(sa, \"Manual link speed/duplex choice not supported\");\n+\t\trc = EINVAL;\n+\t}\n+\n+\tif (conf->lpbk_mode != 0) {\n+\t\tsfc_err(sa, \"Loopback not supported\");\n+\t\trc = EINVAL;\n+\t}\n+\n+\tif (conf->dcb_capability_en != 0) {\n+\t\tsfc_err(sa, \"Priority-based flow control not supported\");\n+\t\trc = EINVAL;\n+\t}\n+\n+\tif (conf->fdir_conf.mode != RTE_FDIR_MODE_NONE) {\n+\t\tsfc_err(sa, \"Flow Director not supported\");\n+\t\trc = EINVAL;\n+\t}\n+\n+\tif (conf->intr_conf.lsc != 0) {\n+\t\tsfc_err(sa, \"Link status change interrupt not supported\");\n+\t\trc = EINVAL;\n+\t}\n+\n+\tif (conf->intr_conf.rxq != 0) {\n+\t\tsfc_err(sa, \"Receive queue interrupt not supported\");\n+\t\trc = EINVAL;\n+\t}\n+\n+\treturn rc;\n+}\n+\n int\n sfc_configure(struct sfc_adapter *sa)\n {\n+\tint rc;\n+\n \tsfc_log_init(sa, \"entry\");\n \n \tSFC_ASSERT(sfc_adapter_is_locked(sa));\n@@ -92,9 +138,18 @@ sfc_configure(struct sfc_adapter *sa)\n \tSFC_ASSERT(sa->state == SFC_ADAPTER_INITIALIZED);\n \tsa->state = SFC_ADAPTER_CONFIGURING;\n \n+\trc = sfc_check_conf(sa);\n+\tif (rc != 0)\n+\t\tgoto fail_check_conf;\n+\n \tsa->state = SFC_ADAPTER_CONFIGURED;\n \tsfc_log_init(sa, \"done\");\n \treturn 0;\n+\n+fail_check_conf:\n+\tsa->state = SFC_ADAPTER_INITIALIZED;\n+\tsfc_log_init(sa, \"failed %d\", rc);\n+\treturn rc;\n }\n \n void\n", "prefixes": [ "dpdk-dev", "v2", "33/55" ] }