From: Yongseok Koh <yskoh@mellanox.com>
To: "Zhang, Qi Z" <qi.z.zhang@intel.com>,
"bluca@debian.org" <bluca@debian.org>
Cc: Andrew Rybchenko <arybchenko@solarflare.com>,
"dev@dpdk.org" <dev@dpdk.org>,
Ivan Malov <Ivan.Malov@oktetlabs.ru>
Subject: Re: [dpdk-dev] Incorrectly applied ethdev: fix queue start to 17.11
Date: Fri, 14 Dec 2018 20:10:42 +0000 [thread overview]
Message-ID: <7587DB79-D422-423E-8935-4D604ABABA34@mellanox.com> (raw)
In-Reply-To: <039ED4275CED7440929022BC67E70611532FBEC4@SHSMSX103.ccr.corp.intel.com>
Will take care of this.
Adding Luca for 16.11.
Thanks!
Yongseok Koh
On Dec 9, 2018, at 11:30 PM, Zhang, Qi Z <qi.z.zhang@intel.com<mailto:qi.z.zhang@intel.com>> wrote:
Your understand is correct, I saw the issue happens on both 17.11 and 16.11 both
From: Andrew Rybchenko [mailto:arybchenko@solarflare.com]
Sent: Monday, December 10, 2018 2:57 PM
To: Zhang, Qi Z <qi.z.zhang@intel.com<mailto:qi.z.zhang@intel.com>>; Yongseok Koh <yskoh@mellanox.com<mailto:yskoh@mellanox.com>>
Cc: dev@dpdk.org<mailto:dev@dpdk.org>; Ivan Malov <Ivan.Malov@oktetlabs.ru<mailto:Ivan.Malov@oktetlabs.ru>>
Subject: Incorrectly applied ethdev: fix queue start to 17.11
Hi,
Do I understand correctly that the following patch was incorrectly applied
on 17.11 since changes are done in queue stop functions instead of
queue start functions? If so, could you fix it, please. Also I think it makes
sense to check other stable branches as well.
We observe regressions because of the changeset.
Andrew.
commit d48890a39af7ae5c2d79dad02dd3ea3df8b6db82
Author: Qi Zhang <qi.z.zhang@intel.com><mailto:qi.z.zhang@intel.com>
Date: Thu Mar 22 20:59:01 2018 +0800
ethdev: fix queue start
[ upstream commit 239c9b435ad4abcac89b5d2480a0c21ce8b8288f ]
Device must be started before start any queue.
Fixes: 0748be2cf9a2 ("ethdev: queue start and stop")
Signed-off-by: Qi Zhang <qi.z.zhang@intel.com><mailto:qi.z.zhang@intel.com>
Acked-by: Konstantin Ananyev <konstantin.ananyev@intel.com><mailto:konstantin.ananyev@intel.com>
prev parent reply other threads:[~2018-12-14 20:10 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-12-10 6:56 Andrew Rybchenko
2018-12-10 7:29 ` Zhang, Qi Z
2018-12-14 20:10 ` Yongseok Koh [this message]
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=7587DB79-D422-423E-8935-4D604ABABA34@mellanox.com \
--to=yskoh@mellanox.com \
--cc=Ivan.Malov@oktetlabs.ru \
--cc=arybchenko@solarflare.com \
--cc=bluca@debian.org \
--cc=dev@dpdk.org \
--cc=qi.z.zhang@intel.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 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).