Linux virtualization list
 help / color / mirror / Atom feed
From: Jiri Pirko <jiri@resnulli.us>
To: Sridhar Samudrala <sridhar.samudrala@intel.com>
Cc: alexander.h.duyck@intel.com, virtio-dev@lists.oasis-open.org,
	mst@redhat.com, kubakici@wp.pl, netdev@vger.kernel.org,
	virtualization@lists.linux-foundation.org, loseweigh@gmail.com,
	aaron.f.brown@intel.com, davem@davemloft.net
Subject: Re: [PATCH net-next v9 3/4] virtio_net: Extend virtio to use VF datapath when available
Date: Sat, 28 Apr 2018 10:24:33 +0200	[thread overview]
Message-ID: <20180428082433.GK5632@nanopsycho.orion> (raw)
In-Reply-To: <1524848820-42258-4-git-send-email-sridhar.samudrala@intel.com>

Fri, Apr 27, 2018 at 07:06:59PM CEST, sridhar.samudrala@intel.com wrote:
>This patch enables virtio_net to switch over to a VF datapath when a VF
>netdev is present with the same MAC address. It allows live migration
>of a VM with a direct attached VF without the need to setup a bond/team
>between a VF and virtio net device in the guest.
>
>The hypervisor needs to enable only one datapath at any time so that
>packets don't get looped back to the VM over the other datapath. When a VF

Why? Both datapaths could be enabled at a time. Why the loop on
hypervisor side would be a problem. This in not an issue for
bonding/team as well.


>is plugged, the virtio datapath link state can be marked as down. The
>hypervisor needs to unplug the VF device from the guest on the source host
>and reset the MAC filter of the VF to initiate failover of datapath to

"reset the MAC filter of the VF" - you mean "set the VF mac"?


>virtio before starting the migration. After the migration is completed,
>the destination hypervisor sets the MAC filter on the VF and plugs it back
>to the guest to switch over to VF datapath.
>
>It uses the generic failover framework that provides 2 functions to create
>and destroy a master failover netdev. When STANDBY feature is enabled, an
>additional netdev(failover netdev) is created that acts as a master device
>and tracks the state of the 2 lower netdevs. The original virtio_net netdev
>is marked as 'standby' netdev and a passthru device with the same MAC is
>registered as 'primary' netdev.
>
>This patch is based on the discussion initiated by Jesse on this thread.
>https://marc.info/?l=linux-virtualization&m=151189725224231&w=2

[...]

  parent reply	other threads:[~2018-04-28  8:24 UTC|newest]

Thread overview: 38+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <1524848820-42258-1-git-send-email-sridhar.samudrala@intel.com>
2018-04-27 17:06 ` [PATCH net-next v9 1/4] virtio_net: Introduce VIRTIO_NET_F_STANDBY feature bit Sridhar Samudrala
2018-04-27 17:06 ` [PATCH net-next v9 2/4] net: Introduce generic failover module Sridhar Samudrala
2018-04-27 17:06 ` [PATCH net-next v9 3/4] virtio_net: Extend virtio to use VF datapath when available Sridhar Samudrala
2018-04-27 17:07 ` [PATCH net-next v9 4/4] netvsc: refactor notifier/event handling code to use the failover framework Sridhar Samudrala
2018-04-27 17:45 ` [PATCH net-next v9 0/4] Enable virtio_net to act as a standby for a passthru device Jiri Pirko
     [not found] ` <20180427174523.GE5632@nanopsycho.orion>
2018-04-27 17:53   ` Samudrala, Sridhar
     [not found]   ` <a60d08cd-6c8b-ea1e-e76c-4decba563e99@intel.com>
2018-04-27 19:38     ` Jiri Pirko
     [not found] ` <1524848820-42258-3-git-send-email-sridhar.samudrala@intel.com>
2018-04-27 17:53   ` [PATCH net-next v9 2/4] net: Introduce generic failover module Jiri Pirko
2018-04-28  8:15   ` Jiri Pirko
2018-04-28  9:06   ` Jiri Pirko
2018-04-30  3:03     ` Samudrala, Sridhar
2018-05-02 16:15     ` Jiri Pirko
     [not found]     ` <20180502161542.GI19250@nanopsycho>
