Re: [PATCH v12 02/17] s390/vfio-ap: decrement reference count to KVM

2020-11-26 Thread Halil Pasic
On Tue, 24 Nov 2020 16:40:01 -0500
Tony Krowiak  wrote:

> Decrement the reference count to KVM when notified that KVM pointer is
> invalidated via the vfio group notifier.

Can you please explain more thoroughly. Is this a bug you found? If
yes do we need to backport it (cc stabe, fixes tag)? 

It doesn't see related to the objective of the series. If not related,
why not spin it separately?


> 
> Signed-off-by: Tony Krowiak 

This s-o-b is probably by accident.

> Signed-off-by: Tony Krowiak 
> ---
>  drivers/s390/crypto/vfio_ap_ops.c | 4 
>  1 file changed, 4 insertions(+)
> 
> diff --git a/drivers/s390/crypto/vfio_ap_ops.c 
> b/drivers/s390/crypto/vfio_ap_ops.c
> index 66fd9784a156..31e39c1f6e56 100644
> --- a/drivers/s390/crypto/vfio_ap_ops.c
> +++ b/drivers/s390/crypto/vfio_ap_ops.c
> @@ -1095,7 +1095,11 @@ static int vfio_ap_mdev_group_notifier(struct 
> notifier_block *nb,
>   matrix_mdev = container_of(nb, struct ap_matrix_mdev, group_notifier);
>  
>   if (!data) {
> + if (matrix_mdev->kvm)
> + kvm_put_kvm(matrix_mdev->kvm);
> +
>   matrix_mdev->kvm = NULL;
> +
>   return NOTIFY_OK;
>   }
>  



[PATCH v12 02/17] s390/vfio-ap: decrement reference count to KVM

2020-11-24 Thread Tony Krowiak
Decrement the reference count to KVM when notified that KVM pointer is
invalidated via the vfio group notifier.

Signed-off-by: Tony Krowiak 
Signed-off-by: Tony Krowiak 
---
 drivers/s390/crypto/vfio_ap_ops.c | 4 
 1 file changed, 4 insertions(+)

diff --git a/drivers/s390/crypto/vfio_ap_ops.c 
b/drivers/s390/crypto/vfio_ap_ops.c
index 66fd9784a156..31e39c1f6e56 100644
--- a/drivers/s390/crypto/vfio_ap_ops.c
+++ b/drivers/s390/crypto/vfio_ap_ops.c
@@ -1095,7 +1095,11 @@ static int vfio_ap_mdev_group_notifier(struct 
notifier_block *nb,
matrix_mdev = container_of(nb, struct ap_matrix_mdev, group_notifier);
 
if (!data) {
+   if (matrix_mdev->kvm)
+   kvm_put_kvm(matrix_mdev->kvm);
+
matrix_mdev->kvm = NULL;
+
return NOTIFY_OK;
}
 
-- 
2.21.1