Re: [PATCH v4 1/2] lpm: add const to lpm arg of rte_lpm_lookup

2022-06-03 Thread Thomas Monjalon
03/06/2022 09:29, Bruce Richardson: > On Thu, Jun 02, 2022 at 10:52:25PM +0200, Thomas Monjalon wrote: > > 01/06/2022 13:15, Stanislaw Kardach: > > > All other rte_lpm_lookup* functions take lpm argument as a const. As the > > > basic rte_lpm_lookup() performs the same function, it should also do >

Re: [PATCH v4 1/2] lpm: add const to lpm arg of rte_lpm_lookup

2022-06-03 Thread Bruce Richardson
On Thu, Jun 02, 2022 at 10:52:25PM +0200, Thomas Monjalon wrote: > 01/06/2022 13:15, Stanislaw Kardach: > > All other rte_lpm_lookup* functions take lpm argument as a const. As the > > basic rte_lpm_lookup() performs the same function, it should also do > > that. > > > > As this function is inline

Re: [PATCH v4 1/2] lpm: add const to lpm arg of rte_lpm_lookup

2022-06-02 Thread Thomas Monjalon
01/06/2022 13:15, Stanislaw Kardach: > All other rte_lpm_lookup* functions take lpm argument as a const. As the > basic rte_lpm_lookup() performs the same function, it should also do > that. > > As this function is inline, no API/ABI change happens. It is an API change and should be noted in the

[PATCH v4 1/2] lpm: add const to lpm arg of rte_lpm_lookup

2022-06-01 Thread Stanislaw Kardach
All other rte_lpm_lookup* functions take lpm argument as a const. As the basic rte_lpm_lookup() performs the same function, it should also do that. As this function is inline, no API/ABI change happens. Signed-off-by: Stanislaw Kardach Acked-by: Vladimir Medvedkin Acked-by: Stephen Hemminger -