Thread (17 messages) flat view 17 messages, 2 authors, 2024-01-11

Re: [PATCH net v2 4/4] dpll: hide "zombie" pins for userspace

From: Jiri Pirko <jiri@resnulli.us>
Date: 2024-01-04 15:07:12

Thu, Jan 04, 2024 at 12:11:32PM CET, arkadiusz.kubalewski@intel.com wrote:

[...]

quoted hunk ↗ jump to hunk
@@ -1179,6 +1195,10 @@ int dpll_nl_pin_set_doit(struct sk_buff *skb, struct genl_info *info)

What about dpll_nl_pin_get_doit(), dpll_nl_pin_id_get_doit()?

I think it would be better to move the check to:
dpll_pin_pre_doit()

{
	struct dpll_pin *pin = info->user_ptr[0];

+	if (!xa_empty(&pin->parent_refs) &&
+	    !dpll_pin_parents_registered(pin))
+		return -ENODEV;
+
	return dpll_pin_set_from_nlattr(pin, info);
}

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