2018-05-02 17:51       ` Samudrala, Sridhar
     [not found]       ` <052e2c60-dc4c-d6a0-0159-fc1821cb4365@intel.com>
2018-05-02 20:30         ` Michael S. Tsirkin
     [not found]         ` <20180502232907-mutt-send-email-mst@kernel.org>
2018-05-02 21:36           ` Samudrala, Sridhar
2018-05-02 21:39         ` Jiri Pirko
2018-05-02 21:39         ` Jiri Pirko
     [not found]   ` <20180428081542.GJ5632@nanopsycho.orion>
2018-04-30  2:47     ` Samudrala, Sridhar
     [not found] ` <1524848820-42258-2-git-send-email-sridhar.samudrala@intel.com>
2018-04-28  7:50   ` [PATCH net-next v9 1/4] virtio_net: Introduce VIRTIO_NET_F_STANDBY feature bit Jiri Pirko
     [not found]   ` <20180428075027.GI5632@nanopsycho.orion>
2018-04-30  2:47     ` Samudrala, Sridhar
     [not found]     ` <9826909e-d6cd-a0a3-142f-6f7f8cf2b5ce@intel.com>
2018-04-30  7:03       ` Jiri Pirko
2018-04-30 19:14         ` Samudrala, Sridhar
     [not found] ` <1524848820-42258-4-git-send-email-sridhar.samudrala@intel.com>
2018-04-28  8:24   ` Jiri Pirko [this message]
2018-04-30  3:00     ` [PATCH net-next v9 3/4] virtio_net: Extend virtio to use VF datapath when available Samudrala, Sridhar
     [not found]     ` <62f3b81b-70f6-6991-8e23-2bf650ecea2d@intel.com>
2018-04-30  7:12       ` Jiri Pirko
     [not found]       ` <20180430071208.GG23854@nanopsycho.orion>
2018-04-30 19:26         ` Samudrala, Sridhar
     [not found]         ` <c965d7f2-9ba6-fc82-ce1c-4d70c3aceb6d@intel.com>
2018-05-01  7:33           ` Jiri Pirko
2018-04-28  9:42   ` Jiri Pirko
     [not found]   ` <20180428094205.GM5632@nanopsycho.orion>
2018-04-29  8:56     ` Siwei Liu
     [not found]     ` <CADGSJ23A4w=MmnP7CneBwk=eouS07HfK4=1UWTk3Dz3gBX0yjg@mail.gmail.com>
2018-04-29 13:45       ` Jiri Pirko
2018-04-30  4:16     ` Samudrala, Sridhar
     [not found]     ` <638e52c7-64db-9d8d-7530-f6b000d3e292@intel.com>
2018-04-30  7:20       ` Jiri Pirko
     [not found]       ` <20180430072034.GH23854@nanopsycho.orion>
2018-05-02  0:20         ` Samudrala, Sridhar
     [not found]         ` <9c6f055c-c665-0601-3973-bce74d72544b@intel.com>
2018-05-02  7:50           ` Jiri Pirko
     [not found]           ` <20180502075021.GC19250@nanopsycho>
2018-05-02 15:34             ` Samudrala, Sridhar
2018-05-02 15:47             ` Michael S. Tsirkin
     [not found]             ` <20180502184326-mutt-send-email-mst@kernel.org>
2018-05-02 16:04               ` Jiri Pirko
     [not found]             ` <c94ce6a9-6f36-675c-cf5b-414454fc2244@intel.com>
2018-05-02 16:05               ` Jiri Pirko

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=20180428082433.GK5632@nanopsycho.orion \
    --to=jiri@resnulli.us \
    --cc=aaron.f.brown@intel.com \
    --cc=alexander.h.duyck@intel.com \
    --cc=davem@davemloft.net \
    --cc=kubakici@wp.pl \
    --cc=loseweigh@gmail.com \
    --cc=mst@redhat.com \
    --cc=netdev@vger.kernel.org \
    --cc=sridhar.samudrala@intel.com \
    --cc=virtio-dev@lists.oasis-open.org \
    --cc=virtualization@lists.linux-foundation.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