From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 0735E360 for ; Fri, 13 Oct 2023 00:15:33 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=kernel.org header.i=@kernel.org header.b="cJxTjZAo" Received: by smtp.kernel.org (Postfix) with ESMTPSA id 2E499C433C7; Fri, 13 Oct 2023 00:15:33 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1697156133; bh=ryPle/1x3mswxx4PaZa3d1IXrtB5t2o1gtURUuXFgfo=; h=Date:From:To:Cc:Subject:In-Reply-To:References:From; b=cJxTjZAowKMrUFCc8h317HW+yYN3hAEuMAxjaMjcaotq4wwk9XY3f+p8lX863qTjM OwZBj4HQ/ZRIVnBpdDNqLcwDtci2+0NkQEvE8762a17d2ik3WaVVR5k9T8PbF685p+ nE9WrKzGsGlfh+ox50233Eey3sPYTKuxkoP8G0ASBgIS0ovdkZTGUvcXX+O5mJj5jT e0QCxItjIwzwRvHptMQ+ddj0BP+KiN8tCYoIGbmpubL3m4SddA/RN7wKN6+onlXjDD V53bybodMfV7FFP2JggBSqPV2NIGbBGVOQiQ1yIs2pWiXOJtI8oxyCNzvr7HhslAx7 e3WKlAEU6rJDw== Date: Thu, 12 Oct 2023 17:15:32 -0700 From: Jakub Kicinski To: Jacob Keller Cc: Jiri Pirko , , , , , Subject: Re: [patch net-next 02/10] tools: ynl-gen: introduce support for bitfield32 attribute type Message-ID: <20231012171532.35515553@kernel.org> In-Reply-To: References: <20231010110828.200709-1-jiri@resnulli.us> <20231010110828.200709-3-jiri@resnulli.us> <20231010115804.761486f1@kernel.org> <20231011095236.5fdca6e2@kernel.org> <20231011112537.2962c8be@kernel.org> 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-Transfer-Encoding: 7bit On Thu, 12 Oct 2023 14:06:57 -0700 Jacob Keller wrote: > >> - it doesn't support "by name" operations so ethtool didn't use it > > > > It follows the original Netlink rule: "all uapi should be well defined in > > enums/defines". > > What's the "by name" operation? Instead of sending the full bit mask sending the list of bits and what state we want them in. And that list can either have bit numbers or names. Looking at ethnl_parse_bit() could be helpful.