From mboxrd@z Thu Jan  1 00:00:00 1970
From: Ferruh Yigit <ferruh.yigit@intel.com>
Subject: Re: [PATCH v4] ethdev: allow returning error on VLAN
 offload configuration
Date: Tue, 10 Oct 2017 06:34:01 +0100
Message-ID: <3ec460b4-b0fd-c9c9-6b9a-dd1beffd5da1@intel.com>
References: <20170825134717.18376-1-dharton@cisco.com>
 <20170901023628.21308-1-dharton@cisco.com>
Mime-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 7bit
Cc: dev@dpdk.org
To: David Harton <dharton@cisco.com>, thomas@monjalon.net,
 ajit.khaparde@broadcom.com, johndale@cisco.com,
 konstantin.ananyev@intel.com, jingjing.wu@intel.com, beilei.xing@intel.com,
 jing.d.chen@intel.com, adrien.mazarguil@6wind.com,
 nelio.laranjeiro@6wind.com, alejandro.lucero@netronome.com,
 hemant.agrawal@nxp.com, rasesh.mody@cavium.com, harish.patil@cavium.com,
 skhare@vmware.com, yliu@fridaylinux.org, maxime.coquelin@redhat.com,
 allain.legacy@windriver.com
Return-path: <dev-bounces@dpdk.org>
Received: from mga03.intel.com (mga03.intel.com [134.134.136.65])
 by dpdk.org (Postfix) with ESMTP id 4F9481B22F
 for <dev@dpdk.org>; Tue, 10 Oct 2017 07:34:04 +0200 (CEST)
In-Reply-To: <20170901023628.21308-1-dharton@cisco.com>
Content-Language: en-US
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 9/1/2017 3:36 AM, David Harton wrote:
> Some devices may not support or fail setting VLAN offload
> configuration based on dynamic circurmstances so the
> vlan_offload_set_t vector is modified to return an int so
> the caller can determine success or not.
> 
> rte_eth_dev_set_vlan_offload is updated to return the
> value provided by the vector when called along with restoring
> the original offload configs on failure.
> 
> Existing vlan_offload_set_t vectors are modified to return
> an int.  Majority of cases return 0 but a few that actually
> can fail now return their failure codes.
> 
> Finally, a vlan_offload_set_t vector is added to virtio
> to facilitate dynamically turning VLAN strip on or off.
> 
> Signed-off-by: David Harton <dharton@cisco.com>

Hi David,

This patch conflicts against latest version, would you mind rebasing it
on top of latest next-net?

Thanks,
ferruh