Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
From:   Vladimir Oltean <vladimir.oltean@nxp.com>
To:     netdev@vger.kernel.org
Cc:     linux-kernel@vger.kernel.org,
        "David S. Miller" <davem@davemloft.net>,
        Eric Dumazet <edumazet@google.com>,
        Jakub Kicinski <kuba@kernel.org>,
        Paolo Abeni <pabeni@redhat.com>,
        Michal Kubecek <mkubecek@suse.cz>,
        Claudiu Manoil <claudiu.manoil@nxp.com>,
        Vinicius Costa Gomes <vinicius.gomes@intel.com>,
        Xiaoliang Yang <xiaoliang.yang_1@nxp.com>,
        Kurt Kanzenbach <kurt@linutronix.de>,
        Rui Sousa <rui.sousa@nxp.com>,
        Ferenc Fejes <ferenc.fejes@ericsson.com>,
        Pranavi Somisetty <pranavi.somisetty@amd.com>,
        Harini Katakam <harini.katakam@amd.com>,
        Colin Foster <colin.foster@in-advantage.com>,
        UNGLinuxDriver@microchip.com,
        Alexandre Belloni <alexandre.belloni@bootlin.com>,
        Andrew Lunn <andrew@lunn.ch>,
        Florian Fainelli <f.fainelli@gmail.com>
Subject: [PATCH v4 net-next 01/12] net: ethtool: netlink: introduce ethnl_update_bool()
Date:   Thu, 19 Jan 2023 14:26:53 +0200
Message-Id: <20230119122705.73054-2-vladimir.oltean@nxp.com>
In-Reply-To: <20230119122705.73054-1-vladimir.oltean@nxp.com>
References: <20230119122705.73054-1-vladimir.oltean@nxp.com>
Content-Transfer-Encoding: 8bit
Content-Type: text/plain
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?us-ascii?Q?t1O+m+canfJghtQVAeLHujY8/CCupuaf4TLLRz/yykjf/SNgbUxxdB6MQL5U?=
 =?us-ascii?Q?OyEgNKZIzyZYIWCiyhC4+7npFDJxzBuCDZQw4OiR5NPkOJPankiIZBTu/upG?=
 =?us-ascii?Q?9A2I6ndi5NfSei99nOlGllR9ndSq9z8p1vQTMrJC2d4lpEiyoAwJ4fcgd4ko?=
 =?us-ascii?Q?yhDpq4uXZrzYpM54r089wJIqYgd1Al8rqSqd0VjAOhuVJ3DpP0bgYmtbMBZy?=
 =?us-ascii?Q?pvZF543r/3AFKNVuPvj4k6CDLPBUKhmvJFST3TGrC1GE/Kko5O3JejRZct5U?=
 =?us-ascii?Q?av8lQkuTSPEh3wRlAftHLrdveBRSY5cnVq7j99ZXat7TLYa7AcxJOrT+acQU?=
 =?us-ascii?Q?Su9DNtZ/pR7/6aYhxRhBw1hV8EOk8alzCf/smjrAsH6NrJFZqLbo+6fNzRkC?=
 =?us-ascii?Q?hIEswCO0bgJHvTpeSxSt3+FIVvM5LxxSfuZiyRs/v9V32IGxEaR9R3dxQbzn?=
 =?us-ascii?Q?sih1lPuTwpJJyzhjvWm1YmTC0uRNbRs67c1xaA7VOu1QT9v5vgYpWgw8zVFt?=
 =?us-ascii?Q?ZiG380T2wz6PsmGQYd71fIAb3HaMFSiJyNu9r0f/5kH4KGFTfodAcj9V8E79?=
 =?us-ascii?Q?n8cD5ALWsnFTVqn2YLwbMqN6w9jXn2BtHoIJ/LpiD2X2HRIAdiEIN53mGMw+?=
 =?us-ascii?Q?96V5decqXQxorDhvG9XpdH231Qsjec3ZYXXvOk7hXMiFIHRUlPL8MCpkZFe+?=
 =?us-ascii?Q?tsPEPpHfKACFjOzU+FtAxIGTASNMxiBO5GQZ88m62GbOiQNfhSmhkHTEUqiV?=
 =?us-ascii?Q?eUrdU5UiD2UvslqviNB7AcWeE4V6hF0t8Oa35MY9GHObdzi043VU0gTuXzHh?=
 =?us-ascii?Q?/7hAj+d359uu5gfJT2iSBZPGkzBP0yND1UEi4zaWF2DTtQA3NafETctW8fP2?=
 =?us-ascii?Q?j96kdl7AjnJQJmZvktnJ5idK3xxPaw9koFoRRs7lKWbzLX/l9SW6OOj3Sj1Y?=
 =?us-ascii?Q?79lpkdDYfdIY/7hoVcsVZjc6uB69O0JBxjkv0HCg5MxtsF6VgNJq8YUJgL5l?=
 =?us-ascii?Q?mUi7XwjTHc8MTm15LalbocCFaoAU/nUlC/6WkTDAEIzp0pK3zOhtTrIqp8RD?=
 =?us-ascii?Q?bq1PCXF++Mg+a5f2AVXDOo3MFh3PAoofmK8s202QS5N9zVbIUv5k+Wz8yaHR?=
 =?us-ascii?Q?Y7DnoU5ytIUcQVuoYeBoGrFTM/FkSjqGRaTzO8iITPFwHLcWOEj5LKOV7eo1?=
 =?us-ascii?Q?4catXYxVXDFFQqhwtiotNSxQmkpdmC73h3osH6jbtHQg/mquccKuTxtQSJr6?=
 =?us-ascii?Q?SyiDvaNrOfuxAYz8rX3HFFO6tVjJeLL097ZXU4D4tGAruOQzsgnF7aiVT8+V?=
 =?us-ascii?Q?d3zuEqBgxoCxdox9NCkSRJCsDuiyelOezTpXIrZcYoAXHPA1/0oUGuSYGc7g?=
 =?us-ascii?Q?A/k1Rp6LOMeDdOJ6UUklSZ0OTUiE4KTKNBiWnySFBhKvFbrYYzuRpVlkom0V?=
 =?us-ascii?Q?RYA1UvXaAh8aTdXjWHuddwiB5p7dNUJ3Yob8MXZg2fpT/n8A5NjTN16Z4+PN?=
 =?us-ascii?Q?zAuauUmxJzs5pHVtT2YGuSQN9LxXOKe2Stwoeut9wd+p1xuAT8EKC9JQHDtE?=
 =?us-ascii?Q?pu1u1rzTOHIvvOMWNFeSG3mP+db1dCSXvCofOAhDlNkLoLIAhqsJdtYh0Stg?=
 =?us-ascii?Q?eA=3D=3D?=
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 01459fc2-d1ed-486b-b30f-08dafa188e20
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB5136.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 19 Jan 2023 12:27:40.1700
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: necYRrzVgHbALeF4mNyxezOKvBtIGf/Whs0+FEPH9W+6iF+cqLI2ul0YL+AoWztDpEp6KGQYNXIM4iHPhOTFxA==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PAXPR04MB9376
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS autolearn=ham
        autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Due to the fact that the kernel-side data structures have been carried
