All of lore.kernel.org
 help / color / mirror / Atom feed
From: Sean Mooney <smooney@redhat.com>
To: Cornelia Huck <cohuck@redhat.com>, Yan Zhao <yan.y.zhao@intel.com>
Cc: "Alex Williamson" <alex.williamson@redhat.com>,
	"Daniel P.Berrangé" <berrange@redhat.com>,
	kvm@vger.kernel.org, libvir-list@redhat.com,
	"Jason Wang" <jasowang@redhat.com>,
	qemu-devel@nongnu.org, kwankhede@nvidia.com, eauger@redhat.com,
	xin-ran.wang@intel.com, corbet@lwn.net,
	openstack-discuss@lists.openstack.org, shaohe.feng@intel.com,
	kevin.tian@intel.com, "Parav Pandit" <parav@mellanox.com>,
	jian-feng.ding@intel.com, dgilbert@redhat.com,
	zhenyuw@linux.intel.com, hejie.xu@intel.com,
	bao.yumeng@zte.com.cn, intel-gvt-dev@lists.freedesktop.org,
	eskultet@redhat.com, "Jiri Pirko" <jiri@mellanox.com>,
	dinechin@redhat.com, devel@ovirt.org
Subject: Re: device compatibility interface for live migration with assigned devices
Date: Fri, 28 Aug 2020 15:04:12 +0100	[thread overview]
Message-ID: <8f5345be73ebf4f8f7f51d6cdc9c2a0d8e0aa45e.camel@redhat.com> (raw)
In-Reply-To: <20200828154741.30cfc1a3.cohuck@redhat.com>

On Fri, 2020-08-28 at 15:47 +0200, Cornelia Huck wrote:
> On Wed, 26 Aug 2020 14:41:17 +0800
> Yan Zhao <yan.y.zhao@intel.com> wrote:
> 
> > previously, we want to regard the two mdevs created with dsa-1dwq x 30 and
> > dsa-2dwq x 15 as compatible, because the two mdevs consist equal resources.
> > 
> > But, as it's a burden to upper layer, we agree that if this condition
> > happens, we still treat the two as incompatible.
> > 
> > To fix it, either the driver should expose dsa-1dwq only, or the target
> > dsa-2dwq needs to be destroyed and reallocated via dsa-1dwq x 30.
> 
> AFAIU, these are mdev types, aren't they? So, basically, any management
> software needs to take care to use the matching mdev type on the target
> system for device creation?

or just do the simple thing of use the same mdev type on the source and dest.
matching mdevtypes is not nessiarly trivial. we could do that but we woudl have
to do that in python rather then sql so it would be slower to do at least today.

we dont currently have the ablity to say the resouce provider must have 1 of these
set of traits. just that we must have a specific trait. this is a feature we have
disucssed a couple of times and delayed untill we really really need it but its not out
of the question that we could add it for this usecase. i suspect however we would do exact
match first and explore this later after the inital mdev migration works.

by the way i was looking at some vdpa reslated matiail today and noticed vdpa devices are nolonger
usign mdevs and and now use a vhost chardev so i guess we will need a completely seperate mechanioum
for vdpa vs mdev migration as a result. that is rather unfortunet but i guess that is life.
> 


WARNING: multiple messages have this Message-ID (diff)
From: Sean Mooney <smooney@redhat.com>
To: Cornelia Huck <cohuck@redhat.com>, Yan Zhao <yan.y.zhao@intel.com>
Cc: kvm@vger.kernel.org, libvir-list@redhat.com,
	"Jason Wang" <jasowang@redhat.com>,
	qemu-devel@nongnu.org, kwankhede@nvidia.com, eauger@redhat.com,
	xin-ran.wang@intel.com, corbet@lwn.net,
	openstack-discuss@lists.openstack.org, shaohe.feng@intel.com,
	kevin.tian@intel.com, "Parav Pandit" <parav@mellanox.com>,
	jian-feng.ding@intel.com, dgilbert@redhat.com,
	zhenyuw@linux.intel.com, hejie.xu@intel.com,
	bao.yumeng@zte.com.cn,
	"Alex Williamson" <alex.williamson@redhat.com>,
	intel-gvt-dev@lists.freedesktop.org,
	"Daniel P.Berrangé" <berrange@redhat.com>,
	eskultet@redhat.com, "Jiri Pirko" <jiri@mellanox.com>,
	dinechin@redhat.com, devel@ovirt.org
