[v7,00/14] fix lcore ID restriction

Message ID 20240326125554.138840-1-sivaprasad.tummala@amd.com (mailing list archive)
Headers
Series fix lcore ID restriction |

Message

Sivaprasad Tummala March 26, 2024, 12:55 p.m. UTC
  With modern CPUs, it is possible to have higher
CPU count thus we can have higher RTE_MAX_LCORES.
In DPDK sample applications, the current config
lcore options are hard limited to 255.

The patchset fixes these constraints by allowing
all lcore IDs up to RTE_MAX_LCORES. Also the rx
queue IDs are increased to support up to 65535.
The port ID constraints were also fixed to support
up to RTE_MAX_ETHPORTS.
  
v7: 
 - updated commit log with rx queue IDs
 - changed format specifier to %PRIu16
 - removed patch unrelated changes 

v6:
 - split queue_id, lcore_id and port_id changes as
   separate patches.
 - updated git commit description on individual
   patches
  
v5:
 - updated lcore_id type to uint32_t

v4:
 - fixed build errors with queue_id type
   in ipsec-secgw
 
v3:
 - updated queue_id type to uint16_t
 
v2:
 - fixed typo with lcore_id type in l3fwd

Sivaprasad Tummala (14):
  examples/l3fwd: fix queue ID restriction
  examples/l3fwd-power: fix queue ID restriction
  examples/l3fwd-graph: fix queue ID restriction
  examples/ipsec-secgw: fix queue ID restriction
  examples/l3fwd: fix lcore ID restriction
  examples/l3fwd-power: fix lcore ID restriction
  examples/l3fwd-graph: fix lcore ID restriction
  examples/ipsec-secgw: fix lcore ID restriction
  examples/qos_sched: fix lcore ID restriction
  examples/vm_power_manager: fix lcore ID restriction
  examples/l3fwd: fix port ID restriction
  examples/l3fwd-power: fix port ID restriction
  examples/l3fwd-graph: fix port ID restriction
  examples/ipsec-secgw: fix port ID restriction

 examples/ipsec-secgw/event_helper.h           |  2 +-
 examples/ipsec-secgw/ipsec-secgw.c            | 40 +++++++-----
 examples/ipsec-secgw/ipsec.c                  |  2 +-
 examples/ipsec-secgw/ipsec.h                  |  6 +-
 examples/ipsec-secgw/ipsec_worker.c           | 10 ++-
 examples/l3fwd-graph/main.c                   | 33 +++++-----
 examples/l3fwd-power/main.c                   | 65 ++++++++++---------
 examples/l3fwd-power/main.h                   |  4 +-
 examples/l3fwd-power/perf_core.c              | 19 ++++--
 examples/l3fwd/l3fwd.h                        |  2 +-
 examples/l3fwd/l3fwd_acl.c                    |  4 +-
 examples/l3fwd/l3fwd_em.c                     |  4 +-
 examples/l3fwd/l3fwd_event.h                  |  2 +-
 examples/l3fwd/l3fwd_fib.c                    |  4 +-
 examples/l3fwd/l3fwd_lpm.c                    |  5 +-
 examples/l3fwd/main.c                         | 42 +++++++-----
 examples/qos_sched/args.c                     |  6 +-
 .../guest_cli/vm_power_cli_guest.c            |  4 +-
 18 files changed, 135 insertions(+), 119 deletions(-)
  

Comments

Sivaprasad Tummala March 26, 2024, 12:55 p.m. UTC | #1
With modern CPUs, it is possible to have higher
CPU count thus we can have higher RTE_MAX_LCORES.
In DPDK sample applications, the current config
lcore options are hard limited to 255.

The patchset fixes these constraints by allowing
all lcore IDs up to RTE_MAX_LCORES. Also the rx
queue IDs are increased to support up to 65535.
The port ID constraints were also fixed to support
up to RTE_MAX_ETHPORTS.
  
v7: 
 - updated commit log with rx queue IDs
 - changed format specifier to %PRIu16
 - removed patch unrelated changes 

v6:
 - split queue_id, lcore_id and port_id changes as
   separate patches.
 - updated git commit description on individual
   patches
  
v5:
 - updated lcore_id type to uint32_t

v4:
 - fixed build errors with queue_id type
   in ipsec-secgw
 
v3:
 - updated queue_id type to uint16_t
 
v2:
 - fixed typo with lcore_id type in l3fwd

Sivaprasad Tummala (14):
  examples/l3fwd: fix queue ID restriction
  examples/l3fwd-power: fix queue ID restriction
  examples/l3fwd-graph: fix queue ID restriction
  examples/ipsec-secgw: fix queue ID restriction
  examples/l3fwd: fix lcore ID restriction
  examples/l3fwd-power: fix lcore ID restriction
  examples/l3fwd-graph: fix lcore ID restriction
  examples/ipsec-secgw: fix lcore ID restriction
  examples/qos_sched: fix lcore ID restriction
  examples/vm_power_manager: fix lcore ID restriction
  examples/l3fwd: fix port ID restriction
  examples/l3fwd-power: fix port ID restriction
  examples/l3fwd-graph: fix port ID restriction
  examples/ipsec-secgw: fix port ID restriction

 examples/ipsec-secgw/event_helper.h           |  2 +-
 examples/ipsec-secgw/ipsec-secgw.c            | 40 +++++++-----
 examples/ipsec-secgw/ipsec.c                  |  2 +-
 examples/ipsec-secgw/ipsec.h                  |  6 +-
 examples/ipsec-secgw/ipsec_worker.c           | 10 ++-
 examples/l3fwd-graph/main.c                   | 33 +++++-----
 examples/l3fwd-power/main.c                   | 65 ++++++++++---------
 examples/l3fwd-power/main.h                   |  4 +-
 examples/l3fwd-power/perf_core.c              | 19 ++++--
 examples/l3fwd/l3fwd.h                        |  2 +-
 examples/l3fwd/l3fwd_acl.c                    |  4 +-
 examples/l3fwd/l3fwd_em.c                     |  4 +-
 examples/l3fwd/l3fwd_event.h                  |  2 +-
 examples/l3fwd/l3fwd_fib.c                    |  4 +-
 examples/l3fwd/l3fwd_lpm.c                    |  5 +-
 examples/l3fwd/main.c                         | 42 +++++++-----
 examples/qos_sched/args.c                     |  6 +-
 .../guest_cli/vm_power_cli_guest.c            |  4 +-
 18 files changed, 135 insertions(+), 119 deletions(-)
  
