From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-pg1-f175.google.com (mail-pg1-f175.google.com [209.85.215.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 27FF71E00A9 for ; Tue, 8 Oct 2024 23:00:45 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.215.175 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728428447; cv=none; b=KCakhgD1iteL9glRihg3yO1j8+vJ899B9YMFYNpjW7errtRhdjL4UR7MXzhyNpLRpSIU7U6VDX4lk2kVpUcIMlaH0S5O0QlmImY3q7jOWO0xJfTyD1NoMjNJcBnrqTk9xJY2eI/4RH/G9TptDbWxqMqkrOOX0VN12Q49npofJ3I= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1728428447; c=relaxed/simple; bh=Cv9bt1/ACmXKfPrd1pnpAyP9ihknxRE9I28Y+VS6+30=; h=Date:From:To:Cc:Subject:Message-ID:References:MIME-Version: Content-Type:Content-Disposition:In-Reply-To; b=PP6luUBnuec47HPcPmkafgS2YJ3bjphPvlWeLJnY1EBlZ0MsKiEQkb0COH1EnP6qGPSwUKa/ODRTeH4xIcbi4ElfxMH2Z9MZiXBQu3/5FPUhVNtO9XpHN7AacvdruHkrfpao6zeUb0v5P+UV4xKULiKV513/jMVod/HFszliI64= 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=YQLOBc51; arc=none smtp.client-ip=209.85.215.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="YQLOBc51" Received: by mail-pg1-f175.google.com with SMTP id 41be03b00d2f7-7e9fd82f1a5so2183221a12.1 for ; Tue, 08 Oct 2024 16:00:45 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=fastly.com; s=google; t=1728428445; x=1729033245; 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=ggxo0hJfdXuejdLNmnLqT0CDSeWl04Q3c4Lz2CofDmo=; b=YQLOBc51BrDsa1a5zeZzhXbRKGZ5irqLhOgdc/hmUd9w5d7iIka5OT42Ufj12XGNQn w14jV71B1hEqQEZ2uKff7lRNYe4iYHoAygRifNLxhf+aQgcD9BrnFOhCIF2BvZ/X0ir8 9kmzHqn6McUQy+o4m10Sg0iQshAnDFVOyNv6w= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1728428445; x=1729033245; 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=ggxo0hJfdXuejdLNmnLqT0CDSeWl04Q3c4Lz2CofDmo=; b=wqFRewowWpCZl3E85EF+GPe7S6aY6XuerafrCo7JSZrYEQ2HZn2hdf3d7nmfzX6JjE jH9a/ZQ/gyvKcHRftrvnj11WgNKJwU4Lwuon9/vi5uoMLTZIx8/2cg2R14NCTPWTZyk7 DvBRgUzaRUTN/43FXjs+NGXoU73yDnC3VJIAglbISKZYYi4A/N+nLVWvc7b8NZQYhYmx Evz3J7F5+Jw1+GwIaObovldKs8aRv98Xsvy52WU92PIO9zxMxe4DWnB22SLt05fqpQKr 0Qq5ijXe51G0OjZV+vKBjPURz1DsxMJ56rFzY6SVv4WT4ywz4Gvptk4Gww6NWp22N/B9 Toyw== X-Forwarded-Encrypted: i=1; AJvYcCXl+MdkqiFros6Kxjas+MPOK94cCIBsN+0N/O3XNsaGuB0bUJp+q0mGDoP2ITM1K4jVBPk3fmOE3XfaM4s=@vger.kernel.org X-Gm-Message-State: AOJu0Yyg6+e1w/CxZ1uyAY+oYFP2XWht/pbv24quciNyi7U3bjGCn/Y7 4RaeGU0590BQbHb3winiV/8iWLSS0DIw+s2PeN+24uFoiUja6e0VStZzC4hCuhQ= X-Google-Smtp-Source: AGHT+IGtCrR4i+f9TZK+BjFld4tVhwASIHDB0qyCNXIxMvlRm4zwHXJPO5kJdqKf8BskWqoStbzm9w== X-Received: by 2002:a17:90a:eb18:b0:2e2:9de2:8563 with SMTP id 98e67ed59e1d1-2e2a253be6amr559916a91.32.1728428445240; Tue, 08 Oct 2024 16:00:45 -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 98e67ed59e1d1-2e2a571fe8asm128302a91.34.2024.10.08.16.00.43 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Tue, 08 Oct 2024 16:00:44 -0700 (PDT) Date: Tue, 8 Oct 2024 16:00:41 -0700 From: Joe Damato To: Jakub Kicinski 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, "David S. Miller" , Eric Dumazet , Paolo Abeni , Donald Hunter , Jesper Dangaard Brouer , Mina Almasry , Xuan Zhuo , Daniel Jurgens , open list Subject: Re: [RFC net-next v4 6/9] netdev-genl: Support setting per-NAPI config values Message-ID: Mail-Followup-To: Joe Damato , Jakub Kicinski , 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, "David S. Miller" , Eric Dumazet , Paolo Abeni , Donald Hunter , Jesper Dangaard Brouer , Mina Almasry , Xuan Zhuo , Daniel Jurgens , open list References: <20241001235302.57609-1-jdamato@fastly.com> <20241001235302.57609-7-jdamato@fastly.com> <20241008151934.58f124f1@kernel.org> 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=us-ascii Content-Disposition: inline In-Reply-To: <20241008151934.58f124f1@kernel.org> On Tue, Oct 08, 2024 at 03:19:34PM -0700, Jakub Kicinski wrote: > On Tue, 8 Oct 2024 11:20:47 -0700 Joe Damato wrote: > > Noticed this while re-reading the code; planning on changing this > > from NLA_S32 to NLA_U32 for v5. > > Make sure you edit the spec, not the output. Looks like there may be > a problem here (napi-id vs id in the attributes). I'm not sure I follow this part, sorry if I'm just missing something here. I was referring to NETDEV_A_NAPI_DEFER_HARD_IRQS which in RFCv4 is listed as NLA_S32 (in this patch): static const struct nla_policy netdev_napi_set_nl_policy[NETDEV_A_NAPI_GRO_FLUSH_TIMEOUT + 1] = { [NETDEV_A_NAPI_ID] = { .type = NLA_U32, }, [NETDEV_A_NAPI_DEFER_HARD_IRQS] = { .type = NLA_S32 }, However, in the yaml spec (patch 2/9): + - + name: defer-hard-irqs + doc: The number of consecutive empty polls before IRQ deferral ends + and hardware IRQs are re-enabled. + type: u32 + checks: + max: s32-max So the type is u32 but with a "checks" to match what happens now in sysfs. That's why I mentioned changing NLA_S32 to NLA_U32. Am I missing something? Not sure what you meant by "napi-id vs id" ? > Make sure you run: ./tools/net/ynl/ynl-regen.sh -f > and the tree is clean afterwards OK, will do.