Patch Detail

HTTP 200 OK
Allow: GET, PUT, PATCH, HEAD, OPTIONS
Content-Type: application/json
Vary: Accept

{
    "id": 614587,
    "url": "http://patchwork.ozlabs.org/api/patches/614587/?format=api",
    "web_url": "http://patchwork.ozlabs.org/project/netdev/patch/20160425183107.11331.67524.stgit@ahduyck-xeon-server/",
    "project": {
        "id": 7,
        "url": "http://patchwork.ozlabs.org/api/projects/7/?format=api",
        "name": "Linux network development",
        "link_name": "netdev",
        "list_id": "netdev.vger.kernel.org",
        "list_email": "netdev@vger.kernel.org",
        "web_url": null,
        "scm_url": null,
        "webscm_url": null,
        "list_archive_url": "",
        "list_archive_url_format": "",
        "commit_url_format": ""
    },
    "msgid": "<20160425183107.11331.67524.stgit@ahduyck-xeon-server>",
    "list_archive_url": null,
    "date": "2016-04-25T18:31:07",
    "name": "[net-next,2/8] gso: Only allow GSO_PARTIAL if we can checksum the inner protocol",
    "commit_ref": null,
    "pull_url": null,
    "state": "deferred",
    "archived": true,
    "hash": "e756f81f70652c19dc20e7ba977126a73e5a7c62",
    "submitter": {
        "id": 67293,
        "url": "http://patchwork.ozlabs.org/api/people/67293/?format=api",
        "name": "Alexander Duyck",
        "email": "aduyck@mirantis.com"
    },
    "delegate": {
        "id": 34,
        "url": "http://patchwork.ozlabs.org/api/users/34/?format=api",
        "username": "davem",
        "first_name": "David",
        "last_name": "Miller",
        "email": "davem@davemloft.net"
    },
    "mbox": "http://patchwork.ozlabs.org/project/netdev/patch/20160425183107.11331.67524.stgit@ahduyck-xeon-server/mbox/",
    "series": [],
    "comments": "http://patchwork.ozlabs.org/api/patches/614587/comments/",
    "check": "pending",
    "checks": "http://patchwork.ozlabs.org/api/patches/614587/checks/",
    "tags": {},
    "related": [],
    "headers": {
        "Return-Path": "<netdev-owner@vger.kernel.org>",
        "X-Original-To": "patchwork-incoming@ozlabs.org",
        "Delivered-To": "patchwork-incoming@ozlabs.org",
        "Received": [
            "from vger.kernel.org (vger.kernel.org [209.132.180.67])\n\tby ozlabs.org (Postfix) with ESMTP id 3qtvtv6wTGz9t5P\n\tfor <patchwork-incoming@ozlabs.org>;\n\tTue, 26 Apr 2016 04:31:19 +1000 (AEST)",
            "(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S933348AbcDYSbL (ORCPT <rfc822;patchwork-incoming@ozlabs.org>);\n\tMon, 25 Apr 2016 14:31:11 -0400",
            "from mail-pa0-f45.google.com ([209.85.220.45]:36396 \"EHLO\n\tmail-pa0-f45.google.com\" rhost-flags-OK-OK-OK-OK) by vger.kernel.org\n\twith ESMTP id S933040AbcDYSbJ (ORCPT\n\t<rfc822;netdev@vger.kernel.org>); Mon, 25 Apr 2016 14:31:09 -0400",
            "by mail-pa0-f45.google.com with SMTP id bt5so14104560pac.3\n\tfor <netdev@vger.kernel.org>; Mon, 25 Apr 2016 11:31:09 -0700 (PDT)",
            "from [192.168.1.143] (static-50-53-29-36.bvtn.or.frontiernet.net.\n\t[50.53.29.36]) by smtp.gmail.com with ESMTPSA id\n\tp78sm30450371pfa.75.2016.04.25.11.31.08\n\t(version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128);\n\tMon, 25 Apr 2016 11:31:08 -0700 (PDT)"
        ],
        "Authentication-Results": "ozlabs.org; dkim=pass (1024-bit key;\n\tunprotected) header.d=mirantis.com header.i=@mirantis.com\n\theader.b=FY8khcA1; dkim-atps=neutral",
        "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=mirantis.com; s=google;\n\th=subject:from:to:date:message-id:in-reply-to:references:user-agent\n\t:mime-version:content-transfer-encoding;\n\tbh=pwmgge87THBUf6/LCNpAGO9hSYXwil/SbIhrtS/Jb6s=;\n\tb=FY8khcA1RYkdscebE44aouc/DR4oY3qZZVedLEs8vLPQaweSvKCjk5GGGZasrUKXdO\n\tNLM2ur6oDtSQV6+v6LSw6OeNUwBahUcyLav4LnKb3AVvzel0y35MO0Dp74PZ7jSKMVgR\n\tIafS0SFK54hAfnKOSiyPIPo+k4MtiBYOzWoaI=",
        "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n\td=1e100.net; s=20130820;\n\th=x-gm-message-state:subject:from:to:date:message-id:in-reply-to\n\t:references:user-agent:mime-version:content-transfer-encoding;\n\tbh=pwmgge87THBUf6/LCNpAGO9hSYXwil/SbIhrtS/Jb6s=;\n\tb=FdxlqULFCsVJOAqa1N2ibmTzQ977m/39t0L7LDFNufXbBdDHIL503CDQgPjGw2RJ0E\n\tlQT+p234TfUz/qOapGV9gXFgg1c6+nConLilEc68JqA7hkIqusSJs9XiL/OBJKZUVGqQ\n\tRyGaZ9uI8FHkrYb7SKt6otK74BEwFsCqY2StKz3ci17bNkA5lt95RggpW8SzhPV0g2Mq\n\twZWSHzdRaqtnWJyLryAdZD1B0cW5kNv7y6lVad6g1g2qdO5TPY8MUiXxLdMetYyeghPE\n\tJOpeHaREE4XMJTGNvxMihH88Fw8DOn4CNlWgh3VjXBUZIpqNpOTcXJSYNQF7cF28fhCb\n\t6BQw==",
        "X-Gm-Message-State": "AOPr4FUPomYF3tp1gu0/HBoUHd+d66b68V96Nsv336GLtzuQzWQoWv1iC1tTTSrbqCEDlfu1",
        "X-Received": "by 10.66.124.226 with SMTP id ml2mr50297369pab.90.1461609069036; \n\tMon, 25 Apr 2016 11:31:09 -0700 (PDT)",
        "Subject": "[net-next PATCH 2/8] gso: Only allow GSO_PARTIAL if we can checksum\n\tthe inner protocol",
        "From": "Alexander Duyck <aduyck@mirantis.com>",
        "To": "talal@mellanox.com, netdev@vger.kernel.org, davem@davemloft.net,\n\tgalp@mellanox.com, ogerlitz@mellanox.com, eranbe@mellanox.com",
        "Date": "Mon, 25 Apr 2016 11:31:07 -0700",
        "Message-ID": "<20160425183107.11331.67524.stgit@ahduyck-xeon-server>",
        "In-Reply-To": "<20160425182442.11331.88349.stgit@ahduyck-xeon-server>",
        "References": "<20160425182442.11331.88349.stgit@ahduyck-xeon-server>",
        "User-Agent": "StGit/0.17.1-dirty",
        "MIME-Version": "1.0",
        "Content-Type": "text/plain; charset=\"utf-8\"",
        "Content-Transfer-Encoding": "7bit",
        "Sender": "netdev-owner@vger.kernel.org",
        "Precedence": "bulk",
        "List-ID": "<netdev.vger.kernel.org>",
        "X-Mailing-List": "netdev@vger.kernel.org"
    },
    "content": "This patch addresses a possible issue that can occur if we get into any odd\ncorner cases where we support TSO for a given protocol but not the checksum\nor scatter-gather offload.  There are few drivers floating around that\nsetup their tunnels this way and by enforcing the checksum piece we can\navoid mangling any frames.\n\nSigned-off-by: Alexander Duyck <aduyck@mirantis.com>\n---\n net/core/skbuff.c |    6 +++---\n 1 file changed, 3 insertions(+), 3 deletions(-)",
    "diff": "diff --git a/net/core/skbuff.c b/net/core/skbuff.c\nindex 7ff7788b0151..d2871d081750 100644\n--- a/net/core/skbuff.c\n+++ b/net/core/skbuff.c\n@@ -3080,8 +3080,7 @@ struct sk_buff *skb_segment(struct sk_buff *head_skb,\n \tunsigned int headroom;\n \tunsigned int len = head_skb->len;\n \t__be16 proto;\n-\tbool csum;\n-\tint sg = !!(features & NETIF_F_SG);\n+\tbool csum, sg;\n \tint nfrags = skb_shinfo(head_skb)->nr_frags;\n \tint err = -ENOMEM;\n \tint i = 0;\n@@ -3093,13 +3092,14 @@ struct sk_buff *skb_segment(struct sk_buff *head_skb,\n \tif (unlikely(!proto))\n \t\treturn ERR_PTR(-EINVAL);\n \n+\tsg = !!(features & NETIF_F_SG);\n \tcsum = !!can_checksum_protocol(features, proto);\n \n \t/* GSO partial only requires that we trim off any excess that\n \t * doesn't fit into an MSS sized block, so take care of that\n \t * now.\n \t */\n-\tif (features & NETIF_F_GSO_PARTIAL) {\n+\tif (sg && csum && (features & NETIF_F_GSO_PARTIAL)) {\n \t\tpartial_segs = len / mss;\n \t\tmss *= partial_segs;\n \t}\n",
    "prefixes": [
        "net-next",
        "2/8"
    ]
}