On Thu, Jun 27, 2019 at 5:45 PM Ferruh Yigit <ferruh.yi...@intel.com> wrote:
>
> On 6/13/2019 1:05 PM, Rastislav Cernay wrote:
> > +Timestamping needs to be enabled during compile time, as there is no way
> > +to check whether a timestamping unit is runnig during run time.
>
> Even if this can't be detected on runtime, it should be possible to configure 
> in
> runtime with device args so need to compile the PMD again to change the 
> behavior.
>
> In this patch default behavior is no timestamps support, same can be done with
> devargs, without any devargs timestamp support disable, but if user provides a
> devarg, like "timestamp=1", feature enabled.
>
> What do you think, can it be possible to convert runtime configuration?

This sounds good. We did not explore the devargs option. We will look
into it, and probably rework this patch and send v2.

Thanks,
  Jan

Reply via email to