From patchwork Thu Feb 16 11:09:13 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Bruce Richardson X-Patchwork-Id: 124061 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 D948641CB1; Thu, 16 Feb 2023 12:09:33 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id C181240EE3; Thu, 16 Feb 2023 12:09:33 +0100 (CET) Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by mails.dpdk.org (Postfix) with ESMTP id 8B40040E0F for ; Thu, 16 Feb 2023 12:09:31 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1676545771; x=1708081771; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=EEYzi+Wq3zNtfeasY5sc+3swznIQLopm5h0cC2I4ctk=; b=l8+BzHGpdEEenHzcYiupadD1sz9onZiyuuAj4jnhXx9VMzvptaVmoMy/ YH34pRSo93GpGfaHQGQf5d4A3e8TSWquBivzGMblKK2wkI/oVGQgSkTN/ gpiTHurUAX7ntq1ew1+tJ0gOR9IukwUSZQ09IxbTj8JayyBmMDyLyz1WT EIL7hIyyq/hSFbWSluwfmcwlK60g39IDEQzBwdFND9obLXAKgRfiGWk6P Q6Uq7cQQT4cc8M8Uz1XHszcUf812uLgSeozq5clRtnSa9CjvsaXUZas67 rCwlEpJuGDl2vUyO46xM0wNxAhmOB5axtFIWh4my0wK1SryMky9EG/WYY w==; X-IronPort-AV: E=McAfee;i="6500,9779,10622"; a="315368137" X-IronPort-AV: E=Sophos;i="5.97,302,1669104000"; d="scan'208";a="315368137" Received: from orsmga001.jf.intel.com ([10.7.209.18]) by orsmga106.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 16 Feb 2023 03:09:26 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6500,9779,10622"; a="702522792" X-IronPort-AV: E=Sophos;i="5.97,302,1669104000"; d="scan'208";a="702522792" Received: from silpixa00401385.ir.intel.com ([10.237.214.22]) by orsmga001.jf.intel.com with ESMTP; 16 Feb 2023 03:09:25 -0800 From: Bruce Richardson To: dev@dpdk.org Cc: fengchengwen@huawei.com, Bruce Richardson Subject: [PATCH v3 0/6] dma/ioat: fix issues with stopping and restarting device Date: Thu, 16 Feb 2023 11:09:13 +0000 Message-Id: <20230216110919.373385-1-bruce.richardson@intel.com> X-Mailer: git-send-email 2.37.2 In-Reply-To: <20230116153714.554470-1-bruce.richardson@intel.com> References: <20230116153714.554470-1-bruce.richardson@intel.com> 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 This patchset fixes a couple of problems with stopping and restarting an ioat DMA device. Following the three fixes, a series of improvements are made to the dmadev unit tests to properly validate that dmadevs work correctly as they are started and stopped, and ensure that no other or future drivers will suffer from issues. v3: * remove unnecessary mempool free on error (patch 4), as noted by Chengwen v2: * extra patch to fix issues with error reporting, as noted by Conor W. Bruce Richardson (6): dma/ioat: fix device stop if no copies done dma/ioat: fix incorrectly set indexes after restart dma/ioat: fix incorrect error reporting on restart test/dmadev: check result for device stop test/dmadev: create separate function for single copy test test/dmadev: add tests for stopping and restarting dev app/test/test_dmadev.c | 171 ++++++++++++++++++++++----------- drivers/dma/ioat/ioat_dmadev.c | 31 ++++-- 2 files changed, 139 insertions(+), 63 deletions(-) --- 2.37.2