From patchwork Fri Feb 2 12:39:45 2024 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Bruce Richardson X-Patchwork-Id: 136322 X-Patchwork-Delegate: jerinj@marvell.com Return-Path: X-Original-To: patchwork@inbox.dpdk.org Delivered-To: patchwork@inbox.dpdk.org Received: from mails.dpdk.org (mails.dpdk.org [217.70.189.124]) by inbox.dpdk.org (Postfix) with ESMTP id 0014543A4F; Fri, 2 Feb 2024 13:40:46 +0100 (CET) Received: from mails.dpdk.org (localhost [127.0.0.1]) by mails.dpdk.org (Postfix) with ESMTP id 2418142E5E; Fri, 2 Feb 2024 13:40:39 +0100 (CET) Received: from mgamail.intel.com (mgamail.intel.com [192.198.163.18]) by mails.dpdk.org (Postfix) with ESMTP id AF92542E63 for ; Fri, 2 Feb 2024 13:40:36 +0100 (CET) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1706877637; x=1738413637; h=from:to:cc:subject:date:message-id:in-reply-to: references:mime-version:content-transfer-encoding; bh=DzYCMLXxu3B262mm4OlS1C0hzLeOmXkEAfiIghGFbSk=; b=e8lhCCJVxuQXj3NB3Re6js9pyXOzHkOJmy5WTIwacvmEp/Hrs6JEF6Kg ZoFSswR1XaenkA3yQlsFClPHmNrohLqXVPMoyFHAQM1d9nqYqg7sSEFYF KKL4cOSbXPyReGagO9saAPWWpu7eAMYRRuaGc61xOQkFKcYPa7X4M5hZP OUeaDUASuXWUreZSFg5KP1X5jY1LXCUP+1/PXktOdKLgq++GWj+JXtPwu NjU1MOHudDHGyhIsSBZ8Xl+mvpFycsnCBHEFn/Un6Y5WIdK9k5vRGoUCo zCKqox4z8luGowb1MTIEHZGWn5IE0yFVe0lm50fYdGqKQ98AxF+oj+eej w==; X-IronPort-AV: E=McAfee;i="6600,9927,10971"; a="54349" X-IronPort-AV: E=Sophos;i="6.05,238,1701158400"; d="scan'208";a="54349" Received: from fmviesa006.fm.intel.com ([10.60.135.146]) by fmvoesa112.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 02 Feb 2024 04:40:36 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="6.05,238,1701158400"; d="scan'208";a="347559" Received: from silpixa00401385.ir.intel.com ([10.237.214.38]) by fmviesa006.fm.intel.com with ESMTP; 02 Feb 2024 04:40:34 -0800 From: Bruce Richardson To: dev@dpdk.org, jerinj@marvell.com, mattias.ronnblom@ericsson.com Cc: abdullah.sevincer@intel.com, sachin.saxena@oss.nxp.com, hemant.agrawal@nxp.com, pbhagavatula@marvell.com, pravin.pathak@intel.com, Bruce Richardson Subject: [PATCH v3 03/11] eventdev: update documentation on device capability flags Date: Fri, 2 Feb 2024 12:39:45 +0000 Message-Id: <20240202123953.77166-4-bruce.richardson@intel.com> X-Mailer: git-send-email 2.40.1 In-Reply-To: <20240202123953.77166-1-bruce.richardson@intel.com> References: <20240119174346.108905-1-bruce.richardson@intel.com> <20240202123953.77166-1-bruce.richardson@intel.com> MIME-Version: 1.0 X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Update the device capability docs, to: * include more cross-references * split longer text into paragraphs, in most cases with each flag having a single-line summary at the start of the doc block * general comment rewording and clarification as appropriate Signed-off-by: Bruce Richardson --- V3: Updated following feedback from Mattias --- lib/eventdev/rte_eventdev.h | 130 +++++++++++++++++++++++++----------- 1 file changed, 92 insertions(+), 38 deletions(-) diff --git a/lib/eventdev/rte_eventdev.h b/lib/eventdev/rte_eventdev.h index 37493464f9..a33024479d 100644 --- a/lib/eventdev/rte_eventdev.h +++ b/lib/eventdev/rte_eventdev.h @@ -253,143 +253,197 @@ struct rte_event; /* Event device capability bitmap flags */ #define RTE_EVENT_DEV_CAP_QUEUE_QOS (1ULL << 0) /**< Event scheduling prioritization is based on the priority and weight - * associated with each event queue. Events from a queue with highest priority - * is scheduled first. If the queues are of same priority, weight of the queues + * associated with each event queue. + * + * Events from a queue with highest priority + * are scheduled first. If the queues are of same priority, weight of the queues * are considered to select a queue in a weighted round robin fashion. * Subsequent dequeue calls from an event port could see events from the same * event queue, if the queue is configured with an affinity count. Affinity * count is the number of subsequent dequeue calls, in which an event port * should use the same event queue if the queue is non-empty * + * NOTE: A device may use both queue prioritization and event prioritization + * (@ref RTE_EVENT_DEV_CAP_EVENT_QOS capability) when making packet scheduling decisions. + * * @see rte_event_queue_setup(), rte_event_queue_attr_set() */ #define RTE_EVENT_DEV_CAP_EVENT_QOS (1ULL << 1) /**< Event scheduling prioritization is based on the priority associated with - * each event. Priority of each event is supplied in *rte_event* structure + * each event. + * + * Priority of each event is supplied in *rte_event* structure * on each enqueue operation. + * If this capability is not set, the priority field of the event structure + * is ignored for each event. * + * NOTE: A device may use both queue prioritization (@ref RTE_EVENT_DEV_CAP_QUEUE_QOS capability) + * and event prioritization when making packet scheduling decisions. + * @see rte_event_enqueue_burst() */ #define RTE_EVENT_DEV_CAP_DISTRIBUTED_SCHED (1ULL << 2) /**< Event device operates in distributed scheduling mode. + * * In distributed scheduling mode, event scheduling happens in HW or - * rte_event_dequeue_burst() or the combination of these two. + * rte_event_dequeue_burst() / rte_event_enqueue_burst() or the combination of these two. * If the flag is not set then eventdev is centralized and thus needs a * dedicated service core that acts as a scheduling thread . * - * @see rte_event_dequeue_burst() + * @see rte_event_dev_service_id_get */ #define RTE_EVENT_DEV_CAP_QUEUE_ALL_TYPES (1ULL << 3) /**< Event device is capable of enqueuing events of any type to any queue. - * If this capability is not set, the queue only supports events of the - * *RTE_SCHED_TYPE_* type that it was created with. * - * @see RTE_SCHED_TYPE_* values + * If this capability is not set, each queue only supports events of the + * *RTE_SCHED_TYPE_* type that it was created with. + * The behaviour when events of other scheduling types are sent to the queue is + * currently undefined. + * + * @see rte_event_enqueue_burst + * @see RTE_SCHED_TYPE_ATOMIC RTE_SCHED_TYPE_ORDERED RTE_SCHED_TYPE_PARALLEL */ #define RTE_EVENT_DEV_CAP_BURST_MODE (1ULL << 4) /**< Event device is capable of operating in burst mode for enqueue(forward, - * release) and dequeue operation. If this capability is not set, application - * still uses the rte_event_dequeue_burst() and rte_event_enqueue_burst() but - * PMD accepts only one event at a time. + * release) and dequeue operation. + * + * If this capability is not set, application + * can still use the rte_event_dequeue_burst() and rte_event_enqueue_burst() but + * PMD accepts or returns only one event at a time. * * @see rte_event_dequeue_burst() rte_event_enqueue_burst() */ #define RTE_EVENT_DEV_CAP_IMPLICIT_RELEASE_DISABLE (1ULL << 5) /**< Event device ports support disabling the implicit release feature, in * which the port will release all unreleased events in its dequeue operation. + * * If this capability is set and the port is configured with implicit release * disabled, the application is responsible for explicitly releasing events - * using either the RTE_EVENT_OP_FORWARD or the RTE_EVENT_OP_RELEASE event + * using either the @ref RTE_EVENT_OP_FORWARD or the @ref RTE_EVENT_OP_RELEASE event * enqueue operations. * * @see rte_event_dequeue_burst() rte_event_enqueue_burst() */ #define RTE_EVENT_DEV_CAP_NONSEQ_MODE (1ULL << 6) -/**< Event device is capable of operating in none sequential mode. The path - * of the event is not necessary to be sequential. Application can change - * the path of event at runtime. If the flag is not set, then event each event - * will follow a path from queue 0 to queue 1 to queue 2 etc. If the flag is - * set, events may be sent to queues in any order. If the flag is not set, the - * eventdev will return an error when the application enqueues an event for a +/**< Event device is capable of operating in non-sequential mode. + * + * The path of the event is not necessary to be sequential. Application can change + * the path of event at runtime and events may be sent to queues in any order. + * + * If the flag is not set, then event each event will follow a path from queue 0 + * to queue 1 to queue 2 etc. + * The eventdev will return an error when the application enqueues an event for a * qid which is not the next in the sequence. */ #define RTE_EVENT_DEV_CAP_RUNTIME_PORT_LINK (1ULL << 7) -/**< Event device is capable of configuring the queue/port link at runtime. +/**< Event device is capable of reconfiguring the queue/port link at runtime. + * * If the flag is not set, the eventdev queue/port link is only can be - * configured during initialization. + * configured during initialization, or by stopping the device and + * then later restarting it after reconfiguration. + * + * @see rte_event_port_link rte_event_port_unlink */ #define RTE_EVENT_DEV_CAP_MULTIPLE_QUEUE_PORT (1ULL << 8) -/**< Event device is capable of setting up the link between multiple queue - * with single port. If the flag is not set, the eventdev can only map a - * single queue to each port or map a single queue to many port. +/**< Event device is capable of setting up links between multiple queues and a single port. + * + * If the flag is not set, each port may only be linked to a single queue, and + * so can only receive events from that queue. + * However, each queue may be linked to multiple ports. + * + * @see rte_event_port_link */ #define RTE_EVENT_DEV_CAP_CARRY_FLOW_ID (1ULL << 9) -/**< Event device preserves the flow ID from the enqueued - * event to the dequeued event if the flag is set. Otherwise, - * the content of this field is implementation dependent. +/**< Event device preserves the flow ID from the enqueued event to the dequeued event. + * + * If this flag is not set, + * the content of the flow-id field in dequeued events is implementation dependent. + * + * @see rte_event_dequeue_burst */ #define RTE_EVENT_DEV_CAP_MAINTENANCE_FREE (1ULL << 10) /**< Event device *does not* require calls to rte_event_maintain(). + * * An event device that does not set this flag requires calls to * rte_event_maintain() during periods when neither * rte_event_dequeue_burst() nor rte_event_enqueue_burst() are called * on a port. This will allow the event device to perform internal * processing, such as flushing buffered events, return credits to a * global pool, or process signaling related to load balancing. + * + * @see rte_event_maintain */ #define RTE_EVENT_DEV_CAP_RUNTIME_QUEUE_ATTR (1ULL << 11) /**< Event device is capable of changing the queue attributes at runtime i.e - * after rte_event_queue_setup() or rte_event_start() call sequence. If this - * flag is not set, eventdev queue attributes can only be configured during + * after rte_event_queue_setup() or rte_event_dev_start() call sequence. + * + * If this flag is not set, event queue attributes can only be configured during * rte_event_queue_setup(). + * + * @see rte_event_queue_setup */ #define RTE_EVENT_DEV_CAP_PROFILE_LINK (1ULL << 12) -/**< Event device is capable of supporting multiple link profiles per event port - * i.e., the value of `rte_event_dev_info::max_profiles_per_port` is greater - * than one. +/**< Event device is capable of supporting multiple link profiles per event port. + * + * + * When set, the value of `rte_event_dev_info::max_profiles_per_port` is greater + * than one, and multiple profiles may be configured and then switched at runtime. + * If not set, only a single profile may be configured, which may itself be + * runtime adjustable (if @ref RTE_EVENT_DEV_CAP_RUNTIME_PORT_LINK is set). + * + * @see rte_event_port_profile_links_set rte_event_port_profile_links_get + * @see rte_event_port_profile_switch + * @see RTE_EVENT_DEV_CAP_RUNTIME_PORT_LINK */ /* Event device priority levels */ #define RTE_EVENT_DEV_PRIORITY_HIGHEST 0 -/**< Highest priority expressed across eventdev subsystem +/**< Highest priority level for events and queues. + * * @see rte_event_queue_setup(), rte_event_enqueue_burst() * @see rte_event_port_link() */ #define RTE_EVENT_DEV_PRIORITY_NORMAL 128 -/**< Normal priority expressed across eventdev subsystem +/**< Normal priority level for events and queues. + * * @see rte_event_queue_setup(), rte_event_enqueue_burst() * @see rte_event_port_link() */ #define RTE_EVENT_DEV_PRIORITY_LOWEST 255 -/**< Lowest priority expressed across eventdev subsystem +/**< Lowest priority level for events and queues. + * * @see rte_event_queue_setup(), rte_event_enqueue_burst() * @see rte_event_port_link() */ /* Event queue scheduling weights */ #define RTE_EVENT_QUEUE_WEIGHT_HIGHEST 255 -/**< Highest weight of an event queue +/**< Highest weight of an event queue. + * * @see rte_event_queue_attr_get(), rte_event_queue_attr_set() */ #define RTE_EVENT_QUEUE_WEIGHT_LOWEST 0 -/**< Lowest weight of an event queue +/**< Lowest weight of an event queue. + * * @see rte_event_queue_attr_get(), rte_event_queue_attr_set() */ /* Event queue scheduling affinity */ #define RTE_EVENT_QUEUE_AFFINITY_HIGHEST 255 -/**< Highest scheduling affinity of an event queue +/**< Highest scheduling affinity of an event queue. + * * @see rte_event_queue_attr_get(), rte_event_queue_attr_set() */ #define RTE_EVENT_QUEUE_AFFINITY_LOWEST 0 -/**< Lowest scheduling affinity of an event queue +/**< Lowest scheduling affinity of an event queue. + * * @see rte_event_queue_attr_get(), rte_event_queue_attr_set() */