From: Yan Zhao <yan.y.zhao@intel.com>
To: "Tian, Kevin" <kevin.tian@intel.com>
Cc: Alex Williamson <alex.williamson@redhat.com>,
"intel-gvt-dev@lists.freedesktop.org"
<intel-gvt-dev@lists.freedesktop.org>,
"kvm@vger.kernel.org" <kvm@vger.kernel.org>,
Zhenyu Wang <zhenyuw@linux.intel.com>
Subject: Re: [PATCH v3 0/2] VFIO mdev aggregated resources handling
Date: Thu, 9 Jul 2020 15:23:34 +0800 [thread overview]
Message-ID: <20200709072334.GA26155@joy-OptiPlex-7040> (raw)
In-Reply-To: <MWHPR11MB1645C5033CB813EBD72CE4FD8C640@MWHPR11MB1645.namprd11.prod.outlook.com>
On Thu, Jul 09, 2020 at 02:53:05AM +0000, Tian, Kevin wrote:
<...>
> > We also can't even seem to agree that type is a necessary requirement
> > for compatibility. Your discussion below of a type-A, which is
> > equivalent to a type-B w/ aggregation set to some value is an example
> > of this. We might also have physical devices with extensions to
> > support migration. These could possibly be compatible with full mdev
> > devices. We have no idea how an administrative tool would discover
> > this other than an exhaustive search across every possible target.
> > That's ugly but feasible when considering a single target host, but
> > completely untenable when considering a datacenter.
>
> If exhaustive search can be done just one-off to build the compatibility
> database for all assignable devices on each node, then it might be
> still tenable in datacenter?
yes, Alex, do you think below behavior to build compatibility database is
acceptable?
management stack could do the exhaustive search in one shot to build the
compatibility database for all devices in every node. Meanwhile, it caches
migration version strings for all tested devices.
when there's a newly created/attached device, management stack could write
every cached strings to migration version attribute of the newly
created/attached device in order to update the migration compatibility
database. Then it caches the migration version string of the newly
created/attached device as well.
once a device attribute is modified, e.g. after changing its aggregation
count or updating its parent driver, update its cached migration version
string and update the compatibility database by testing against migration
version attribute of this device.
Thanks
Yan
next prev parent reply other threads:[~2020-07-09 7:34 UTC|newest]
Thread overview: 34+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-03-26 5:41 [PATCH v2 0/2] VFIO mdev aggregated resources handling Zhenyu Wang
2020-03-26 5:41 ` [PATCH v2 1/2] Documentation/driver-api/vfio-mediated-device.rst: update for aggregation support Zhenyu Wang
2020-03-26 8:17 ` Tian, Kevin
2020-03-26 8:21 ` Zhenyu Wang
2020-03-27 6:16 ` Tian, Kevin
2020-03-27 6:21 ` Zhenyu Wang
2020-03-26 5:41 ` [PATCH v2 2/2] drm/i915/gvt: mdev aggregation type Zhenyu Wang
2020-03-27 7:48 ` Tian, Kevin
2020-03-27 8:12 ` Zhenyu Wang
2020-03-27 8:44 ` Tian, Kevin
2020-03-27 8:58 ` Zhenyu Wang
2020-03-27 9:31 ` Tian, Kevin
2020-04-08 5:58 ` [PATCH v3 0/2] VFIO mdev aggregated resources handling Zhenyu Wang
2020-07-07 23:28 ` Tian, Kevin
2020-07-08 1:06 ` Alex Williamson
2020-07-08 1:54 ` Zhenyu Wang
2020-07-08 3:38 ` Yan Zhao
2020-07-08 3:40 ` Yan Zhao
2020-07-08 4:17 ` Alex Williamson
2020-07-08 6:31 ` Tian, Kevin
2020-07-08 9:54 ` Zhenyu Wang
2020-07-08 18:48 ` Alex Williamson
2020-07-09 2:53 ` Tian, Kevin
2020-07-09 7:23 ` Yan Zhao [this message]
2020-07-09 20:22 ` Alex Williamson
2020-07-10 1:58 ` Yan Zhao
2020-07-10 15:00 ` Alex Williamson
2020-07-09 17:28 ` Alex Williamson
2020-07-10 2:09 ` Tian, Kevin
2020-07-10 6:29 ` Yan Zhao
2020-07-10 15:12 ` Alex Williamson
2020-07-13 0:59 ` Yan Zhao
2020-04-08 5:58 ` [PATCH v3 1/2] Documentation/driver-api/vfio-mediated-device.rst: update for aggregation support Zhenyu Wang
2020-04-08 5:58 ` [PATCH v3 2/2] drm/i915/gvt: mdev aggregation type Zhenyu Wang
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=20200709072334.GA26155@joy-OptiPlex-7040 \
--to=yan.y.zhao@intel.com \
--cc=alex.williamson@redhat.com \
--cc=intel-gvt-dev@lists.freedesktop.org \
--cc=kevin.tian@intel.com \
--cc=kvm@vger.kernel.org \
--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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox