[PATCH v17 02/20] netlink: add new family to manage ULP_DDP enablement and stats
Jakub Kicinski
kuba at kernel.org
Tue Oct 24 07:59:08 PDT 2023
On Tue, 24 Oct 2023 15:58:22 +0200 Jiri Pirko wrote:
> >+definitions:
> >+ -
> >+ type: enum
> >+ name: cap
> >+ entries:
> >+ - nvme-tcp
> >+ - nvme-tcp-ddgst-rx
> >+
> >+uapi-header: linux/ulp_ddp_nl.h
>
> Not needed.
> Hmm, Jakub, why this is not only allowed in genetlink-legacy?
Agreed that it's not needed here.
There's a non-trivial chance of name collisions between new families
and existing headers under include/uapi/linux. Since this is a C/C++
-only artifact and "modern" languages will simply ignore it allowing it
seemed like the right choice.
> >+ -
> >+ name: pad
> >+ type: pad
> >+ -
> >+ name: rx-nvmeotcp-sk-add
> >+ doc: Sockets successfully configured for NVMeTCP offloading.
> >+ type: u64
Everything you have as u64 should now be uint, and pads can then go
away.
More information about the Linux-nvme
mailing list