Cover Letter Detail
Show a cover letter.
GET /api/covers/1257844/?format=api
{ "id": 1257844, "url": "http://patchwork.ozlabs.org/api/covers/1257844/?format=api", "web_url": "http://patchwork.ozlabs.org/project/intel-wired-lan/cover/20200318230102.36952-1-andre.guedes@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": "<20200318230102.36952-1-andre.guedes@intel.com>", "list_archive_url": null, "date": "2020-03-18T23:00:50", "name": "[00/12] igc: Refactor MAC address filtering code", "submitter": { "id": 72323, "url": "http://patchwork.ozlabs.org/api/people/72323/?format=api", "name": "Andre Guedes", "email": "andre.guedes@intel.com" }, "mbox": "http://patchwork.ozlabs.org/project/intel-wired-lan/cover/20200318230102.36952-1-andre.guedes@intel.com/mbox/", "series": [ { "id": 165243, "url": "http://patchwork.ozlabs.org/api/series/165243/?format=api", "web_url": "http://patchwork.ozlabs.org/project/intel-wired-lan/list/?series=165243", "date": "2020-03-18T23:00:52", "name": "igc: Refactor MAC address filtering code", "version": 1, "mbox": "http://patchwork.ozlabs.org/series/165243/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/covers/1257844/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; spf=pass (sender SPF authorized)\n\tsmtp.mailfrom=osuosl.org (client-ip=140.211.166.136;\n\thelo=silver.osuosl.org;\n\tenvelope-from=intel-wired-lan-bounces@osuosl.org;\n\treceiver=<UNKNOWN>)", "ozlabs.org;\n\tdmarc=fail (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 ECDHE-RSA-AES256-GCM-SHA384 (256/256\n\tbits)) (No client certificate requested)\n\tby ozlabs.org (Postfix) with ESMTPS id 48jQXm0Mkfz9sQt\n\tfor <incoming@patchwork.ozlabs.org>;\n\tThu, 19 Mar 2020 10:02:14 +1100 (AEDT)", "from localhost (localhost [127.0.0.1])\n\tby silver.osuosl.org (Postfix) with ESMTP id D3E022040F;\n\tWed, 18 Mar 2020 23:02:11 +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 6kfxLKR4wIIk; Wed, 18 Mar 2020 23:02:09 +0000 (UTC)", "from ash.osuosl.org (ash.osuosl.org [140.211.166.34])\n\tby silver.osuosl.org (Postfix) with ESMTP id 61B91203E5;\n\tWed, 18 Mar 2020 23:02:09 +0000 (UTC)", "from fraxinus.osuosl.org (smtp4.osuosl.org [140.211.166.137])\n\tby ash.osuosl.org (Postfix) with ESMTP id BE4921BF335\n\tfor <intel-wired-lan@lists.osuosl.org>;\n\tWed, 18 Mar 2020 23:02:07 +0000 (UTC)", "from localhost (localhost [127.0.0.1])\n\tby fraxinus.osuosl.org (Postfix) with ESMTP id B36C286644\n\tfor <intel-wired-lan@lists.osuosl.org>;\n\tWed, 18 Mar 2020 23:02:07 +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 HUBa9hQnq6SG for <intel-wired-lan@lists.osuosl.org>;\n\tWed, 18 Mar 2020 23:02:07 +0000 (UTC)", "from mga17.intel.com (mga17.intel.com [192.55.52.151])\n\tby fraxinus.osuosl.org (Postfix) with ESMTPS id 26F3F86610\n\tfor <intel-wired-lan@lists.osuosl.org>;\n\tWed, 18 Mar 2020 23:02:07 +0000 (UTC)", "from orsmga006.jf.intel.com ([10.7.209.51])\n\tby fmsmga107.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; \n\t18 Mar 2020 16:02:06 -0700", "from ymtomada-mobl.amr.corp.intel.com ([10.251.21.179])\n\tby orsmga006.jf.intel.com with ESMTP; 18 Mar 2020 16:02:06 -0700" ], "X-Virus-Scanned": [ "amavisd-new at osuosl.org", "amavisd-new at osuosl.org" ], "X-Greylist": "domain auto-whitelisted by SQLgrey-1.7.6", "IronPort-SDR": [ "UNQ6oiZ9noxb7ZpZSMGriCq5gBAB6YE6z8mA7DzdtagxooSDmJv+7DbmFUR+LwZKKjkJoRxnXi\n\tLJa+lh+Eha+A==", "lp4f2PoxKqNBpAFV1mR15u/DumLtUTJSTSNJAL+Q2JspejcDfqdTuVZaDr8YQhgm5xoJ3n+EQI\n\t3JQwULe9dESw==" ], "X-Amp-Result": "SKIPPED(no attachment in message)", "X-Amp-File-Uploaded": "False", "X-ExtLoop1": "1", "X-IronPort-AV": "E=Sophos;i=\"5.70,569,1574150400\"; d=\"scan'208\";a=\"248341766\"", "From": "Andre Guedes <andre.guedes@intel.com>", "To": "intel-wired-lan@lists.osuosl.org", "Date": "Wed, 18 Mar 2020 16:00:50 -0700", "Message-Id": "<20200318230102.36952-1-andre.guedes@intel.com>", "X-Mailer": "git-send-email 2.25.0", "MIME-Version": "1.0", "Subject": "[Intel-wired-lan] [PATCH 00/12] igc: Refactor MAC address filtering\n\tcode", "X-BeenThere": "intel-wired-lan@osuosl.org", "X-Mailman-Version": "2.1.29", "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>", "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 all,\n\nThe MAC address filtering support from IGC driver has some duplicate code and\nconvoluted logic, making it harder to read, debug, and maintain.\n\nAs a follow-up to the bug fixes already applied to dev-queue, this patchset\nrefactors the MAC address filtering code to address those issues. It also takes\nthe opportunity to improve documentation from some key functions and clean up\ncomments that are not applicable to IGC, and fix minor issues along the way.\n\nIn summary, the new MAC address filtering code is organized as follows:\n\n\t* Filters are added and deleted only via igc_add_mac_filter() and\n\t igc_del_mac_filter() APIs. These APIs manage the adapter->mac_table.\n\t They are defined in igc_main.c and used in both igc_main.c and\n\t igc_ethtool.c.\n\n\t* Filter configuration in hardware is handled by igc_set_mac_filter_\n\t hw() and igc_clear_mac_filter_hw() local helpers in igc_main.c.\n\n\t* IGC_MAC_STATE_QUEUE_STEERING flag as well as igc_*_mac_steering_\n\t filter() are gone.\n\nBest regards,\n\nAndre\n\nAndre Guedes (12):\n igc: Remove duplicate code in MAC filtering logic\n igc: Check unsupported flag in igc_add_mac_filter()\n igc: Change igc_add_mac_filter() returning value\n igc: Fix igc_uc_unsync()\n igc: Refactor igc_rar_set_index()\n igc: Improve address check in igc_del_mac_filter()\n igc: Remove 'queue' check in igc_del_mac_filter()\n igc: Remove IGC_MAC_STATE_QUEUE_STEERING\n igc: Remove igc_*_mac_steering_filter() wrappers\n igc: Refactor igc_mac_entry_can_be_used()\n igc: Refactor igc_del_mac_filter()\n igc: Add debug messages to MAC filter code\n\n drivers/net/ethernet/intel/igc/igc.h | 11 +-\n drivers/net/ethernet/intel/igc/igc_ethtool.c | 22 +-\n drivers/net/ethernet/intel/igc/igc_main.c | 372 ++++++++-----------\n 3 files changed, 169 insertions(+), 236 deletions(-)" }