Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/2215951/?format=api
{ "id": 2215951, "url": "http://patchwork.ozlabs.org/api/patches/2215951/?format=api", "web_url": "http://patchwork.ozlabs.org/project/intel-wired-lan/patch/20260325151615.1407182-2-ade@amicon.ru/", "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": "<20260325151615.1407182-2-ade@amicon.ru>", "list_archive_url": null, "date": "2026-03-25T15:16:03", "name": "[net-next,v2,1/2] e1000: limit endianness conversion to boundary words", "commit_ref": null, "pull_url": null, "state": "changes-requested", "archived": false, "hash": "8b2dfcef9e18586687d369c9f5257ec971d03c0e", "submitter": { "id": 92902, "url": "http://patchwork.ozlabs.org/api/people/92902/?format=api", "name": "Agalakov Daniil", "email": "ade@amicon.ru" }, "delegate": null, "mbox": "http://patchwork.ozlabs.org/project/intel-wired-lan/patch/20260325151615.1407182-2-ade@amicon.ru/mbox/", "series": [ { "id": 497462, "url": "http://patchwork.ozlabs.org/api/series/497462/?format=api", "web_url": "http://patchwork.ozlabs.org/project/intel-wired-lan/list/?series=497462", "date": "2026-03-25T15:16:02", "name": "e1000/e1000e: limit endianness conversion to boundary words", "version": 2, "mbox": "http://patchwork.ozlabs.org/series/497462/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/patches/2215951/comments/", "check": "pending", "checks": "http://patchwork.ozlabs.org/api/patches/2215951/checks/", "tags": {}, "related": [], "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@legolas.ozlabs.org", "intel-wired-lan@lists.osuosl.org" ], "Authentication-Results": [ "legolas.ozlabs.org;\n\tdkim=pass (2048-bit key;\n unprotected) header.d=osuosl.org header.i=@osuosl.org header.a=rsa-sha256\n header.s=default header.b=EolbawgY;\n\tdkim-atps=neutral", "legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=osuosl.org\n (client-ip=140.211.166.136; helo=smtp3.osuosl.org;\n envelope-from=intel-wired-lan-bounces@osuosl.org;\n receiver=patchwork.ozlabs.org)" ], "Received": [ "from smtp3.osuosl.org (smtp3.osuosl.org [140.211.166.136])\n\t(using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits)\n\t key-exchange x25519 server-signature ECDSA (secp384r1) server-digest SHA384)\n\t(No client certificate requested)\n\tby legolas.ozlabs.org (Postfix) with ESMTPS id 4fgr9J2bpDz1xy3\n\tfor <incoming@patchwork.ozlabs.org>; Thu, 26 Mar 2026 02:16:52 +1100 (AEDT)", "from localhost (localhost [127.0.0.1])\n\tby smtp3.osuosl.org (Postfix) with ESMTP id 5E4F3606B1;\n\tWed, 25 Mar 2026 15:16:50 +0000 (UTC)", "from smtp3.osuosl.org ([127.0.0.1])\n by localhost (smtp3.osuosl.org [127.0.0.1]) (amavis, port 10024) with ESMTP\n id qzzr6KEqU7-p; Wed, 25 Mar 2026 15:16:49 +0000 (UTC)", "from lists1.osuosl.org (lists1.osuosl.org [140.211.166.142])\n\tby smtp3.osuosl.org (Postfix) with ESMTP id 0F61860668;\n\tWed, 25 Mar 2026 15:16:49 +0000 (UTC)", "from smtp2.osuosl.org (smtp2.osuosl.org [IPv6:2605:bc80:3010::133])\n by lists1.osuosl.org (Postfix) with ESMTP id 54395353\n for <intel-wired-lan@lists.osuosl.org>; Wed, 25 Mar 2026 15:16:47 +0000 (UTC)", "from localhost (localhost [127.0.0.1])\n by smtp2.osuosl.org (Postfix) with ESMTP id 8D21B4018E\n for <intel-wired-lan@lists.osuosl.org>; Wed, 25 Mar 2026 15:16:46 +0000 (UTC)", "from smtp2.osuosl.org ([127.0.0.1])\n by localhost (smtp2.osuosl.org [127.0.0.1]) (amavis, port 10024) with ESMTP\n id MSBaaw2HQ7fb for <intel-wired-lan@lists.osuosl.org>;\n Wed, 25 Mar 2026 15:16:46 +0000 (UTC)", "from mail.amicon.ru (mail.amicon.ru [77.108.111.100])\n by smtp2.osuosl.org (Postfix) with ESMTPS id 887044014E\n for <intel-wired-lan@lists.osuosl.org>; Wed, 25 Mar 2026 15:16:45 +0000 (UTC)", "from example.com (172.16.2.46) by mail.amicon.lan (192.168.0.59)\n with Microsoft SMTP Server (version=TLS1_2,\n cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.2.1544.27; Wed, 25 Mar\n 2026 18:16:40 +0300" ], "X-Virus-Scanned": [ "amavis at osuosl.org", "amavis at osuosl.org" ], "X-Comment": "SPF check N/A for local connections - client-ip=140.211.166.142;\n helo=lists1.osuosl.org; envelope-from=intel-wired-lan-bounces@osuosl.org;\n receiver=<UNKNOWN> ", "DKIM-Filter": [ "OpenDKIM Filter v2.11.0 smtp3.osuosl.org 0F61860668", "OpenDKIM Filter v2.11.0 smtp2.osuosl.org 887044014E" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed; d=osuosl.org;\n\ts=default; t=1774451809;\n\tbh=YOpNw8QIFMjTgGrsjFQXv+M9MSE61bNxQgNOBC3TR5g=;\n\th=From:To:CC:Date:In-Reply-To:References:Subject:List-Id:\n\t List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe:\n\t From;\n\tb=EolbawgYPNqzPRBcyEdWeb2xmtiYtGDag60Q6mmBtyZCvvnaLT6brL0xE/M6uveMK\n\t Bgbe6vYtIb6RUM57O96Y8uSQ8OEIrboH3fLtrGcfTXLbACn0GTsVR6Y4J8gkKhjD3D\n\t 5Pu96Bki/r2m7+WVaG4lpB2Kvu4hLXbVM9IqT5L/YX3xUm4ng4CDGJ4LeTss0JDBov\n\t 9j/TvAHsk1CFQvi/54xmJkBZg1cMdTCd1EK4WH+uQxMx3YS3hgJxZf6CLCjYc+GquP\n\t xPyGpQ28IzLhzMgX8cGNosPrD7Ec17RfDsrUVgWxHuKipR9oaSpsvCx9CgPBOfDWbb\n\t pERYCrStBPraw==", "Received-SPF": "Pass (mailfrom) identity=mailfrom; client-ip=77.108.111.100;\n helo=mail.amicon.ru; envelope-from=ade@amicon.ru; receiver=<UNKNOWN>", "DMARC-Filter": "OpenDMARC Filter v1.4.2 smtp2.osuosl.org 887044014E", "Content-Transfer-Encoding": "8bit", "Content-Type": "text/plain", "From": "Agalakov Daniil <ade@amicon.ru>", "To": "Tony Nguyen <anthony.l.nguyen@intel.com>", "CC": "Agalakov Daniil <ade@amicon.ru>, Przemek Kitszel\n <przemyslaw.kitszel@intel.com>, Andrew Lunn <andrew+netdev@lunn.ch>, \"David\n S. Miller\" <davem@davemloft.net>, Eric Dumazet <edumazet@google.com>, \"Jakub\n Kicinski\" <kuba@kernel.org>, Paolo Abeni <pabeni@redhat.com>,\n <intel-wired-lan@lists.osuosl.org>, <netdev@vger.kernel.org>,\n <linux-kernel@vger.kernel.org>, <lvc-project@linuxtesting.org>, \"Daniil\n Iskhakov\" <dish@amicon.ru>, Roman Razov <rrv@amicon.ru>", "Date": "Wed, 25 Mar 2026 18:16:03 +0300", "Message-ID": "<20260325151615.1407182-2-ade@amicon.ru>", "X-Mailer": "git-send-email 2.51.0", "In-Reply-To": "<20260325151615.1407182-1-ade@amicon.ru>", "References": "<a70a49a0-43f8-4908-993a-50999bf0dfa8@intel.com>\n <20260325151615.1407182-1-ade@amicon.ru>", "MIME-Version": "1.0", "X-Originating-IP": "[172.16.2.46]", "X-ClientProxiedBy": "mail.amicon.lan (192.168.0.59) To mail.amicon.lan\n (192.168.0.59)", "X-Mailman-Original-DKIM-Signature": "v=1; a=rsa-sha256; d=amicon.ru; s=mail;\n c=simple/simple;\n t=1774451801; h=from:subject:to:date:message-id;\n bh=Wlg44DqnfJaqyWBN6Op2t8hnzKVeLag5dyQfgjVI3tU=;\n b=jaFw6gAkRx5+HcHWMEsrrq7gqnPDy08whPC1GNH52qCB4NWMZ7YbX2OsrO80Znjd07XNaYQkFfT\n d7FE6cmXpRyX7yEPRRcYMDj5BW+oAIu94/9a2+rtPiMPcJ7ej0S2F9gE7vVnVnVuPE0ifPcROVqTj\n y17KYvecF6kJ3NPIo0R/MGtQE9jQl5v6tbVuguCnJ7GCChaxf78O1bpJIv9uY8wlDPRDr9opouJVp\n kRoHxS8qGEpgslGOYN2kNsSD6RA4FVLtJ6Pk2qGZo//AnN1U+DxjkwsLWMT9YsFKxNDT7eDXw0rCw\n MlyjsptYTWt+SI04SV5gfW3n+aWM3OcXttyg==", "X-Mailman-Original-Authentication-Results": [ "smtp2.osuosl.org;\n dmarc=pass (p=quarantine dis=none)\n header.from=amicon.ru", "smtp2.osuosl.org;\n dkim=pass (2048-bit key) header.d=amicon.ru header.i=@amicon.ru\n header.a=rsa-sha256 header.s=mail header.b=jaFw6gAk" ], "Subject": "[Intel-wired-lan] [PATCH net-next v2 1/2] e1000: limit endianness\n conversion to boundary words", "X-BeenThere": "intel-wired-lan@osuosl.org", "X-Mailman-Version": "2.1.30", "Precedence": "list", "List-Id": "Intel Wired Ethernet Linux Kernel Driver Development\n <intel-wired-lan.osuosl.org>", "List-Unsubscribe": "<https://lists.osuosl.org/mailman/options/intel-wired-lan>,\n <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 <mailto:intel-wired-lan-request@osuosl.org?subject=subscribe>", "Errors-To": "intel-wired-lan-bounces@osuosl.org", "Sender": "\"Intel-wired-lan\" <intel-wired-lan-bounces@osuosl.org>" }, "content": "[Why]\nIn e1000_set_eeprom(), the eeprom_buff is allocated to hold a range of\nwords. However, only the boundary words (the first and the last) are\npopulated from the EEPROM if the write request is not word-aligned.\nThe words in the middle of the buffer remain uninitialized because they\nare intended to be completely overwritten by the new data via memcpy().\n\nThe previous implementation had a loop that performed le16_to_cpus()\non the entire buffer. This resulted in endianness conversion being\nperformed on uninitialized memory for all interior words.\n\nFix this by converting the endianness only for the boundary words\nimmediately after they are successfully read from the EEPROM.\n\nFound by Linux Verification Center (linuxtesting.org) with SVACE.\n\nCo-developed-by: Iskhakov Daniil <dish@amicon.ru>\nSigned-off-by: Iskhakov Daniil <dish@amicon.ru>\nSigned-off-by: Agalakov Daniil <ade@amicon.ru>\n---\nv2:\n - Split from the original bugfix series and targeted at 'net-text'.\n - Removed the Fixes: tag; limiting the conversion scope is an\n improvement to avoid unnecessary processing of uninitialized memory.\n - Improved commit description for clarity.\n\n drivers/net/ethernet/intel/e1000/e1000_ethtool.c | 11 +++++++----\n 1 file changed, 7 insertions(+), 4 deletions(-)", "diff": "diff --git a/drivers/net/ethernet/intel/e1000/e1000_ethtool.c b/drivers/net/ethernet/intel/e1000/e1000_ethtool.c\nindex ab232b3fbbd0..38b1f91823ef 100644\n--- a/drivers/net/ethernet/intel/e1000/e1000_ethtool.c\n+++ b/drivers/net/ethernet/intel/e1000/e1000_ethtool.c\n@@ -496,6 +496,10 @@ static int e1000_set_eeprom(struct net_device *netdev,\n \t\t */\n \t\tret_val = e1000_read_eeprom(hw, first_word, 1,\n \t\t\t\t\t &eeprom_buff[0]);\n+\n+\t\t/* Device's eeprom is always little-endian, word addressable */\n+\t\tle16_to_cpus(&eeprom_buff[0]);\n+\n \t\tptr++;\n \t}\n \tif (((eeprom->offset + eeprom->len) & 1) && (ret_val == 0)) {\n@@ -504,11 +508,10 @@ static int e1000_set_eeprom(struct net_device *netdev,\n \t\t */\n \t\tret_val = e1000_read_eeprom(hw, last_word, 1,\n \t\t\t\t\t &eeprom_buff[last_word - first_word]);\n-\t}\n \n-\t/* Device's eeprom is always little-endian, word addressable */\n-\tfor (i = 0; i < last_word - first_word + 1; i++)\n-\t\tle16_to_cpus(&eeprom_buff[i]);\n+\t\t/* Device's eeprom is always little-endian, word addressable */\n+\t\tle16_to_cpus(&eeprom_buff[last_word - first_word]);\n+\t}\n \n \tmemcpy(ptr, bytes, eeprom->len);\n \n", "prefixes": [ "net-next", "v2", "1/2" ] }