From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH 6/7] net/qede: fix maximum VF count to 0 Date: Mon, 19 Dec 2016 14:38:21 +0100 Message-ID: <2614732.lzO8dLuuld@xps13> References: <1480733039-13046-1-git-send-email-harish.patil@qlogic.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: Harish Patil , dev@dpdk.org, Dept-Eng DPDK Dev To: Ferruh Yigit Return-path: Received: from mail-wj0-f169.google.com (mail-wj0-f169.google.com [209.85.210.169]) by dpdk.org (Postfix) with ESMTP id 3255CF94A for ; Mon, 19 Dec 2016 14:38:23 +0100 (CET) Received: by mail-wj0-f169.google.com with SMTP id v7so150000871wjy.2 for ; Mon, 19 Dec 2016 05:38:23 -0800 (PST) In-Reply-To: List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 2016-12-12 18:13, Ferruh Yigit: > On 12/12/2016 5:47 PM, Harish Patil wrote: > >> btw, while checking feature list, I have seen qede_vf supports SR-IOV, > >> is that correct? > > > > Yes. The supported combination for SR-IOV is VF driver (qede PMD) with PF > > driver (qede linux driver). > > So you are using SR-IOV feature set in VF driver, as meaning VF driver > support exists. I don't know what SR-IOV feature mean for VF drivers. > Some other VF drivers not has this feature flag set. > > CC'ed Thomas for help, if this is the intention of the feature flag, it > is OK. Good question. I wonder where better describe the meaning of this row in the features table. Maybe it would be clearer by splitting in 2 rows: - SR-IOV VF - SR-IOV PF