Subject: Re: device compatibility interface for live migration with assigned devices
Date: Fri, 28 Aug 2020 15:04:12 +0100	[thread overview]
Message-ID: <8f5345be73ebf4f8f7f51d6cdc9c2a0d8e0aa45e.camel@redhat.com> (raw)
In-Reply-To: <20200828154741.30cfc1a3.cohuck@redhat.com>

On Fri, 2020-08-28 at 15:47 +0200, Cornelia Huck wrote:
> On Wed, 26 Aug 2020 14:41:17 +0800
> Yan Zhao <yan.y.zhao@intel.com> wrote:
> 
> > previously, we want to regard the two mdevs created with dsa-1dwq x 30 and
> > dsa-2dwq x 15 as compatible, because the two mdevs consist equal resources.
> > 
> > But, as it's a burden to upper layer, we agree that if this condition
> > happens, we still treat the two as incompatible.
> > 
> > To fix it, either the driver should expose dsa-1dwq only, or the target
> > dsa-2dwq needs to be destroyed and reallocated via dsa-1dwq x 30.
> 
> AFAIU, these are mdev types, aren't they? So, basically, any management
> software needs to take care to use the matching mdev type on the target
> system for device creation?

or just do the simple thing of use the same mdev type on the source and dest.
matching mdevtypes is not nessiarly trivial. we could do that but we woudl have
to do that in python rather then sql so it would be slower to do at least today.

we dont currently have the ablity to say the resouce provider must have 1 of these
set of traits. just that we must have a specific trait. this is a feature we have
disucssed a couple of times and delayed untill we really really need it but its not out
of the question that we could add it for this usecase. i suspect however we would do exact
match first and explore this later after the inital mdev migration works.

by the way i was looking at some vdpa reslated matiail today and noticed vdpa devices are nolonger
usign mdevs and and now use a vhost chardev so i guess we will need a completely seperate mechanioum
for vdpa vs mdev migration as a result. that is rather unfortunet but i guess that is life.
> 



  reply	other threads:[~2020-08-28 14:04 UTC|newest]

