public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: Max Krasnyansky <maxk@qti.qualcomm.com>
To: Jason Wang <jasowang@redhat.com>
Cc: <davem@davemloft.net>, <mst@redhat.com>, <netdev@vger.kernel.org>,
	<linux-kernel@vger.kernel.org>, <edumazet@google.com>,
	<krkumar2@in.ibm.com>, <ernesto.martin@viasat.com>,
	<haixiao@juniper.net>
Subject: Re: [net-next v5 0/7] Multiqueue support in tuntap
Date: Thu, 1 Nov 2012 19:18:11 -0700	[thread overview]
Message-ID: <50932D63.3080308@qti.qualcomm.com> (raw)
In-Reply-To: <1351748762-3455-1-git-send-email-jasowang@redhat.com>

On 10/31/2012 10:45 PM, Jason Wang wrote:
> Hello All:
> 
> This is an update of multiqueue support in tuntap from V3. Please consider to
> merge.
> 
> The main idea for this series is to let tun/tap device to be benefited from
> multiqueue network cards and multi-core host. We used to have a single queue for
> tuntap which could be a bottleneck in a multiqueue/core environment. So this
> series let the device could be attched with multiple sockets and expose them
> through fd to the userspace as multiqueues. The sereis were orignally designed
> to serve as backend for multiqueue virtio-net in KVM, but the design is generic
> for other application to be used.
> 
> Some quick overview of the design:
> 
> - Moving socket from tun_device to tun_file.
> - Allowing multiple sockets to be attached to a tun/tap devices.
> - Using RCU to synchronize the data path and system call.
> - Two new ioctls were added for the usespace to attach and detach socket to the
>   device.
> - API compatibility were maintained without userspace notable changes, so legacy
>   userspace that only use one queue won't need any changes.
> - A flow(rxhash) to queue table were maintained by tuntap which choose the txq
>   based on the last rxq where it comes.

I'm still trying to wrap my head around the new locking/RCU stuff but it looks like
Paul and others already looked at it.

Otherwise looks good to me.

btw In the description above you really meant allowing for attaching multiple file
descriptors not sockets.

Thanks
Max



      parent reply	other threads:[~2012-11-02  2:18 UTC|newest]

Thread overview: 11+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-11-01  5:45 [net-next v5 0/7] Multiqueue support in tuntap Jason Wang
2012-11-01  5:45 ` [net-next v5 1/7] tuntap: log the unsigned informaiton with %u Jason Wang
2012-11-01  5:45 ` [net-next v5 2/7] tuntap: move socket to tun_file Jason Wang
2012-11-01  5:45 ` [net-next v5 3/7] tuntap: RCUify dereferencing between tun_struct and tun_file Jason Wang
2012-11-01  5:45 ` [net-next v5 4/7] tuntap: introduce multiqueue flags Jason Wang
2012-11-01  5:46 ` [net-next v5 5/7] tuntap: multiqueue support Jason Wang
2012-11-01  5:46 ` [net-next v5 6/7] tuntap: add ioctl to attach or detach a file form tuntap device Jason Wang
2012-11-12 15:11   ` Michael S. Tsirkin
2012-11-01  5:46 ` [net-next v5 7/7] tuntap: choose the txq based on rxq Jason Wang
2012-11-01 15:14 ` [net-next v5 0/7] Multiqueue support in tuntap David Miller
2012-11-02  2:18 ` Max Krasnyansky [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=50932D63.3080308@qti.qualcomm.com \
    --to=maxk@qti.qualcomm.com \
    --cc=davem@davemloft.net \
    --cc=edumazet@google.com \
    --cc=ernesto.martin@viasat.com \
    --cc=haixiao@juniper.net \
    --cc=jasowang@redhat.com \
    --cc=krkumar2@in.ibm.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mst@redhat.com \
    --cc=netdev@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox