From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f202.google.com (mail-pf1-f202.google.com [209.85.210.202]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 49FB11B4154 for ; Thu, 12 Dec 2024 23:33:58 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.202 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1734046439; cv=none; b=nfYvjszLHY+vIg5UtSkIxBYgtUNWYPxPUrQkMgGmt5wIOxwURVAOckDI/ND/fZeLlfHY/0XCL1t++ZCc4lODlpKrbHRnu+KIvL7mF+MW4D2PHXUoiPcorR6shSyQrU0CE82scPOZU4+Nwgxs8Sg0RX6fzxke5osddtAUWA0elPE= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1734046439; c=relaxed/simple; bh=jEJnjScO/YSbRCtvxKFizyayCq8HRzZLyh/em8BEj5k=; h=Date:Mime-Version:Message-ID:Subject:From:To:Cc:Content-Type; b=c03TveA5LlkuEh/Afs6lDmnYVsJ8xQCNVFN3n60I46VQhNxTIj7cUW2PQwnGykvJ0bg6o/VfqSK9TDyI86WygwdtxMTiUSXLqhoHpvTCIdoggwQR56xOJQxFqIIZr0NScnhhgtrv/kD6j3p2PgENrZlVzgezquIjnD66EsNJj2E= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com; spf=pass smtp.mailfrom=flex--brianvv.bounces.google.com; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b=MyFEk7bZ; arc=none smtp.client-ip=209.85.210.202 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=google.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=flex--brianvv.bounces.google.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=google.com header.i=@google.com header.b="MyFEk7bZ" Received: by mail-pf1-f202.google.com with SMTP id d2e1a72fcca58-7273a901821so975365b3a.2 for ; Thu, 12 Dec 2024 15:33:58 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=20230601; t=1734046437; x=1734651237; darn=vger.kernel.org; h=cc:to:from:subject:message-id:mime-version:date:from:to:cc:subject :date:message-id:reply-to; bh=gLVftiiQq4akEB+OBVvUaxKCKfRMsg/7Sv0m+78/QB4=; b=MyFEk7bZxW1aGPcb7a/WlOl8aQ+tAGvbclQ4UXDOMgshUfTFyG+YsEHcOO2CnK8SkM cEXY6xZJfibjBSd3CJEZQEm3LzUUWVMuKobeesSJrq/7a/j9sjhpCcMwZbldTTKLJ2ZW 3clIc/g14r3Ff91D3lijc38isVe0+7T71LxQK6yIM8iDxlDbateSQVT/oQRzSbfC2ojH u6j8RdsbEuOE2l1El4tOX2WBrwAc4J+PWOtoy4CTjAIPKdBGavEKYtAG8u9ew/F9epRu 7SH7AkLi2hEOXX2nscacxdCvS25qEl/9wr+JsBEWZ8PKDjyRShPjIiPTyQVGqaFCULqn vwnw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1734046437; x=1734651237; h=cc:to:from:subject:message-id:mime-version:date:x-gm-message-state :from:to:cc:subject:date:message-id:reply-to; bh=gLVftiiQq4akEB+OBVvUaxKCKfRMsg/7Sv0m+78/QB4=; b=rAZ/Ex45/VmRyEdEG+s2J7kyNLd2hIgfawZBEh94xLCZ15VAcTgw2H+0aHnDp7KrCx 2IGC9lTINTSf702F/7Tjxv4M3QgPY6BzpOn/9p+eMwXJMJNsScbR+hBNCm4n3npG5y4d ptTAj5UcTnR0XJLp5LXouOd1xmhgLzNXlPQrGpl5aizuGf5Rzu6hhgrkB0uE8FmIcBmI eVZt2k4Ov3kL1+aZ2gCd6OEtoY+beaar8BBwnzg2vprWSN4Dv7zQEG5GcMZZ9bonxcef Y/Z5CLHtamZ1/gWf5vFFElp4klGpxD+k72tzwMHWiTfly4vhBL/hk/9QoSk9cVcDXabW hjfw== X-Forwarded-Encrypted: i=1; AJvYcCXdPnlJfYlwyBVj+amc02X4OEX0ShtiUPtZ6Wr+piby7nJsykmknwZC49IxeReCxuO69TH0o1PuBH7sbao=@vger.kernel.org X-Gm-Message-State: AOJu0YzalI89HgTExvtbtm1FJktja9L/KQdze/ECK8Tb5DEidc8xSFoT HIOsOf4UYHihDZq1KvmgeS6Z9rT/6WJ2VeMz6EFr1E37GKBZYMRdlf2X/XfwyP+X/htDn6WiJqd MJZvXmw== X-Google-Smtp-Source: AGHT+IGEXM7+fNeWc2fIXBSFJ8pi3T0k9Tu+WFppiiRiTyRZ3FGf2nCxK3Dn/BUITxXbwDo18e0BTVZea9s3 X-Received: from pfbhs6.prod.google.com ([2002:a05:6a00:6906:b0:725:9e76:b700]) (user=brianvv job=prod-delivery.src-stubby-dispatcher) by 2002:a05:6a00:895:b0:725:d9b6:3932 with SMTP id d2e1a72fcca58-7290c1df088mr644347b3a.15.1734046437615; Thu, 12 Dec 2024 15:33:57 -0800 (PST) Date: Thu, 12 Dec 2024 23:33:30 +0000 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: Mime-Version: 1.0 X-Mailer: git-send-email 2.47.1.613.gc27f4b7a9f-goog Message-ID: <20241212233333.3743239-1-brianvv@google.com> Subject: [iwl-next PATCH v3 0/3] IDPF Virtchnl: Enhance error reporting & fix locking/workqueue issues From: Brian Vazquez To: Brian Vazquez , Tony Nguyen , Przemek Kitszel , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , intel-wired-lan@lists.osuosl.org Cc: David Decotigny , Vivek Kumar , Anjali Singhai , Sridhar Samudrala , linux-kernel@vger.kernel.org, netdev@vger.kernel.org, emil.s.tantilov@intel.com, Brian Vazquez Content-Type: text/plain; charset="UTF-8" This patch series addresses several IDPF virtchnl issues: * Improved error reporting for better diagnostics. * Fixed locking sequence in virtchnl message handling to avoid potential race conditions. * Converted idpf workqueues to unbound to prevent virtchnl processing delays under heavy load. Previously, CPU-bound kworkers for virtchnl processing could be starved, leading to transaction timeouts and connection failures. This was particularly problematic when IRQ traffic and user space processes contended for the same CPU. By making the workqueues unbound, we ensure virtchnl processing is not tied to a specific CPU, improving responsiveness even under high system load. --- V3: - Taking over Manoj's v2 series - Dropped "idpf: address an rtnl lock splat in tx timeout recovery path" it needs more rework and will be submitted later - Addresed nit typo - Addresed checkpatch.pl errors and warnings V2: - Dropped patch from Willem - RCS/RCT variable naming - Improved commit message on feedback v1: https://lore.kernel.org/netdev/20240813182747.1770032-2-manojvishy@google.com/T/ Manoj Vishwanathan (2): idpf: Acquire the lock before accessing the xn->salt idpf: add more info during virtchnl transaction time out Marco Leogrande (1): idpf: convert workqueues to unbound drivers/net/ethernet/intel/idpf/idpf_main.c | 15 ++++++++++----- drivers/net/ethernet/intel/idpf/idpf_virtchnl.c | 16 +++++++++++----- 2 files changed, 21 insertions(+), 10 deletions(-) -- 2.47.1.613.gc27f4b7a9f-goog