All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jerin Jacob <jerin.jacob@caviumnetworks.com>
To: "Elo, Matias (Nokia - FI/Espoo)" <matias.elo@nokia.com>
Cc: "dev@dpdk.org" <dev@dpdk.org>,
	"Van Haaren, Harry" <harry.van.haaren@intel.com>,
	mattias.ronnblom@ericsson.com
Subject: Re: eventdev: rte_event_dev_start() all queues are linked requirement
Date: Mon, 30 Jul 2018 13:35:08 +0530	[thread overview]
Message-ID: <20180730080507.GA15830@jerin> (raw)
In-Reply-To: <21037D32-C2B9-40FF-92E6-55F6FA725E24@nokia.com>

-----Original Message-----
> Date: Mon, 30 Jul 2018 07:38:27 +0000
> From: "Elo, Matias (Nokia - FI/Espoo)" <matias.elo@nokia.com>
> To: "dev@dpdk.org" <dev@dpdk.org>
> CC: "jerin.jacob@caviumnetworks.com" <jerin.jacob@caviumnetworks.com>, "Van
>  Haaren, Harry" <harry.van.haaren@intel.com>
> Subject: eventdev: rte_event_dev_start() all queues are linked requirement
> x-mailer: Apple Mail (2.3445.9.1)
> 

+ mattias.ronnblom@ericsson.com as his SW driver is scheduled for next
release.

> 
> Hi,
> 
> rte_event_dev_start() requires that all queues have to be linked, which makes
> writing applications which link/unlink queues at runtime cumbersome.
> E.g. the application has to dummy link all queues before rte_event_dev_start()
> and then unlink them after the function call. This alone wouldn't be a big issue
> but rte_event_dev_start() may also be called inside
> rte_event_eth_rx_adapter_create() implementation causing additional complexity.
> 
> To me this check seems more like eventdev implementation specific limitation,
> which should be solved by the particular implementation and not enforced by the
> API to all applications. From an application point of view enqueueing events to
> an unlinked queue and expecting something meaningful to happen is an error
> anyway. So, would it be conceivable to remove this particular requirement?

For HW drivers, It is OK remove the particular requirement. But, If
there is an issue(performance/functionality) for enabling such feature
in SW driver. I would like keep that constraint to keep eventdev as
abstraction for both SW and HW driver. Harry and/or Mattias.ronnblom can
comment from SW driver perspective.

> 
> -Matias

  reply	other threads:[~2018-07-30  8:15 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-07-30  7:38 eventdev: rte_event_dev_start() all queues are linked requirement Elo, Matias (Nokia - FI/Espoo)
2018-07-30  8:05 ` Jerin Jacob [this message]
2018-07-30  9:15   ` Van Haaren, Harry

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20180730080507.GA15830@jerin \
    --to=jerin.jacob@caviumnetworks.com \
    --cc=dev@dpdk.org \
    --cc=harry.van.haaren@intel.com \
    --cc=matias.elo@nokia.com \
    --cc=mattias.ronnblom@ericsson.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.