From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from smtp.kernel.org (aws-us-west-2-korg-mail-1.web.codeaurora.org [10.30.226.201]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 42143229692; Thu, 12 Dec 2024 16:46:57 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=10.30.226.201 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1734022017; cv=none; b=GZ7vWsSpnc/gz3tj4yPHhPPxsxJXGEjFalF2Vb9XnQ4lA8YJ+vf3hC6cj+WVp/nLIIeo2EvRuPQ/3Z5XEX/WUQSKdxF2KELWAKnF5RvE0UvwvkahS1/OE0rVE5aPRoUemnS+TcMxkOilEeTsvZxJ4IA3YbkBj4EKnFTdhc/0/Cw= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1734022017; c=relaxed/simple; bh=frNd6jk6jgPeszZG+3XPYCGJFpgSH4IvN4QkBG3zbXc=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=T5BGqd1dIcuudPmxbCWa75nByQnB1DXJWWN+FwN0uT3Z6nbRfkHBv3xeFXMvzT5wc/PKnY6hzh7W2BTU5DVayOcSYOBrX+9jb0qxPPmowa3tghiE5mnaJFsCx+I+WRmXZw+JFMMPGHm2jP2mlOSpQG7mKpxP+2DQP0Nn1r4nvNo= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b=x8pYCy7/; arc=none smtp.client-ip=10.30.226.201 Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=linuxfoundation.org header.i=@linuxfoundation.org header.b="x8pYCy7/" Received: by smtp.kernel.org (Postfix) with ESMTPSA id A2560C4CECE; Thu, 12 Dec 2024 16:46:56 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=linuxfoundation.org; s=korg; t=1734022017; bh=frNd6jk6jgPeszZG+3XPYCGJFpgSH4IvN4QkBG3zbXc=; h=From:To:Cc:Subject:Date:In-Reply-To:References:From; b=x8pYCy7/Y5E6lwDlLa70PWArqfUBBU/wmzv3xgygbDaXBXel/sDR04TnfexitDeXK fhUa/xvnuOk95HjuvnD9k9VNk690zLGZrgSCgwu0P7mK3rLrog3FMX08dBBcNip58E J07I6HySH6VTf8XaVS4zHPg9UZtqBh6fbV8GIZB4= From: Greg Kroah-Hartman To: stable@vger.kernel.org Cc: Greg Kroah-Hartman , patches@lists.linux.dev, Ahsan Atta , Giovanni Cabiddu , Herbert Xu , Sasha Levin Subject: [PATCH 5.15 094/565] crypto: qat - remove faulty arbiter config reset Date: Thu, 12 Dec 2024 15:54:49 +0100 Message-ID: <20241212144315.199955262@linuxfoundation.org> X-Mailer: git-send-email 2.47.1 In-Reply-To: <20241212144311.432886635@linuxfoundation.org> References: <20241212144311.432886635@linuxfoundation.org> User-Agent: quilt/0.67 X-stable: review X-Patchwork-Hint: ignore Precedence: bulk X-Mailing-List: stable@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: 8bit 5.15-stable review patch. If anyone has any objections, please let me know. ------------------ From: Ahsan Atta [ Upstream commit 70199359902f1c7187dcb28a1be679a7081de7cc ] Resetting the service arbiter config can cause potential issues related to response ordering and ring flow control check in the event of AER or device hang. This is because it results in changing the default response ring size from 32 bytes to 16 bytes. The service arbiter config reset also disables response ring flow control check. Thus, by removing this reset we can prevent the service arbiter from being configured inappropriately, which leads to undesired device behaviour in the event of errors. Fixes: 7afa232e76ce ("crypto: qat - Intel(R) QAT DH895xcc accelerator") Signed-off-by: Ahsan Atta Reviewed-by: Giovanni Cabiddu Signed-off-by: Herbert Xu Signed-off-by: Sasha Levin --- drivers/crypto/qat/qat_common/adf_hw_arbiter.c | 4 ---- 1 file changed, 4 deletions(-) diff --git a/drivers/crypto/qat/qat_common/adf_hw_arbiter.c b/drivers/crypto/qat/qat_common/adf_hw_arbiter.c index 64e4596a24f40..fd39cbcdec039 100644 --- a/drivers/crypto/qat/qat_common/adf_hw_arbiter.c +++ b/drivers/crypto/qat/qat_common/adf_hw_arbiter.c @@ -90,10 +90,6 @@ void adf_exit_arb(struct adf_accel_dev *accel_dev) hw_data->get_arb_info(&info); - /* Reset arbiter configuration */ - for (i = 0; i < ADF_ARB_NUM; i++) - WRITE_CSR_ARB_SARCONFIG(csr, arb_off, i, 0); - /* Unmap worker threads to service arbiters */ for (i = 0; i < hw_data->num_engines; i++) WRITE_CSR_ARB_WT2SAM(csr, arb_off, wt_off, i, 0); -- 2.43.0