Cover Letter Detail
Show a cover letter.
GET /api/covers/892984/?format=api
{ "id": 892984, "url": "http://patchwork.ozlabs.org/api/covers/892984/?format=api", "web_url": "http://patchwork.ozlabs.org/project/intel-wired-lan/cover/20180329210751.11531-1-vinicius.gomes@intel.com/", "project": { "id": 46, "url": "http://patchwork.ozlabs.org/api/projects/46/?format=api", "name": "Intel Wired Ethernet development", "link_name": "intel-wired-lan", "list_id": "intel-wired-lan.osuosl.org", "list_email": "intel-wired-lan@osuosl.org", "web_url": "", "scm_url": "", "webscm_url": "", "list_archive_url": "", "list_archive_url_format": "", "commit_url_format": "" }, "msgid": "<20180329210751.11531-1-vinicius.gomes@intel.com>", "list_archive_url": null, "date": "2018-03-29T21:07:41", "name": "[next-queue,v6,00/10] igb: offloading of receive filters", "submitter": { "id": 72272, "url": "http://patchwork.ozlabs.org/api/people/72272/?format=api", "name": "Vinicius Costa Gomes", "email": "vinicius.gomes@intel.com" }, "mbox": "http://patchwork.ozlabs.org/project/intel-wired-lan/cover/20180329210751.11531-1-vinicius.gomes@intel.com/mbox/", "series": [ { "id": 36560, "url": "http://patchwork.ozlabs.org/api/series/36560/?format=api", "web_url": "http://patchwork.ozlabs.org/project/intel-wired-lan/list/?series=36560", "date": "2018-03-29T21:07:48", "name": "igb: offloading of receive filters", "version": 6, "mbox": "http://patchwork.ozlabs.org/series/36560/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/covers/892984/comments/", "headers": { "Return-Path": "<intel-wired-lan-bounces@osuosl.org>", "X-Original-To": [ "incoming@patchwork.ozlabs.org", "intel-wired-lan@lists.osuosl.org" ], "Delivered-To": [ "patchwork-incoming@bilbo.ozlabs.org", "intel-wired-lan@lists.osuosl.org" ], "Authentication-Results": [ "ozlabs.org;\n\tspf=pass (mailfrom) smtp.mailfrom=osuosl.org\n\t(client-ip=140.211.166.136; helo=silver.osuosl.org;\n\tenvelope-from=intel-wired-lan-bounces@osuosl.org;\n\treceiver=<UNKNOWN>)", "ozlabs.org;\n\tdmarc=none (p=none dis=none) header.from=intel.com" ], "Received": [ "from silver.osuosl.org (smtp3.osuosl.org [140.211.166.136])\n\t(using TLSv1.2 with cipher AECDH-AES256-SHA (256/256 bits))\n\t(No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 40By6P0PxFz9s19\n\tfor <incoming@patchwork.ozlabs.org>;\n\tFri, 30 Mar 2018 08:09:00 +1100 (AEDT)", "from localhost (localhost [127.0.0.1])\n\tby silver.osuosl.org (Postfix) with ESMTP id C00D33071C;\n\tThu, 29 Mar 2018 21:08:57 +0000 (UTC)", "from silver.osuosl.org ([127.0.0.1])\n\tby localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024)\n\twith ESMTP id 4e4fats5hWY1; Thu, 29 Mar 2018 21:08:55 +0000 (UTC)", "from ash.osuosl.org (ash.osuosl.org [140.211.166.34])\n\tby silver.osuosl.org (Postfix) with ESMTP id 0C44022767;\n\tThu, 29 Mar 2018 21:08:55 +0000 (UTC)", "from fraxinus.osuosl.org (smtp4.osuosl.org [140.211.166.137])\n\tby ash.osuosl.org (Postfix) with ESMTP id 247051C0BD3\n\tfor <intel-wired-lan@lists.osuosl.org>;\n\tThu, 29 Mar 2018 21:08:54 +0000 (UTC)", "from localhost (localhost [127.0.0.1])\n\tby fraxinus.osuosl.org (Postfix) with ESMTP id 1FAA08868A\n\tfor <intel-wired-lan@lists.osuosl.org>;\n\tThu, 29 Mar 2018 21:08:54 +0000 (UTC)", "from fraxinus.osuosl.org ([127.0.0.1])\n\tby localhost (.osuosl.org [127.0.0.1]) (amavisd-new, port 10024)\n\twith ESMTP id wVP7UKXAR3E4 for <intel-wired-lan@lists.osuosl.org>;\n\tThu, 29 Mar 2018 21:08:53 +0000 (UTC)", "from mga09.intel.com (mga09.intel.com [134.134.136.24])\n\tby fraxinus.osuosl.org (Postfix) with ESMTPS id 586EB88AFD\n\tfor <intel-wired-lan@lists.osuosl.org>;\n\tThu, 29 Mar 2018 21:08:53 +0000 (UTC)", "from fmsmga005.fm.intel.com ([10.253.24.32])\n\tby orsmga102.jf.intel.com with ESMTP/TLS/DHE-RSA-AES256-GCM-SHA384;\n\t29 Mar 2018 14:08:52 -0700", "from ellie.jf.intel.com (HELO localhost.localdomain)\n\t([10.54.70.23])\n\tby fmsmga005.fm.intel.com with ESMTP; 29 Mar 2018 14:08:52 -0700" ], "X-Virus-Scanned": [ "amavisd-new at osuosl.org", "amavisd-new at osuosl.org" ], "X-Greylist": "domain auto-whitelisted by SQLgrey-1.7.6", "X-Amp-Result": "SKIPPED(no attachment in message)", "X-Amp-File-Uploaded": "False", "X-ExtLoop1": "1", "X-IronPort-AV": "E=Sophos;i=\"5.48,378,1517904000\"; d=\"scan'208\";a=\"216093108\"", "From": "Vinicius Costa Gomes <vinicius.gomes@intel.com>", "To": "intel-wired-lan@lists.osuosl.org", "Date": "Thu, 29 Mar 2018 14:07:41 -0700", "Message-Id": "<20180329210751.11531-1-vinicius.gomes@intel.com>", "X-Mailer": "git-send-email 2.16.3", "Subject": "[Intel-wired-lan] [next-queue PATCH v6 00/10] igb: offloading of\n\treceive filters", "X-BeenThere": "intel-wired-lan@osuosl.org", "X-Mailman-Version": "2.1.24", "Precedence": "list", "List-Id": "Intel Wired Ethernet Linux Kernel Driver Development\n\t<intel-wired-lan.osuosl.org>", "List-Unsubscribe": "<https://lists.osuosl.org/mailman/options/intel-wired-lan>, \n\t<mailto:intel-wired-lan-request@osuosl.org?subject=unsubscribe>", "List-Archive": "<http://lists.osuosl.org/pipermail/intel-wired-lan/>", "List-Post": "<mailto:intel-wired-lan@osuosl.org>", "List-Help": "<mailto:intel-wired-lan-request@osuosl.org?subject=help>", "List-Subscribe": "<https://lists.osuosl.org/mailman/listinfo/intel-wired-lan>, \n\t<mailto:intel-wired-lan-request@osuosl.org?subject=subscribe>", "Cc": "netdev@vger.kernel.org, jesus.sanchez-palencia@intel.com", "MIME-Version": "1.0", "Content-Type": "text/plain; charset=\"us-ascii\"", "Content-Transfer-Encoding": "7bit", "Errors-To": "intel-wired-lan-bounces@osuosl.org", "Sender": "\"Intel-wired-lan\" <intel-wired-lan-bounces@osuosl.org>" }, "content": "Hi,\n\nChanges from v5:\n - Filters can now be added for local MAC addresses, when removed,\n they return to their initial configuration (thanks for the testing\n Aaron);\n\nChanges from v4:\n - Added a new bit to the MAC address filters internal\n representation to mean that some filters are steering filters (i.e.\n they direct traffic to queues);\n - And, this is only supported in i210;\n\nChanges from v3:\n - Addressed review comments from Aaron F. Brown and\n Jakub Kicinski;\n\nChanges from v2:\n - Addressed review comments from Jakub Kicinski, mostly about coding\n style adjustments and more consistent error reporting;\n\nChanges from v1:\n - Addressed review comments from Alexander Duyck and Florian\n Fainelli;\n - Adding and removing cls_flower filters are now proposed in the same\n patch;\n - cls_flower filters are kept in a separated list from \"ethtool\"\n filters (so that section of the original cover letter is no longer\n valid);\n - The patch adding support for ethtool filters is now independent from\n the rest of the series;\n\nOriginal cover letter:\n\nThis series enables some ethtool and tc-flower filters to be offloaded\nto igb-based network controllers. This is useful when the system\nconfigurator want to steer kinds of traffic to a specific hardware\nqueue.\n\nThe first two commits are bug fixes.\n\nThe basis of this series is to export the internal API used to\nconfigure address filters, so they can be used by ethtool, and\nextending the functionality so an source address can be handled.\n\nThen, we enable the tc-flower offloading implementation to re-use the\nsame infrastructure as ethtool, and storing them in the per-adapter\n\"nfc\" (Network Filter Config?) list. But for consistency, for\ndestructive access they are separated, i.e. an filter added by\ntc-flower can only be removed by tc-flower, but ethtool can read them\nall.\n\nOnly support for VLAN Prio, Source and Destination MAC Address, and\nEthertype is enabled for now.\n\nOpen question:\n - igb is initialized with the number of traffic classes as 1, if we\n want to use multiple traffic classes we need to increase this value,\n the only way I could find is to use mqprio (for example). Should igb\n be initialized with, say, the number of queues as its \"num_tc\"?\n\n\nVinicius Costa Gomes (10):\n igb: Fix not adding filter elements to the list\n igb: Fix queue selection on MAC filters on i210\n igb: Enable the hardware traffic class feature bit for igb models\n igb: Add support for MAC address filters specifying source addresses\n igb: Add support for enabling queue steering in filters\n igb: Allow filters to be added for the local MAC address\n igb: Enable nfc filters to specify MAC addresses\n igb: Add MAC address support for ethtool nftuple filters\n igb: Add the skeletons for tc-flower offloading\n igb: Add support for adding offloaded clsflower filters\n\n drivers/net/ethernet/intel/igb/e1000_defines.h | 2 +\n drivers/net/ethernet/intel/igb/igb.h | 13 +\n drivers/net/ethernet/intel/igb/igb_ethtool.c | 65 ++++-\n drivers/net/ethernet/intel/igb/igb_main.c | 370 ++++++++++++++++++++++++-\n 4 files changed, 433 insertions(+), 17 deletions(-)\n\n--\n2.16.3" }