From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v2 1/3] ether: Add API to support setting TX rate for queue and VF Date: Wed, 11 Jun 2014 01:02:26 +0200 Message-ID: <2006926.ZSHy59qidU@xps13> References: <1401090331-18455-1-git-send-email-changchun.ouyang@intel.com> <2115230.WNnKFPWGt0@xps13> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev-VfR2kkLFssw@public.gmane.org To: "Ouyang, Changchun" Return-path: In-Reply-To: List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org Sender: "dev" Hi Changchun, 2014-06-05 03:30, Ouyang, Changchun: > As we can see below, There are already 4 existing functions for vf in the header file: > rte_ethdev.h:int rte_eth_dev_set_vf_rxmode(uint8_t port, uint16_t vf, uint16_t rx_mode, > rte_ethdev.h:rte_eth_dev_set_vf_tx(uint8_t port,uint16_t vf, uint8_t on); > rte_ethdev.h:rte_eth_dev_set_vf_rx(uint8_t port,uint16_t vf, uint8_t on); > rte_ethdev.h:rte_eth_dev_set_vf_vlan_filter(uint8_t port, uint16_t vlan_id, > > So do we have plan to move them or remove them as they are all for VF specifically? I'm not very happy with these functions. I feel it's too specific because only related to igb/ixgbe features. We should try to isolate these things elsewhere. The ethdev API should be generic. It's not the right time to debate about this but I'd like to have such cleanup in roadmap. > If no, why we can accept those functions, but not accept the rte_eth_set_vf_rate_limit? :-) Let's accept it while thinking together to a future nice cleanup. -- Thomas