Patch Detail
get:
Show a patch.
patch:
Update a patch.
put:
Update a patch.
GET /api/patches/134182/?format=api
{ "id": 134182, "url": "http://patches.dpdk.org/api/patches/134182/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/patch/20231113202833.12900-6-jspewock@iol.unh.edu/", "project": { "id": 1, "url": "http://patches.dpdk.org/api/projects/1/?format=api", "name": "DPDK", "link_name": "dpdk", "list_id": "dev.dpdk.org", "list_email": "dev@dpdk.org", "web_url": "http://core.dpdk.org", "scm_url": "git://dpdk.org/dpdk", "webscm_url": "http://git.dpdk.org/dpdk", "list_archive_url": "https://inbox.dpdk.org/dev", "list_archive_url_format": "https://inbox.dpdk.org/dev/{}", "commit_url_format": "" }, "msgid": "<20231113202833.12900-6-jspewock@iol.unh.edu>", "list_archive_url": "https://inbox.dpdk.org/dev/20231113202833.12900-6-jspewock@iol.unh.edu", "date": "2023-11-13T20:28:10", "name": "[v3,5/7] dts: add optional packet filtering to scapy sniffer", "commit_ref": null, "pull_url": null, "state": "superseded", "archived": true, "hash": "3bc62b5f2a690198626aefaeb6d609b47abedfe9", "submitter": { "id": 2772, "url": "http://patches.dpdk.org/api/people/2772/?format=api", "name": "Jeremy Spewock", "email": "jspewock@iol.unh.edu" }, "delegate": { "id": 1, "url": "http://patches.dpdk.org/api/users/1/?format=api", "username": "tmonjalo", "first_name": "Thomas", "last_name": "Monjalon", "email": "thomas@monjalon.net" }, "mbox": "http://patches.dpdk.org/project/dpdk/patch/20231113202833.12900-6-jspewock@iol.unh.edu/mbox/", "series": [ { "id": 30280, "url": "http://patches.dpdk.org/api/series/30280/?format=api", "web_url": "http://patches.dpdk.org/project/dpdk/list/?series=30280", "date": "2023-11-13T20:28:05", "name": "dts: Port scatter suite over", "version": 3, "mbox": "http://patches.dpdk.org/series/30280/mbox/" } ], "comments": "http://patches.dpdk.org/api/patches/134182/comments/", "check": "success", "checks": "http://patches.dpdk.org/api/patches/134182/checks/", "tags": {}, "related": [], "headers": { "Return-Path": "<dev-bounces@dpdk.org>", "X-Original-To": "patchwork@inbox.dpdk.org", "Delivered-To": "patchwork@inbox.dpdk.org", "Received": [ "from mails.dpdk.org (mails.dpdk.org [217.70.189.124])\n\tby inbox.dpdk.org (Postfix) with ESMTP id CFB7A4331E;\n\tMon, 13 Nov 2023 21:29:19 +0100 (CET)", "from mails.dpdk.org (localhost [127.0.0.1])\n\tby mails.dpdk.org (Postfix) with ESMTP id 3BCD140A67;\n\tMon, 13 Nov 2023 21:28:57 +0100 (CET)", "from mail-wm1-f97.google.com (mail-wm1-f97.google.com\n [209.85.128.97]) by mails.dpdk.org (Postfix) with ESMTP id 8C40D402E6\n for <dev@dpdk.org>; Mon, 13 Nov 2023 21:28:53 +0100 (CET)", "by mail-wm1-f97.google.com with SMTP id\n 5b1f17b1804b1-40839807e82so28613005e9.0\n for <dev@dpdk.org>; Mon, 13 Nov 2023 12:28:53 -0800 (PST)", "from postal.iol.unh.edu (postal.iol.unh.edu.\n [2606:4100:3880:1234::84]) by smtp-relay.gmail.com with ESMTPS id\n r22-20020a05600c159600b0040654dc5913sm5582331wmf.26.2023.11.13.12.28.52\n (version=TLS1_2 cipher=ECDHE-ECDSA-AES128-GCM-SHA256 bits=128/128);\n Mon, 13 Nov 2023 12:28:53 -0800 (PST)", "from iol.unh.edu (unknown\n [IPv6:2606:4100:3880:1271:90f9:1b64:f6e6:867f])\n by postal.iol.unh.edu (Postfix) with ESMTP id DBAA4605C345;\n Mon, 13 Nov 2023 15:28:51 -0500 (EST)" ], "DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=iol.unh.edu; s=unh-iol; t=1699907333; x=1700512133; darn=dpdk.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=hdJVf3ypWMnb9Qt4IiEySc5/AHylOmI4+9ZLJPRV+8A=;\n b=CzEyDqBfZrtprRxg4XcGvjei2mFp+W9dgogniWGJ4Uz0DsRCKSE7ZuBK19CgK2Pxtr\n oXemV9uWHH7jj4x5jNNK3vN0oL/974o3AF+339ZxyiQ0AmLF1z3I9n8p7CwGsVBMlswI\n 3PzBVfrUeH9qfi2VSRkQJHskMLqni5KhPqOlk=", "X-Google-DKIM-Signature": "v=1; a=rsa-sha256; c=relaxed/relaxed;\n d=1e100.net; s=20230601; t=1699907333; x=1700512133;\n h=content-transfer-encoding:mime-version:references:in-reply-to\n :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc\n :subject:date:message-id:reply-to;\n bh=hdJVf3ypWMnb9Qt4IiEySc5/AHylOmI4+9ZLJPRV+8A=;\n b=LXxfqkeHVRtklB+6iXpIgDkWIlpMhwby1s909dBmMtk/o97pxXiEYN9cVTnhr+1eOh\n 4RoyG016z0RtwZG6d5m9bpIl2zqZoySgwfU1zrwMfOjgY0cUzmYf8gn91XHAe2pHrhrJ\n 2BZd32CiF9PvidEjM7lVgRe6wFH1SqqCpTChmMy5aS1h4ljszVjSDrr426QIpCuR/SAk\n yeqeYrLlRFVCa4HKllW4Tc2VQCKTkIhIdeYnMSLDH20OpntV80GiNAQd6lDrSiuAd0KR\n HJ7YMwhSMVO8BGtKugRluqARbWcGJQTj2kPL2H8OtMR6ZhS2wNMufCi2riJq5ztk9K1B\n QVVQ==", "X-Gm-Message-State": "AOJu0YzrCt5cMaAxTcA/odGIuLazZBY3OpcUQ1uKoCFDVGxI8KsezYNJ\n 2MZjSnl9VMzBnEhzfb5CkM0vE32zGi2s7+gQIYRV8U0r81Fv2sn6eoDfczKm1KShjQ==", "X-Google-Smtp-Source": "\n AGHT+IE2EelwBhJOpnr8+luaWmXM4uqOULMXNlbIgr3aMUzPhqXN7Jw5bLIGSU4Qg2HoD6oKbx+27NpemTdS", "X-Received": "by 2002:a05:600c:5025:b0:3fe:d67d:5040 with SMTP id\n n37-20020a05600c502500b003fed67d5040mr465943wmr.5.1699907333126;\n Mon, 13 Nov 2023 12:28:53 -0800 (PST)", "X-Relaying-Domain": "iol.unh.edu", "From": "jspewock@iol.unh.edu", "To": "Honnappa.Nagarahalli@arm.com, juraj.linkes@pantheon.tech,\n thomas@monjalon.net, wathsala.vithanage@arm.com, probb@iol.unh.edu,\n paul.szczepanek@arm.com, yoan.picchi@foss.arm.com, ferruh.yigit@amd.com,\n andrew.rybchenko@oktetlabs.ru", "Cc": "dev@dpdk.org,\n\tJeremy Spewock <jspewock@iol.unh.edu>", "Subject": "[PATCH v3 5/7] dts: add optional packet filtering to scapy sniffer", "Date": "Mon, 13 Nov 2023 15:28:10 -0500", "Message-ID": "<20231113202833.12900-6-jspewock@iol.unh.edu>", "X-Mailer": "git-send-email 2.42.0", "In-Reply-To": "<20231113202833.12900-1-jspewock@iol.unh.edu>", "References": "<20231113202833.12900-1-jspewock@iol.unh.edu>", "MIME-Version": "1.0", "Content-Transfer-Encoding": "8bit", "X-BeenThere": "dev@dpdk.org", "X-Mailman-Version": "2.1.29", "Precedence": "list", "List-Id": "DPDK patches and discussions <dev.dpdk.org>", "List-Unsubscribe": "<https://mails.dpdk.org/options/dev>,\n <mailto:dev-request@dpdk.org?subject=unsubscribe>", "List-Archive": "<http://mails.dpdk.org/archives/dev/>", "List-Post": "<mailto:dev@dpdk.org>", "List-Help": "<mailto:dev-request@dpdk.org?subject=help>", "List-Subscribe": "<https://mails.dpdk.org/listinfo/dev>,\n <mailto:dev-request@dpdk.org?subject=subscribe>", "Errors-To": "dev-bounces@dpdk.org" }, "content": "From: Jeremy Spewock <jspewock@iol.unh.edu>\n\nAdded the options to filter out LLDP and ARP packets when\nsniffing for packets with scapy. This was done using BPF filters to\nensure that the noise these packets provide does not interfere with test\ncases.\n\nSigned-off-by: Jeremy Spewock <jspewock@iol.unh.edu>\n---\n dts/framework/test_suite.py | 13 +++++++++++--\n .../testbed_model/capturing_traffic_generator.py | 12 +++++++++++-\n dts/framework/testbed_model/scapy.py | 11 ++++++++++-\n dts/framework/testbed_model/tg_node.py | 4 +++-\n 4 files changed, 35 insertions(+), 5 deletions(-)", "diff": "diff --git a/dts/framework/test_suite.py b/dts/framework/test_suite.py\nindex 3b890c0451..3222f172f3 100644\n--- a/dts/framework/test_suite.py\n+++ b/dts/framework/test_suite.py\n@@ -152,7 +152,11 @@ def _configure_ipv4_forwarding(self, enable: bool) -> None:\n self.sut_node.configure_ipv4_forwarding(enable)\n \n def send_packet_and_capture(\n- self, packet: Packet, duration: float = 1\n+ self,\n+ packet: Packet,\n+ duration: float = 1,\n+ no_lldp: bool = True,\n+ no_arp: bool = True,\n ) -> list[Packet]:\n \"\"\"\n Send a packet through the appropriate interface and\n@@ -162,7 +166,12 @@ def send_packet_and_capture(\n \"\"\"\n packet = self._adjust_addresses(packet)\n return self.tg_node.send_packet_and_capture(\n- packet, self._tg_port_egress, self._tg_port_ingress, duration\n+ packet,\n+ self._tg_port_egress,\n+ self._tg_port_ingress,\n+ duration,\n+ no_lldp,\n+ no_arp,\n )\n \n def get_expected_packet(self, packet: Packet) -> Packet:\ndiff --git a/dts/framework/testbed_model/capturing_traffic_generator.py b/dts/framework/testbed_model/capturing_traffic_generator.py\nindex ab98987f8e..0a0d0f7d0d 100644\n--- a/dts/framework/testbed_model/capturing_traffic_generator.py\n+++ b/dts/framework/testbed_model/capturing_traffic_generator.py\n@@ -52,6 +52,8 @@ def send_packet_and_capture(\n send_port: Port,\n receive_port: Port,\n duration: float,\n+ no_lldp: bool,\n+ no_arp: bool,\n capture_name: str = _get_default_capture_name(),\n ) -> list[Packet]:\n \"\"\"Send a packet, return received traffic.\n@@ -71,7 +73,7 @@ def send_packet_and_capture(\n A list of received packets. May be empty if no packets are captured.\n \"\"\"\n return self.send_packets_and_capture(\n- [packet], send_port, receive_port, duration, capture_name\n+ [packet], send_port, receive_port, duration, no_lldp, no_arp, capture_name\n )\n \n def send_packets_and_capture(\n@@ -80,6 +82,8 @@ def send_packets_and_capture(\n send_port: Port,\n receive_port: Port,\n duration: float,\n+ no_lldp: bool,\n+ no_arp: bool,\n capture_name: str = _get_default_capture_name(),\n ) -> list[Packet]:\n \"\"\"Send packets, return received traffic.\n@@ -93,6 +97,8 @@ def send_packets_and_capture(\n send_port: The egress port on the TG node.\n receive_port: The ingress port in the TG node.\n duration: Capture traffic for this amount of time after sending the packets.\n+ no_lldp: Option to disable capturing LLDP packets\n+ no_arp: Option to disable capturing ARP packets\n capture_name: The name of the .pcap file where to store the capture.\n \n Returns:\n@@ -108,6 +114,8 @@ def send_packets_and_capture(\n send_port,\n receive_port,\n duration,\n+ no_lldp,\n+ no_arp,\n )\n \n self._logger.debug(\n@@ -123,6 +131,8 @@ def _send_packets_and_capture(\n send_port: Port,\n receive_port: Port,\n duration: float,\n+ no_lldp: bool,\n+ no_arp: bool,\n ) -> list[Packet]:\n \"\"\"\n The extended classes must implement this method which\ndiff --git a/dts/framework/testbed_model/scapy.py b/dts/framework/testbed_model/scapy.py\nindex af0d4dbb25..58f01af21a 100644\n--- a/dts/framework/testbed_model/scapy.py\n+++ b/dts/framework/testbed_model/scapy.py\n@@ -69,6 +69,7 @@ def scapy_send_packets_and_capture(\n send_iface: str,\n recv_iface: str,\n duration: float,\n+ sniff_filter: str,\n ) -> list[bytes]:\n \"\"\"RPC function to send and capture packets.\n \n@@ -90,6 +91,7 @@ def scapy_send_packets_and_capture(\n iface=recv_iface,\n store=True,\n started_callback=lambda *args: scapy.all.sendp(scapy_packets, iface=send_iface),\n+ filter=sniff_filter,\n )\n sniffer.start()\n time.sleep(duration)\n@@ -264,10 +266,16 @@ def _send_packets_and_capture(\n send_port: Port,\n receive_port: Port,\n duration: float,\n+ no_lldp: bool,\n+ no_arp: bool,\n capture_name: str = _get_default_capture_name(),\n ) -> list[Packet]:\n binary_packets = [packet.build() for packet in packets]\n-\n+ sniff_filter = []\n+ if no_lldp:\n+ sniff_filter.append(\"ether[12:2] != 0x88cc\")\n+ if no_arp:\n+ sniff_filter.append(\"ether[12:2] != 0x0806\")\n xmlrpc_packets: list[\n xmlrpc.client.Binary\n ] = self.rpc_server_proxy.scapy_send_packets_and_capture(\n@@ -275,6 +283,7 @@ def _send_packets_and_capture(\n send_port.logical_name,\n receive_port.logical_name,\n duration,\n+ \" && \".join(sniff_filter),\n ) # type: ignore[assignment]\n \n scapy_packets = [Ether(packet.data) for packet in xmlrpc_packets]\ndiff --git a/dts/framework/testbed_model/tg_node.py b/dts/framework/testbed_model/tg_node.py\nindex 27025cfa31..98e55b7831 100644\n--- a/dts/framework/testbed_model/tg_node.py\n+++ b/dts/framework/testbed_model/tg_node.py\n@@ -56,6 +56,8 @@ def send_packet_and_capture(\n send_port: Port,\n receive_port: Port,\n duration: float = 1,\n+ no_lldp: bool = True,\n+ no_arp: bool = True,\n ) -> list[Packet]:\n \"\"\"Send a packet, return received traffic.\n \n@@ -73,7 +75,7 @@ def send_packet_and_capture(\n A list of received packets. May be empty if no packets are captured.\n \"\"\"\n return self.traffic_generator.send_packet_and_capture(\n- packet, send_port, receive_port, duration\n+ packet, send_port, receive_port, duration, no_lldp, no_arp\n )\n \n def close(self) -> None:\n", "prefixes": [ "v3", "5/7" ] }