From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by smtp.lore.kernel.org (Postfix) with ESMTP id 1CFC5C5DF74 for ; Tue, 18 Aug 2026 04:58:34 +0000 (UTC) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 09AD840294; Tue, 18 Aug 2026 06:58:34 +0200 (CEST) Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.15]) by mails.dpdk.org (Postfix) with ESMTP id F10D54028E for ; Tue, 18 Aug 2026 06:58:32 +0200 (CEST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1787029113; x=1818565113; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=I+WUzz6TYdb7Vy9/ftC399ga5P0cKDo4SKP7Fb9XRUM=; b=GjJ90ciLqUxW48ZDtbPd3cD7YphBTidpY7QWWpeHehIPiIaIyLOM2W95 oKgpDG8/kQCKG8/lCYRiSQe3+q0aOaFjRMjxqKu3EZDFwXtauVAznhoeG ljBKzkUM+8XMJfxZWEM1143oRNIRtH2Mmij6uxIQOt+DIgbwbaAmJYL2J /AcTRhrkDU+gWHh+8VgSNXKlwH4R9JGn2+tykU0aBu53cHTT5NRfZCUlI pbkdCg8R/P2rSRl0P9x+2drblXPWF2n1jHjwZEYTe8k080fjlGvIjzxOz r/0XH+dcsRFzI8VgR1rpF9h06GtFo3CNw8g5Kg0P6y++S944fxtJzRXWa g==; X-CSE-ConnectionGUID: /UN1fcgVQiOVhsnNsxDWaQ== X-CSE-MsgGUID: QEpai1rQSdSqJqGz3QhP7Q== X-IronPort-AV: E=McAfee;i="6800,10657,11878"; a="87632994" X-IronPort-AV: E=Sophos;i="6.25,230,1779174000"; d="scan'208";a="87632994" Received: from orviesa003.jf.intel.com ([10.64.159.143]) by fmvoesa109.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 17 Aug 2026 21:58:32 -0700 X-CSE-ConnectionGUID: XKJeI2mZSI2PLhSz0UVCfA== X-CSE-MsgGUID: zqQtlE+2QZCxrsMmSYZanw== X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.25,230,1779174000"; d="scan'208";a="268611241" Received: from pae-14.iind.intel.com ([10.190.203.153]) by orviesa003.jf.intel.com with ESMTP; 17 Aug 2026 21:58:30 -0700 From: Anurag Mandal To: dev@dpdk.org Cc: bruce.richardson@intel.com, vladimir.medvedkin@intel.com, ciara.loftus@intel.com, Anurag Mandal Subject: [PATCH v4 0/7] net/iavf: harden reset recovery and data path on link flap Date: Tue, 18 Aug 2026 04:57:25 +0000 Message-Id: X-Mailer: git-send-email 2.34.1 In-Reply-To: References: 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 List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org V4-changes: - Addressed Bruce Richardson's review comment on Patch-3 - Removed Tx drain on link down V3-changes: - Addressed Ciara Loftus's review comments on Fixes tag - Added missing watchdog enable in Patch-6 - Removed extra space in commit msg of Patch-5 V2-changes: - Addressed Ciara Loftus's review comments. - Split Watchdog and flow flush from other patches. - Shortened multiple comments and commit msg - Prevented iavf_resume_pending_start() looping During PF-initiated reset or a remote/ToR switch link-flap, the VF might miss the reset event, race on the no_poll gate, leak in-flight Tx descriptors, and stay down if dev_start ran before the PF VSI was ready. This patch series build on the earlier reset-recovery fixes with the following: - Reset detection: complement the ARQLEN1 check with VFGEN_RSTAT (VIRTCHNL_VFR_INPROGRESS) and poll at a 5 ms interval, matching the kernel iavf driver, so fast ARQ flips are not missed. When the VFR is still not observed, recover anyway instead of bailing out, keeping PF and VF state in sync. - no_poll: make the flag atomic (RTE_ATOMIC) with release/acquire ordering so the data-plane lcores observe gate changes reliably. - Tx drain: add iavf_dev_tx_drain() to flush in-flight Tx descriptors on impending-reset events before teardown, preventing MDD events and descriptor leaks. - Deferred start: when dev_start fails during recovery (PF VSI inactive), defer it via start_pending and resume on the next link-up event so the VF comes back without manual intervention. - AdminQ: discard zeroed (opcode 0) descriptors seen during PF-initiated resets to avoid the "Request 0 is not supported" log flood. - Watchdog: keep watchdog armed for the whole reset window - Flow flush: skip flow flush during PF-initiated reset Anurag Mandal (7): net/iavf: discard empty AdminQ descriptors on reset net/iavf: defer device start when PF VSI not ready net/iavf: drain in-flight Tx before reset net/iavf: change no_poll flag to atomic net/iavf: improve VF reset detection on fast ARQ flip net/iavf: keep watchdog armed for the whole reset window net/iavf: skip flow flush during PF-initiated reset drivers/net/intel/iavf/iavf.h | 5 +- drivers/net/intel/iavf/iavf_ethdev.c | 108 +++++++++++++++++++++++---- drivers/net/intel/iavf/iavf_rxtx.c | 108 ++++++++++++++++++++++++++- drivers/net/intel/iavf/iavf_rxtx.h | 6 ++ drivers/net/intel/iavf/iavf_vchnl.c | 20 ++++- 5 files changed, 227 insertions(+), 20 deletions(-) -- 2.34.1