{"id":2235057,"url":"http://patchwork.ozlabs.org/api/1.2/covers/2235057/?format=json","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=json","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=json","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=json","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":"","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 ; 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 ; 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 ; 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 ;\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 ; Fri, 8 May 2026 13:43:30 +0000 (UTC)","by mail-oa1-x35.google.com with SMTP id\n 586e51a60fabf-4303eb92930so1450665fac.3\n for ; 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= ","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=","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 ","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":"","List-Unsubscribe":",\n ","List-Archive":"","List-Post":"","List-Help":"","List-Subscribe":",\n ","Cc":"dev@openvswitch.org, linux-kselftest@vger.kernel.org,\n Minxi Hou , 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\" "},"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(-)"}