RE: [EXTERNAL] [PATCH v2] eventdev: make flag types consistent with API

[Jerin Jacob <jerinj@marvell.com>]

> -----Original Message-----
> From: Mattias Rönnblom <mattias.ronnblom@ericsson.com>
> Sent: Thursday, June 26, 2025 7:49 PM
> To: Jerin Jacob <jerinj@marvell.com>
> Cc: dev@dpdk.org; Mattias Rönnblom <hofors@lysator.liu.se>; Mattias
> Rönnblom <mattias.ronnblom@ericsson.com>
> Subject: [EXTERNAL] [PATCH v2] eventdev: make flag types consistent with API
> 
> Make RTE_EVENT_DEV_CAP_* flags' type consistent with the
> rte_event_dev_info. event_dev_cap's type. Make
> RTE_EVENT_DEV_CFG_PER_DEQUEUE_TIMEOUT consistent with
> rte_event_dev_config. event_dev_cfg. Make RTE_EVENT_QUEUE_CFG_*
> consistent with rte_event_queue_conf. event_queue_cfg.
> ZjQcmQRYFpfptBannerStart Prioritize security for external emails:
> Confirm sender and content safety before clicking links or opening attachments
> <https://us-phishalarm-
> ewt.proofpoint.com/EWT/v1/CRVmXkqW!ts3Z1f8UAnUatM-
> cOfxa7o5TEJ5_ESrFodeuUmP-LDt8WyHeBy6iQNTRjY0ARSdwSD-
> aXy_SlFdQEn8cMtiJdlzWks4ySLxEclA-S8gCKA$>
> Report Suspicious
> 
> ZjQcmQRYFpfptBannerEnd
> Make RTE_EVENT_DEV_CAP_* flags' type consistent with the
> rte_event_dev_info.event_dev_cap's type.
> 
> Make RTE_EVENT_DEV_CFG_PER_DEQUEUE_TIMEOUT consistent with
> rte_event_dev_config.event_dev_cfg.
> 
> Make RTE_EVENT_QUEUE_CFG_* consistent with
> rte_event_queue_conf.event_queue_cfg.
> 
> Make RTE_EVENT_PORT_CFG_* consistent with
> rte_event_port_conf.event_port_cfg.
> 
> Make RTE_EVENT_TIMER_ADAPTER_CAP_* consistent with the caps parameter
> in rte_event_eth_rx_adapter_caps_get().
> 
> The flags were all unsigned long longs (64 bits), but the event_dev_cap,
> event_dev_cfg, event_queue_cfg, and event_port_cfg fields and the caps
> parameter are all uint32_t. This change makes the flags uint32_ts.
> 
> Signed-off-by: Mattias Rönnblom <mattias.ronnblom@ericsson.com>

Reviewed-by: Jerin Jacob <jerinj@marvell.com>

Updated the git commit as follows and applied to dpdk-next-net-eventdev/for-main. Thanks

    eventdev: fix flag types consistency

    Make RTE_EVENT_DEV_CAP_* flags' type consistent with the
    rte_event_dev_info.event_dev_cap's type.

    Make RTE_EVENT_DEV_CFG_PER_DEQUEUE_TIMEOUT consistent with
    rte_event_dev_config.event_dev_cfg.

    Make RTE_EVENT_QUEUE_CFG_* consistent with
    rte_event_queue_conf.event_queue_cfg.

    Make RTE_EVENT_PORT_CFG_* consistent with
    rte_event_port_conf.event_port_cfg.

    Make RTE_EVENT_TIMER_ADAPTER_CAP_* consistent with the caps parameter
    in rte_event_eth_rx_adapter_caps_get().

    The flags were all unsigned long longs (64 bits), but the
    event_dev_cap, event_dev_cfg, event_queue_cfg, and event_port_cfg
    fields and the caps parameter are all uint32_t. This change makes the
    flags uint32_ts.

    Fixes: 71f238432865 ("eventdev: introduce event driven programming model")
    Fixes: 73e6b8c93dca ("eventdev: introduce burst mode capability")
    Fixes: ec36d881f56d ("eventdev: add implicit release disable capability")
    Fixes: 29653a7bfec1 ("eventdev: extend capability flags")
    Fixes: bd99189724b8 ("eventdev: negate maintenance capability flag")
    Fixes: d007a7f39de3 ("eventdev: introduce link profiles")
    Fixes: acc65ee307f7 ("eventdev: introduce event pre-scheduling")
    Fixes: c1bdd86d04d1 ("eventdev: add event port pre-schedule modify")
    Fixes: acc65ee307f7 ("eventdev: introduce event pre-scheduling")
    Fixes: 47d05b292820 ("eventdev: add timer adapter common code")
    Cc: stable@dpdk.org

    Signed-off-by: Mattias Rönnblom <mattias.ronnblom@ericsson.com>
    Reviewed-by: Jerin Jacob <jerinj@marvell.com>