All of lore.kernel.org
 help / color / mirror / Atom feed
From: Yuanhan Liu <yuanhan.liu@linux.intel.com>
To: Wei Dai <wei.dai@intel.com>
Cc: wenzhuo.lu@intel.com, thomas@monjalon.net,
	harish.patil@cavium.com, rasesh.mody@cavium.com,
	stephen.hurd@broadcom.com, ajit.khaparde@broadcom.com,
	helin.zhang@intel.com, konstantin.ananyev@intel.com,
	jingjing.wu@intel.com, jing.d.chen@intel.com,
	adrien.mazarguil@6wind.com, nelio.laranjeiro@6wind.com,
	bruce.richardson@intel.com, maxime.coquelin@redhat.com,
	shepard.siegel@atomicrules.com, ed.czeck@atomicrules.com,
	john.miller@atomicrules.com, dev@dpdk.org, stable@dpdk.org
Subject: Re: [PATCH v6 1/3] ethdev: fix adding invalid MAC addr
Date: Wed, 3 May 2017 09:46:01 +0800	[thread overview]
Message-ID: <20170503014601.GC3005@yliu-dev.sh.intel.com> (raw)
In-Reply-To: <1493729065-17090-2-git-send-email-wei.dai@intel.com>

On Tue, May 02, 2017 at 08:44:23PM +0800, Wei Dai wrote:
> Some customers find adding MAC addr to VF sometimes can fail,
> but it is still stored in dev->data->mac_addrs[ ]. So this
> can lead to some errors that assumes the non-zero entry in
> dev->data->mac_addrs[ ] is valid.
> Following acknowledgements are from specific NIC PMD
> maintainer for their managing part.
> 
> Fixes: af75078fece3 ("first public release")


> Cc: stable@dpdk.org

Just a note, this patch changes API. It should not be backported to a
stable/LTS release, even though it fixes something.

	--yliu

  parent reply	other threads:[~2017-05-03  1:46 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-04-30  4:11 [PATCH v5 0/3] MAC address fail to be added shouldn't be stored Wei Dai
2017-04-30  4:11 ` [PATCH v5 1/3] ethdev: fix adding invalid MAC addr Wei Dai
2017-04-30  4:11 ` [PATCH v5 2/3] doc: change type of return value of adding " Wei Dai
2017-04-30  4:11 ` [PATCH v5 3/3] app/testpmd: add a command to add many MAC addrs Wei Dai
2017-04-30 22:59 ` [PATCH v5 0/3] MAC address fail to be added shouldn't be stored Thomas Monjalon
2017-05-02 12:44 ` [PATCH v6 " Wei Dai
2017-05-02 12:44   ` [PATCH v6 1/3] ethdev: fix adding invalid MAC addr Wei Dai
2017-05-03  0:57     ` Lu, Wenzhuo
2017-05-03  1:46     ` Yuanhan Liu [this message]
2017-05-05  0:16       ` Dai, Wei
2017-05-02 12:44   ` [PATCH v6 2/3] doc: change type of return value of adding " Wei Dai
2017-05-02 12:44   ` [PATCH v6 3/3] app/testpmd: add a command to add many MAC addrs Wei Dai
2017-05-05  0:39   ` [PATCH v7 0/3] MAC address fail to be added shouldn't be stored Wei Dai
2017-05-05  0:40     ` [PATCH v7 1/3] ethdev: fix adding invalid MAC addr Wei Dai
2017-05-05  1:46       ` Yang, Qiming
2017-05-05 14:07         ` Thomas Monjalon
2017-05-05 14:21       ` Thomas Monjalon
2017-05-07  4:39         ` Yuanhan Liu
2017-05-05 14:24       ` Thomas Monjalon
2017-05-05  0:40     ` [PATCH v7 2/3] doc: change type of return value of adding " Wei Dai
2017-05-05 14:23       ` Thomas Monjalon
2017-05-05  0:40     ` [PATCH v7 3/3] app/testpmd: add a command to add many MAC addrs Wei Dai
2017-05-05 14:24       ` Thomas Monjalon
2017-05-08  1:01         ` Wu, Jingjing
2017-05-08  6:17           ` Wu, Jingjing

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=20170503014601.GC3005@yliu-dev.sh.intel.com \
    --to=yuanhan.liu@linux.intel.com \
    --cc=adrien.mazarguil@6wind.com \
    --cc=ajit.khaparde@broadcom.com \
    --cc=bruce.richardson@intel.com \
    --cc=dev@dpdk.org \
    --cc=ed.czeck@atomicrules.com \
    --cc=harish.patil@cavium.com \
    --cc=helin.zhang@intel.com \
    --cc=jing.d.chen@intel.com \
    --cc=jingjing.wu@intel.com \
    --cc=john.miller@atomicrules.com \
    --cc=konstantin.ananyev@intel.com \
    --cc=maxime.coquelin@redhat.com \
    --cc=nelio.laranjeiro@6wind.com \
    --cc=rasesh.mody@cavium.com \
    --cc=shepard.siegel@atomicrules.com \
    --cc=stable@dpdk.org \
    --cc=stephen.hurd@broadcom.com \
    --cc=thomas@monjalon.net \
    --cc=wei.dai@intel.com \
    --cc=wenzhuo.lu@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 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.