From mboxrd@z Thu Jan  1 00:00:00 1970
From: Thomas Monjalon <thomas.monjalon-pdR9zngts4EAvxtiuMwx3w@public.gmane.org>
Subject: Re: [PATCH v2] eal_pci: Fix max_vfs missing for none
	igb_uio driver
Date: Fri, 13 Feb 2015 10:20:11 +0100
Message-ID: <7191659.yLKdrT87cL@xps13>
References: <1423653001-11660-1-git-send-email-michael.qiu@intel.com>
 <1893948.saDpN2FujX@xps13>
 <533710CFB86FA344BFBF2D6802E60286CE8F2C@SHSMSX101.ccr.corp.intel.com>
Mime-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7Bit
Cc: dev-VfR2kkLFssw@public.gmane.org
To: "Qiu, Michael" <michael.qiu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
Return-path: <dev-bounces-VfR2kkLFssw@public.gmane.org>
In-Reply-To: <533710CFB86FA344BFBF2D6802E60286CE8F2C-0J0gbvR4kThpB2pF5aRoyrfspsVTdybXVpNB7YpNyf8@public.gmane.org>
List-Id: patches and discussions about DPDK <dev.dpdk.org>
List-Unsubscribe: <http://dpdk.org/ml/options/dev>,
 <mailto:dev-request-VfR2kkLFssw@public.gmane.org?subject=unsubscribe>
List-Archive: <http://dpdk.org/ml/archives/dev/>
List-Post: <mailto:dev-VfR2kkLFssw@public.gmane.org>
List-Help: <mailto:dev-request-VfR2kkLFssw@public.gmane.org?subject=help>
List-Subscribe: <http://dpdk.org/ml/listinfo/dev>,
 <mailto:dev-request-VfR2kkLFssw@public.gmane.org?subject=subscribe>
Errors-To: dev-bounces-VfR2kkLFssw@public.gmane.org
Sender: "dev" <dev-bounces-VfR2kkLFssw@public.gmane.org>

2015-02-13 02:56, Qiu, Michael:
> On 2/13/2015 12:26 AM, Thomas Monjalon wrote:
> > 2015-02-11 21:25, Michael Qiu:
> >> max_vfs will only be created by igb_uio driver, for other
> >> drivers like vfio or pci_uio_generic, max_vfs will miss.
> >>
> >> But sriov_numvfs is not driver related, just get the vf numbers
> >> from that field.
> >>
> >> Signed-off-by: Michael Qiu <michael.qiu-ral2JQCrhuEAvxtiuMwx3w@public.gmane.org>
> > [...]
> >>  	snprintf(filename, sizeof(filename), "%s/max_vfs", dirname);
> >>  	if (!access(filename, F_OK) &&
> >> -	    eal_parse_sysfs_value(filename, &tmp) == 0) {
> >> +	    eal_parse_sysfs_value(filename, &tmp) == 0)
> >>  		dev->max_vfs = (uint16_t)tmp;
> >> +	else {
> >> +		/* for none igb_uio driver, need kernel
> > I think it's a typo. Do you mean "for non igb_uio driver"?
> 
> Hi, Thomas
> 
> Yes, you are right.
> 
> Did I need to send out V3 patch for this typo?

No, thanks. Your agreement is enough. I'll fix it.