Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/12087/?format=api
{ "id": 12087, "url": "http://patches.dpdk.org/api/patches/12087/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/1460669598-3122-1-git-send-email-thomas.monjalon@6wind.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": "<1460669598-3122-1-git-send-email-thomas.monjalon@6wind.com>", "list_archive_url": "https://inbox.dpdk.org/dev/1460669598-3122-1-git-send-email-thomas.monjalon@6wind.com", "date": "2016-04-14T21:33:18", "name": "[dpdk-dev] pci: remove deprecated specific config", "commit_ref": null, "pull_url": null, "state": "accepted", "archived": true, "hash": "b6f2274565c1b5bd0f67def58c94aa98fdf2428e", "submitter": { "id": 1, "url": "http://patches.dpdk.org/api/people/1/?format=api", "name": "Thomas Monjalon", "email": "thomas.monjalon@6wind.com" }, "delegate": null, "mbox": "http://patches.dpdk.org/project/dpdk/patch/1460669598-3122-1-git-send-email-thomas.monjalon@6wind.com/mbox/", "series": [], "comments": "http://patches.dpdk.org/api/patches/12087/comments/", "check": "pending", "checks": "http://patches.dpdk.org/api/patches/12087/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 7D7132C6E;\n\tThu, 14 Apr 2016 23:33:39 +0200 (CEST)", "from mail-wm0-f45.google.com (mail-wm0-f45.google.com\n\t[74.125.82.45]) by dpdk.org (Postfix) with ESMTP id 252C92BD4\n\tfor <dev@dpdk.org>; Thu, 14 Apr 2016 23:33:38 +0200 (CEST)", "by mail-wm0-f45.google.com with SMTP id v188so6282351wme.1\n\tfor <dev@dpdk.org>; Thu, 14 Apr 2016 14:33:38 -0700 (PDT)", "from XPS13.localdomain (245.111.75.86.rev.sfr.net. [86.75.111.245])\n\tby smtp.gmail.com with ESMTPSA id\n\ts10sm42980588wjp.3.2016.04.14.14.33.36\n\t(version=TLSv1/SSLv3 cipher=OTHER);\n\tThu, 14 Apr 2016 14:33:37 -0700 (PDT)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=6wind-com.20150623.gappssmtp.com; s=20150623;\n\th=from:to:cc:subject:date:message-id;\n\tbh=2rOOpBMn3ZcvjmlAEbi7IIcAIuOPct6PSh6FAm1KmUs=;\n\tb=BaDDbap5N93H3YTfxznFYJtwsbr5ymFYkkApXDBjAVEgHyVpG2RqWn7p7a70HhjHxK\n\tBWXM890CxMlcsHFynzsdj7WXGE2NCUrVrN2iJVfp03yi7Oi4nUvRFkPEFUkCSKMkoXEW\n\tQnM3pznzmSrquf7RpslPyncLv46QYI+WfeizTfKiu6/JkY9B+GlPKr0d9Og1z2Xm6veX\n\tIaWdSMbVxIW968YNytKuaHHtLRCwfDqX3iu+oGscT4eIKieinMxJ/oltUXHbYO/p67HT\n\t0ZoyOAwFb+0T0/hR9KQ3uV0rQRgTcVPWB+xBHyO7U44eyhCF/DZmH75Kf0qq7x7fQErG\n\t1mcw==", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20130820;\n\th=x-gm-message-state:from:to:cc:subject:date:message-id;\n\tbh=2rOOpBMn3ZcvjmlAEbi7IIcAIuOPct6PSh6FAm1KmUs=;\n\tb=aGWKwGA0gyZrT2ND2V4yXakSyMeMWf0T0jhcfkK5eJYUib0nPtKbmXVssELmm4l7al\n\tfAuNEJhEqKlWG4F1s+XgPJvEtivov9LVbm/zTfmAt/7de/I7n1+0PvXEVaAdOtipada8\n\tDBLj06j5HKs9MNGkvPwxjaj3429Q8btNG4+dkL8qqZTnse6watzH5Co2hTSdpPkGM691\n\tUmNQk0ig54LWwxetXvetMie8OGB1ShjLx9/KyGfvGc2lUM8juxxeElZep95E1Q2IKOkW\n\t0YmjqUmFBWTpwn5V0YWDh/mwU1QNso+o3/KZA5N3aqzWOj67M446pAtP7M//VmEeaFvn\n\ttkaA==", "X-Gm-Message-State": "AOPr4FUnA3JK4LTxTOXB/g6zsdQxp2BGmL+xIjYAGBHtM5KYflUkpo/7sX6bGKwg9UsyKMb8", "X-Received": "by 10.28.136.19 with SMTP id k19mr639828wmd.11.1460669617952;\n\tThu, 14 Apr 2016 14:33:37 -0700 (PDT)", "From": "Thomas Monjalon <thomas.monjalon@6wind.com>", "To": "helin.zhang@intel.com", "Cc": "dev@dpdk.org", "Date": "Thu, 14 Apr 2016 23:33:18 +0200", "Message-Id": "<1460669598-3122-1-git-send-email-thomas.monjalon@6wind.com>", "X-Mailer": "git-send-email 2.7.0", "Subject": "[dpdk-dev] [PATCH] pci: remove deprecated specific config", "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": "The driver i40e was using a specific PCI config before the release 16.04.\nFrom 16.04, it is always enabled in i40e (commit 56465cfaf).\nThe API has been deprecated in the commit 68f77593823cab.\nThe igb_uio implementation has been deprecated in commit b7cf8e155.\nThe config helper - through igb_uio sysfs entries - is now removed.\n\nSigned-off-by: Thomas Monjalon <thomas.monjalon@6wind.com>\n---\n config/common_base | 8 -----\n doc/guides/linux_gsg/enable_func.rst | 15 ---------\n doc/guides/rel_notes/deprecation.rst | 7 -----\n lib/librte_eal/common/include/rte_pci.h | 14 ---------\n lib/librte_eal/linuxapp/eal/eal_pci.c | 12 -------\n lib/librte_eal/linuxapp/igb_uio/igb_uio.c | 52 -------------------------------\n 6 files changed, 108 deletions(-)", "diff": "diff --git a/config/common_base b/config/common_base\nindex 0124e86..1a54e4c 100644\n--- a/config/common_base\n+++ b/config/common_base\n@@ -101,14 +101,6 @@ CONFIG_RTE_MALLOC_DEBUG=n\n CONFIG_RTE_EAL_PMD_PATH=\"\"\n \n #\n-# Special configurations in PCI Config Space for high performance\n-# They are all deprecated, and will be removed later.\n-#\n-CONFIG_RTE_PCI_CONFIG=n\n-CONFIG_RTE_PCI_EXTENDED_TAG=\"\"\n-CONFIG_RTE_PCI_MAX_READ_REQUEST_SIZE=0\n-\n-#\n # Compile Environment Abstraction Layer to support Vmware TSC map\n #\n CONFIG_RTE_LIBRTE_EAL_VMWARE_TSC_MAP_SUPPORT=y\ndiff --git a/doc/guides/linux_gsg/enable_func.rst b/doc/guides/linux_gsg/enable_func.rst\nindex 076770f..ec0e04d 100644\n--- a/doc/guides/linux_gsg/enable_func.rst\n+++ b/doc/guides/linux_gsg/enable_func.rst\n@@ -186,21 +186,6 @@ Check with the local Intel's Network Division application engineers for firmware\n The base driver to support firmware version of FVL3E will be integrated in the next\n DPDK release, so currently the validated firmware version is 4.2.6.\n \n-Enabling Extended Tag\n-~~~~~~~~~~~~~~~~~~~~~\n-\n-PCI configuration of ``extended_tag`` has big impact on small packet size\n-performance of 40G ports. Enabling ``extended_tag`` can help 40G port to\n-achieve the best performance, especially for small packet size.\n-\n-* Disabling/enabling ``extended_tag`` can be done in some BIOS implementations.\n-\n-* If BIOS does not enable it, and does not support changing it, tools\n- (e.g. ``setpci`` on Linux) can be used to enable or disable ``extended_tag``.\n-\n-* From release 16.04, ``extended_tag`` is enabled by default during port\n- initialization, users don't need to care about that anymore.\n-\n Use 16 Bytes RX Descriptor Size\n ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~\n \ndiff --git a/doc/guides/rel_notes/deprecation.rst b/doc/guides/rel_notes/deprecation.rst\nindex a3fdbb1..c78cde7 100644\n--- a/doc/guides/rel_notes/deprecation.rst\n+++ b/doc/guides/rel_notes/deprecation.rst\n@@ -20,13 +20,6 @@ Deprecation Notices\n do not need to care about the kind of devices that are being used, making it\n easier to add new buses later.\n \n-* The EAL function pci_config_space_set is deprecated in release 16.04\n- and will be removed from 16.07.\n- Macros CONFIG_RTE_PCI_CONFIG, CONFIG_RTE_PCI_EXTENDED_TAG and\n- CONFIG_RTE_PCI_MAX_READ_REQUEST_SIZE will be removed.\n- The /sys entries extended_tag and max_read_request_size created by igb_uio\n- will be removed.\n-\n * ABI changes are planned for struct rte_pci_id, i.e., add new field ``class``.\n This new added ``class`` field can be used to probe pci device by class\n related info. This change should impact size of struct rte_pci_id and struct\ndiff --git a/lib/librte_eal/common/include/rte_pci.h b/lib/librte_eal/common/include/rte_pci.h\nindex e692094..9f2301d 100644\n--- a/lib/librte_eal/common/include/rte_pci.h\n+++ b/lib/librte_eal/common/include/rte_pci.h\n@@ -577,20 +577,6 @@ void rte_eal_pci_ioport_read(struct rte_pci_ioport *p,\n void rte_eal_pci_ioport_write(struct rte_pci_ioport *p,\n \t\t\t const void *data, size_t len, off_t offset);\n \n-#ifdef RTE_PCI_CONFIG\n-#include <rte_common.h>\n-/**\n- * Set special config space registers for performance purpose.\n- * It is deprecated, as all configurations have been moved into\n- * each PMDs respectively.\n- *\n- * @param dev\n- * A pointer to a rte_pci_device structure describing the device\n- * to use\n- */\n-void pci_config_space_set(struct rte_pci_device *dev) __rte_deprecated;\n-#endif /* RTE_PCI_CONFIG */\n-\n #ifdef __cplusplus\n }\n #endif\ndiff --git a/lib/librte_eal/linuxapp/eal/eal_pci.c b/lib/librte_eal/linuxapp/eal/eal_pci.c\nindex dbf12a8..bdc08a0 100644\n--- a/lib/librte_eal/linuxapp/eal/eal_pci.c\n+++ b/lib/librte_eal/linuxapp/eal/eal_pci.c\n@@ -481,18 +481,6 @@ error:\n \treturn -1;\n }\n \n-#ifdef RTE_PCI_CONFIG\n-/*\n- * It is deprecated, all its configurations have been moved into\n- * each PMD respectively.\n- */\n-void\n-pci_config_space_set(__rte_unused struct rte_pci_device *dev)\n-{\n-\tRTE_LOG(DEBUG, EAL, \"Nothing here, as it is deprecated\\n\");\n-}\n-#endif\n-\n /* Read PCI config space. */\n int rte_eal_pci_read_config(const struct rte_pci_device *device,\n \t\t\t void *buf, size_t len, off_t offset)\ndiff --git a/lib/librte_eal/linuxapp/igb_uio/igb_uio.c b/lib/librte_eal/linuxapp/igb_uio/igb_uio.c\nindex 72b2692..45a5720 100644\n--- a/lib/librte_eal/linuxapp/igb_uio/igb_uio.c\n+++ b/lib/librte_eal/linuxapp/igb_uio/igb_uio.c\n@@ -81,62 +81,10 @@ store_max_vfs(struct device *dev, struct device_attribute *attr,\n \treturn err ? err : count;\n }\n \n-#ifdef RTE_PCI_CONFIG\n-static ssize_t\n-show_extended_tag(struct device *dev, struct device_attribute *attr, char *buf)\n-{\n-\tdev_info(dev, \"Deprecated\\n\");\n-\n-\treturn 0;\n-}\n-\n-static ssize_t\n-store_extended_tag(struct device *dev,\n-\t\t struct device_attribute *attr,\n-\t\t const char *buf,\n-\t\t size_t count)\n-{\n-\tdev_info(dev, \"Deprecated\\n\");\n-\n-\treturn 0;\n-}\n-\n-static ssize_t\n-show_max_read_request_size(struct device *dev,\n-\t\t\t struct device_attribute *attr,\n-\t\t\t char *buf)\n-{\n-\tdev_info(dev, \"Deprecated\\n\");\n-\n-\treturn 0;\n-}\n-\n-static ssize_t\n-store_max_read_request_size(struct device *dev,\n-\t\t\t struct device_attribute *attr,\n-\t\t\t const char *buf,\n-\t\t\t size_t count)\n-{\n-\tdev_info(dev, \"Deprecated\\n\");\n-\n-\treturn 0;\n-}\n-#endif\n-\n static DEVICE_ATTR(max_vfs, S_IRUGO | S_IWUSR, show_max_vfs, store_max_vfs);\n-#ifdef RTE_PCI_CONFIG\n-static DEVICE_ATTR(extended_tag, S_IRUGO | S_IWUSR, show_extended_tag,\n-\tstore_extended_tag);\n-static DEVICE_ATTR(max_read_request_size, S_IRUGO | S_IWUSR,\n-\tshow_max_read_request_size, store_max_read_request_size);\n-#endif\n \n static struct attribute *dev_attrs[] = {\n \t&dev_attr_max_vfs.attr,\n-#ifdef RTE_PCI_CONFIG\n-\t&dev_attr_extended_tag.attr,\n-\t&dev_attr_max_read_request_size.attr,\n-#endif\n \tNULL,\n };\n \n", "prefixes": [ "dpdk-dev" ] }