> -----Original Message-----
> From: Ajmera, Megha <[email protected]>
> Sent: Thursday, October 6, 2022 8:01 PM
> To: [email protected]; Singh, Jasvinder <[email protected]>;
> Dumitrescu, Cristian <[email protected]>
> Cc: [email protected]
> Subject: [PATCH 2/3] sched: removed unused subport field in hqos profile
>
> Removed unused subport field from profile.cfg
> Correctly using subport profile id in subport config load.
>
> Fixes: 802d214dc880 ("examples/qos_sched: update subport rate
> dynamically")
> Cc: [email protected]
>
> Signed-off-by: Megha Ajmera <[email protected]>
> ---
> examples/qos_sched/cfg_file.c | 2 +-
> examples/qos_sched/profile.cfg | 2 --
> 2 files changed, 1 insertion(+), 3 deletions(-)
>
> diff --git a/examples/qos_sched/cfg_file.c b/examples/qos_sched/cfg_file.c
> index 3d5d75fcf0..ca871d3287 100644
> --- a/examples/qos_sched/cfg_file.c
> +++ b/examples/qos_sched/cfg_file.c
> @@ -157,7 +157,7 @@ cfg_load_subport_profile(struct rte_cfgfile *cfg,
>
> profiles = rte_cfgfile_num_sections(cfg, "subport profile",
> sizeof("subport profile") - 1);
> - subport_params[0].n_pipe_profiles = profiles;
> + port_params.n_subport_profiles = profiles;
>
> for (i = 0; i < profiles; i++) {
> char sec_name[32];
> diff --git a/examples/qos_sched/profile.cfg
> b/examples/qos_sched/profile.cfg
> index c9ec187c93..e8de101b6c 100644
> --- a/examples/qos_sched/profile.cfg
> +++ b/examples/qos_sched/profile.cfg
> @@ -26,8 +26,6 @@ number of subports per port = 1
> number of pipes per subport = 4096
> queue sizes = 64 64 64 64 64 64 64 64 64 64 64 64 64
>
> -subport 0-8 = 0 ; These subports are configured with subport
> profile
> 0
> -
> [subport profile 0]
> tb rate = 1250000000 ; Bytes per second
> tb size = 1000000 ; Bytes
> --
> 2.25.1
Hi Megha,
Please fix:
-the title: "sched: fix number of subport profiles".
-add Cc: [email protected], not Cc to me; you need to add me in the "To:" line of
the email
Acked-by: Cristian Dumitrescu <[email protected]>
Regards,
Cristian
