From: Pavan Nikhilesh Bhagavatula <pbhagavatula@marvell.com>
To: Apeksha Gupta <apeksha.gupta@nxp.com>,
"jerin.jacob@caviumnetworks.com" <jerin.jacob@caviumnetworks.com>
Cc: "thomas@monjalon.net" <thomas@monjalon.net>,
"dev@dpdk.org" <dev@dpdk.org>,
"hemant.agrawal@nxp.com" <hemant.agrawal@nxp.com>,
"nipun.gupta@nxp.com" <nipun.gupta@nxp.com>,
"stable@dpdk.org" <stable@dpdk.org>
Subject: Re: [dpdk-dev] [PATCH v3 1/2] app/test-eventdev: Fix pipeline atq
Date: Wed, 22 Jul 2020 01:13:36 +0000 [thread overview]
Message-ID: <BN6PR18MB1140F17780B8D753E1D5582FDE790@BN6PR18MB1140.namprd18.prod.outlook.com> (raw)
In-Reply-To: <20200713114406.4383-1-apeksha.gupta@nxp.com>
>Subject: [dpdk-dev] [PATCH v3 1/2] app/test-eventdev: Fix pipeline atq
./devtools/check-git-log.sh
Wrong headline uppercase:
app/test-eventdev: Fix pipeline atq
Having a more informative commit title and message would be helpful,
Something like:
" app/eventdev: fix capability check in pipeline atq test"
>
>if-check is required to check the capability of all type queue.
"Add all type queue capability check before configuring event device
for pipeline atq test."
>
>Fixes: 6bf570a9911 ("app/eventdev: add pipeline atq test")
>Cc: stable@dpdk.org
>
>Signed-off-by: Apeksha Gupta <apeksha.gupta@nxp.com>
With above changes
Acked-by: Pavan Nikhilesh <pbhagavatula@marvell.com>
>---
>v3:
>* Added patch
>1. event/dpaa2: Add all type queue capability flag
>
>v2:
>* Removed unrequired patches
>1. app/test-eventdev: Enhancing perf-queue packet flow
>2. app/test-eventdev: Removed unwanted checks
>3. event/dpaa2: Add all type queue capability flag
>4. app/test-eventdev: Enhancing perf-atq packet flow
>5. app/test-eventdev: fix eventdev queues
>---
> app/test-eventdev/test_pipeline_atq.c | 2 ++
> 1 file changed, 2 insertions(+)
>
>diff --git a/app/test-eventdev/test_pipeline_atq.c b/app/test-
>eventdev/test_pipeline_atq.c
>index 8e8686c145..0872b25b53 100644
>--- a/app/test-eventdev/test_pipeline_atq.c
>+++ b/app/test-eventdev/test_pipeline_atq.c
>@@ -495,6 +495,8 @@ pipeline_atq_capability_check(struct
>evt_options *opt)
> evt_nr_active_lcores(opt->wlcores),
> dev_info.max_event_ports);
> }
>+ if (!evt_has_all_types_queue(opt->dev_id))
>+ return false;
>
> return true;
> }
>--
>2.17.1
next prev parent reply other threads:[~2020-07-22 1:13 UTC|newest]
Thread overview: 22+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-07-01 6:06 [dpdk-dev] [PATCH 1/6] app/test-eventdev: Enhancing perf-queue packet flow Apeksha Gupta
2020-07-01 6:06 ` [dpdk-dev] [PATCH 2/6] app/test-eventdev: Removed unwanted checks Apeksha Gupta
2020-07-01 6:06 ` [dpdk-dev] [PATCH 3/6] event/dpaa2: Add all type queue capability flag Apeksha Gupta
2020-07-01 6:06 ` [dpdk-dev] [PATCH 4/6] app/test-eventdev: Fix pipeline atq Apeksha Gupta
2020-07-10 13:08 ` [dpdk-dev] [PATCH v2] " Apeksha Gupta
2020-07-13 11:44 ` [dpdk-dev] [PATCH v3 1/2] " Apeksha Gupta
2020-07-13 11:44 ` [dpdk-dev] [PATCH v3 2/2] event/dpaa2: Add all type queue capability flag Apeksha Gupta
2020-07-21 17:51 ` Jerin Jacob
2020-07-23 13:36 ` Nipun Gupta
2020-07-24 5:24 ` Jerin Jacob
2020-07-23 17:04 ` [dpdk-dev] [PATCH v4 2/2] event/dpaa2: add " Apeksha Gupta
2020-07-22 1:13 ` Pavan Nikhilesh Bhagavatula [this message]
2020-07-22 7:48 ` [dpdk-dev] [PATCH v3 1/2] app/test-eventdev: Fix pipeline atq Thomas Monjalon
2020-07-23 17:02 ` [dpdk-dev] [PATCH v4 1/2] app/test-eventdev: fix capability check in pipeline ATQ test Apeksha Gupta
2020-07-01 6:06 ` [dpdk-dev] [PATCH 5/6] app/test-eventdev: Enhancing perf-atq packet flow Apeksha Gupta
2020-07-02 3:25 ` Pavan Nikhilesh Bhagavatula
2020-07-02 14:46 ` [dpdk-dev] [EXT] " Apeksha Gupta
2020-07-01 6:06 ` [dpdk-dev] [PATCH 6/6] app/test-eventdev: fix eventdev queues Apeksha Gupta
2020-07-02 3:27 ` Pavan Nikhilesh Bhagavatula
2020-07-09 13:05 ` [dpdk-dev] [EXT] " Apeksha Gupta
2020-07-02 3:24 ` [dpdk-dev] [PATCH 1/6] app/test-eventdev: Enhancing perf-queue packet flow Pavan Nikhilesh Bhagavatula
2020-07-02 14:46 ` [dpdk-dev] [EXT] " Apeksha Gupta
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=BN6PR18MB1140F17780B8D753E1D5582FDE790@BN6PR18MB1140.namprd18.prod.outlook.com \
--to=pbhagavatula@marvell.com \
--cc=apeksha.gupta@nxp.com \
--cc=dev@dpdk.org \
--cc=hemant.agrawal@nxp.com \
--cc=jerin.jacob@caviumnetworks.com \
--cc=nipun.gupta@nxp.com \
--cc=stable@dpdk.org \
--cc=thomas@monjalon.net \
/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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).