Re: [PATCH 1/1] devlink: Fixup port identifiers for 'port param show'

[Jiri Pirko <jiri@resnulli.us>]

Regarding subject, please put "iproute2" alongside with "patch" into the
brackets.


Tue, Mar 19, 2024 at 11:09:53PM CET, jorcrous@amazon.com wrote:
>Commit 70faecdca8f5 ("devlink: implement dump selector for devlink objects show commands")
>inadvertently removed DL_OP_HANDLEP from the required flags so that
>port definitions no longer worked:
>
>  $ devlink port param show pci/0000:01:00.0/52 name bc_kbyte_per_sec_rate
>  Devlink identification ("bus_name/dev_name") expected

Yeah, nobody should care as there is no in-tree kernel implementation of
this :)


>
>Return DL_OP_HANDLEP to the mask so the code correctly goes down the
>dl_argv_handle_both() path and handles both types of identifiers.
>


Provide proper fixes tag please.

>Signed-off-by: Jordan Crouse <jorcrous@amazon.com>

>---
> devlink/devlink.c | 3 ++-
> 1 file changed, 2 insertions(+), 1 deletion(-)
>
>diff --git a/devlink/devlink.c b/devlink/devlink.c
>index dbeb6e39..355e521c 100644
>--- a/devlink/devlink.c
>+++ b/devlink/devlink.c
>@@ -5050,7 +5050,8 @@ static int cmd_port_param_show(struct dl *dl)
> 
> 	err = dl_argv_parse_with_selector(dl, &flags,
> 					  DEVLINK_CMD_PORT_PARAM_GET,
>-					  DL_OPT_HANDLE | DL_OPT_PARAM_NAME, 0,
>+					  DL_OPT_HANDLE | DL_OPT_HANDLEP |
>+					  DL_OPT_PARAM_NAME, 0,

Should be only: DL_OPT_HANDLEP | DL_OPT_PARAM_NAME, Remove
DL_OPT_HANDLE.


pw-bot: cr

> 					  DL_OPT_HANDLE | DL_OPT_HANDLEP, 0);
> 	if (err)
> 		return err;
>-- 
>2.40.1
>
>