eventdev/timer: rearrange struct fields

Rearrange fields in rte_event_timer data structure to remove holes.
Also, remove use of volatile from rte_event_timer.

Signed-off-by: Pavan Nikhilesh <pbhagavatula@marvell.com>

diff --git a/doc/guides/rel_notes/release_21_11.rst b/doc/guides/rel_notes/release_21_11.rst
index 2af6aca..d81c2f2 100644 (file)
--- a/doc/guides/rel_notes/release_21_11.rst
+++ b/doc/guides/rel_notes/release_21_11.rst
@@ -441,6 +441,8 @@ ABI Changes
   accessed directly by user any more. This change is transparent to both
   applications and PMDs.
 
   accessed directly by user any more. This change is transparent to both
   applications and PMDs.
 

+* eventdev: Re-arranged fields in ``rte_event_timer`` to remove holes.
+

 
 Known Issues
 ------------
 
 Known Issues
 ------------

diff --git a/lib/eventdev/rte_event_timer_adapter.h b/lib/eventdev/rte_event_timer_adapter.h
index cad6d3b..1551741 100644 (file)
--- a/lib/eventdev/rte_event_timer_adapter.h
+++ b/lib/eventdev/rte_event_timer_adapter.h
@@ -475,8 +475,6 @@ struct rte_event_timer {
         *  - op: RTE_EVENT_OP_NEW
         *  - event_type: RTE_EVENT_TYPE_TIMER
         */
         *  - op: RTE_EVENT_OP_NEW
         *  - event_type: RTE_EVENT_TYPE_TIMER
         */

-       volatile enum rte_event_timer_state state;
-       /**< State of the event timer. */

        uint64_t timeout_ticks;
        /**< Expiry timer ticks expressed in number of *timer_ticks_ns* from
         * now.
        uint64_t timeout_ticks;
        /**< Expiry timer ticks expressed in number of *timer_ticks_ns* from
         * now.


@@ -488,6 +486,8 @@ struct rte_event_timer {
         * implementation specific values to share between the arm and cancel
         * operations.  The application should not modify this field.
         */
         * implementation specific values to share between the arm and cancel
         * operations.  The application should not modify this field.
         */

+       enum rte_event_timer_state state;
+       /**< State of the event timer. */

        uint8_t user_meta[0];
        /**< Memory to store user specific metadata.
         * The event timer adapter implementation should not modify this area.
        uint8_t user_meta[0];
        /**< Memory to store user specific metadata.
         * The event timer adapter implementation should not modify this area.