Thread overview: 227+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-07-13 23:29 device compatibility interface for live migration with assigned devices Yan Zhao
2020-07-13 23:29 ` Yan Zhao
2020-07-14 10:21 ` Daniel P. Berrangé
2020-07-14 10:21   ` Daniel P. Berrangé
2020-07-14 12:33   ` Sean Mooney
2020-07-14 12:33     ` Sean Mooney
     [not found]     ` <20200714110148.0471c03c@x1.home>
     [not found]       ` <eb705c72cdc8b6b8959b6ebaeeac6069a718d524.camel@redhat.com>
2020-07-14 21:15         ` Sean Mooney
2020-07-14 21:15           ` Sean Mooney
2020-07-14 16:16   ` Alex Williamson
2020-07-14 16:16     ` Alex Williamson
2020-07-14 16:47     ` Daniel P. Berrangé
2020-07-14 16:47       ` Daniel P. Berrangé
2020-07-14 20:47       ` Alex Williamson
2020-07-14 20:47         ` Alex Williamson
2020-07-15  9:16         ` Daniel P. Berrangé
2020-07-15  9:16           ` Daniel P. Berrangé
2020-07-14 17:19     ` Dr. David Alan Gilbert
2020-07-14 17:19       ` Dr. David Alan Gilbert
2020-07-14 20:59       ` Alex Williamson
2020-07-14 20:59         ` Alex Williamson
2020-07-15  7:37         ` Alex Xu
2020-07-17 15:18           ` Alex Williamson
2020-07-17 15:18             ` Alex Williamson
2020-07-15  8:20         ` Yan Zhao
2020-07-15  8:20           ` Yan Zhao
2020-07-15  8:49           ` Feng, Shaohe
2020-07-15  8:49             ` Feng, Shaohe
2020-07-15  9:21           ` Alex Xu
2020-07-17 14:59           ` Alex Williamson
2020-07-17 14:59             ` Alex Williamson
2020-07-17 18:03             ` Dr. David Alan Gilbert
2020-07-17 18:03               ` Dr. David Alan Gilbert
2020-07-17 18:30               ` Alex Williamson
2020-07-17 18:30                 ` Alex Williamson
2020-07-15  8:23         ` Dr. David Alan Gilbert
2020-07-15  8:23           ` Dr. David Alan Gilbert
2020-07-15  7:23     ` Alex Xu
2020-07-16  4:16 ` Jason Wang
2020-07-16  4:16   ` Jason Wang
2020-07-16  8:32   ` Yan Zhao
2020-07-16  8:32     ` Yan Zhao
2020-07-16  9:30     ` Jason Wang
2020-07-17 16:12     ` Alex Williamson
2020-07-17 16:12       ` Alex Williamson
2020-07-20  3:41       ` Jason Wang
2020-07-20  3:41         ` Jason Wang
2020-07-20 10:39         ` Sean Mooney
2020-07-20 10:39           ` Sean Mooney
2020-07-21  2:11           ` Jason Wang
2020-07-21  2:11             ` Jason Wang
2020-07-21  0:51       ` Yan Zhao
2020-07-21  0:51         ` Yan Zhao
2020-07-27  7:24         ` Yan Zhao
2020-07-27 22:23           ` Alex Williamson
2020-07-29  8:05             ` Yan Zhao
2020-07-29 11:28               ` Sean Mooney
2020-07-29 11:28                 ` Sean Mooney
2020-07-29 19:12                 ` Alex Williamson
2020-07-29 19:12                   ` Alex Williamson
2020-07-30  3:41                   ` Yan Zhao
2020-07-30  3:41                     ` Yan Zhao
2020-07-30 13:24                     ` Sean Mooney
2020-07-30 13:24                       ` Sean Mooney
2020-07-30 17:29                     ` Alex Williamson
2020-07-30 17:29                       ` Alex Williamson
2020-08-04  8:37                       ` Yan Zhao
2020-08-04  8:37                         ` Yan Zhao
2020-08-05  9:44                         ` Dr. David Alan Gilbert
2020-08-05  9:44                           ` Dr. David Alan Gilbert
2020-07-30  1:56                 ` Yan Zhao
2020-07-30  1:56                   ` Yan Zhao
2020-07-30 13:14                   ` Sean Mooney
2020-07-30 13:14                     ` Sean Mooney
2020-08-04 16:35               ` Cornelia Huck
2020-08-04 16:35                 ` Cornelia Huck
2020-08-05  2:22                 ` Jason Wang
2020-08-05  2:22                   ` Jason Wang
2020-08-05  2:16                   ` Yan Zhao
2020-08-05  2:16                     ` Yan Zhao
2020-08-05  2:41                     ` Jason Wang
2020-08-05  2:41                       ` Jason Wang
2020-08-05  7:56                       ` Jiri Pirko
2020-08-05  7:56                         ` Jiri Pirko
2020-08-05  8:02                         ` Jason Wang
2020-08-05  8:02                           ` Jason Wang
2020-08-05  9:33                           ` Yan Zhao
2020-08-05  9:33                             ` Yan Zhao
2020-08-05 10:53                             ` Jiri Pirko
2020-08-05 10:53                               ` Jiri Pirko
2020-08-05 11:35                               ` Sean Mooney
2020-08-05 11:35                                 ` Sean Mooney
2020-08-07 11:59                                 ` Cornelia Huck
2020-08-07 11:59                                   ` Cornelia Huck
2020-08-13 15:33                                   ` Cornelia Huck
2020-08-13 15:33                                     ` Cornelia Huck
2020-08-13 19:02                                     ` Eric Farman
2020-08-13 19:02                                       ` Eric Farman
2020-08-17  6:38                                       ` Cornelia Huck
2020-08-17  6:38                                         ` Cornelia Huck
2020-08-10  7:46                               ` Yan Zhao
2020-08-10  7:46                                 ` Yan Zhao
2020-08-13  4:24                                 ` Jason Wang
2020-08-13  4:24                                   ` Jason Wang
2020-08-14  5:16                                   ` Yan Zhao
2020-08-14  5:16                                     ` Yan Zhao
2020-08-14 12:30                                     ` Sean Mooney
2020-08-14 12:30                                       ` Sean Mooney
2020-08-17  1:52                                       ` Yan Zhao
2020-08-17  1:52                                         ` Yan Zhao
2020-08-18  3:24                                     ` Jason Wang
2020-08-18  3:24                                       ` Jason Wang
2020-08-18  8:55                                       ` Daniel P. Berrangé
2020-08-18  8:55                                         ` Daniel P. Berrangé
2020-08-18  9:06                                         ` Cornelia Huck
2020-08-18  9:06                                           ` Cornelia Huck
2020-08-18  9:24                                           ` Daniel P. Berrangé
2020-08-18  9:24                                             ` Daniel P. Berrangé
2020-08-18  9:38                                             ` Cornelia Huck
2020-08-18  9:38                                               ` Cornelia Huck
     [not found]                                         ` <3a073222-dcfe-c02d-198b-29f6a507b2e1@redhat.com>
2020-08-18  9:16                                           ` Daniel P. Berrangé
2020-08-18  9:16                                             ` Daniel P. Berrangé
2020-08-18  9:36                                             ` Cornelia Huck
2020-08-18  9:36                                               ` Cornelia Huck
2020-08-18  9:39                                               ` Parav Pandit
2020-08-18  9:39                                                 ` Parav Pandit
2020-08-19  3:30                                                 ` Yan Zhao
2020-08-19  3:30                                                   ` Yan Zhao
2020-08-19  5:58                                                   ` Parav Pandit
2020-08-19  5:58                                                     ` Parav Pandit
2020-08-19  9:41                                                     ` Jason Wang
2020-08-19  9:41                                                       ` Jason Wang
2020-08-19  6:57                                                   ` [ovirt-devel] " Jason Wang
2020-08-19  6:57                                                     ` Jason Wang
2020-08-19  6:59                                                     ` Yan Zhao
2020-08-19  6:59                                                       ` Yan Zhao
2020-08-19  7:39                                                       ` Jason Wang
2020-08-19  7:39                                                         ` Jason Wang
2020-08-19  8:13                                                         ` Yan Zhao
2020-08-19  8:13                                                           ` Yan Zhao
2020-08-19  9:28                                                           ` Jason Wang
2020-08-19  9:28                                                             ` Jason Wang
2020-08-20 12:27                                                             ` Cornelia Huck
2020-08-20 12:27                                                               ` Cornelia Huck
2020-08-21  3:14                                                               ` Jason Wang
2020-08-21  3:14                                                                 ` Jason Wang
2020-08-21 14:52                                                                 ` Cornelia Huck
2020-08-21 14:52                                                                   ` Cornelia Huck
2020-08-31  3:07                                                                   ` Jason Wang
2020-08-31  3:07                                                                     ` Jason Wang
2020-08-19 17:50                                                   ` Alex Williamson
2020-08-19 17:50                                                     ` Alex Williamson
2020-08-20  0:18                                                     ` Yan Zhao
2020-08-20  0:18                                                       ` Yan Zhao
2020-08-20  3:13                                                       ` Alex Williamson
2020-08-20  3:13                                                         ` Alex Williamson
2020-08-20  3:09                                                         ` Yan Zhao
2020-08-20  3:09                                                           ` Yan Zhao
2020-08-19  2:54                                               ` Jason Wang
2020-08-19  2:54                                                 ` Jason Wang
2020-08-20  0:39                                               ` Yan Zhao
2020-08-20  0:39                                                 ` Yan Zhao
2020-08-20  1:29                                                 ` Sean Mooney
2020-08-20  1:29                                                   ` Sean Mooney
2020-08-20  4:01                                                   ` Yan Zhao
2020-08-20  4:01                                                     ` Yan Zhao
2020-08-20  5:16                                                     ` Sean Mooney
2020-08-20  5:16                                                       ` Sean Mooney
2020-08-20  6:27                                                       ` Yan Zhao
2020-08-20  6:27                                                         ` Yan Zhao
2020-08-20 13:24                                                         ` Sean Mooney
2020-08-20 13:24                                                           ` Sean Mooney
2020-08-26  8:54                                                           ` Yan Zhao
2020-08-26  8:54                                                             ` Yan Zhao
2020-08-20  3:22                                                 ` Alex Williamson
2020-08-20  3:22                                                   ` Alex Williamson
2020-08-20  3:16                                                   ` Yan Zhao
2020-08-20  3:16                                                     ` Yan Zhao
2020-08-25 14:39                                                     ` Cornelia Huck
2020-08-25 14:39                                                       ` Cornelia Huck
2020-08-26  6:41                                                       ` Yan Zhao
2020-08-26  6:41                                                         ` Yan Zhao
2020-08-28 13:47                                                         ` Cornelia Huck
2020-08-28 13:47                                                           ` Cornelia Huck
2020-08-28 14:04                                                           ` Sean Mooney [this message]
2020-08-28 14:04                                                             ` Sean Mooney
2020-08-31  4:43                                                             ` Yan Zhao
2020-08-31  4:43                                                               ` Yan Zhao
2020-09-08 14:41                                                               ` Cornelia Huck
2020-09-08 14:41                                                                 ` Cornelia Huck
2020-09-09  2:13                                                                 ` Yan Zhao
2020-09-09  2:13                                                                   ` Yan Zhao
2020-09-10 12:38                                                                   ` Cornelia Huck
2020-09-10 12:38                                                                     ` Cornelia Huck
2020-09-10 12:50                                                                     ` Sean Mooney
2020-09-10 12:50                                                                       ` Sean Mooney
2020-09-10 18:02                                                                       ` Alex Williamson
2020-09-10 18:02                                                                         ` Alex Williamson
2020-09-11  0:56                                                                         ` Yan Zhao
2020-09-11  0:56                                                                           ` Yan Zhao
2020-09-11 10:08                                                                           ` Cornelia Huck
2020-09-11 10:08                                                                             ` Cornelia Huck
2020-09-11 10:18                                                                             ` Tian, Kevin
2020-09-11 10:18                                                                               ` Tian, Kevin
2020-09-11 16:51                                                                           ` Alex Williamson
2020-09-11 16:51                                                                             ` Alex Williamson
2020-09-14 13:48                                                                             ` Zeng, Xin
2020-09-14 13:48                                                                               ` Zeng, Xin
2020-09-14 14:44                                                                               ` Alex Williamson
2020-09-14 14:44                                                                                 ` Alex Williamson
2020-09-15  7:46                                                                                 ` Zeng, Xin
2020-09-09  5:37                                                               ` Yan Zhao
2020-09-09  5:37                                                                 ` Yan Zhao
2020-08-31  2:23                                                           ` Yan Zhao
2020-08-19  2:38                                             ` Jason Wang
2020-08-19  2:38                                               ` Jason Wang
2020-08-18  9:32                                           ` Parav Pandit
2020-08-18  9:32                                             ` Parav Pandit
2020-08-19  2:45                                             ` Jason Wang
2020-08-19  2:45                                               ` Jason Wang
2020-08-19  5:26                                               ` Parav Pandit
2020-08-19  5:26                                                 ` Parav Pandit
2020-08-19  6:48                                                 ` Jason Wang
2020-08-19  6:48                                                   ` Jason Wang
2020-08-19  6:53                                                   ` Parav Pandit
2020-08-19  6:53                                                     ` Parav Pandit
2020-07-29 19:05             ` Dr. David Alan Gilbert
2020-07-29 19:05               ` Dr. David Alan Gilbert

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=8f5345be73ebf4f8f7f51d6cdc9c2a0d8e0aa45e.camel@redhat.com \
    --to=smooney@redhat.com \
    --cc=alex.williamson@redhat.com \
    --cc=bao.yumeng@zte.com.cn \
    --cc=berrange@redhat.com \
    --cc=cohuck@redhat.com \
    --cc=corbet@lwn.net \
    --cc=devel@ovirt.org \
    --cc=dgilbert@redhat.com \
    --cc=dinechin@redhat.com \
    --cc=eauger@redhat.com \
    --cc=eskultet@redhat.com \
    --cc=hejie.xu@intel.com \
    --cc=intel-gvt-dev@lists.freedesktop.org \
    --cc=jasowang@redhat.com \
    --cc=jian-feng.ding@intel.com \
    --cc=jiri@mellanox.com \
    --cc=kevin.tian@intel.com \
    --cc=kvm@vger.kernel.org \
    --cc=kwankhede@nvidia.com \
    --cc=libvir-list@redhat.com \
    --cc=openstack-discuss@lists.openstack.org \
    --cc=parav@mellanox.com \
    --cc=qemu-devel@nongnu.org \
    --cc=shaohe.feng@intel.com \
    --cc=xin-ran.wang@intel.com \
    --cc=yan.y.zhao@intel.com \
    --cc=zhenyuw@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.