DPDK patches and discussions
 help / color / mirror / Atom feed
From: Maxime Coquelin <maxime.coquelin@redhat.com>
To: "Michael S. Tsirkin" <mst@redhat.com>
Cc: Ciara Loftus <ciara.loftus@intel.com>,
	mark.b.kavanagh@intel.com, Flavio Leitner <fleitner@redhat.com>,
	Daniele Di Proietto <diproiettod@vmware.com>,
	"dev@openvswitch.org" <dev@openvswitch.org>,
	Kevin Traynor <ktraynor@redhat.com>,
	"Daniel P. Berrange" <berrange@redhat.com>,
	Yuanhan Liu <yuanhan.liu@linux.intel.com>,
	"dev@dpdk.org" <dev@dpdk.org>,
	"libvir-list@redhat.com" <libvir-list@redhat.com>,
	sean.k.mooney@intel.com
Subject: Re: [dpdk-dev] [RFC] Vhost-user backends cross-version migration support
Date: Fri, 3 Feb 2017 18:22:08 +0100	[thread overview]
Message-ID: <bb3a766d-1c62-7ad3-a570-2aa70ed2dad0@redhat.com> (raw)
In-Reply-To: <20170203172140-mutt-send-email-mst@kernel.org>



On 02/03/2017 04:34 PM, Michael S. Tsirkin wrote:
> On Fri, Feb 03, 2017 at 03:11:10PM +0100, Maxime Coquelin wrote:
>> Hi,
>>
>> On 02/01/2017 09:35 AM, Maxime Coquelin wrote:
>>> Hi,
>>>
>>>  Few months ago, Michael reported a problem about migrating VMs relying
>>> on vhost-user between hosts supporting different backend versions:
>>>  - Message-Id: <20161011173526-mutt-send-email-mst@kernel.org>
>>>  - https://lists.gnu.org/archive/html/qemu-devel/2016-10/msg03026.html
>>>
>>>  The goal of this thread is to draft a proposal based on the outcomes
>>> of discussions with contributors of the different parties (DPDK/OVS
>>> /libvirt/...).
>>
>> Thanks the first feedback. It seems to converge that this is Nova's
>> role, but not Libvirt one to manage these versions from management tool
>> layer.
>
>
> I think the conclusion is not that it should go up the stack.  I think
> this will just get broken all the time.  No one understands versions and
> stuff. Even QEMU developers get confused and break compatibility once in
> a while.

Is it that difficult for OVS to know with which versions of OVS it is
compatible with, and propose a way to be compatible with older versions?

 From Nova perspective, it just has to collect supported versions
strings, and select the most recent common one.

> My conclusion is that doing it from OVS side is wrong.  Migration is not
> an OVS thing, it's a QEMU thing, and libvirt abstracts QEMU.    People
> just want migration to work, ok? It's our job to do it, we do not really
> need a "make things work" flag.
>
> If libvirt does not want to use the vhost-user protocol (which sounds
> reasonable, it's rather complex) how about qemu providing a small
> utility to query the port?  We could output json or whatever.

Doing this, it would be libvirt role to compare key/value pairs in the 
json output, right? If so, it would mean teaching libvirt to compare
things it knows nothing about, that are likely to evolve over time.

Also, it means the port will have already been created, so I'm not
clear how we would manage compatibility version.

>
> This can help with MTU as well.
>
> And maybe it will help with nowait support - if someone uses the utility
> to dump backend config once, QEMU can later start the device without
> feature queries.

Regards,
Maxime

      parent reply	other threads:[~2017-02-03 17:22 UTC|newest]

Thread overview: 30+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-02-01  8:35 Maxime Coquelin
2017-02-01  9:14 ` [dpdk-dev] [libvirt] " Michal Privoznik
2017-02-01  9:43   ` Daniel P. Berrange
2017-02-01 11:33     ` Maxime Coquelin
2017-02-01 11:41       ` Daniel P. Berrange
2017-02-01 22:32         ` Michael S. Tsirkin
2017-02-02 14:14         ` Maxime Coquelin
2017-02-02 15:06           ` Daniel P. Berrange
2017-02-02 16:21             ` Michael S. Tsirkin
2017-02-02 17:10               ` Daniel P. Berrange
2017-02-02 17:20                 ` Michael S. Tsirkin
2017-02-02 17:29                   ` Daniel P. Berrange
2017-02-02 17:31                     ` Michael S. Tsirkin
2017-02-02 18:21                       ` Daniel P. Berrange
2017-02-02 18:27                         ` Michael S. Tsirkin
2017-02-03  9:27                           ` Daniel P. Berrange
2017-02-03  9:41                             ` Maxime Coquelin
2017-02-03 10:11                               ` Daniel P. Berrange
2017-02-03 11:36                                 ` Maxime Coquelin
2017-02-02 16:47             ` Laine Stump
2017-02-02 17:09               ` Michael S. Tsirkin
2017-02-02 17:13                 ` Daniel P. Berrange
2017-02-02 17:16                 ` Maxime Coquelin
2017-02-03  9:12                   ` Michal Privoznik
2017-02-03 17:40                     ` Laine Stump
2017-02-03 14:11 ` [dpdk-dev] " Maxime Coquelin
2017-02-03 15:34   ` Michael S. Tsirkin
2017-02-03 15:54     ` Daniel P. Berrange
2017-02-03 16:10       ` Michael S. Tsirkin
2017-02-03 17:22     ` Maxime Coquelin [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=bb3a766d-1c62-7ad3-a570-2aa70ed2dad0@redhat.com \
    --to=maxime.coquelin@redhat.com \
    --cc=berrange@redhat.com \
    --cc=ciara.loftus@intel.com \
    --cc=dev@dpdk.org \
    --cc=dev@openvswitch.org \
    --cc=diproiettod@vmware.com \
    --cc=fleitner@redhat.com \
    --cc=ktraynor@redhat.com \
    --cc=libvir-list@redhat.com \
    --cc=mark.b.kavanagh@intel.com \
    --cc=mst@redhat.com \
    --cc=sean.k.mooney@intel.com \
    --cc=yuanhan.liu@linux.intel.com \
    /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).