Hi Shijith,

Please see a question in-line:

> -----Original Message-----
> From: Shijith Thotton <sthot...@marvell.com>
> Sent: Monday, March 8, 2021 2:46 PM
> To: Carrillo, Erik G <erik.g.carri...@intel.com>
> Cc: Shijith Thotton <sthot...@marvell.com>; Pavan Nikhilesh
> <pbhagavat...@marvell.com>; Jerin Jacob <jer...@marvell.com>;
> dev@dpdk.org
> Subject: [PATCH 1/3] eventdev: introduce adapter flags for periodic mode
> 
> A timer adapter in periodic mode can be used to arm periodic timers.
> This patch adds flags used to advertise capability and configure timer adapter
> in periodic mode. Capability flag should be set for adapters which support
> periodic mode.
> 
> Below is a programming sequence on the usage:
>       /* check for periodic mode support by reading capability. */
>       rte_event_timer_adapter_caps_get(...);
> 
>       /* create adapter in periodic mode by setting periodic flag
>          (RTE_EVENT_TIMER_ADAPTER_F_PERIODIC) and resolution. */
>       rte_event_timer_adapter_create_ext(...);

It looks like periodic support is an operating mode of the adapter itself, and 
that all timers created with a periodic adapter instance will be periodic 
timers.

Is it possible to instead have "periodic/single-shot" be an attribute of an 
event timer itself, such that a single adapter instance could support either 
type of timer?

Thanks,
Erik

> 
>       /* arm periodic timer of configured resolution */
>       rte_event_timer_arm_burst(...);
> 
>       /* timer event will be periodically generated at configured
>          resolution till cancel is called. */
>       while (running) { rte_event_dequeue_burst(...); }
> 
>       /* cancel periodic timer which stops generating events */
>       rte_event_timer_cancel_burst(...);
> 
> Signed-off-by: Shijith Thotton <sthot...@marvell.com>
> ---

Reply via email to