[PATCH] drivers: remove compile-time option for IEEE 1588

Ferruh Yigit ferruh.yigit at amd.com
Fri Jun 23 16:00:21 CEST 2023


On 2/3/2023 1:28 PM, Thomas Monjalon wrote:
> The option RTE_LIBRTE_IEEE1588 has no effect on any library
> unlike its name.
> 
> Also we are suppose to enable/disable features dynamically,
> not at compilation time.
> 
> And the best is that this macro is neither documented,
> nor in rte_config.h.
> 
> It looks to be a mistake keeping this flag, so it is removed,
> meaning always enabled.
> PS: it is disabling vector paths of some drivers.
> 

PTP (IEEE1588) processing brings additional overhead to datapath.

Agree that it is not good to have undocumented compile macro, but just
removing it may cause performance degradation.


It may be possible to have separate burst function that supports PTP and
driver configures it when application explicitly request it with a new
offload flag (although it is not exactly an offload), what do you think?




More information about the dev mailing list