Ferruh Yigit April 25, 2024, 12:31 p.m. UTC | #2
On 3/26/2024 12:55 PM, Sivaprasad Tummala wrote:
> With modern CPUs, it is possible to have higher
> CPU count thus we can have higher RTE_MAX_LCORES.
> In DPDK sample applications, the current config
> lcore options are hard limited to 255.
> 
> The patchset fixes these constraints by allowing
> all lcore IDs up to RTE_MAX_LCORES. Also the rx
> queue IDs are increased to support up to 65535.
> The port ID constraints were also fixed to support
> up to RTE_MAX_ETHPORTS.
>   
> v7: 
>  - updated commit log with rx queue IDs
>  - changed format specifier to %PRIu16
>  - removed patch unrelated changes 
> 
> v6:
>  - split queue_id, lcore_id and port_id changes as
>    separate patches.
>  - updated git commit description on individual
>    patches
>   
> v5:
>  - updated lcore_id type to uint32_t
> 
> v4:
>  - fixed build errors with queue_id type
>    in ipsec-secgw
>  
> v3:
>  - updated queue_id type to uint16_t
>  
> v2:
>  - fixed typo with lcore_id type in l3fwd
> 
> Sivaprasad Tummala (14):
>   examples/l3fwd: fix queue ID restriction
>   examples/l3fwd-power: fix queue ID restriction
>   examples/l3fwd-graph: fix queue ID restriction
>   examples/ipsec-secgw: fix queue ID restriction
>   examples/l3fwd: fix lcore ID restriction
>   examples/l3fwd-power: fix lcore ID restriction
>   examples/l3fwd-graph: fix lcore ID restriction
>   examples/ipsec-secgw: fix lcore ID restriction
>   examples/qos_sched: fix lcore ID restriction
>   examples/vm_power_manager: fix lcore ID restriction
>   examples/l3fwd: fix port ID restriction
>   examples/l3fwd-power: fix port ID restriction
>   examples/l3fwd-graph: fix port ID restriction
>   examples/ipsec-secgw: fix port ID restriction
>

Recheck-request: iol-unit-amd64-testing
  
Ferruh Yigit April 30, 2024, 1:47 p.m. UTC | #3
On 4/25/2024 1:31 PM, Ferruh Yigit wrote:
> On 3/26/2024 12:55 PM, Sivaprasad Tummala wrote:
>> With modern CPUs, it is possible to have higher
>> CPU count thus we can have higher RTE_MAX_LCORES.
>> In DPDK sample applications, the current config
>> lcore options are hard limited to 255.
>>
>> The patchset fixes these constraints by allowing
>> all lcore IDs up to RTE_MAX_LCORES. Also the rx
>> queue IDs are increased to support up to 65535.
>> The port ID constraints were also fixed to support
>> up to RTE_MAX_ETHPORTS.
>>   
>> v7: 
>>  - updated commit log with rx queue IDs
>>  - changed format specifier to %PRIu16
>>  - removed patch unrelated changes 
>>
>> v6:
>>  - split queue_id, lcore_id and port_id changes as
>>    separate patches.
>>  - updated git commit description on individual
>>    patches
>>   
>> v5:
>>  - updated lcore_id type to uint32_t
>>
>> v4:
>>  - fixed build errors with queue_id type
>>    in ipsec-secgw
>>  
>> v3:
>>  - updated queue_id type to uint16_t
>>  
>> v2:
>>  - fixed typo with lcore_id type in l3fwd
>>
>> Sivaprasad Tummala (14):
>>   examples/l3fwd: fix queue ID restriction
>>   examples/l3fwd-power: fix queue ID restriction
>>   examples/l3fwd-graph: fix queue ID restriction
>>   examples/ipsec-secgw: fix queue ID restriction
>>   examples/l3fwd: fix lcore ID restriction
>>   examples/l3fwd-power: fix lcore ID restriction
>>   examples/l3fwd-graph: fix lcore ID restriction
>>   examples/ipsec-secgw: fix lcore ID restriction
>>   examples/qos_sched: fix lcore ID restriction
>>   examples/vm_power_manager: fix lcore ID restriction
>>   examples/l3fwd: fix port ID restriction
>>   examples/l3fwd-power: fix port ID restriction
>>   examples/l3fwd-graph: fix port ID restriction
>>   examples/ipsec-secgw: fix port ID restriction
>>
> 
> Recheck-request: iol-unit-amd64-testing
>

Recheck-request: iol-sample-apps-testing