Re: [PATCH] eventdev/timer: fix overflow issue

2023-01-25 Thread Stephen Hemminger
On Wed, 25 Jan 2023 10:37:16 +0530 Jerin Jacob wrote: > Looks like Stephan replied to v1 hence comment is not showing up here > https://patches.dpdk.org/project/dpdk/patch/20230124204555.3022361-1-erik.g.carri...@intel.com/ > @Erik Gabriel Carrillo Can be following moved to slow path ? > + cycles

Re: [PATCH] eventdev/timer: fix overflow issue

2023-01-24 Thread Jerin Jacob
On Wed, Jan 25, 2023 at 8:17 AM Stephen Hemminger wrote: > > On Tue, 24 Jan 2023 10:09:45 -0600 > Erik Gabriel Carrillo wrote: > > > The software timer adapter converts event timer timeout ticks to a > > number of CPU cycles at which an rte_timer should expire. The > > computation uses integer op

Re: [PATCH] eventdev/timer: fix overflow issue

2023-01-24 Thread Stephen Hemminger
On Tue, 24 Jan 2023 10:09:45 -0600 Erik Gabriel Carrillo wrote: > The software timer adapter converts event timer timeout ticks to a > number of CPU cycles at which an rte_timer should expire. The > computation uses integer operations that can result in overflow. > > Use floating point operation

[PATCH] eventdev/timer: fix overflow issue

2023-01-24 Thread Erik Gabriel Carrillo
The software timer adapter converts event timer timeout ticks to a number of CPU cycles at which an rte_timer should expire. The computation uses integer operations that can result in overflow. Use floating point operations instead to perform the computation, and convert the final result back to a