From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f181.google.com (mail-pl1-f181.google.com [209.85.214.181]) (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 E56966FE16 for ; Thu, 10 Oct 2024 16:07:41 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.181 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728576463; cv=none; b=O0yaE6q59HOZZdch5fBE1Tc+DrGIQEyXnMLFykeo2uHkAnEVw7KeivKR0suSPGy8xk+I8UAEQIDJD/sC4DzmJNsxDd5+VU8QO2gdgzPRjgf4gn1FZBENKyJzfh4nKs1E7xc5oaySvmVg1KAm9C089U+bhZvGbQxeRh3FgzF4WI8= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728576463; c=relaxed/simple; bh=gMJJQSlsn8Yh8reIJdg3CtduncAlSvBM+SH5wCCRMpg=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=i//tgWfJwCoCOpvaKiAcJRLyZcGs9aeSjWxwWdregUarwg92sjWRj79gVtXfZ5PxlJ8YHzfYAfbfykAp2JdtZWq0iHgdyJpYOjTrSWJKMIwFJXUR79Lnc++cwxbK0r34DkBtR6shGHpsBVbN2sG9WT1uQFUwjXSVZ0S6/rA6aDs= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=fastly.com; spf=pass smtp.mailfrom=fastly.com; dkim=pass (1024-bit key) header.d=fastly.com header.i=@fastly.com header.b=GsTrJMkU; arc=none smtp.client-ip=209.85.214.181 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=fastly.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=fastly.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=fastly.com header.i=@fastly.com header.b="GsTrJMkU" Received: by mail-pl1-f181.google.com with SMTP id d9443c01a7336-20b9b35c7c7so7185005ad.1 for ; Thu, 10 Oct 2024 09:07:41 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fastly.com; s=google; t=1728576461; x=1729181261; darn=vger.kernel.org; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:mail-followup-to:message-id:subject:cc:to :from:date:from:to:cc:subject:date:message-id:reply-to; bh=lLDmBuPbqdFWywlMJb7s5WtOj7TP+t2aXk4dAI9ZYPM=; b=GsTrJMkU25PTamlCMwj87VBDSg08a9TYsWC/cOPWfuVOfRvnrParif5IpeDbCg4MKw wHJ/vAKC1dIdXOs1HCvL9FNUt+WJhcZnLPmPOdzXcizhVnHsxxDZcOn2FrwYYfbJa/X2 3aNilAGkKW3DcnVXRiG8wt/fKZJLJDwx9hdW8= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1728576461; x=1729181261; h=in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:mail-followup-to:message-id:subject:cc:to :from:date:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=lLDmBuPbqdFWywlMJb7s5WtOj7TP+t2aXk4dAI9ZYPM=; b=igOoHqjxytHcO0dOk0vppiM7W6WRt9E9AXVg9Fdo5+0UsupTrEAXzF7K+AZjtdRVQQ tSG06NhXM9iOEqetrubYo0fnorWAkseBcuFjwl2dWH7sevmxRlH+NvGasvmFqRKmBcPE 7obLpo0VoVDh9sSPCSbmyJE6efwv6DZrdXw7iEm1cwtgkePedu5EW0uhrImJgbO0wkai RUTwjrooxMSRnDglVZxpd+FJ7KBT5ZA0CVEYJTNcJgPwgJMEPu48M914UkUC8FTwj/5r vu7PWQTgv1CW2nczPrnSrc1uGQSajutbOB0e9LtCudWV1Effu6AlpxYq1IAbaxVOaRVB 046A== X-Forwarded-Encrypted: i=1; AJvYcCVuLe6IbS9TopwJedwJMse0e6Wx3NmGVJ+M2xhvjgHtbnD9QRZpNyxtzxNIaUzLtCtKMmMe5Nkie6xojyI=@vger.kernel.org X-Gm-Message-State: AOJu0YyBC5j68sBQUf9PB8vc0Yl8ZGh6FLyAa2U9w3Aqf3PRV/WG6IYH ILmEnB6fbMe7V8/SdGcpEWksm60AA9evyj11W6kX0goCDMYR8psUUOIpm1ejya0= X-Google-Smtp-Source: AGHT+IG0fUbGFEzJUGz4lr95LVIQt209ymVKDCNv+HN/QpGQnTewWumSlw9CcDSQqGqrSbXg5wKhWw== X-Received: by 2002:a17:903:32c9:b0:20b:43f8:d764 with SMTP id d9443c01a7336-20c6375c6e4mr98324265ad.8.1728576461240; Thu, 10 Oct 2024 09:07:41 -0700 (PDT) Received: from LQ3V64L9R2 (c-24-6-151-244.hsd1.ca.comcast.net. [24.6.151.244]) by smtp.gmail.com with ESMTPSA id d9443c01a7336-20c8c33ce22sm10984635ad.256.2024.10.10.09.07.39 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 10 Oct 2024 09:07:40 -0700 (PDT) Date: Thu, 10 Oct 2024 09:07:37 -0700 From: Joe Damato To: Eric Dumazet Cc: netdev@vger.kernel.org, mkarsten@uwaterloo.ca, skhawaja@google.com, sdf@fomichev.me, bjorn@rivosinc.com, amritha.nambiar@intel.com, sridhar.samudrala@intel.com, willemdebruijn.kernel@gmail.com, Tariq Toukan , "David S. Miller" , Jakub Kicinski , Paolo Abeni , "open list:MELLANOX MLX4 core VPI driver" , open list Subject: Re: [net-next v5 9/9] mlx4: Add support for persistent NAPI config to RX CQs Message-ID: Mail-Followup-To: Joe Damato , Eric Dumazet , netdev@vger.kernel.org, mkarsten@uwaterloo.ca, skhawaja@google.com, sdf@fomichev.me, bjorn@rivosinc.com, amritha.nambiar@intel.com, sridhar.samudrala@intel.com, willemdebruijn.kernel@gmail.com, Tariq Toukan , "David S. Miller" , Jakub Kicinski , Paolo Abeni , "open list:MELLANOX MLX4 core VPI driver" , open list References: <20241009005525.13651-1-jdamato@fastly.com> <20241009005525.13651-10-jdamato@fastly.com> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: On Thu, Oct 10, 2024 at 06:28:59AM +0200, Eric Dumazet wrote: > On Wed, Oct 9, 2024 at 2:56 AM Joe Damato wrote: > > > > Use netif_napi_add_config to assign persistent per-NAPI config when > > initializing RX CQ NAPIs. > > > > Presently, struct napi_config only has support for two fields used for > > RX, so there is no need to support them with TX CQs, yet. > > > > Signed-off-by: Joe Damato > > --- > > nit: technically, the napi_defer_hard_irqs could benefit TX completions as well. That's true - I think I missed updating this commit message when I realized it. I can correct the commit message while retaining your Reviewed-by for the v6. Note: This adds to the confusion I have around the support for allocating max(rxqs, txqs) config structs; it would seem we'll be missing config structure for some queues if the system is configured to use the maximum number of each? Perhaps that configuration is uncommon enough that it doesn't matter? > Reviewed-by: Eric Dumazet