over from the ioctl-based ethtool, we are now in the situation where we
have an ethnl_update_bool32() function, but the plain function that
operates on a boolean value kept in an actual u8 netlink attribute
doesn't exist.

With new ethtool features that are exposed solely over netlink, the
kernel data structures will use the "bool" type, so we will need this
kind of helper. Introduce it now; it's needed for things like
verify-disabled for the MAC merge configuration.

Signed-off-by: Vladimir Oltean <vladimir.oltean@nxp.com>
---
v2->v4: none
v1->v2: rewrite commit message

 net/ethtool/netlink.h | 26 ++++++++++++++++++++++++++
 1 file changed, 26 insertions(+)

diff --git a/net/ethtool/netlink.h b/net/ethtool/netlink.h
index f271266f6e28..f675f62fe181 100644
--- a/net/ethtool/netlink.h
+++ b/net/ethtool/netlink.h
@@ -111,6 +111,32 @@ static inline void ethnl_update_u8(u8 *dst, const struct nlattr *attr,
 	*mod = true;
 }
 
+/**
+ * ethnl_update_bool() - update bool from NLA_U8 attribute
+ * @dst:  value to update
+ * @attr: netlink attribute with new value or null
+ * @mod:  pointer to bool for modification tracking
+ *
+ * Use the u8 value from NLA_U8 netlink attribute @attr to set bool variable
+ * pointed to by @dst to false (if zero) or 1 (if not); do nothing if @attr is
+ * null. Bool pointed to by @mod is set to true if this function changed the
+ * logical value of *dst, otherwise it is left as is.
+ */
+static inline void ethnl_update_bool(bool *dst, const struct nlattr *attr,
+				     bool *mod)
+{
+	u8 val;
+
+	if (!attr)
+		return;
+	val = !!nla_get_u8(attr);
+	if (*dst == val)
+		return;
+
+	*dst = val;
+	*mod = true;
+}
+
 /**
  * ethnl_update_bool32() - update u32 used as bool from NLA_U8 attribute
  * @dst:  value to update
-- 
2.34.1