From: "Michael S. Tsirkin" <mst@redhat.com>
To: Jesse Brandeburg <jesse.brandeburg@intel.com>
Cc: achiad shochat <achiad.mellanox@gmail.com>,
Jakub Kicinski <jakub.kicinski@netronome.com>,
Hannes Frederic Sowa <hannes@redhat.com>,
Sridhar Samudrala <sridhar.samudrala@intel.com>,
Alexander Duyck <alexander.duyck@gmail.com>,
virtualization@lists.linux-foundation.org,
Shannon Nelson <shannon.nelson@oracle.com>,
Achiad <achiad@mellanox.com>,
Peter Waskiewicz Jr <peter.waskiewicz.jr@intel.com>,
netdev <netdev@vger.kernel.org>,
Anjali Singhai Jain <anjali.singhai@intel.com>,
Andy Gospodarek <gospo@broadcom.com>,
Or Gerlitz <gerlitz.or@gmail.com>
Subject: Re: [RFC] virtio-net: help live migrate SR-IOV devices
Date: Wed, 6 Dec 2017 00:05:22 +0200 [thread overview]
Message-ID: <20171205235457-mutt-send-email-mst@kernel.org> (raw)
In-Reply-To: <20171205135226.00002b68@intel.com>
On Tue, Dec 05, 2017 at 01:52:26PM -0800, Jesse Brandeburg wrote:
> On Tue, 5 Dec 2017 21:20:07 +0200
> "Michael S. Tsirkin" <mst@redhat.com> wrote:
>
> > On Tue, Dec 05, 2017 at 11:59:17AM +0200, achiad shochat wrote:
> > > Then we'll have a single solution for both netvsc and virtio (and any
> > > other PV device).
> > > And we could handle the VF DMA dirt issue agnostically.
> >
> > For the record, I won't block patches adding this kist to virtio
> > on the basis that they must be generic. It's not a lot
> > of code, implementation can come first, prettify later.
>
> Thanks, based on this discussion we're going to work on improving
> virtio-net first, but some of Achiad's points are good. I don't believe
> it should block the virtio work however.
>
> In particular I'm really interested in figuring out how we can get to
> the point that virtio is able to make or implement some smart decisions
> about which NIC to pick for traffic delivery (it's own paravirt path or
> the passthorugh device path), if Achiad wants to develop the idea into
> some code, I'd be interested to review it.
>
> > But we do need to have a discussion about how devices are paired.
> > I am not sure using just MAC works. E.g. some passthrough
> > devices don't give host ability to set the MAC.
> > Are these worth worrying about?
>
> I personally don't think that will be much of a problem, if a
> certain device has that issue, can't we just have the virtio-net device
> pick up the MAC address of the passthrough device?
Then what do you do after you have migrated to another box?
The PT device there likely has a different MAC.
> As long as they match
> things should work OK. It at least is an initial way to do the
> configuration that has at least some traction as workable, as proved by
> the Microsoft design.
Yes - that design just implements what people have been doing for years
using bond so of course it's workable.
> FWIW, the Intel SR-IOV devices all accept a hypervisor/host provided
> MAC address.
For VFs you often can program the MAC through the PF, but you typically
can't do this for PFs. Or as another example consider nested virt with a
VF passed through. PF isn't there within L1 guest so can't be used to
program the mac of the VF.
Still, we can always start small and require same mac, add other ways
to address issues later as we come up with them.
--
MST
next prev parent reply other threads:[~2017-12-05 22:05 UTC|newest]
Thread overview: 25+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20171128112722.00003716@intel.com>
2017-11-30 3:29 ` [RFC] virtio-net: help live migrate SR-IOV devices Jason Wang
2017-11-30 3:51 ` Jakub Kicinski
2017-11-30 4:10 ` Stephen Hemminger
2017-11-30 4:21 ` Jakub Kicinski
2017-11-30 13:54 ` Michael S. Tsirkin
2017-11-30 20:48 ` Jakub Kicinski
2017-12-01 5:13 ` Michael S. Tsirkin
2017-11-30 8:08 ` achiad shochat
2017-11-30 14:11 ` Michael S. Tsirkin
2017-12-01 20:08 ` Shannon Nelson
2017-12-03 5:05 ` Michael S. Tsirkin
2017-12-03 9:14 ` achiad shochat
2017-12-03 17:35 ` Stephen Hemminger
2017-12-04 9:51 ` achiad shochat
2017-12-04 16:30 ` Alexander Duyck
2017-12-05 9:59 ` achiad shochat
2017-12-05 19:20 ` Michael S. Tsirkin
2017-12-05 21:52 ` Jesse Brandeburg
2017-12-05 22:05 ` Michael S. Tsirkin [this message]
2017-12-07 7:28 ` achiad shochat
2017-12-07 16:45 ` Alexander Duyck
2017-12-07 16:53 ` Michael S. Tsirkin
2017-12-05 22:29 ` Jakub Kicinski
2017-12-05 22:41 ` Stephen Hemminger
2017-11-30 14:14 ` Michael S. Tsirkin
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=20171205235457-mutt-send-email-mst@kernel.org \
--to=mst@redhat.com \
--cc=achiad.mellanox@gmail.com \
--cc=achiad@mellanox.com \
--cc=alexander.duyck@gmail.com \
--cc=anjali.singhai@intel.com \
--cc=gerlitz.or@gmail.com \
--cc=gospo@broadcom.com \
--cc=hannes@redhat.com \
--cc=jakub.kicinski@netronome.com \
--cc=jesse.brandeburg@intel.com \
--cc=netdev@vger.kernel.org \
--cc=peter.waskiewicz.jr@intel.com \
--cc=shannon.nelson@oracle.com \
--cc=sridhar.samudrala@intel.com \
--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;
as well as URLs for NNTP newsgroup(s).