On 8/7/2024 8:27 AM, Xu, Rosen wrote:
> Hi,
>
>> -----Original Message-----
<...>
>> Subject: [PATCH 1/4] ethdev: make parameters to TM node add fn constant
>>
>> The function to add a new scheduling node in rte_tm should not (and does
>> not) modify the actual node parameters passed in via struct pointer. We
>> should guarantee this by marking the parameter pointer as const. This allows
>> SW to create multiple scheduling nodes using the same parameter struct
>> without having to reset it each time.
>>
>> Signed-off-by: Bruce Richardson <bruce.richard...@intel.com>
>> ---
>> drivers/net/ipn3ke/ipn3ke_tm.c | 4 ++--
>> diff --git a/drivers/net/ipn3ke/ipn3ke_tm.c
>> b/drivers/net/ipn3ke/ipn3ke_tm.c index 0260227900..cffe1fdaa4 100644
>> --- a/drivers/net/ipn3ke/ipn3ke_tm.c
>> +++ b/drivers/net/ipn3ke/ipn3ke_tm.c
>> @@ -1010,7 +1010,7 @@ ipn3ke_tm_tdrop_profile_delete(struct
>> rte_eth_dev *dev, static int
>> ipn3ke_tm_node_add_check_parameter(uint32_t tm_id,
>> uint32_t node_id, uint32_t parent_node_id, uint32_t priority,
>> - uint32_t weight, uint32_t level_id, struct rte_tm_node_params
>> *params,
>> + uint32_t weight, uint32_t level_id, const struct rte_tm_node_params
>> +*params,
>> struct rte_tm_error *error)
>> {
>> uint32_t level_of_node_id;
>> @@ -1168,7 +1168,7 @@ ipn3ke_tm_node_add_check_mount(uint32_t
>> tm_id, static int ipn3ke_tm_node_add(struct rte_eth_dev *dev,
>> uint32_t node_id, uint32_t parent_node_id, uint32_t priority,
>> - uint32_t weight, uint32_t level_id, struct rte_tm_node_params
>> *params,
>> + uint32_t weight, uint32_t level_id, const struct rte_tm_node_params
>> +*params,
>> struct rte_tm_error *error)
>> {
>> struct ipn3ke_hw *hw = IPN3KE_DEV_PRIVATE_TO_HW(dev); diff --
>> /** @internal Traffic manager node delete */
>> --
>> 2.43.0
>
> Reviewed-by: Rosen Xu <rosen...@intel.com>
>
Acked-by: Ferruh Yigit <ferruh.yi...@amd.com>
