From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mail-wg0-f45.google.com (mail-wg0-f45.google.com [74.125.82.45]) by dpdk.org (Postfix) with ESMTP id 7E5E4689B for ; Wed, 24 Dec 2014 09:01:41 +0100 (CET) Received: by mail-wg0-f45.google.com with SMTP id b13so10897499wgh.18 for ; Wed, 24 Dec 2014 00:01:41 -0800 (PST) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:message-id:date:from:user-agent:mime-version:to :cc:subject:references:in-reply-to:content-type; bh=d3ybF88z79GcEjGLPXam0vp6yOYUY0p0wp6WsVtjGpE=; b=B/oDZhh674+ajUFUSq5aSPWR81dsx/qeh1z8LkarV+sGkzmPxdAK8tFJaG6zc+Iv9B gCaoRYJuskOTzUubWnYP+lpUyARutXvVbmYVX1XqvsTWRv44+YQIMXzvMg+0SmYaITwR MaQxSxCsUQAGG8mj3gAMzZe9dcob7w0n7aKwsHbFVBHeb7FVe961+g34cS2Xp1/wdc82 wproESynYN2k9NQ+6a95yOw+RvjG1h4xfFyM4vdfZkuZy2/wWSV0krWXT7G4qQpP539/ L6RH05rBdjMrGG0CPSe7LdpVTIdLB5imCZOHLfG7+5Dhe0gfxwKte0DZjUmZPTyX+Kpe iUfg== X-Gm-Message-State: ALoCoQnXNSWge7piyP0lgK2QJ5uBp/BgemqyhFAOQcv8a3NsUmA/Ld3c1lXAcogCWMmmKdVSU/Kh X-Received: by 10.194.61.168 with SMTP id q8mr59686245wjr.53.1419408101282; Wed, 24 Dec 2014 00:01:41 -0800 (PST) Received: from [10.0.0.1] (bzq-79-178-154-55.red.bezeqint.net. [79.178.154.55]) by mx.google.com with ESMTPSA id e7sm30631243wjx.31.2014.12.24.00.01.40 (version=TLSv1.2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Wed, 24 Dec 2014 00:01:40 -0800 (PST) Message-ID: <549A72E3.9060606@cloudius-systems.com> Date: Wed, 24 Dec 2014 10:01:39 +0200 From: Vlad Zolotarov User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.3.0 MIME-Version: 1.0 To: "Ouyang, Changchun" References: <1418897516-25918-1-git-send-email-vladz@cloudius-systems.com> <5492D13C.4020006@cloudius-systems.com> In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit X-Content-Filtered-By: Mailman/MimeDel 2.1.15 Cc: "dev@dpdk.org" Subject: Re: [dpdk-dev] [PATCH 0/6] Enable VF RSS for Niantic X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 24 Dec 2014 08:01:41 -0000 On 12/24/14 08:37, Ouyang, Changchun wrote: > > Hi, > > I have v3 patch for this to resolve your comments, > > We can discuss on v3 patch if there are further comments. > I'm looking at your v3 patchset right now... ;) > Thanks > > Changchun > > *From:*Vladislav Zolotarov [mailto:vladz@cloudius-systems.com] > *Sent:* Sunday, December 21, 2014 4:02 PM > *To:* Ouyang, Changchun > *Cc:* dev@dpdk.org > *Subject:* RE: [PATCH 0/6] Enable VF RSS for Niantic > > > On Dec 19, 2014 3:35 AM, "Ouyang, Changchun" > > wrote: > > > > My response as below. > > > > > > > > From: Vlad Zolotarov [mailto:vladz@cloudius-systems.com > ] > > Sent: Thursday, December 18, 2014 9:06 PM > > To: dev@dpdk.org ; Ouyang, Changchun > > Subject: Re: [PATCH 0/6] Enable VF RSS for Niantic > > > > > > > > > > > > On 12/18/14 12:11, Vlad Zolotarov wrote: > >> > >> From: Changchun Ouyang > > >> > >> > >> > >> This patch enables VF RSS for Niantic, which allow each VF having > at most 4 queues. > >> > >> The actual queue number per VF depends on the number of VF: > >> > >> VF number from 1~32: 4 queues per VF; > >> > >> VF number from 33~max vf num: 2 queues per VF; > >> > >> > >> > >> On host, to enable VF RSS functionality, mq mode should be set as > ETH_MQ_RX_VMDQ_RSS > >> > >> or ETH_MQ_RX_RSS mode, and SRIOV mode should be activated. > >> > >> It also needs config VF RSS information like hash function, RSS > key, RSS key length. > > > > > > This patch series is missing a few things: > > > > Taking into the consideration the number of Rx queues requested by a > user in the rte_eth_dev_configure(). > > > > Changchun: yes, will have a v2 to consider it, as I say in my > previous response. > > I must have joined the mailing list after your v2 series. I'll take a > look in the web for it. > > > > > dev->dev_ops->reta_query used by a rte_eth_dev_rss_reta_query() is > still not initialized for a VF. Thus there is no way to query the RSS > table > > > > Changchun: do you mean query rss table on guest? > > Of course. > > Niantic doesn’t have separate reta for a specific vf, the reta is > shared by pf and all vf. > > AFAIK the reta is statically divided between all available functions. > > So, first of all we need a way to know that is the size of a single VF > partition in order to know how many RSS queues may be configured. > > Then we need an ability to query/set the contents of the corresponding > reta partition like we can for PF. > > > > > Do you think of returning all reta contents for each vf make sense? > Or any other insight here? > > > > rte_eth_dev_info_get() returns reta_size == 0 when called for a VF > function. > > > > Changchun: same as above. > > > > Your insight here are welcome here. > > > > thanks > > > > Changchun > > > > >