Thread (29 messages) 29 messages, 5 authors, 2023-08-02

Re: [net-next/RFC PATCH v1 4/4] netdev-genl: Add support for exposing napi info from netdev

From: Simon Horman <hidden>
Date: 2023-06-02 15:47:28

On Thu, Jun 01, 2023 at 10:42:41AM -0700, Amritha Nambiar wrote:
Add support in ynl/netdev.yaml for napi related information. The
netdev structure tracks all the napi instances and napi fields.
The napi instances and associated queue[s] can be retrieved this way.

Refactored netdev-genl to support exposing napi<->queue[s] mapping
that is retained in a netdev.
Hi Amritha,

This feels like it should be two patches to me.
Though it is not something I feel strongly about.
Signed-off-by: Amritha Nambiar <redacted>
...
+static int
+netdev_nl_dev_napi_prepare_fill(struct net_device *netdev,
+				struct sk_buff **pskb, u32 portid, u32 seq,
+				int flags, u32 cmd, enum netdev_nl_type type)
+{
+	struct nlmsghdr *nlh;
+	struct sk_buff *skb = *pskb;
+	bool last = false;
+	int index = 0;
+	void *hdr;
+	int err;
+
nit: please use reverse xmas tree - longest line to shortest - for
     local variable declarations in (new) Networking code.

...
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help