[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20240507171058.6ec1308b@kernel.org>
Date: Tue, 7 May 2024 17:10:58 -0700
From: Jakub Kicinski <kuba@...nel.org>
To: Antonio Quartulli <antonio@...nvpn.net>
Cc: netdev@...r.kernel.org, Sergey Ryazanov <ryazanov.s.a@...il.com>, Paolo
Abeni <pabeni@...hat.com>, Eric Dumazet <edumazet@...gle.com>, Andrew Lunn
<andrew@...n.ch>, Esben Haabendal <esben@...nix.com>
Subject: Re: [PATCH net-next v3 03/24] ovpn: add basic netlink support
On Mon, 6 May 2024 03:16:16 +0200 Antonio Quartulli wrote:
> + name: nonce_tail_size
nit: typically we hyphenate the names in YAML and C codegen replaces
the hyphens with underscores (and converts to uppercase)
> + exact-len: OVPN_NONCE_TAIL_SIZE
speaking of which - is the codegen buggy or this can be nonce_tail_size?
(or rather nonce-tail-size)
> + -
> + name: pad
> + type: pad
You shouldn't need this, now that we have uint.
replace nla_put_u64_64bit() with nla_put_uint().
Unfortunately libnl hasn't caught up so you may need to open code
the getter a little in user space CLI.
BTW I'd also bump the packet counters to uint.
Doesn't cost much if they don't grow > 32b and you never know..
> + request:
> + attributes:
> + - ifname
> + - mode
> + reply:
> + attributes:
> + - ifname
The attribute lists
> + struct net_device *dev;
> + int ifindex;
> +
> + if (!attrs[OVPN_A_IFINDEX])
GENL_REQ_ATTR_CHECK()
> + return ERR_PTR(-EINVAL);
> +
> + ifindex = nla_get_u32(attrs[OVPN_A_IFINDEX]);
> +
> + dev = dev_get_by_index(net, ifindex);
> + if (!dev)
> + return ERR_PTR(-ENODEV);
> +
> + if (!ovpn_dev_is_valid(dev))
> + goto err_put_dev;
> +
> + return dev;
> +
> +err_put_dev:
> + dev_put(dev);
NL_SET_BAD_ATTR(info->extack, ...[OVPN_A_IFINDEX])
> + return ERR_PTR(-EINVAL);
Powered by blists - more mailing lists