Received: by 2002:a05:7412:31a9:b0:e2:908c:2ebd with SMTP id et41csp3345193rdb; Wed, 13 Sep 2023 09:13:52 -0700 (PDT) X-Google-Smtp-Source: AGHT+IGw566xam80JMytyDL7nWjMtnCVZsQo1EgHyFeVBepVX1seFXPgNC7YVroD16SOGuh4yE1I X-Received: by 2002:a17:90a:53e6:b0:26b:6a2f:7d90 with SMTP id y93-20020a17090a53e600b0026b6a2f7d90mr2759675pjh.23.1694621631833; Wed, 13 Sep 2023 09:13:51 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1694621631; cv=none; d=google.com; s=arc-20160816; b=dV/gVU2uwakxBfSF4MsNsDvF5i3Li6onWsQjdntLYs2nF8Qnc2f+Ia7CFfS1NLcMs5 u+aZNEqlRT0/mFGj+6eW21+mlRT93te2Zpvr/eudI50MwHf1EBS3RkOAom51Dq7KJh9m KY5D3Qhve2gR9alUTqlbwZ51Hl0a1vj6CncmLHKwt2kVi2id9XeUq5ZNzdZUkek7Fe+7 eu4h6mP3A4zjuPBzMwV0mfDQbVlwUkL/rkY4agoF+Ghc50FBNhDROTlD01tnWSz9bcx7 K6KuhYNuwOBL9uEfFSd8Mq3SUZbHcyJe6aNbRt3NZ/hr1ofE2uFkcQyWNyQTLo9OYyKF l9Dg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:mime-version :references:in-reply-to:message-id:date:subject:cc:to:from :dkim-signature; bh=+yS4A9C6bDkm88UVUGu+D7xxJ8zAuI4Ip9K+7iZOJkk=; fh=sNagsEwRY2A1GXf/CzjygzPzk1OrfXbRTZcBRQTEQUI=; b=oR99c5t54MigPNbHbWM/Dyn7xaQvKmLk4q/o09vXqmpGfvS8AgphOwp+J/0KKgwYEW SjIMrCkhgU887CzW00SX1dJIII7De4KsdD+zKJYilbIYZIvSGlzYbrICSBwymv6zKocN KbXVzB1F1jC6172jpaVCTrUIJDsHXi9Yz3Dz/UlwzfpXn420GpUsh6PmEOHWKQJW+0Bm 7mELg/5oO4qaLNIM9L1JA/lBFfqXQjLmC8MN/njaqth0/IJtEtG8aG9Oc0q//pwhzmnm 1IEsb7Th7JFQxtid7eoOGsV8N2B6uFvYRN67KlKdSQgdOhxGdFAs43tBkkBloqYsLlyU jTzg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@ibm.com header.s=pp1 header.b=Q65SckCt; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=NONE dis=NONE) header.from=ibm.com Return-Path: Received: from lipwig.vger.email (lipwig.vger.email. [23.128.96.33]) by mx.google.com with ESMTPS id x1-20020a170902a38100b001b82a4d3ea7si7266211pla.249.2023.09.13.09.13.51 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Wed, 13 Sep 2023 09:13:51 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) client-ip=23.128.96.33; Authentication-Results: mx.google.com; dkim=pass header.i=@ibm.com header.s=pp1 header.b=Q65SckCt; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.33 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=NONE dis=NONE) header.from=ibm.com Received: from out1.vger.email (depot.vger.email [IPv6:2620:137:e000::3:0]) by lipwig.vger.email (Postfix) with ESMTP id 1F0E1825B48E; Wed, 13 Sep 2023 06:06:58 -0700 (PDT) X-Virus-Status: Clean X-Virus-Scanned: clamav-milter 0.103.10 at lipwig.vger.email Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S239093AbjIMNGo (ORCPT + 99 others); Wed, 13 Sep 2023 09:06:44 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:58136 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S231326AbjIMNGn (ORCPT ); Wed, 13 Sep 2023 09:06:43 -0400 Received: from mx0b-001b2d01.pphosted.com (mx0b-001b2d01.pphosted.com [148.163.158.5]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 72D9019B1; Wed, 13 Sep 2023 06:06:39 -0700 (PDT) Received: from pps.filterd (m0353723.ppops.net [127.0.0.1]) by mx0a-001b2d01.pphosted.com (8.17.1.19/8.17.1.19) with ESMTP id 38DCuuJ1031677; Wed, 13 Sep 2023 13:06:37 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ibm.com; h=from : to : cc : subject : date : message-id : in-reply-to : references : mime-version : content-transfer-encoding; s=pp1; bh=+yS4A9C6bDkm88UVUGu+D7xxJ8zAuI4Ip9K+7iZOJkk=; b=Q65SckCto85+tqnP9QTRrqimdxzZM3ZS7NNwdfMVmZMGhsvy3xW2cFz8tIOVK8Com3VJ grPIxoZ0ImwYtHxHROd7k7e2Q7BwyXaj5b47jW1XSpuJ3Wf04CGFHHAB/OxyJZ+xsTY4 Zmc9m9ayL1kBDk6DV+D/SQhu1Vl/Xs0OLqb5u2Sb/Akdp6JuL1EwPIJAMEY0NXVGyh8Q L+/l1C3ff6U9r2tIG9OsaBXcRteTY+Vt6ALoGcF9qgxaLzi6k3NFEAUFMluLQ72noQvJ uifNVSob3+OKsXQdVD2K1DZmrrQOb4FizzyMM2b4MmqSrxAAFyRlJ+8S7EZ1WKXT86Ug 1w== Received: from pps.reinject (localhost [127.0.0.1]) by mx0a-001b2d01.pphosted.com (PPS) with ESMTPS id 3t3dhggddd-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 13 Sep 2023 13:06:36 +0000 Received: from m0353723.ppops.net (m0353723.ppops.net [127.0.0.1]) by pps.reinject (8.17.1.5/8.17.1.5) with ESMTP id 38DD6W7E028030; Wed, 13 Sep 2023 13:06:34 GMT Received: from ppma23.wdc07v.mail.ibm.com (5d.69.3da9.ip4.static.sl-reverse.com [169.61.105.93]) by mx0a-001b2d01.pphosted.com (PPS) with ESMTPS id 3t3dhggd9c-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 13 Sep 2023 13:06:34 +0000 Received: from pps.filterd (ppma23.wdc07v.mail.ibm.com [127.0.0.1]) by ppma23.wdc07v.mail.ibm.com (8.17.1.19/8.17.1.19) with ESMTP id 38DAxqSm002755; Wed, 13 Sep 2023 13:06:30 GMT Received: from smtprelay04.dal12v.mail.ibm.com ([172.16.1.6]) by ppma23.wdc07v.mail.ibm.com (PPS) with ESMTPS id 3t14hm32tv-1 (version=TLSv1.2 cipher=ECDHE-RSA-AES256-GCM-SHA384 bits=256 verify=NOT); Wed, 13 Sep 2023 13:06:30 +0000 Received: from smtpav01.dal12v.mail.ibm.com (smtpav01.dal12v.mail.ibm.com [10.241.53.100]) by smtprelay04.dal12v.mail.ibm.com (8.14.9/8.14.9/NCO v10.0) with ESMTP id 38DD6TS113369908 (version=TLSv1/SSLv3 cipher=DHE-RSA-AES256-GCM-SHA384 bits=256 verify=OK); Wed, 13 Sep 2023 13:06:29 GMT Received: from smtpav01.dal12v.mail.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 3D40558059; Wed, 13 Sep 2023 13:06:29 +0000 (GMT) Received: from smtpav01.dal12v.mail.ibm.com (unknown [127.0.0.1]) by IMSVA (Postfix) with ESMTP id 4042A58058; Wed, 13 Sep 2023 13:06:28 +0000 (GMT) Received: from li-2c1e724c-2c76-11b2-a85c-ae42eaf3cb3d.ibm.com.com (unknown [9.61.101.13]) by smtpav01.dal12v.mail.ibm.com (Postfix) with ESMTP; Wed, 13 Sep 2023 13:06:28 +0000 (GMT) From: Tony Krowiak To: linux-s390@vger.kernel.org, linux-kernel@vger.kernel.org, kvm@vger.kernel.org Cc: jjherne@linux.ibm.com, pasic@linux.ibm.com, alex.williamson@redhat.com, borntraeger@linux.ibm.com, kwankhede@nvidia.com, frankja@linux.ibm.com, imbrenda@linux.ibm.com, david@redhat.com, Anthony Krowiak , stable@vger.kernel.org Subject: [PATCH 1/2] s390/vfio-ap: unpin pages on gisc registration failure Date: Wed, 13 Sep 2023 09:06:21 -0400 Message-ID: <20230913130626.217665-2-akrowiak@linux.ibm.com> X-Mailer: git-send-email 2.41.0 In-Reply-To: <20230913130626.217665-1-akrowiak@linux.ibm.com> References: <20230913130626.217665-1-akrowiak@linux.ibm.com> MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-TM-AS-GCONF: 00 X-Proofpoint-GUID: wsF5irEh6MPDqm-9JoGtN0h0tWuxxoYU X-Proofpoint-ORIG-GUID: ka6gNPFCTGmE6hlryg6mAw6brf4XiIzB X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.267,Aquarius:18.0.980,Hydra:6.0.601,FMLib:17.11.176.26 definitions=2023-09-13_06,2023-09-13_01,2023-05-22_02 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 mlxscore=0 malwarescore=0 bulkscore=0 phishscore=0 clxscore=1011 mlxlogscore=999 lowpriorityscore=0 suspectscore=0 priorityscore=1501 impostorscore=0 spamscore=0 adultscore=0 classifier=spam adjust=0 reason=mlx scancount=1 engine=8.12.0-2308100000 definitions=main-2309130101 Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (lipwig.vger.email [0.0.0.0]); Wed, 13 Sep 2023 06:06:58 -0700 (PDT) X-Spam-Status: No, score=-0.8 required=5.0 tests=DKIM_SIGNED,DKIM_VALID, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_HELO_NONE, SPF_PASS autolearn=unavailable autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lipwig.vger.email From: Anthony Krowiak In the vfio_ap_irq_enable function, after the page containing the notification indicator byte (NIB) is pinned, the function attempts to register the guest ISC. If registration fails, the function sets the status response code and returns without unpinning the page containing the NIB. In order to avoid a memory leak, the NIB should be unpinned before returning from the vfio_ap_irq_enable function. Fixes: 783f0a3ccd79 ("s390/vfio-ap: add s390dbf logging to the vfio_ap_irq_enable function") Signed-off-by: Janosch Frank Signed-off-by: Anthony Krowiak Cc: --- drivers/s390/crypto/vfio_ap_ops.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/s390/crypto/vfio_ap_ops.c b/drivers/s390/crypto/vfio_ap_ops.c index 4db538a55192..9cb28978c186 100644 --- a/drivers/s390/crypto/vfio_ap_ops.c +++ b/drivers/s390/crypto/vfio_ap_ops.c @@ -457,6 +457,7 @@ static struct ap_queue_status vfio_ap_irq_enable(struct vfio_ap_queue *q, VFIO_AP_DBF_WARN("%s: gisc registration failed: nisc=%d, isc=%d, apqn=%#04x\n", __func__, nisc, isc, q->apqn); + vfio_unpin_pages(&q->matrix_mdev->vdev, nib, 1); status.response_code = AP_RESPONSE_INVALID_GISA; return status; } -- 2.41.0