From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from smtp1.osuosl.org (smtp1.osuosl.org [140.211.166.138]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 17DFFCCF9E3 for ; Mon, 10 Nov 2025 23:46:51 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp1.osuosl.org (Postfix) with ESMTP id D2E44818A1; Mon, 10 Nov 2025 23:46:50 +0000 (UTC) X-Virus-Scanned: amavis at osuosl.org Received: from smtp1.osuosl.org ([127.0.0.1]) by localhost (smtp1.osuosl.org [127.0.0.1]) (amavis, port 10024) with ESMTP id jvpNxgDZ9WnR; Mon, 10 Nov 2025 23:46:50 +0000 (UTC) X-Comment: SPF check N/A for local connections - client-ip=140.211.166.142; helo=lists1.osuosl.org; envelope-from=intel-wired-lan-bounces@osuosl.org; receiver= DKIM-Filter: OpenDKIM Filter v2.11.0 smtp1.osuosl.org E36D881785 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=osuosl.org; s=default; t=1762818409; bh=vVYJpfphQEEEjec/RD/qNfWPctFjvU06h/A56kUxSZ0=; h=Date:From:To:Cc:In-Reply-To:References:Subject:List-Id: List-Unsubscribe:List-Archive:List-Post:List-Help:List-Subscribe: From; b=gV2cDs9/HQEdn4VH1ZHwRmleY6WXQRqTpOS7JyoUY/79E4LBbe9RUNQmKp6vENt78 RDBQxOqvUKTjEBwuDSCiQTT9KBIqccgFLBAj3kbUv4jAZfY6bFWMVhATKtwX3hQ4PH Wm5RVKTPH+zBYLGOlKZwTy5icksoD9f/mlaTyA5Y8G6XoRkTK5TWFyAJuMNAsGinCl ub76Fg6idQc+U7jFeWyGWDFzhzMMfDd3uaKQmq73mltNp26XIuSEzYGdFcZQ5fHCqp Cmmw1+QLsN6KiQlMS2SQbnrqW+xDNbwP0hDh8dDG9Z0pSB6Bob4/yNeVLQYpvt+Ob/ nlcBUaM1jVjVg== Received: from lists1.osuosl.org (lists1.osuosl.org [140.211.166.142]) by smtp1.osuosl.org (Postfix) with ESMTP id E36D881785; Mon, 10 Nov 2025 23:46:49 +0000 (UTC) Received: from smtp2.osuosl.org (smtp2.osuosl.org [140.211.166.133]) by lists1.osuosl.org (Postfix) with ESMTP id ABC82FA for ; Mon, 10 Nov 2025 23:46:48 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp2.osuosl.org (Postfix) with ESMTP id 9D30F406DB for ; Mon, 10 Nov 2025 23:46:48 +0000 (UTC) X-Virus-Scanned: amavis at osuosl.org Received: from smtp2.osuosl.org ([127.0.0.1]) by localhost (smtp2.osuosl.org [127.0.0.1]) (amavis, port 10024) with ESMTP id ydorhXjqT1JN for ; Mon, 10 Nov 2025 23:46:47 +0000 (UTC) Received-SPF: Pass (mailfrom) identity=mailfrom; client-ip=172.234.252.31; helo=sea.source.kernel.org; envelope-from=kuba@kernel.org; receiver= DMARC-Filter: OpenDMARC Filter v1.4.2 smtp2.osuosl.org F32F0406D6 DKIM-Filter: OpenDKIM Filter v2.11.0 smtp2.osuosl.org F32F0406D6 Received: from sea.source.kernel.org (sea.source.kernel.org [172.234.252.31]) by smtp2.osuosl.org (Postfix) with ESMTPS id F32F0406D6 for ; Mon, 10 Nov 2025 23:46:46 +0000 (UTC) Received: from smtp.kernel.org (transwarp.subspace.kernel.org [100.75.92.58]) by sea.source.kernel.org (Postfix) with ESMTP id 599E744595; Mon, 10 Nov 2025 23:46:46 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 28E6FC4CEF5; Mon, 10 Nov 2025 23:46:44 +0000 (UTC) Date: Mon, 10 Nov 2025 15:46:43 -0800 From: Jakub Kicinski To: Jiri Pirko Cc: Saeed Mahameed , Daniel Zahka , "David S. Miller" , Eric Dumazet , Paolo Abeni , Simon Horman , Jonathan Corbet , Srujana Challa , Bharat Bhushan , Herbert Xu , Brett Creeley , Andrew Lunn , Michael Chan , Pavan Chebbi , Tony Nguyen , Przemek Kitszel , Sunil Goutham , Linu Cherian , Geetha sowjanya , Jerin Jacob , hariprasad , Subbaraya Sundeep , Tariq Toukan , Saeed Mahameed , Leon Romanovsky , Mark Bloch , Ido Schimmel , Petr Machata , Manish Chopra , Maxime Coquelin , Alexandre Torgue , Siddharth Vadapalli , Roger Quadros , Loic Poulain , Sergey Ryazanov , Johannes Berg , Vladimir Oltean , Michal Swiatkowski , Aleksandr Loktionov , Dave Ertman , Vlad Dumitrescu , "Russell King (Oracle)" , Alexander Sverdlin , Lorenzo Bianconi , netdev@vger.kernel.org, linux-doc@vger.kernel.org, intel-wired-lan@lists.osuosl.org, linux-rdma@vger.kernel.org, linux-stm32@st-md-mailman.stormreply.com, linux-arm-kernel@lists.infradead.org, linux-omap@vger.kernel.org Message-ID: <20251110154643.66d15800@kernel.org> In-Reply-To: References: <20251107204347.4060542-1-daniel.zahka@gmail.com> <20251107204347.4060542-3-daniel.zahka@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Mailman-Original-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1762818406; bh=4e1bgpUV/iimHsy8hCdENWKdiFeyy00W/2NkO5sEUAI=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=oP2yrdDG0/mUFvFidbKQaKe6gJ9F/49YmJX2dk2zIRQVnLBAUt5Bk09EXUhtxHglT RAHcKDqMcD6UbGffywoOsfuUPYOzyut01P+a224sUF9Q/lCufFp9yx3N1QHe8hHzQ1 h1Gcdd3ZW0XNvLVdJLo0EU9+29ceM4l9zpj0MH6jJCfTtB7C1/yevHRx8d/7412s8d I7WAj66CpWa5b2UnoISnNm/NI/L7rkfg3Llh78AlgAm4kaqySG0bWidh4Tqr8v6nCf UIrKMk8d7m0dQaOHa+KAQEtIcdyh5ignOrmz8lLJaeASJcxf/Y4qB4NfmGN21hdN5y KC7cNoFQCREBw== X-Mailman-Original-Authentication-Results: smtp2.osuosl.org; dmarc=pass (p=quarantine dis=none) header.from=kernel.org X-Mailman-Original-Authentication-Results: smtp2.osuosl.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.a=rsa-sha256 header.s=k20201202 header.b=oP2yrdDG Subject: Re: [Intel-wired-lan] [PATCH net-next v3 2/2] net/mlx5: implement swp_l4_csum_mode via devlink params X-BeenThere: intel-wired-lan@osuosl.org X-Mailman-Version: 2.1.30 Precedence: list List-Id: Intel Wired Ethernet Linux Kernel Driver Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-wired-lan-bounces@osuosl.org Sender: "Intel-wired-lan" On Sun, 9 Nov 2025 11:46:37 +0100 Jiri Pirko wrote: > >So, I checked a couple of flows internally, and it seems this allows > >some flexibility in the FW to decide later on which mode to pick, > >based on other parameters, which practically means > >"user has no preference on this param". Driver can only find out > >after boot, when it reads the runtime capabilities, but still > >this is a bug, by the time the driver reads this (in devlink), the > >default value should've already been determined by FW, so FW must > >return the actual runtime value. Which can only be one of the following > > I don't think it is correct to expose the "default" as a value. > > On read, user should see the configured value, either "full_csum" or > "l4_only". Reporting "default" to the user does not make any sense. > On write, user should pass either "full_csum" or "l4_only". Why we would > ever want to pass "default"? FWIW I agree that this feels a bit odd. Should the default be a flag attr? On get flag being present means the value is the FW default (no override present). On set passing the flag means user wants to reset to FW default (remove override)? > Regardless this patch, since this is param to be reflected on fw reboot > (permanent cmode), I think it would be nice to expose indication if > param value passed to user currently affects the fw, or if it is going > to be applied after fw reboot. Perhaps a simple bool attr would do? IIUC we're basically talking about user having no information that the update is pending? Could this be done by the core? Core can do a ->get prior to calling ->set and if the ->set succeeds and cmode != runtime record that the update is pending? That feels very separate from the series tho, there are 3 permanent params in mlx5, already. Is there something that makes this one special?