Re: [PATCH 2/2] s390/vfio-ap: add status attribute to AP queue device's sysfs dir

[Tony Krowiak <akrowiak@linux.ibm.com>]

On 10/28/21 3:40 AM, Harald Freudenberger wrote:
> On 27.10.21 18:48, Tony Krowiak wrote:
>> This patch adds a sysfs 'status' attribute to a queue device when it is
>> bound to the vfio_ap device driver. The field displays a string indicating
>> the status of the queue device:
>>
>> Status String:  Indicates:
>> -------------   ---------
>> "assigned"      the queue is assigned to an mdev, but is not in use by a
>>                  KVM guest.
>> "in use"        the queue is assigned to an mdev and is in use by a KVM
>>                  guest.
>> "unassigned"    the queue is not assigned to an mdev.
>>
>> The status string will be displayed by the 'lszcrypt' command if the queue
>> device is bound to the vfio_ap device driver.
>>
>> Signed-off-by: Harald Freudenberger <freude@linux.ibm.com>
>> [akrowiak@linux.ibm.com: added check for queue in use by guest]
>> Signed-off-by: Tony Krowiak <akrowiak@linux.ibm.com>Looks good to me, add my "Reviewed-by: Harald Freudenberger <freude@linux.ibm.com>"

You already have a Signed-off-by since you created the original version 
of this
patch. I added my Signed-off-by for adding the "in use" status value. 
Should I still
add a Reviewed-by for you?