From patchwork Tue Apr 23 09:12:48 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Patchwork-Submitter: =?utf-8?q?Juraj_Linke=C5=A1?= X-Patchwork-Id: 886 Return-Path: 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]) by inbox.dpdk.org (Postfix) with ESMTP id 5083143EE8; Tue, 23 Apr 2024 11:12:56 +0200 (CEST) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id D8E6D40E54; Tue, 23 Apr 2024 11:12:55 +0200 (CEST) Received: from mail-ed1-f41.google.com (mail-ed1-f41.google.com [209.85.208.41]) by mails.dpdk.org (Postfix) with ESMTP id C6C1D402AB for ; Tue, 23 Apr 2024 11:12:54 +0200 (CEST) Received: by mail-ed1-f41.google.com with SMTP id 4fb4d7f45d1cf-572040691c9so3866706a12.1 for ; Tue, 23 Apr 2024 02:12:54 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=pantheon.tech; s=google; t=1713863574; x=1714468374; darn=dpdk.org; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:from:to:cc:subject:date:message-id:reply-to; bh=oM/gbQE4irneB0PaPuksfAvU4E8y/0Rqs9K/v5w/1Co=; b=VUvFGoYcVREmxw5DB/GOiVPRWBQUOgYOZlbLtQ/4ZQukGJTI125Z/l63chXs6PudVE 05wgaOSqEGesX3P13ku34cD2jsZfIe5BD51LwbM1ucJ4hZyQSLNZbzYj7FisM6ee3t9M U1i2e7yIBjd1oB2OKnTcdciArSNyDr/VVmcQW43qUIjZiAteherKxy6tGaDlB7ajiS9W zY43kHM145WPwJpjtnRyCGZydVif2eoEx6ULlCeGNFyC8LMELHTmiKV8iA6Cy6hSNOMA bmuq8YssLYxG3cRiTbxi4u1rHEXbGJxNLOH6KAQAuuNcF+kERfoauS/6vvkxzNBLYBt9 0APA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1713863574; x=1714468374; h=content-transfer-encoding:mime-version:message-id:date:subject:cc :to:from:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=oM/gbQE4irneB0PaPuksfAvU4E8y/0Rqs9K/v5w/1Co=; b=iEx5Mh8stxD49qlcIDG79MEAFNkujJWlafOn9InRtJXToZQ8h1S8mreMRtXuOJ6o50 MzqvFDhD8MUUQHUs2xVaEhDs7Cxswwpn6oDxxQUAvsVPWaJUF0lf9D9l3C4dx42POJB7 AveDwNn11CMThjGHy/QA/cmVub+48G4CefYI5nDb5XoHiQn+/Z8XTXLa50A5jLITpnQg AMXjXYYtmxgUMe3xrmw7YxGMSTbOcQWpY4QJ8BklEY7uDT/4kahmJkHhM/2IkmyOGh22 J/Hi0bz/JW51IlGicusgm3M/zkpqaAEPuWyo4k+pqtyKKrz5RxduM2z1M1jYYFteYqw+ PYvA== X-Gm-Message-State: AOJu0YxZRYgDwiMovXx6bbAQOcacQEHXVCOF0/GcZUq4ecI07S79L7bu vmK2WxSwzlbXleH222wFqWcEamCvsFQv3YBnXlhDbm6Vpa8oLk4iFxTJSos6IIo= X-Google-Smtp-Source: AGHT+IE4TcixW72DvtsGiDFeFIvWHwnIyxEhAaTfsC4vAGGMiw+B7q1n14/d0boV77RTweTXCZxRxA== X-Received: by 2002:a50:8e0c:0:b0:571:bf62:81ce with SMTP id 12-20020a508e0c000000b00571bf6281cemr1811165edw.9.1713863574209; Tue, 23 Apr 2024 02:12:54 -0700 (PDT) Received: from jlinkes-PT-Latitude-5530.pantheon.local (81.89.53.154.host.vnet.sk. [81.89.53.154]) by smtp.gmail.com with ESMTPSA id be12-20020a0564021a2c00b0056bf7f92346sm6440534edb.50.2024.04.23.02.12.53 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 23 Apr 2024 02:12:53 -0700 (PDT) From: =?utf-8?q?Juraj_Linke=C5=A1?= To: thomas@monjalon.net, Honnappa.Nagarahalli@arm.com, jspewock@iol.unh.edu, probb@iol.unh.edu, paul.szczepanek@arm.com, Luca.Vizzarro@arm.com, npratte@iol.unh.edu Cc: dev@dpdk.org, =?utf-8?q?Juraj_Linke=C5=A1?= Subject: [PATCH v1 0/4] node and inheritance improvements Date: Tue, 23 Apr 2024 11:12:48 +0200 Message-Id: <20240423091252.62924-1-juraj.linkes@pantheon.tech> X-Mailer: git-send-email 2.34.1 MIME-Version: 1.0 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org There are two areas that are unified: The super() calls, where the arguments were removed as they're not needed in single inheritance. The classes derived from object don't need to explicitly state that. The tg node execution setup and teardown was omitted and is fixed in this series. And probably the most important part if the cleanup of execution and build target setup and teardown. Build targets are relevant only for sut nodes, so it's been moved there, same for vdevs. The execution setup/teardown code was modified so that subclasses use super() to extend the base methods. Juraj Linkeš (4): dts: add tg node execution setup and teardown dts: unify class inheritance from object dts: unify super calls dts: refine pre-test setup and teardown steps dts/framework/remote_session/testpmd_shell.py | 2 +- dts/framework/runner.py | 2 + dts/framework/test_result.py | 16 ++--- dts/framework/test_suite.py | 2 +- dts/framework/testbed_model/cpu.py | 4 +- dts/framework/testbed_model/node.py | 65 ++----------------- dts/framework/testbed_model/sut_node.py | 53 ++++++++++----- dts/framework/testbed_model/tg_node.py | 4 +- dts/framework/testbed_model/virtual_device.py | 2 +- dts/framework/utils.py | 4 +- 10 files changed, 61 insertions(+), 93 deletions(-)