Cover Letter Detail
Show a cover letter.
GET /api/1.2/covers/2235057/?format=api
{ "id": 2235057, "url": "http://patchwork.ozlabs.org/api/1.2/covers/2235057/?format=api", "web_url": "http://patchwork.ozlabs.org/project/openvswitch/cover/20260508134308.2857449-1-houminxi@gmail.com/", "project": { "id": 47, "url": "http://patchwork.ozlabs.org/api/1.2/projects/47/?format=api", "name": "Open vSwitch", "link_name": "openvswitch", "list_id": "ovs-dev.openvswitch.org", "list_email": "ovs-dev@openvswitch.org", "web_url": "http://openvswitch.org/", "scm_url": "git@github.com:openvswitch/ovs.git", "webscm_url": "https://github.com/openvswitch/ovs", "list_archive_url": "", "list_archive_url_format": "", "commit_url_format": "" }, "msgid": "<20260508134308.2857449-1-houminxi@gmail.com>", "list_archive_url": null, "date": "2026-05-08T13:43:06", "name": "[ovs-dev,net-next,v8,0/2] selftests: openvswitch: add pop_vlan test", "submitter": { "id": 93141, "url": "http://patchwork.ozlabs.org/api/1.2/people/93141/?format=api", "name": "Minxi Hou", "email": "houminxi@gmail.com" }, "mbox": "http://patchwork.ozlabs.org/project/openvswitch/cover/20260508134308.2857449-1-houminxi@gmail.com/mbox/", "series": [ { "id": 503392, "url": "http://patchwork.ozlabs.org/api/1.2/series/503392/?format=api", "web_url": "http://patchwork.ozlabs.org/project/openvswitch/list/?series=503392", "date": "2026-05-08T13:43:06", "name": "selftests: openvswitch: add pop_vlan test", "version": 8, "mbox": "http://patchwork.ozlabs.org/series/503392/mbox/" } ], "comments": "http://patchwork.ozlabs.org/api/covers/2235057/comments/", "headers": { "Return-Path": "<ovs-dev-bounces@openvswitch.org>", "X-Original-To": [ "incoming@patchwork.ozlabs.org", "dev@openvswitch.org" ], "Delivered-To": [ "patchwork-incoming@legolas.ozlabs.org", "ovs-dev@lists.linuxfoundation.org" ], "Authentication-Results": [ "legolas.ozlabs.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key;\n unprotected) header.d=gmail.com header.i=@gmail.com header.a=rsa-sha256\n header.s=20251104 header.b=OWnBju9v;\n\tdkim-atps=neutral", "legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=openvswitch.org\n (client-ip=2605:bc80:3010::133; helo=smtp2.osuosl.org;\n envelope-from=ovs-dev-bounces@openvswitch.org; receiver=patchwork.ozlabs.org)", "smtp2.osuosl.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key,\n unprotected) header.d=gmail.com header.i=@gmail.com header.a=rsa-sha256\n header.s=20251104 header.b=OWnBju9v", "smtp1.osuosl.org;\n dmarc=pass (p=none dis=none) header.from=gmail.com", "smtp1.osuosl.org;\n dkim=pass (2048-bit key) header.d=gmail.com header.i=@gmail.com\n header.a=rsa-sha256 header.s=20251104 header.b=OWnBju9v" ], "Received": [ "from smtp2.osuosl.org (smtp2.osuosl.org [IPv6:2605:bc80:3010::133])\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 4gBr1V23Rwz1yKd\n\tfor <incoming@patchwork.ozlabs.org>; Fri, 08 May 2026 23:43:41 +1000 (AEST)", "from localhost (localhost [127.0.0.1])\n\tby smtp2.osuosl.org (Postfix) with ESMTP id B74C540E34;\n\tFri, 8 May 2026 13:43:34 +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 tP5eeemBlwBA; Fri, 8 May 2026 13:43:32 +0000 (UTC)", "from lists.linuxfoundation.org (lf-lists.osuosl.org [140.211.9.56])\n\tby smtp2.osuosl.org (Postfix) with ESMTPS id 72F4F406B5;\n\tFri, 8 May 2026 13:43:32 +0000 (UTC)", "from lf-lists.osuosl.org (localhost [127.0.0.1])\n\tby lists.linuxfoundation.org (Postfix) with ESMTP id 564E5C04EB;\n\tFri, 8 May 2026 13:43:32 +0000 (UTC)", "from smtp1.osuosl.org (smtp1.osuosl.org [140.211.166.138])\n by lists.linuxfoundation.org (Postfix) with ESMTP id 3B172C04E7\n for <dev@openvswitch.org>; Fri, 8 May 2026 13:43:31 +0000 (UTC)", "from localhost (localhost [127.0.0.1])\n by smtp1.osuosl.org (Postfix) with ESMTP id 19D9583FF6\n for <dev@openvswitch.org>; Fri, 8 May 2026 13:43:31 +0000 (UTC)", "from smtp1.osuosl.org ([127.0.0.1])\n by localhost (smtp1.osuosl.org [127.0.0.1]) (amavis, port 10024) with ESMTP\n id HkooEb9X7Q1o for <dev@openvswitch.org>;\n Fri, 8 May 2026 13:43:30 +0000 (UTC)", "from mail-oa1-x35.google.com (mail-oa1-x35.google.com\n [IPv6:2001:4860:4864:20::35])\n by smtp1.osuosl.org (Postfix) with ESMTPS id 39DA583D21\n for <dev@openvswitch.org>; Fri, 8 May 2026 13:43:30 +0000 (UTC)", "by mail-oa1-x35.google.com with SMTP id\n 586e51a60fabf-4303eb92930so1450665fac.3\n for <dev@openvswitch.org>; Fri, 08 May 2026 06:43:30 -0700 (PDT)", "from houminxi ([72.244.37.221]) by smtp.gmail.com with ESMTPSA id\n 006d021491bc7-69b25c76cc4sm1119108eaf.6.2026.05.08.06.43.22\n (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n Fri, 08 May 2026 06:43:26 -0700 (PDT)" ], "X-Virus-Scanned": [ "amavis at osuosl.org", "amavis at osuosl.org" ], "X-Comment": "SPF check N/A for local connections - client-ip=140.211.9.56;\n helo=lists.linuxfoundation.org;\n envelope-from=ovs-dev-bounces@openvswitch.org; receiver=<UNKNOWN> ", "DKIM-Filter": [ "OpenDKIM Filter v2.11.0 smtp2.osuosl.org 72F4F406B5", "OpenDKIM Filter v2.11.0 smtp1.osuosl.org 39DA583D21" ], "Received-SPF": "Pass (mailfrom) identity=mailfrom;\n client-ip=2001:4860:4864:20::35; helo=mail-oa1-x35.google.com;\n envelope-from=houminxi@gmail.com; receiver=<UNKNOWN>", "DMARC-Filter": "OpenDMARC Filter v1.4.2 smtp1.osuosl.org 39DA583D21", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=gmail.com; s=20251104; t=1778247809; x=1778852609; darn=openvswitch.org;\n h=content-transfer-encoding:mime-version:message-id:date:subject:cc\n :to:from:from:to:cc:subject:date:message-id:reply-to;\n bh=Iq3rziezZOyZlc85+/KkZ62IiSWPXFI0NfzP5OCCW9I=;\n b=OWnBju9vTKa3L1r2vGZoNzrawL0BGkLB9QH0f3e7FhBrfWCzjKT3bePryeI9cH3Drb\n fzMdoZOp5BmMC3Um6NGw5yxrOYIzwwHO9fm0xZhSYfWeTLYXUW4kb8V4XbIQ6gFdFgjx\n OaLQsvFnbHoiQ30LupspXXVbopulmz+1Rz3PaNd07zcdjj0Qg6UJWGGgsY4+8YijlYWX\n 4r07R9CZHt27mLe0zEGOkJ8MGQf1TZrYOBgs6v6vtxCin5Ys7CUVJ4OdI7nvOz+8zIWj\n 6f5dKBKyvDkdyIorIH+DEs4GAOU5sPt6FdZ95kP/9K6FIN2fAHJ3efe0/iasaumMZXxH\n J6Eg==", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20251104; t=1778247809; x=1778852609;\n h=content-transfer-encoding:mime-version:message-id:date:subject:cc\n :to:from:x-gm-gg:x-gm-message-state:from:to:cc:subject:date\n :message-id:reply-to;\n bh=Iq3rziezZOyZlc85+/KkZ62IiSWPXFI0NfzP5OCCW9I=;\n b=rYhOf1xdS1BURTBtqfoGZenuhoZdTyYFvpUiiPEHgafMRfF+fjIgvWsLNjglc40B0Q\n lkOGWzg64GvJYwD+tio0EsHAv2UxOJ4sF9M8e1x7Fo+HFq7xKqXqqABVaB4qwXHXZNB1\n 4fkY50vngXd4/9LW/cib7xPY3Ep3JfLwDXWWx/wWgZsKK297mM4kw7gweICrJRV8vFSd\n gSV0URADiVXKnDZo7RvHtL8Nc/PZBm0J7ZE7bReDUfJ5yuha809JnNQHRHmB+kx7eX/D\n a5F57OnFzZnef2+yJ05QdRe5iC0A4Hbpi0wbBff23phK0QBRXmBJqzwWHpaLNbYbC5nJ\n 7JQw==", "X-Forwarded-Encrypted": "i=1;\n AFNElJ9rJGhanQBe9RvUwjwKnvirv/XdV+Ryz1QjzfCMNYTJHe7yG8+Eycq09RUOIqvcM55Ylv8=@openvswitch.org", "X-Gm-Message-State": "AOJu0Yw8saB3RwHfALu5a24674Y0f0mMWDkqs78E7zJfNHK1cBdY1Cig\n mIArTF3XFQi8J5IfcC31Hu7Oncna5S5+3v6pcwMnKuSmmabrFecDcI7K", "X-Gm-Gg": "AeBDievru5coh0r8FU4W8On5XjEZYhBML1OAxNR/t9zEFYFqc5dyAZpqrgjZkovS8uj\n HhRUrg4D8M+cw0IJ2o4rKzmhZxwsg6kU2HeLinBJEhq5IwXy/XR/OmBno0v9xJHjAQYmNNWQFfP\n Afwz6WMdu0qjWqwFcjpBrUvSHMbOOQzZBiY48wr0HNqkl55vNwc7mnybD/GRRBIcJXEDWwmpoZD\n IBXkVacD7QRfyqR5ZwUqZcomKkVgRsa4FLK2A3nQ8w78KcLcvAnfVkhg3qedPPPy/eJYrjmYzUw\n CbXm6L3NVLRKmtSINp7bWC1+33ZEAcKIPBuIM7CiFPDGu8Ko7TE5NPNrQQDurKEemzq1DghIZT4\n CTooNRGIayuDQouc22muGireSD6IJK/41PVxgEsTGlfy2VkJOQDVXbDG+5LFoRQ3au7B8dndXUH\n GcxkCB1NkttPlviTg=", "X-Received": "by 2002:a05:6820:4b0a:b0:699:a391:44cd with SMTP id\n 006d021491bc7-69b25b787camr1402565eaf.23.1778247807957;\n Fri, 08 May 2026 06:43:27 -0700 (PDT)", "From": "Minxi Hou <houminxi@gmail.com>", "To": "netdev@vger.kernel.org", "Date": "Fri, 8 May 2026 21:43:06 +0800", "Message-ID": "<20260508134308.2857449-1-houminxi@gmail.com>", "X-Mailer": "git-send-email 2.53.0", "MIME-Version": "1.0", "Subject": "[ovs-dev] [PATCH net-next v8 0/2] selftests: openvswitch: add\n pop_vlan test", "X-BeenThere": "ovs-dev@openvswitch.org", "X-Mailman-Version": "2.1.30", "Precedence": "list", "List-Id": "<ovs-dev.openvswitch.org>", "List-Unsubscribe": "<https://mail.openvswitch.org/mailman/options/ovs-dev>,\n <mailto:ovs-dev-request@openvswitch.org?subject=unsubscribe>", "List-Archive": "<http://mail.openvswitch.org/pipermail/ovs-dev/>", "List-Post": "<mailto:ovs-dev@openvswitch.org>", "List-Help": "<mailto:ovs-dev-request@openvswitch.org?subject=help>", "List-Subscribe": "<https://mail.openvswitch.org/mailman/listinfo/ovs-dev>,\n <mailto:ovs-dev-request@openvswitch.org?subject=subscribe>", "Cc": "dev@openvswitch.org, linux-kselftest@vger.kernel.org,\n Minxi Hou <houminxi@gmail.com>, linux-kernel@vger.kernel.org,\n i.maximets@ovn.org, edumazet@google.com, horms@kernel.org, kuba@kernel.org,\n pabeni@redhat.com, shuah@kernel.org, davem@davemloft.net", "Content-Type": "text/plain; charset=\"us-ascii\"", "Content-Transfer-Encoding": "7bit", "Errors-To": "ovs-dev-bounces@openvswitch.org", "Sender": "\"dev\" <ovs-dev-bounces@openvswitch.org>" }, "content": "Add test_pop_vlan() to verify OVS kernel datapath pop_vlan action\ncorrectly strips 802.1Q VLAN tags from frames.\n\nPatch 1 extends ovs-dpctl.py with vlan(vid=X,pcp=Y,cfi=Z) formatting\nand parsing, plus an encap_ovskey subclass for safe ENCAP NLA decoding.\nIt changes OVS_KEY_ATTR_VLAN type from uint16 to be16 to match\nthe kernel __be16 wire format.\nIt also adds push_vlan action support (parse/format with range\nvalidation) and removes the unnecessary MAX_ENCAP_DEPTH limit.\nPatch 2 adds the selftest using purely ping-based verification with\na push_vlan return flow for symmetric bidirectional testing.\n\nTested with vng on x86_64, all OVS selftests pass (including new\ntest_pop_vlan).\n\nv8:\n - consistent error message prefix: push_vlan() instead of\n push_vlan (matches vlan() and encap() style)\n - unify encap() error messages for clarity\n - add empty-string guards for MAC address variables\n - remove slot number comments from encap_ovskey nla_map\n - remove explicit modprobe 8021q pre-flight check\nv7: https://lore.kernel.org/netdev/20260507131541.2331771-1-houminxi@gmail.com/\n - remove slot number comments from encap_ovskey nla_map, keep\n only comments explaining differences from base ovskey class\n - remove explicit modprobe 8021q pre-flight check (ip link add\n type vlan auto-loads the module)\nv6: https://lore.kernel.org/netdev/20260506131218.1880852-1-houminxi@gmail.com/\n - fix non-ASCII characters (em dashes) in comments and commit\n messages\nv5: https://lore.kernel.org/netdev/20260505124957.1239812-1-houminxi@gmail.com/\n - add push_vlan action class, dpstr format and parse with range\n validation (vid 0-4095, pcp 0-7, tpid 0-0xFFFF, CFI forced to 1)\n - remove MAX_ENCAP_DEPTH constant and depth tracking (bracket-depth\n counter in encap parser already handles nesting)\n - remove start_capture/stop_capture helpers and tcpdump/pcap\n verification -- use ping success/failure instead\n - remove modprobe/netns pre-flight checks (other tests don't do this)\n - remove ethtool VLAN offload disable (unnecessary for veth)\n - add push_vlan return flow for symmetric bidirectional ping\n - use ovs_sbx wrapper for ping commands (consistent with siblings)\nv4: https://lore.kernel.org/netdev/20260504123713.555461-1-houminxi@gmail.com/\n - fix all checkpatch line-length warnings in new code\n - fix pylint W0707: use explicit exception chaining (from exc)\nv3: https://lore.kernel.org/netdev/20260503120946.51869-1-houminxi@gmail.com/\n - encap_ovskey: MPLS type \"ovs_key_mpls\" -> \"array(ovs_key_mpls)\"\n - encap_ovskey: PRIORITY/IN_PORT set to \"none\" (metadata, not in ENCAP)\n - _vlan_dpstr: cfi=0 falls back to tci=0x%04x for round-trip safety\n - encap parse(): check return value for unrecognized trailing content\n - vlan parser: boundary check + raise-from for exception chaining\n - start_capture: || return $? to propagate ksft_skip correctly\n - on_exit: moved after resource creation, not before\n - ping success: changed from NOTE to FAIL + return 1\n - VLAN interface creation: added || return 1 error propagation\n - netns probe: distinguish EEXIST from missing CONFIG_NET_NS\n - sbx_add: || return $ksft_skip -> || return $? (match sibling tests)\nv2: https://lore.kernel.org/netdev/20260501133924.3100680-1-houminxi@gmail.com/\n\nMinxi Hou (2):\n selftests: openvswitch: add vlan() and encap() flow string parsing\n selftests: openvswitch: add pop_vlan test\n\n .../selftests/net/openvswitch/openvswitch.sh | 77 +++++\n .../selftests/net/openvswitch/ovs-dpctl.py | 322 +++++++++++++++++-\n 2 files changed, 389 insertions(+), 10 deletions(-)" }