Thread (7 messages) flat view 7 messages, 2 authors, 2021-01-03

Re: [PATCH iproute2 1/3] dcb: Set values with RTM_SETDCB type

From: Petr Machata <hidden>
Date: 2021-01-03 10:49:24

Stephen Hemminger [off-list ref] writes:
On Sat,  2 Jan 2021 00:25:50 +0100
Petr Machata [off-list ref] wrote:
quoted
dcb currently sends all netlink messages with a type RTM_GETDCB, even the
set ones. Change to the appropriate type.

Signed-off-by: Petr Machata <redacted>
---
 dcb/dcb.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/dcb/dcb.c b/dcb/dcb.c
index adec57476e1d..f5c62790e27e 100644
--- a/dcb/dcb.c
+++ b/dcb/dcb.c
@@ -177,7 +177,7 @@ int dcb_set_attribute(struct dcb *dcb, const char *dev, int attr, const void *da
 	struct nlattr *nest;
 	int ret;
 
-	nlh = dcb_prepare(dcb, dev, RTM_GETDCB, DCB_CMD_IEEE_SET);
+	nlh = dcb_prepare(dcb, dev, RTM_SETDCB, DCB_CMD_IEEE_SET);
 
 	nest = mnl_attr_nest_start(nlh, DCB_ATTR_IEEE);
 	mnl_attr_put(nlh, attr, data_len, data);
Should I add fixes tag to this?
Ha, I forgot that Fixes: tags are a thing in iproute2. Sorry about that,
I'll resend.
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help