From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kirti Wankhede Subject: Re: [libvirt] [RFC PATCH 0/2] Add new mdev type for aggregated resources Date: Thu, 21 Jun 2018 19:57:38 +0530 Message-ID: References: <20180620074039.10539-1-zhenyuw@linux.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Cc: intel-gvt-dev@lists.freedesktop.org, kevin.tian@intel.com To: Zhenyu Wang , , Return-path: In-Reply-To: <20180620074039.10539-1-zhenyuw@linux.intel.com> Content-Language: en-US List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: libvir-list-bounces@redhat.com Errors-To: libvir-list-bounces@redhat.com List-Id: kvm.vger.kernel.org On 6/20/2018 1:10 PM, Zhenyu Wang wrote: > Current mdev device create interface depends on fixed mdev type, which get uuid > from user to create instance of mdev device. If user wants to use customized > number of resource for mdev device, then only can create new mdev type for that > which may not be flexible. > > To allow to create user defined resources for mdev, this RFC trys > to extend mdev create interface by adding new "instances=xxx" parameter > following uuid, for target mdev type if aggregation is supported, it can > create new mdev device which contains resources combined by number of > instances, e.g > > echo ",instances=10" > create This seems orthogonal to the way mdev types are meant to be used. Vendor driver can provide the possible types to provide flexibility to the users. Secondly, not always all resources defined for a particular mdev type can be multiplied, for example, a mdev type for vGPU that supports 2 heads, that can't be multiplied to use with 20 heads. Thanks, Kirti > > VM manager e.g libvirt can check mdev type with "aggregation" attribute > which can support this setting. And new sysfs attribute "instances" is > created for each mdev device to show allocated number. Default number > of 1 or no "instances" file can be used for compatibility check. > > This RFC trys to create new KVMGT type with minimal vGPU resources which > can be combined with "instances=x" setting to allocate for user wanted resources. > > Zhenyu Wang (2): > vfio/mdev: Add new instances parameters for mdev create > drm/i915/gvt: Add new aggregation type > > drivers/gpu/drm/i915/gvt/gvt.c | 26 ++++++++++++--- > drivers/gpu/drm/i915/gvt/gvt.h | 14 +++++--- > drivers/gpu/drm/i915/gvt/kvmgt.c | 9 +++-- > drivers/gpu/drm/i915/gvt/vgpu.c | 56 ++++++++++++++++++++++++++++---- > drivers/s390/cio/vfio_ccw_ops.c | 3 +- > drivers/vfio/mdev/mdev_core.c | 11 ++++--- > drivers/vfio/mdev/mdev_private.h | 6 +++- > drivers/vfio/mdev/mdev_sysfs.c | 42 ++++++++++++++++++++---- > include/linux/mdev.h | 3 +- > samples/vfio-mdev/mbochs.c | 3 +- > samples/vfio-mdev/mdpy.c | 3 +- > samples/vfio-mdev/mtty.c | 3 +- > 12 files changed, 141 insertions(+), 38 deletions(-) >