From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pl1-f175.google.com (mail-pl1-f175.google.com [209.85.214.175]) (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 531B7191F92 for ; Tue, 29 Oct 2024 15:03:24 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.214.175 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1730214206; cv=none; b=SQin1gBLXEDrzaokqmXNSROrDpHQNJoRiYJVn3eXQBpbonZLVMvl0gCqKd6IyP/lBdNR+WpRrjk9VNbEghlnyoMul2ISQc/7TnX/2G319WU2ST/YQpN3r3UYhHJ9pS6DZSyK29fUnE145rczc0iUg4MCR5tbrw0PIBNaU442NwI= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1730214206; c=relaxed/simple; bh=NBSC41srgKPt9bg/2Qz/7PgkEc3c+cfTUpjZyDXQUXI=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=CCMJU7Omv76kyYXWthRhHFmBvcTIyQdJoxPsIsx8SQH54RTpJZADf3NiMtyO2i4NI5oVsTeOsYSLEFfH8Ad/ySFeua9PpNPtGYCLSkHmq0zwDAi5zV4EkyCGXj9WzJsqPnUH8EDcToaqp+4rKNygPgI9z8CvY+9Mwj7wOVB/C3c= 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=MEL6gOA5; arc=none smtp.client-ip=209.85.214.175 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="MEL6gOA5" Received: by mail-pl1-f175.google.com with SMTP id d9443c01a7336-20c8b557f91so51595615ad.2 for ; Tue, 29 Oct 2024 08:03:24 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fastly.com; s=google; t=1730214204; x=1730819004; 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=agAKt0IT7G723lrXGrtqJgZBhbv7SHC0IN5lHsfmwYg=; b=MEL6gOA51kVLII2DrX5XlCZ45ift+ozAjPNP1QUrIAfg+Cscpr79lUtPOwcfK8UHZY bcFKwJf9zAK++TP9/IgY1eu1aBWMDsdxjChIfoIpGAJQXRcejE3eshWYuNORz/CfiWVo xdwLQiyGHL2X6Sa2uatbuFBEc7LE3UhOWf7+o= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1730214204; x=1730819004; 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=agAKt0IT7G723lrXGrtqJgZBhbv7SHC0IN5lHsfmwYg=; b=QugiH2drJzEMeLiF9YzCuhQTQVEXNcDDCLkw2BAaaeCiIFIv65PNxEpQzvJeXNn6XP BcU0aOrt+C+j/Tdj8z/pMayJ62XCf6KVnwslDekuDRAzUY134A5+yHodHZfdoryJn5iU CLpGyoW5guVRgE7xGL6UVh5uWzSKW9XnLoVgEDOi4GrDikoDiMCrhkYPNg/tZQTEEeVn yhbfNxYjTQ1zje18ipE7kWB0FyShyVk2ZC9xMCB7s3LE1HAJB56suXWBdTa+cytdylhR PRZ+8EKU18k+82Bmu76u7oDiejW1TuVIRdFxFZBixvV5IfiOu+3XniHv8k+NQ4oMsMN/ I2rg== X-Forwarded-Encrypted: i=1; AJvYcCVjFmrEd28dWgBOI2MTujuvrYyGzvuRHQbxXoL7udIjmATUJJDUgbt4cE6bK9EFBRDT9DatldgFDvo=@vger.kernel.org X-Gm-Message-State: AOJu0YyB6lN5Lsi7KKIu6XSSMwDb/1tT1cUaJ5Xz8sncMca9KZ1FHSTU bwnNuyQC3A2ZJwmkl9gQ7FoiL1kHSsMjxDTGWltX2apxwp8MK+tuDfWhIEWJE7M= X-Google-Smtp-Source: AGHT+IFCKnHDgGtsA90IcNnhxnGU7fXTHsLH84y6r6k1evSEfeb5nttjlUCgjLc1TGCn9mq7Trci/w== X-Received: by 2002:a17:902:7790:b0:20c:fb47:5c05 with SMTP id d9443c01a7336-210c6c6d996mr123687415ad.46.1730214202479; Tue, 29 Oct 2024 08:03:22 -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-210bc02e941sm66967365ad.204.2024.10.29.08.03.19 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 29 Oct 2024 08:03:22 -0700 (PDT) Date: Tue, 29 Oct 2024 08:03:18 -0700 From: Joe Damato To: Paolo Abeni Cc: netdev@vger.kernel.org, namangulati@google.com, edumazet@google.com, amritha.nambiar@intel.com, sridhar.samudrala@intel.com, sdf@fomichev.me, peter@typeblog.net, m2shafiei@uwaterloo.ca, bjorn@rivosinc.com, hch@infradead.org, willy@infradead.org, willemdebruijn.kernel@gmail.com, skhawaja@google.com, kuba@kernel.org, Alexander Lobakin , Alexander Viro , "open list:BPF [MISC] :Keyword:(?:b|_)bpf(?:b|_)" , Christian Brauner , "David S. Miller" , Donald Hunter , Jan Kara , Jesper Dangaard Brouer , Jiri Pirko , Johannes Berg , Jonathan Corbet , "open list:DOCUMENTATION" , "open list:FILESYSTEMS (VFS and infrastructure)" , open list , Lorenzo Bianconi , Martin Karsten , Mina Almasry , Sebastian Andrzej Siewior , Xuan Zhuo Subject: Re: [PATCH net-next v2 0/6] Suspend IRQs during application busy periods Message-ID: Mail-Followup-To: Joe Damato , Paolo Abeni , netdev@vger.kernel.org, namangulati@google.com, edumazet@google.com, amritha.nambiar@intel.com, sridhar.samudrala@intel.com, sdf@fomichev.me, peter@typeblog.net, m2shafiei@uwaterloo.ca, bjorn@rivosinc.com, hch@infradead.org, willy@infradead.org, willemdebruijn.kernel@gmail.com, skhawaja@google.com, kuba@kernel.org, Alexander Lobakin , Alexander Viro , "open list:BPF [MISC] :Keyword:(?:b|_)bpf(?:b|_)" , Christian Brauner , "David S. Miller" , Donald Hunter , Jan Kara , Jesper Dangaard Brouer , Jiri Pirko , Johannes Berg , Jonathan Corbet , "open list:DOCUMENTATION" , "open list:FILESYSTEMS (VFS and infrastructure)" , open list , Lorenzo Bianconi , Martin Karsten , Mina Almasry , Sebastian Andrzej Siewior , Xuan Zhuo References: <20241021015311.95468-1-jdamato@fastly.com> <57a0e1e7-1079-4055-8072-d9105b70103f@redhat.com> Precedence: bulk X-Mailing-List: linux-doc@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: <57a0e1e7-1079-4055-8072-d9105b70103f@redhat.com> On Tue, Oct 29, 2024 at 11:25:18AM +0100, Paolo Abeni wrote: > On 10/21/24 03:52, Joe Damato wrote: > > Greetings: > > > > Welcome to v2, see changelog below. [...] > > The changes makes sense to me, and I could not find any obvious issue in > the patches. Thanks for taking a look. > I think this deserve some - even basic - self-tests coverage. Note that > you can enable GRO on veth devices to make NAPI instances avail there. > > Possibly you could opt for a drivers/net defaulting to veth usage and > allowing the user to select real H/W via env variables. Could we send a selftest in a follow up? I am asking because we've jumped through a number of hoops to get to this point: 1. Added support for per-NAPI config settings [1] to address Eric's concern in the v1, which took several revisions and was stalled due to a merge window. 2. Added support for netdev-genl to numerous drivers so that this would be usable in many different environments (gve, ena, tg3, e1000, e1000e, igc...) [2] [3] [4] [5] [6] 3. We didn't get any feedback about adding a selftest in the RFC or v1 [7]. I think all busy poll methods currently in the kernel need selftests (including the existing busy poll method) and its not clear to me currently how much work it'll be to get veth or netdevsim working to a point where we could re-submit this with a selftest that would be considered reasonable enough. FWIW, neither my previous series on the epoll ioctl nor the per NAPI settings were rejected due to lack of selftests, but I did add a simple selftest for the epoll ioctl later [8]. What do you think? Could we get this merged without having to get selftests working and come back with a selftest as separate change (like I did for the epoll ioctl) ? [1]: https://lore.kernel.org/lkml/20241011184527.16393-1-jdamato@fastly.com/ [2]: https://lore.kernel.org/lkml/20240930210731.1629-1-jdamato@fastly.com/ [3]: https://lore.kernel.org/lkml/20241002001331.65444-1-jdamato@fastly.com/ [4]: https://lore.kernel.org/netdev/20241009175509.31753-1-jdamato@fastly.com/ [5]: https://lore.kernel.org/netdev/20240930171232.1668-1-jdamato@fastly.com/ [6]: https://lore.kernel.org/lkml/20241028195243.52488-1-jdamato@fastly.com/ [7]: https://lore.kernel.org/all/20240823173103.94978-1-jdamato@fastly.com/ [8]: https://lore.kernel.org/netdev/171528362770.20134.14528995105510778643.git-patchwork-notify@kernel.org/T/