Subject: Re: [PATCH net-next v2 06/10] tools/net/ynl: Add support for netlink-raw families

From: Jakub Kicinski <[email-masked]>
Date: Wed, 16 Aug 2023 08:29:08 -0700

On Tue, 15 Aug 2023 20:42:50 +0100 Donald Hunter wrote:
> Refactor the ynl code to encapsulate protocol specifics into
> NetlinkProtocol and GenlProtocol.

Looks good, but do we also need some extra plumbing to decode extack
for classic netlink correctly?  Basically shouldn't _decode_extack()
also move to proto? Or we can parameterize it? All we really need there
is to teach it how much of fixed headers parser needs to skip to get to
attributes, really (which, BTW is already kinda buggy for genl families
with fixed headers).