Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/1.2/patches/2235058/?format=api
{ "id": 2235058, "url": "http://patchwork.ozlabs.org/api/1.2/patches/2235058/?format=api", "web_url": "http://patchwork.ozlabs.org/project/openvswitch/patch/20260508134308.2857449-2-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-2-houminxi@gmail.com>", "list_archive_url": null, "date": "2026-05-08T13:43:07", "name": "[ovs-dev,net-next,v8,1/2] selftests: openvswitch: add vlan() and encap() flow string parsing", "commit_ref": null, "pull_url": null, "state": "new", "archived": false, "hash": "54e4b63c856ce65743dc24397eb188474608af9a", "submitter": { "id": 93141, "url": "http://patchwork.ozlabs.org/api/1.2/people/93141/?format=api", "name": "Minxi Hou", "email": "houminxi@gmail.com" }, "delegate": null, "mbox": "http://patchwork.ozlabs.org/project/openvswitch/patch/20260508134308.2857449-2-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/patches/2235058/comments/", "check": "pending", "checks": "http://patchwork.ozlabs.org/api/patches/2235058/checks/", "tags": {}, "related": [], "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=bUY2PYx3;\n\tdkim-atps=neutral", "legolas.ozlabs.org;\n spf=pass (sender SPF authorized) smtp.mailfrom=openvswitch.org\n (client-ip=2605:bc80:3010::137; helo=smtp4.osuosl.org;\n envelope-from=ovs-dev-bounces@openvswitch.org; receiver=patchwork.ozlabs.org)", "smtp4.osuosl.org;\n\tdkim=fail reason=\"signature verification failed\" (2048-bit key)\n header.d=gmail.com header.i=@gmail.com header.a=rsa-sha256 header.s=20251104\n header.b=bUY2PYx3", "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=bUY2PYx3" ], "Received": [ "from smtp4.osuosl.org (smtp4.osuosl.org [IPv6:2605:bc80:3010::137])\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 4gBr1V2fdRz1yKm\n\tfor <incoming@patchwork.ozlabs.org>; Fri, 08 May 2026 23:43:42 +1000 (AEST)", "from localhost (localhost [127.0.0.1])\n\tby smtp4.osuosl.org (Postfix) with ESMTP id B399941BC0;\n\tFri, 8 May 2026 13:43:40 +0000 (UTC)", "from smtp4.osuosl.org ([127.0.0.1])\n by localhost (smtp4.osuosl.org [127.0.0.1]) (amavis, port 10024) with ESMTP\n id YiT0AJXNb5dj; Fri, 8 May 2026 13:43:39 +0000 (UTC)", "from lists.linuxfoundation.org (lf-lists.osuosl.org\n [IPv6:2605:bc80:3010:104::8cd3:938])\n\tby smtp4.osuosl.org (Postfix) with ESMTPS id 2BD5C41BC4;\n\tFri, 8 May 2026 13:43:39 +0000 (UTC)", "from lf-lists.osuosl.org (localhost [127.0.0.1])\n\tby lists.linuxfoundation.org (Postfix) with ESMTP id 0907AC04EC;\n\tFri, 8 May 2026 13:43:39 +0000 (UTC)", "from smtp1.osuosl.org (smtp1.osuosl.org [IPv6:2605:bc80:3010::138])\n by lists.linuxfoundation.org (Postfix) with ESMTP id 1980BC04EB\n for <dev@openvswitch.org>; Fri, 8 May 2026 13:43:38 +0000 (UTC)", "from localhost (localhost [127.0.0.1])\n by smtp1.osuosl.org (Postfix) with ESMTP id 47C42842DB\n for <dev@openvswitch.org>; Fri, 8 May 2026 13:43:37 +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 TKk9vO2YYNeG for <dev@openvswitch.org>;\n Fri, 8 May 2026 13:43:35 +0000 (UTC)", "from mail-ot1-x343.google.com (mail-ot1-x343.google.com\n [IPv6:2607:f8b0:4864:20::343])\n by smtp1.osuosl.org (Postfix) with ESMTPS id E0FE9842C3\n for <dev@openvswitch.org>; Fri, 8 May 2026 13:43:34 +0000 (UTC)", "by mail-ot1-x343.google.com with SMTP id\n 46e09a7af769-7dcd9061b1aso1714930a34.2\n for <dev@openvswitch.org>; Fri, 08 May 2026 06:43:34 -0700 (PDT)", "from houminxi ([72.244.37.221]) by smtp.gmail.com with ESMTPSA id\n 006d021491bc7-69b25c76cc4sm1119108eaf.6.2026.05.08.06.43.28\n (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256);\n Fri, 08 May 2026 06:43:32 -0700 (PDT)" ], "X-Virus-Scanned": [ "amavis at osuosl.org", "amavis at osuosl.org" ], "X-Comment": "SPF check N/A for local connections -\n client-ip=2605:bc80:3010:104::8cd3:938; helo=lists.linuxfoundation.org;\n envelope-from=ovs-dev-bounces@openvswitch.org; receiver=<UNKNOWN> ", "DKIM-Filter": [ "OpenDKIM Filter v2.11.0 smtp4.osuosl.org 2BD5C41BC4", "OpenDKIM Filter v2.11.0 smtp1.osuosl.org E0FE9842C3" ], "Received-SPF": "Pass (mailfrom) identity=mailfrom;\n client-ip=2607:f8b0:4864:20::343; helo=mail-ot1-x343.google.com;\n envelope-from=houminxi@gmail.com; receiver=<UNKNOWN>", "DMARC-Filter": "OpenDMARC Filter v1.4.2 smtp1.osuosl.org E0FE9842C3", "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=gmail.com; s=20251104; t=1778247813; x=1778852613; darn=openvswitch.org;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:cc:to:from:from:to:cc:subject:date\n :message-id:reply-to;\n bh=+k8jCx+4owYWbxzMTzfLIwLSG5wN8p2PhTA/UFqYTeI=;\n b=bUY2PYx3Dv4yazKSoPDdITjOqMn7UvxlaQ/coLrgLeUTawOTOflrk5hO9CHjw8HcT+\n f6TJ9ECnA41loKQAUD3zQuFGH5l4lPxzPoRKvnuTg8bXlxix60XBqaq6DgjmEDZ/3oOs\n pzLHNs9V/EHgJX5NV/gQu9dZoPCpyyWHLdxESUTss0MrBnEft+mu0LiPK5Tc1emWSBor\n f6bIAFeMvl2NiAcCjlqb352HTMwY9k7t8OumAOs4PGFGs828hbh+35KdEgRn9sepxN/+\n 8Ep1I5uL6QE8Oexr5NccA08tZkWMRcggT9ATJ7eH2Edb6NtnW8IzhIfUXCeBdDuGaEp6\n NHUg==", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20251104; t=1778247813; x=1778852613;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from\n :to:cc:subject:date:message-id:reply-to;\n bh=+k8jCx+4owYWbxzMTzfLIwLSG5wN8p2PhTA/UFqYTeI=;\n b=WcQJ19D1kBnImklrb4X1+DK4lsWhLKOuKAAkJKmDV1BThAiKllCySf756KACUX2BuT\n IvR0BjFXeV6Ujsfg5zoRTFqweAR/94PqqpXv/Wp4JlILk7Gc5Wob4urhc9NNcBNv9rZD\n IjzfrVVTm7rmOVO0qRFzStbwaR+Aoh1yDSxMweSaHki2yW2H4ak4Ul9sH6FWVbV3VbmD\n e5NVfSpykT/9wiM+/d/h0DJFotGLRMk3TRYL3cUHgjHO2fDOL8jBkgJPuQHvSa2GMLj9\n qfUPnw9lSS5Lss8XRI+XL+b1oN9VG0ORpytzmhtsyBYGPHDCxulxMH+z+LcCCcp8iw0G\n vJFg==", "X-Forwarded-Encrypted": "i=1;\n AFNElJ/rtCuLieYS/iif2q67q5+TItoo3hE3nOUNM4URT728nalQGYG850W4hb2FehrLWxAGhRU=@openvswitch.org", "X-Gm-Message-State": "AOJu0Yx4DPv1qeKIQeROOoUVMwZiyu1QaaZOLEX3aJ9tBfcDUY7HI1rs\n kSjCog2Y8O4pUMX6M6XBvP14MYtdCv1mG1nnYRbrN6UVmQ12q1pu6MBK", "X-Gm-Gg": "AeBDievGdHJthmj7/jg2KnU+zWVuHzpSuvQdDwJ6jcJUn34zGwAF+OCMb8rUMBPra/W\n wjSK44UaeNRJmNJ+X43R6VTL4YVLHmL1jQw7GHvOvEK4KWaumeO8gVtP1NiMxHkgUYFYg5x8IRH\n GvIOfvbIvtz43TgZN/6fVn2YvprM1PjkHMiyjKpZaPuBqlbI570MMgYI/x66zNhaUa7qHKbUXdC\n iJmkTkftyja3AZ9tp1J1CIdd8UmRbwyOoOrrJEw9x89woXLJec8KoCc0JfYnlTKZuoexT8Gs40n\n 7RNRunFCzCXy8K+i4dXv4AdiqbVXvM3/f9GZ9+2OC201qZXIGIR7L5lyyNTkNdQY2kXnW0/+XJI\n b7yzDb/2tBaD/ot3SG2InN0FaCL6sKefojhgoi0eO5LFBdBf3rou437MVKzC0k13WLyMwyem9o5\n vB5LSOoe9i4IFlTwU=", "X-Received": "by 2002:a05:6820:81c6:b0:688:7dbe:5ff7 with SMTP id\n 006d021491bc7-69998d6381amr7317612eaf.56.1778247813275;\n Fri, 08 May 2026 06:43:33 -0700 (PDT)", "From": "Minxi Hou <houminxi@gmail.com>", "To": "netdev@vger.kernel.org", "Date": "Fri, 8 May 2026 21:43:07 +0800", "Message-ID": "<20260508134308.2857449-2-houminxi@gmail.com>", "X-Mailer": "git-send-email 2.53.0", "In-Reply-To": "<20260508134308.2857449-1-houminxi@gmail.com>", "References": "<20260508134308.2857449-1-houminxi@gmail.com>", "MIME-Version": "1.0", "Subject": "[ovs-dev] [PATCH net-next v8 1/2] selftests: openvswitch: add\n vlan() and encap() flow string parsing", "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 VLAN TCI formatting and parsing support to ovs-dpctl.py:\n\n- Add _vlan_dpstr() to decompose TCI into vid/pcp/cfi fields,\n with raw tci=0x%04x fallback when cfi=0 for round-trip safety.\n- Add _parse_vlan_from_flowstr() boundary check for missing ')'.\n- Add encap_ovskey subclass restricting nla_map to L2-L4 attributes\n (slots 0-21) that appear inside 802.1Q ENCAP, with metadata\n attributes set to \"none\".\n- Check parse() return value for unrecognized trailing content.\n- Support callable format functions in dpstr() output.\n- Change OVS_KEY_ATTR_VLAN type from uint16 to be16 to match the\n kernel __be16 wire format; uint16 decodes in host byte order,\n which gives wrong values on little-endian architectures.\n- Change OVS_KEY_ATTR_ENCAP type from none to encap_ovskey to\n enable recursive parsing of 802.1Q encapsulated flow keys.\n- Add push_vlan action class with fields matching kernel struct\n ovs_action_push_vlan (vlan_tpid, vlan_tci as network-order u16).\n- Add push_vlan dpstr format and parse with range validation\n (vid 0-4095, pcp 0-7, tpid 0-0xFFFF) and CFI forced to 1.\n- Remove MAX_ENCAP_DEPTH constant and depth tracking -- the\n bracket-depth counter in the encap parser already handles\n nesting; the global depth limit was unnecessary.\n\nSigned-off-by: Minxi Hou <houminxi@gmail.com>\n---\n .../selftests/net/openvswitch/ovs-dpctl.py | 322 +++++++++++++++++-\n 1 file changed, 312 insertions(+), 10 deletions(-)", "diff": "diff --git a/tools/testing/selftests/net/openvswitch/ovs-dpctl.py b/tools/testing/selftests/net/openvswitch/ovs-dpctl.py\nindex 848f61fdcee0..75a6e3861242 100644\n--- a/tools/testing/selftests/net/openvswitch/ovs-dpctl.py\n+++ b/tools/testing/selftests/net/openvswitch/ovs-dpctl.py\n@@ -370,7 +370,7 @@ class ovsactions(nla):\n (\"OVS_ACTION_ATTR_OUTPUT\", \"uint32\"),\n (\"OVS_ACTION_ATTR_USERSPACE\", \"userspace\"),\n (\"OVS_ACTION_ATTR_SET\", \"ovskey\"),\n- (\"OVS_ACTION_ATTR_PUSH_VLAN\", \"none\"),\n+ (\"OVS_ACTION_ATTR_PUSH_VLAN\", \"push_vlan\"),\n (\"OVS_ACTION_ATTR_POP_VLAN\", \"flag\"),\n (\"OVS_ACTION_ATTR_SAMPLE\", \"sample\"),\n (\"OVS_ACTION_ATTR_RECIRC\", \"uint32\"),\n@@ -427,6 +427,9 @@ class ovsactions(nla):\n \n return actstr\n \n+ class push_vlan(nla):\n+ fields = ((\"vlan_tpid\", \"!H\"), (\"vlan_tci\", \"!H\"))\n+\n class sample(nla):\n nla_flags = NLA_F_NESTED\n \n@@ -633,6 +636,14 @@ class ovsactions(nla):\n print_str += \"ct_clear\"\n elif field[0] == \"OVS_ACTION_ATTR_POP_VLAN\":\n print_str += \"pop_vlan\"\n+ elif field[0] == \"OVS_ACTION_ATTR_PUSH_VLAN\":\n+ datum = self.get_attr(field[0])\n+ tpid = datum[\"vlan_tpid\"]\n+ tci = datum[\"vlan_tci\"]\n+ vid = tci & 0x0FFF\n+ pcp = (tci >> 13) & 0x7\n+ print_str += \"push_vlan(vid=%d,pcp=%d\" \\\n+ \",tpid=0x%04x)\" % (vid, pcp, tpid)\n elif field[0] == \"OVS_ACTION_ATTR_POP_ETH\":\n print_str += \"pop_eth\"\n elif field[0] == \"OVS_ACTION_ATTR_POP_NSH\":\n@@ -726,7 +737,57 @@ class ovsactions(nla):\n actstr = actstr[strspn(actstr, \", \") :]\n parsed = True\n \n- if parse_starts_block(actstr, \"clone(\", False):\n+ if parse_starts_block(actstr, \"push_vlan(\", False):\n+ actstr = actstr[len(\"push_vlan(\"):]\n+ vid = 0\n+ pcp = 0\n+ tpid = 0x8100\n+ if \")\" not in actstr:\n+ raise ValueError(\n+ \"push_vlan(): missing ')'\")\n+ paren = actstr.index(\")\")\n+ if not actstr[:paren].strip():\n+ raise ValueError(\"push_vlan(): no fields\")\n+ for kv in actstr[:paren].split(\",\"):\n+ if \"=\" not in kv:\n+ raise ValueError(\n+ \"push_vlan(): bad field '%s'\"\n+ % kv.strip())\n+ k = kv[:kv.index(\"=\")].strip()\n+ v = kv[kv.index(\"=\") + 1:].strip()\n+ if k == \"vid\":\n+ vid = int(v, 0)\n+ if vid < 0 or vid > 0xFFF:\n+ raise ValueError(\n+ \"push_vlan(): vid=%d out of \"\n+ \"range (0-4095)\" % vid)\n+ elif k == \"pcp\":\n+ pcp = int(v, 0)\n+ if pcp < 0 or pcp > 7:\n+ raise ValueError(\n+ \"push_vlan(): pcp=%d out of \"\n+ \"range (0-7)\" % pcp)\n+ elif k == \"tpid\":\n+ tpid = int(v, 0)\n+ if tpid < 0 or tpid > 0xFFFF:\n+ raise ValueError(\n+ \"push_vlan(): tpid=0x%x out \"\n+ \"of range (0-0xffff)\" % tpid)\n+ else:\n+ raise ValueError(\n+ \"push_vlan(): unknown key '%s'\"\n+ % k)\n+ tci = (vid & 0x0FFF) | ((pcp & 0x7) << 13) \\\n+ | 0x1000\n+ pvact = self.push_vlan()\n+ pvact[\"vlan_tpid\"] = tpid\n+ pvact[\"vlan_tci\"] = tci\n+ self[\"attrs\"].append(\n+ [\"OVS_ACTION_ATTR_PUSH_VLAN\", pvact])\n+ actstr = actstr[paren + 1:]\n+ parsed = True\n+\n+ elif parse_starts_block(actstr, \"clone(\", False):\n parencount += 1\n subacts = ovsactions()\n actstr = actstr[len(\"clone(\"):]\n@@ -901,11 +962,11 @@ class ovskey(nla):\n nla_flags = NLA_F_NESTED\n nla_map = (\n (\"OVS_KEY_ATTR_UNSPEC\", \"none\"),\n- (\"OVS_KEY_ATTR_ENCAP\", \"none\"),\n+ (\"OVS_KEY_ATTR_ENCAP\", \"encap_ovskey\"),\n (\"OVS_KEY_ATTR_PRIORITY\", \"uint32\"),\n (\"OVS_KEY_ATTR_IN_PORT\", \"uint32\"),\n (\"OVS_KEY_ATTR_ETHERNET\", \"ethaddr\"),\n- (\"OVS_KEY_ATTR_VLAN\", \"uint16\"),\n+ (\"OVS_KEY_ATTR_VLAN\", \"be16\"),\n (\"OVS_KEY_ATTR_ETHERTYPE\", \"be16\"),\n (\"OVS_KEY_ATTR_IPV4\", \"ovs_key_ipv4\"),\n (\"OVS_KEY_ATTR_IPV6\", \"ovs_key_ipv6\"),\n@@ -1636,6 +1697,194 @@ class ovskey(nla):\n class ovs_key_mpls(nla):\n fields = ((\"lse\", \">I\"),)\n \n+ # 802.1Q CFI (Canonical Format Indicator) bit, always set for Ethernet\n+ _VLAN_CFI_MASK = 0x1000\n+\n+ @staticmethod\n+ def _vlan_dpstr(tci):\n+ \"\"\"Format VLAN TCI as vid=X,pcp=Y,cfi=Z or tci=0xNNNN.\n+\n+ When cfi=1 (standard Ethernet VLAN), outputs decomposed\n+ vid/pcp/cfi fields. When cfi=0 (truncated VLAN header),\n+ falls back to raw tci=0x%04x to ensure round-trip\n+ correctness: the parser auto-adds cfi=1 for vid/pcp\n+ format, so cfi=0 would be lost on re-parse.\"\"\"\n+ vid = tci & 0x0FFF\n+ pcp = (tci >> 13) & 0x7\n+ cfi = (tci >> 12) & 0x1\n+ if cfi:\n+ return \"vid=%d,pcp=%d,cfi=%d\" % (vid, pcp, cfi)\n+ return \"tci=0x%04x\" % tci\n+\n+ @staticmethod\n+ def _parse_vlan_from_flowstr(flowstr):\n+ \"\"\"Parse vlan(tci=X) or vlan(vid=X[,pcp=Y,cfi=Z]) from flowstr.\n+\n+ Returns (remaining_flowstr, key_tci, mask_tci).\n+ TCI values use standard bit layout (VID bits 0-11,\n+ CFI bit 12, PCP bits 13-15); byte order conversion to\n+ big-endian happens in pyroute2 be16 NLA serialization.\n+ The mask covers only the fields the caller specified:\n+ vid -> 0x0FFF, pcp -> 0xE000, cfi -> 0x1000, tci -> 0xFFFF.\n+\n+ The tci= key sets the raw TCI bitfield (no CFI validation) to allow\n+ non-Ethernet use cases. Use cfi=1 for standard Ethernet VLAN matching.\n+ \"\"\"\n+ tci = 0\n+ mask = 0\n+ has_tci = False\n+ has_vid = has_pcp = has_cfi = False\n+ _tci_mix_err = \"vlan(): 'tci' cannot be mixed \" \\\n+ \"with 'vid'/'pcp'/'cfi'\"\n+ first = True\n+ while True:\n+ flowstr = flowstr.lstrip()\n+ if not flowstr:\n+ raise ValueError(\"vlan(): missing ')'\")\n+ if flowstr[0] == ')':\n+ break\n+ if not first:\n+ flowstr = flowstr[1:] # skip ','\n+ if not flowstr:\n+ raise ValueError(\"vlan(): missing ')' after trailing comma\")\n+ flowstr = flowstr.lstrip()\n+ if flowstr and flowstr[0] == ')':\n+ break\n+ if flowstr and flowstr[0] == ',':\n+ raise ValueError(\n+ \"vlan(): empty or extra comma in field list\")\n+ first = False\n+\n+ eq = flowstr.find('=')\n+ if eq == -1:\n+ raise ValueError(\n+ \"vlan(): expected key=value, got '%s'\" % flowstr)\n+ key = flowstr[:eq].strip()\n+ flowstr = flowstr[eq + 1:]\n+\n+ end = flowstr.find(',')\n+ end2 = flowstr.find(')')\n+ if end == -1 and end2 == -1:\n+ raise ValueError(\"vlan(): missing ')'\")\n+ if end == -1 or (end2 != -1 and end2 < end):\n+ end = end2\n+ val = flowstr[:end].strip()\n+ flowstr = flowstr[end:]\n+\n+ if not val:\n+ raise ValueError(\"vlan(): empty value for key '%s'\" % key)\n+ try:\n+ v = int(val, 16) if val.startswith(('0x', '0X')) else int(val)\n+ except ValueError as exc:\n+ raise ValueError(\n+ \"vlan(): invalid value '%s' for key '%s'\"\n+ % (val, key)) from exc\n+\n+ if key == 'tci':\n+ if has_tci:\n+ raise ValueError(\"vlan(): duplicate 'tci'\")\n+ if has_vid or has_pcp or has_cfi:\n+ raise ValueError(_tci_mix_err)\n+ if v > 0xFFFF or v < 0:\n+ raise ValueError(\"vlan(): tci=0x%x out of range\" % v)\n+ tci = v\n+ mask = 0xFFFF\n+ has_tci = True\n+ elif key == 'vid':\n+ if has_tci:\n+ raise ValueError(_tci_mix_err)\n+ if has_vid:\n+ raise ValueError(\"vlan(): duplicate 'vid'\")\n+ if v < 0 or v > 0xFFF:\n+ raise ValueError(\"vlan(): vid=%d out of range (0-4095)\" % v)\n+ tci |= v\n+ mask |= 0x0FFF\n+ has_vid = True\n+ elif key == 'pcp':\n+ if has_tci:\n+ raise ValueError(_tci_mix_err)\n+ if has_pcp:\n+ raise ValueError(\"vlan(): duplicate 'pcp'\")\n+ if v < 0 or v > 7:\n+ raise ValueError(\"vlan(): pcp=%d out of range (0-7)\" % v)\n+ tci |= (v & 0x7) << 13\n+ mask |= 0xE000\n+ has_pcp = True\n+ elif key == 'cfi':\n+ if has_tci:\n+ raise ValueError(_tci_mix_err)\n+ if has_cfi:\n+ raise ValueError(\"vlan(): duplicate 'cfi'\")\n+ if v != 1:\n+ raise ValueError(\"vlan(): cfi must be 1 for Ethernet\")\n+ tci |= ovskey._VLAN_CFI_MASK\n+ mask |= ovskey._VLAN_CFI_MASK\n+ has_cfi = True\n+ else:\n+ raise ValueError(\"vlan(): unknown key '%s'\" % key)\n+\n+ flowstr = flowstr[1:] # skip ')'\n+ # Catch immediate '))' (user error). A ')' after ',' is consumed\n+ # by parse()'s strspn(flowstr, \"), \") inter-field separator stripping.\n+ if flowstr.lstrip().startswith(')'):\n+ raise ValueError(\"vlan(): unmatched ')'\")\n+ # parse() strips trailing ',', ')', ' ' as inter-field separators,\n+ # so we do not need to call strspn here.\n+\n+ if mask == 0:\n+ raise ValueError(\"vlan(): no fields specified, \"\n+ \"use vlan(vid=X[,pcp=Y,cfi=Z]) or vlan(tci=X)\")\n+ if not has_tci:\n+ tci |= ovskey._VLAN_CFI_MASK\n+ mask |= ovskey._VLAN_CFI_MASK\n+ return flowstr, tci, mask\n+\n+ @staticmethod\n+ def _parse_encap_from_flowstr(flowstr):\n+ \"\"\"Parse encap(inner_flow) from flowstr.\n+\n+ Returns (remaining_flowstr, inner_key_dict, inner_mask_dict)\n+ where each dict has an 'attrs' key for recursive NLA encoding.\n+ Parenthesis-depth tracking handles nested encap() calls but not\n+ quoted strings containing literal parentheses.\n+ \"\"\"\n+ depth = 1\n+ end = -1\n+ for i, c in enumerate(flowstr):\n+ if c == '(':\n+ depth += 1\n+ elif c == ')':\n+ depth -= 1\n+ if depth < 0:\n+ raise ValueError(\n+ \"encap(): unmatched ')' at position %d\" % i)\n+ if depth == 0:\n+ end = i\n+ break\n+\n+ if end == -1:\n+ if depth > 1:\n+ raise ValueError(\"encap(): missing ')' in nested encap\")\n+ raise ValueError(\"encap(): missing ')'\")\n+\n+ inner_str = flowstr[:end].strip()\n+ if not inner_str:\n+ raise ValueError(\"encap(): empty inner flow\")\n+\n+ flowstr = flowstr[end + 1:]\n+ if flowstr.lstrip().startswith(')'):\n+ raise ValueError(\"encap(): unmatched ')' after encap()\")\n+\n+ inner_key = encap_ovskey()\n+ inner_mask = encap_ovskey()\n+ remaining = inner_key.parse(inner_str, inner_mask)\n+ if remaining and re.search(r'[^\\s,)]', remaining):\n+ raise ValueError(\n+ \"encap(): unrecognized trailing \"\n+ \"content '%s'\" % remaining.strip())\n+\n+ return flowstr, inner_key, inner_mask\n+\n def parse(self, flowstr, mask=None):\n for field in (\n (\"OVS_KEY_ATTR_PRIORITY\", \"skb_priority\", intparse),\n@@ -1657,6 +1906,16 @@ class ovskey(nla):\n \"eth_type\",\n lambda x: intparse(x, \"0xffff\"),\n ),\n+ (\n+ \"OVS_KEY_ATTR_VLAN\",\n+ \"vlan\",\n+ ovskey._parse_vlan_from_flowstr,\n+ ),\n+ (\n+ \"OVS_KEY_ATTR_ENCAP\",\n+ \"encap\",\n+ ovskey._parse_encap_from_flowstr,\n+ ),\n (\n \"OVS_KEY_ATTR_IPV4\",\n \"ipv4\",\n@@ -1794,6 +2053,9 @@ class ovskey(nla):\n True,\n ),\n (\"OVS_KEY_ATTR_ETHERNET\", None, None, False, False),\n+ (\"OVS_KEY_ATTR_VLAN\", \"vlan\", ovskey._vlan_dpstr,\n+ lambda x: False, True),\n+ (\"OVS_KEY_ATTR_ENCAP\", None, None, False, False),\n (\n \"OVS_KEY_ATTR_ETHERTYPE\",\n \"eth_type\",\n@@ -1821,22 +2083,61 @@ class ovskey(nla):\n v = self.get_attr(field[0])\n if v is not None:\n m = None if mask is None else mask.get_attr(field[0])\n+ fmt = field[2] # str format or callable\n if field[4] is False:\n print_str += v.dpstr(m, more)\n print_str += \",\"\n else:\n if m is None or field[3](m):\n- print_str += field[1] + \"(\"\n- print_str += field[2] % v\n- print_str += \"),\"\n+ val = fmt(v) if callable(fmt) else fmt % v\n+ print_str += field[1] + \"(\" + val + \"),\"\n elif more or m != 0:\n- print_str += field[1] + \"(\"\n- print_str += (field[2] % v) + \"/\" + (field[2] % m)\n- print_str += \"),\"\n+ if callable(fmt):\n+ val = fmt(v) + \"/\" + fmt(m)\n+ else:\n+ val = (fmt % v) + \"/\" + (fmt % m)\n+ print_str += field[1] + \"(\" + val + \"),\"\n \n return print_str\n \n \n+class encap_ovskey(ovskey):\n+ \"\"\"Inner flow key attributes valid inside 802.1Q ENCAP.\n+\n+ Only L2-L4 key attributes (slots 0-21) appear inside ENCAP.\n+ Metadata-only attributes (SKB_MARK, DP_HASH, RECIRC_ID, etc.)\n+ are set to \"none\" -- they never appear inside ENCAP per\n+ ovs_nla_put_vlan() in net/openvswitch/flow_netlink.c.\n+\n+ nla_map indexes must match OVS_KEY_ATTR_* enum values in\n+ include/uapi/linux/openvswitch.h.\n+ \"\"\"\n+ nla_map = (\n+ (\"OVS_KEY_ATTR_UNSPEC\", \"none\"),\n+ (\"OVS_KEY_ATTR_ENCAP\", \"none\"), # placeholder, parsed by ovskey\n+ (\"OVS_KEY_ATTR_PRIORITY\", \"none\"), # skb metadata, not in ENCAP\n+ (\"OVS_KEY_ATTR_IN_PORT\", \"none\"), # skb metadata, not in ENCAP\n+ (\"OVS_KEY_ATTR_ETHERNET\", \"ethaddr\"),\n+ (\"OVS_KEY_ATTR_VLAN\", \"be16\"),\n+ (\"OVS_KEY_ATTR_ETHERTYPE\", \"be16\"),\n+ (\"OVS_KEY_ATTR_IPV4\", \"ovs_key_ipv4\"),\n+ (\"OVS_KEY_ATTR_IPV6\", \"ovs_key_ipv6\"),\n+ (\"OVS_KEY_ATTR_TCP\", \"ovs_key_tcp\"),\n+ (\"OVS_KEY_ATTR_UDP\", \"ovs_key_udp\"),\n+ (\"OVS_KEY_ATTR_ICMP\", \"ovs_key_icmp\"),\n+ (\"OVS_KEY_ATTR_ICMPV6\", \"ovs_key_icmpv6\"),\n+ (\"OVS_KEY_ATTR_ARP\", \"ovs_key_arp\"),\n+ (\"OVS_KEY_ATTR_ND\", \"ovs_key_nd\"),\n+ (\"OVS_KEY_ATTR_SKB_MARK\", \"none\"), # metadata, not in ENCAP\n+ (\"OVS_KEY_ATTR_TUNNEL\", \"none\"), # tunnel metadata, not in ENCAP\n+ (\"OVS_KEY_ATTR_SCTP\", \"ovs_key_sctp\"),\n+ (\"OVS_KEY_ATTR_TCP_FLAGS\", \"be16\"),\n+ (\"OVS_KEY_ATTR_DP_HASH\", \"none\"), # metadata, not in ENCAP\n+ (\"OVS_KEY_ATTR_RECIRC_ID\", \"none\"), # metadata, not in ENCAP\n+ (\"OVS_KEY_ATTR_MPLS\", \"array(ovs_key_mpls)\"),\n+ )\n+\n+\n class OvsPacket(GenericNetlinkSocket):\n OVS_PACKET_CMD_MISS = 1 # Flow table miss\n OVS_PACKET_CMD_ACTION = 2 # USERSPACE action\n@@ -2576,6 +2877,7 @@ def print_ovsdp_full(dp_lookup_rep, ifindex, ndb=NDB(), vpl=OvsVport()):\n \n \n def main(argv):\n+ nlmsg_atoms.encap_ovskey = encap_ovskey\n nlmsg_atoms.ovskey = ovskey\n nlmsg_atoms.ovsactions = ovsactions\n \n", "prefixes": [ "ovs-dev", "net-next", "v8", "1/2" ] }