From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ben Hutchings Subject: Re: [net-next RFC PATCH 2/5] tuntap: simple flow director support Date: Tue, 6 Dec 2011 17:31:44 +0000 Message-ID: <1323192704.2772.0.camel@bwh-desktop> References: <20111205085603.6116.65101.stgit@dhcp-8-146.nay.redhat.com> <20111205085857.6116.99252.stgit@dhcp-8-146.nay.redhat.com> <1323115763.2887.12.camel@bwh-desktop> <4EDDC27D.9050608@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <4EDDC27D.9050608@redhat.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: virtualization-bounces@lists.linux-foundation.org Errors-To: virtualization-bounces@lists.linux-foundation.org To: Jason Wang Cc: krkumar2@in.ibm.com, kvm@vger.kernel.org, mst@redhat.com, netdev@vger.kernel.org, virtualization@lists.linux-foundation.org, levinsasha928@gmail.com List-Id: virtualization@lists.linuxfoundation.org On Tue, 2011-12-06 at 15:21 +0800, Jason Wang wrote: > On 12/06/2011 04:09 AM, Ben Hutchings wrote: > > On Mon, 2011-12-05 at 16:58 +0800, Jason Wang wrote: > >> This patch adds a simple flow director to tun/tap device. It is just a > >> page that contains the hash to queue mapping which could be changed by > >> user-space. The backend (tap/macvtap) would query this table to get > >> the desired queue of a packets when it send packets to userspace. > > This is just flow hashing (RSS), not flow steering. > > > >> The page address were set through a new kind of ioctl - TUNSETFD and > >> were pinned until device exit or another new page were specified. > > [...] > > > > You should implement ethtool ETHTOOL_{G,S}RXFHINDIR instead. > > > > Ben. > > > > I'm not fully understanding this. The page belongs to guest, and the > idea is to let guest driver can easily change any entry. Looks like if > ethtool_set_rxfh_indir() is used, this kind of change is not easy as it > needs one copy and can only accept the whole table as its parameters. Sorry, yes, I was misreading this. Ben. -- Ben Hutchings, Staff Engineer, Solarflare Not speaking for my employer; that's the marketing department's job. They asked us to note that Solarflare product names are trademarked.