From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pf1-f174.google.com (mail-pf1-f174.google.com [209.85.210.174]) (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 09BE814EC55 for ; Tue, 22 Oct 2024 19:55:02 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.210.174 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1729626904; cv=none; b=fVB/Dg+wIHPLRWPhIugdq+Vy+Rpu0/CTTVPSaj3+towBHZsBeV14ii0X4S5pdx+rHAqBcKxI1AyY9u2I3eW+a7ax6rSfOOfqRGufWTEeoFwKTAj1gvFbE2usY6gUAee9CCah6tfGSOhsBQ3aZhzhBGSW9+3hi31s8Vr6wA2zfcs= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1729626904; c=relaxed/simple; bh=OwNBECyO8a4KyyWaokcQ5DrkDXHEf/AVG2cFGODOWbw=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=oML1eF2MRjDF4EnPW8ToA4FvmX3c47HWQ9ttn3CO1l8H5C4r9WeOYEApFcfRecTxW+khM63pmQlO3Oe06OGpztN2jXaFccBG/95DTdbGUxrvD1JTpLUQHOkQ8auId1kDeCcA5+tYDL2zlSthmyZ87Ka67UEfzuTAFAcEylhgRxQ= 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=IaivukWo; arc=none smtp.client-ip=209.85.210.174 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="IaivukWo" Received: by mail-pf1-f174.google.com with SMTP id d2e1a72fcca58-71e4244fdc6so4292868b3a.0 for ; Tue, 22 Oct 2024 12:55:02 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fastly.com; s=google; t=1729626902; x=1730231702; darn=vger.kernel.org; h=in-reply-to: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=GyiwtrdDNttqs/EmE5JLMsYdKcmXilzmJKF+u4QMJ3k=; b=IaivukWoBQvm+COOLj8X/MyxG65oIZGqscCxUuhoxZdcVuSl+LueqG6NFnsBoLOk3k J84085qjbv79KatZEA0bDPQX6QataLqxWcLIks99NQ4U4wBX/oKLO8ZcoekwaJdsvHfe OmTepm22hNnq2+tz68B+ay4PWroUTM3bQxflE= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1729626902; x=1730231702; h=in-reply-to: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=GyiwtrdDNttqs/EmE5JLMsYdKcmXilzmJKF+u4QMJ3k=; b=ezLS3m7AUv3cmgSBH76T6N00X6Vb+EWDesdNuuVyWd4G/L/6Zj/QQNDDMkCyS5kTzm KrgvF2/hDnsuWIjhiCkwA/Js/l/BOJrnogpRPH1YbXecHfOAKMOtwnHTiDxTIj/CMpCP vxnqXKfwKLcbNHiXCgeJ3qjQxg35qY+21D5Xibz0gTBbWKCZVUSdXlSsrLXOzsykv21E uikpYOVENlQt+A4rea3BgaEC37R+YgfnqF/N9VfgiEHu84QkLT9FubEzGEQ1HOiL0ZVZ ZbZw8JQS2/55lYJOw2kLDPcn6OAyp5hnfRIvMIww9SLFd1u7eB6WuWOOAFoaTGL9c+mL rZhw== X-Gm-Message-State: AOJu0YwN9OGnLE2yI7s+2rHaeVdH6AeE6CYHRHEppgKqSPFlxdJIbzvb yo62CYUme1Dz/U2cbk/GwqtStZ0Qk5m2CtOWh5ChBIDReWBDoRbmfHDYXn5yT9c= X-Google-Smtp-Source: AGHT+IHOS5ZW9hfqejqTSv0uUEcgV0zqvj28qCLjL54AXpj9P6o2oacvHMmutTEyUR5UBufxHBYjiw== X-Received: by 2002:a05:6a00:14cf:b0:71e:f4:dbc with SMTP id d2e1a72fcca58-72030cefe05mr409232b3a.25.1729626902274; Tue, 22 Oct 2024 12:55:02 -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 d2e1a72fcca58-71ec14159d4sm5085891b3a.210.2024.10.22.12.55.00 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 22 Oct 2024 12:55:01 -0700 (PDT) Date: Tue, 22 Oct 2024 12:54:59 -0700 From: Joe Damato To: Jacob Keller Cc: netdev@vger.kernel.org, kurt@linutronix.de, vinicius.gomes@intel.com, Tony Nguyen , Przemek Kitszel , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , "moderated list:INTEL ETHERNET DRIVERS" , open list Subject: Re: [net-next v3 1/2] igc: Link IRQs to NAPI instances Message-ID: Mail-Followup-To: Joe Damato , Jacob Keller , netdev@vger.kernel.org, kurt@linutronix.de, vinicius.gomes@intel.com, Tony Nguyen , Przemek Kitszel , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , "moderated list:INTEL ETHERNET DRIVERS" , open list References: <20241018171343.314835-1-jdamato@fastly.com> <20241018171343.314835-2-jdamato@fastly.com> Precedence: bulk X-Mailing-List: netdev@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: On Tue, Oct 22, 2024 at 11:50:15AM -0700, Jacob Keller wrote: > > > On 10/18/2024 10:13 AM, Joe Damato wrote: > > Link IRQs to NAPI instances via netdev-genl API so that users can query > > this information with netlink. > > > > Compare the output of /proc/interrupts (noting that IRQ 144 is the > > "other" IRQ which does not appear to have a NAPI instance): > > > > Minor nit: 144 doesn't appear in either output, and it seems like this > intended to indicate 128? > > We think its a typo as the 144 appears in the data from the second commit. > > I can make a note here to fix this typo when sending after we finish > validation, if there's no other issues. Yes, that's an error on my part. Sorry about that. I re-ran the patch after updating it and amended the commit message, but forgot to update '144' to be '128'. Based on the e1000 bug report that came in [1], I'm going to take another look at the igc patches to make sure the paths where the queue mapping happens (in Patch 2) are all in paths where rtnl is held as I attempted to do for e1000 [2]. [1]: https://lore.kernel.org/netdev/8cf62307-1965-46a0-a411-ff0080090ff9@yandex.ru/ [2]: https://lore.kernel.org/netdev/20241022172153.217890-1-jdamato@fastly.com/T/#u