From mboxrd@z Thu Jan  1 00:00:00 1970
From: Yuanhan Liu <yuanhan.liu@linux.intel.com>
Subject: Re: [PATCH v6 1/3] ethdev: fix adding invalid MAC addr
Date: Wed, 3 May 2017 09:46:01 +0800
Message-ID: <20170503014601.GC3005@yliu-dev.sh.intel.com>
References: <1493525507-56304-1-git-send-email-wei.dai@intel.com>
 <1493729065-17090-1-git-send-email-wei.dai@intel.com>
 <1493729065-17090-2-git-send-email-wei.dai@intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=us-ascii
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
To: Wei Dai <wei.dai@intel.com>
Return-path: <dev-bounces@dpdk.org>
Content-Disposition: inline
In-Reply-To: <1493729065-17090-2-git-send-email-wei.dai@intel.com>
List-Id: DPDK patches and discussions <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request@dpdk.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev@dpdk.org>
List-Help: <mailto:dev-request@dpdk.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request@dpdk.org?subject=subscribe>
Errors-To: dev-bounces@dpdk.org
Sender: "dev" <dev-bounces@dpdk.org>

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