Re: [PATCH v1 13/18] vfio/mdev: Consolidate all the device_api sysfs into the core code

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Subject: Re: [PATCH v1 13/18] vfio/mdev: Consolidate all the device_api sysfs into the core code
From: Christoph Hellwig <hch@xxxxxxxxxxxxx>
Date: Thu, 2 Jun 2022 23:36:13 -0700
Cc: Matthew Rosato <mjrosato@xxxxxxxxxxxxx>, Jason Gunthorpe <jgg@xxxxxxxxxx>, Alex Williamson <alex.williamson@xxxxxxxxxx>, Liu Yi L <yi.l.liu@xxxxxxxxx>, Halil Pasic <pasic@xxxxxxxxxxxxx>, kvm@xxxxxxxxxxxxxxx, linux-s390@xxxxxxxxxxxxxxx, Kirti Wankhede <kwankhede@xxxxxxxxxx>, Jonathan Corbet <corbet@xxxxxxx>, Zhenyu Wang <zhenyuw@xxxxxxxxxxxxxxx>, Zhi Wang <zhi.a.wang@xxxxxxxxx>, Tony Krowiak <akrowiak@xxxxxxxxxxxxx>, Jason Herne <jjherne@xxxxxxxxxxxxx>, intel-gvt-dev@xxxxxxxxxxxxxxxxxxxxx
In-reply-to: <20220602171948.2790690-14-farman@linux.ibm.com>
References: <20220602171948.2790690-1-farman@linux.ibm.com> <20220602171948.2790690-14-farman@linux.ibm.com>

On Thu, Jun 02, 2022 at 07:19:43PM +0200, Eric Farman wrote:
> From: Jason Gunthorpe <jgg@xxxxxxxxxx>
> 
> Every driver just emits a static string, simply feed it through the ops
> and provide a standard sysfs show function.

Looks good:

Reviewed-by: Christoph Hellwig <hch@xxxxxx>

References:
- [PATCH v1 00/18] VFIO ccw/mdev rework
  - From: Eric Farman
- [PATCH v1 13/18] vfio/mdev: Consolidate all the device_api sysfs into the core code
  - From: Eric Farman

Prev by Date: [PATCH 8/8] vfio/mdev: remove mtype_get_parent_dev
Next by Date: [PATCH v11 18/19] KVM: s390: pv: avoid export before import if possible
Previous by thread: [PATCH v1 13/18] vfio/mdev: Consolidate all the device_api sysfs into the core code
Next by thread: Re: [PATCH v1 13/18] vfio/mdev: Consolidate all the device_api sysfs into the core code
Index(es):
- Date
- Thread

[Index of Archives] [KVM ARM] [KVM ia64] [KVM ppc] [Virtualization Tools] [Spice Development] [Libvirt] [Libvirt Users] [Linux USB Devel] [Linux Audio Users] [Yosemite Questions] [Linux Kernel] [Linux SCSI] [XFree86]