From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from out1-smtp.messagingengine.com (out1-smtp.messagingengine.com [66.111.4.25]) by dpdk.org (Postfix) with ESMTP id 1DAEAA49F for ; Thu, 25 Jan 2018 15:58:55 +0100 (CET) Received: from compute1.internal (compute1.nyi.internal [10.202.2.41]) by mailout.nyi.internal (Postfix) with ESMTP id AB4E420E92; Thu, 25 Jan 2018 09:58:53 -0500 (EST) Received: from frontend1 ([10.202.2.160]) by compute1.internal (MEProxy); Thu, 25 Jan 2018 09:58:53 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=monjalon.net; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to:x-me-sender :x-me-sender:x-sasl-enc; s=mesmtp; bh=perrHmdymoNNdQ09j7MhRjNsPI jwlAbp14S4H/OeKik=; b=g/4uA6wlGL1o5A2qnH/q2MTd+NsKNJp7XZCl/Z5E+i XCZSgaF6AwbD0s94GW6Ol37K6uxBnSSkYyPW2CEre5E/qY2hpyg8pEHmA1lXC7/F g3TLLH1d/yLgLz4JBp0w0ZLnDowez5/ZiKANxbL0VCul9PxNrFZumPvMZ/YWIPh+ I= DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d= messagingengine.com; h=cc:content-transfer-encoding:content-type :date:from:in-reply-to:message-id:mime-version:references :subject:to:x-me-sender:x-me-sender:x-sasl-enc; s=fm1; bh=perrHm dymoNNdQ09j7MhRjNsPIjwlAbp14S4H/OeKik=; b=iH6ioNI05e0YA/ks5XwvOf Ky32iAxZi/oKccQgR0e3witSZ7WP0YCAywIvmEq+jjRU2AoTuCn1Qr5H7TBVT/1x E4O0CrH0Pw9hTcQkH+dWQ83nJxLuPTTSqGhJoQWh4FPV3zvkvnz6deWqXBujJJGN 4Cz4AYt9epVjV1XhlRUmbN/BioIMMNN9BJgk5C78IXpGjxtNZngXurIfoXSxvFYK ALvVOatmKLIPchii7usm200OLkTyFgJXL3smkwqWM8Tf0+aGncwZIhmlw+0A4bsX /raevs3J8pK3ULsBoudKDgUcWHYsbux5Bo37HTmi/2VUNzYxFi+J+VKrjYtpkHgg == X-ME-Sender: Received: from xps.localnet (184.203.134.77.rev.sfr.net [77.134.203.184]) by mail.messagingengine.com (Postfix) with ESMTPA id 52CAC7E13A; Thu, 25 Jan 2018 09:58:53 -0500 (EST) From: Thomas Monjalon To: Yuanhan Liu Cc: =?ISO-8859-1?Q?Ga=EBtan?= Rivet , Ferruh Yigit , dev@dpdk.org Date: Thu, 25 Jan 2018 15:58:11 +0100 Message-ID: <8785061.Eg1DoYrWuq@xps> In-Reply-To: <20180125144142.GA29540@yliu-mob> References: <1516114218-21501-1-git-send-email-yliu@fridaylinux.org> <4586426.ySiLvJ04ZT@xps> <20180125144142.GA29540@yliu-mob> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" Subject: Re: [dpdk-dev] [PATCH] doc: document the new devargs syntax X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 25 Jan 2018 14:58:55 -0000 25/01/2018 15:41, Yuanhan Liu: > On Wed, Jan 24, 2018 at 05:57:34PM +0100, Thomas Monjalon wrote: > > 24/01/2018 16:04, Yuanhan Liu: > > > On Wed, Jan 24, 2018 at 11:37:31AM +0100, Thomas Monjalon wrote: > > > > 24/01/2018 11:36, Yuanhan Liu: > > > > > On Wed, Jan 24, 2018 at 11:21:44AM +0100, Thomas Monjalon wrote: > > > > > > 24/01/2018 10:28, Yuanhan Liu: > > > > > > > On Wed, Jan 24, 2018 at 09:19:10AM +0100, Thomas Monjalon wrote: > > > > > > > > 24/01/2018 07:43, Yuanhan Liu: > > > > > > > > > On Tue, Jan 23, 2018 at 03:29:34PM +0100, Thomas Monjalon wrote: > > > > > > > > > > 23/01/2018 13:46, Yuanhan Liu: > > > > > > > > > > > If port not found, then the whole string will be used for dev attachment. > > > > > > > > > > > It means we are attaching a port with PCI BDF == 04.00.0 AND > > > > > > > > > > > port == 0 (the 2nd port will not be attached). > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > And here is how the devargs would look like if "matching;settings" is > > > > > > > > > > > being used: > > > > > > > > > > > > > > > > > > > > > > bus=pci,id=04:00.0/class=eth,port=0;bus=pci,id=04:00.0/class=eth,port=0/driver=mlx4,mlx4_arg_A=val,... > > > > > > > > > > > > > > > > > > > > > > The part before ";" will be used for lookup and the later part will be > > > > > > > > > > > used for attachment. It should work. It just looks redundant. > > > > > > > > > > > > > > > > > > > > It does not have to be redundant. > > > > > > > > > > It can be: > > > > > > > > > > bus=pci,id=04:00.0/class=eth,port=0;driver=mlx4,mlx4_arg1=settings1,... > > > > > > > > > > > > > > > > > > I knew you would make such reply :) > > > > > > > > > Then there is a contradiction. According your suggestion, the "port=0" belongs > > > > > > > > > to the matching section, but it also has to be used in the settings section. > > > > > > > > > > > > > > > > If port=0 is matched, it is already set, right? > > > > > > > > > > > > > > Yes. > > > > > > > > > > > > > > > Why it needs to be in settings? > > > > > > > > > > > > > > But I was talking the case it's not matched, say it's not probed and here > > > > > > > we do hotplug. > > > > > > > > > > > > I don't understand. > > > > > > Anyway, the port property should be read-only. > > > > > > > > > > All proberties should be read-only. > > > > > > > > > > > Are we talking about the dev_port from the Linux kernel? > > > > > > > > > > Yes. And it can be used for probing one port only (out of 2 ports in a NIC) > > > > > at probe stage. So, at this stage, it's a setting but not a match. > > > > > > > > No it's a match! > > > > > > > > A settings is changing data in the port. > > > > > > So I see that's your definition about the "settings". What I think is > > > everything needed for driver initiation are settings. > > > > > > For example, one proposed interface for VF rep is the "vf_id" property, > > > Similar to "port" property we have just discussed above, it's used for > > > probing one specific VR rep for the given VF id. > > > > > > You can say it's a match here, just like the "port" property. > > > > > > But note that "vf_id" could be a range, to enable multiple VF reps. > > > The semantics looks like "setting" more than "match". > > > > Not sure why it would look like settings. > > > > > Another example is from the failsafe PMD that Gaetan had mentioned: > > > > > > driver=failsafe,dev(bus=pci,id=00:02.0),fd(/some/file/) > > > > > > They (dev and fd) should belong the "setting" section, for 2 reasons: > > > > > > - they should not be used for matching > > > - they are used for failsafe PMD initiation > > > > Yes these ones are settings. > > > > > But it belongs "match", according to your definition about "settings", > > > because it doesn't change data in the port. > > > > No, it changes data, dev() is adding a slave, and fd() is adding > > a file descriptor. > > Then "port" should be settings, as it's used for adding a port. > Moreover, "vf_id=0-3" is definitely settings (according your definition), > as it's going to add 4 VF rep ports. They are properties to identify a device. Question we should ask is which device is identified by this string? In the failsafe case, it is different because at first, we identify the failsafe vdev itself. So the slaves are not part of this. Slaves id are settings to bind another device to the failsafe. > > So we agree these are settings. > > > > > That also means, the word "settings" might not be well named. It's > > > probably better to name it "drvargs". > > > > I disagree. But it's only naming. > > Settings can be class settings, not only driver settings. > > And driver properties can be matching or settings. > > So "drvargs" does not make sense.