Received: by 2002:a25:ef43:0:0:0:0:0 with SMTP id w3csp673664ybm; Wed, 27 May 2020 05:25:37 -0700 (PDT) X-Google-Smtp-Source: ABdhPJwTNEwnkH1iaKD8FO+5/cvWT24JxFY8RX1nmy6CSUOXCJkCTPiogLZXc+pJjJTTFGu+NdwB X-Received: by 2002:a17:906:f98f:: with SMTP id li15mr6085240ejb.259.1590582337749; Wed, 27 May 2020 05:25:37 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1590582337; cv=none; d=google.com; s=arc-20160816; b=JUXCEtlu+8JQNU4yyVy1sNj1XeAyBtHlzhuaTS35bdfdkaNCUVFmUTF0pP+HEq8EAV A4g5GnQJq2tWZF9a0/2QnbrnJ0X/IUpWJXdS7IapcccXsZpoYs0L/K55FtJaluvSzLGQ ONaK3RAKVCRmnnRsC1OfLH1qnX1FLgWaMRLt8N/dtDOKO/ymLZKBffeAu90dikQMj95T 3UBR8USyILfGikKJg8D8/lWmOyviq4Ug1cF6J7Ahkl37YCmrWZMG/us3v70ZFeDK8UjC j5SEserNqE20yq6bKq0LdUDeahBmXv9NEGD+HwJWbg7NBQg8Spvo5RCCu4wwlyKsfRVR PUjQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:dkim-signature; bh=ri+/lIeSNOTM3l+FNff6gZorR0Kh5OVSvczQZBkHThk=; b=eTIGaNIupAKLPcvL6gHCgqZsL7yATYxUbPj2MYsqbTz9QYv8FX1vg94dsnLr5erleu tI1COC4L0FjPP3Ji23VNLnKKmVHno3qcQWpNEGWcEeYcpQ8pQ5duo3XBa7LiuBjJdaRI WBC9u5B7CDFx+SN78LWF7BVGS8PpooaCWKf69CTe8J+hPFi1f+LEDjvbTt8GaAu9H5iB xA4C0K3GI2JSKdfr2ywwiCyrQSRgAnFtHO0kwcWl6liMuit8B0MwGYmGwPTgMWP7YM+m 8IjKHSuR8wBT8DkxFNsu6To8lT3IyKMlS9lJYtEmPa3BIxXMAaKbYxqEEv3x5oTlWIh8 bf2g== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@ti.com header.s=ti-com-17Q1 header.b=PllSCQhX; spf=pass (google.com: domain of linux-crypto-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-crypto-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=NONE dis=NONE) header.from=ti.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id z8si1916965eja.225.2020.05.27.05.25.14; Wed, 27 May 2020 05:25:37 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-crypto-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=pass header.i=@ti.com header.s=ti-com-17Q1 header.b=PllSCQhX; spf=pass (google.com: domain of linux-crypto-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-crypto-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=NONE dis=NONE) header.from=ti.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1729352AbgE0MYv (ORCPT + 99 others); Wed, 27 May 2020 08:24:51 -0400 Received: from lelv0142.ext.ti.com ([198.47.23.249]:35620 "EHLO lelv0142.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1729407AbgE0MYu (ORCPT ); Wed, 27 May 2020 08:24:50 -0400 Received: from fllv0035.itg.ti.com ([10.64.41.0]) by lelv0142.ext.ti.com (8.15.2/8.15.2) with ESMTP id 04RCOjv6005915; Wed, 27 May 2020 07:24:45 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1590582285; bh=ri+/lIeSNOTM3l+FNff6gZorR0Kh5OVSvczQZBkHThk=; h=From:To:CC:Subject:Date:In-Reply-To:References; b=PllSCQhXoZ4b+1HiStnGp4SOPH9Fyue9XuL3kXPAg0aP9xFc3+ZbcVYI1bxz9laAO BewHr3+wj3odfNIaQoawSpg0Fw3T0W51yxS/+uiUF5nDnkXM4VTHY4Y/9S8kXIcXK0 /6v7KN87/VqS0+IQ9D6XU6b3+P+twuX8YyMFLWDw= Received: from DLEE112.ent.ti.com (dlee112.ent.ti.com [157.170.170.23]) by fllv0035.itg.ti.com (8.15.2/8.15.2) with ESMTP id 04RCOjGF064318; Wed, 27 May 2020 07:24:45 -0500 Received: from DLEE114.ent.ti.com (157.170.170.25) by DLEE112.ent.ti.com (157.170.170.23) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1979.3; Wed, 27 May 2020 07:24:44 -0500 Received: from fllv0040.itg.ti.com (10.64.41.20) by DLEE114.ent.ti.com (157.170.170.25) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1979.3 via Frontend Transport; Wed, 27 May 2020 07:24:45 -0500 Received: from sokoban.ti.com (ileax41-snat.itg.ti.com [10.172.224.153]) by fllv0040.itg.ti.com (8.15.2/8.15.2) with ESMTP id 04RCOYRj101979; Wed, 27 May 2020 07:24:43 -0500 From: Tero Kristo To: , , CC: Subject: [RESEND PATCHv3 6/7] crypto: omap-aes: prevent unregistering algorithms twice Date: Wed, 27 May 2020 15:24:28 +0300 Message-ID: <20200527122429.14888-7-t-kristo@ti.com> X-Mailer: git-send-email 2.17.1 In-Reply-To: <20200527122429.14888-1-t-kristo@ti.com> References: <20200527122429.14888-1-t-kristo@ti.com> MIME-Version: 1.0 Content-Type: text/plain X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 Sender: linux-crypto-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-crypto@vger.kernel.org Most of the OMAP family SoCs contain two instances for AES core, which causes the remove callbacks to be also done twice when driver is removed. Fix the algorithm unregister callbacks to take into account the number of algorithms still registered to avoid removing these twice. Signed-off-by: Tero Kristo --- drivers/crypto/omap-aes.c | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/crypto/omap-aes.c b/drivers/crypto/omap-aes.c index 824ddf2a66ff..b5aff20c5900 100644 --- a/drivers/crypto/omap-aes.c +++ b/drivers/crypto/omap-aes.c @@ -1269,13 +1269,17 @@ static int omap_aes_remove(struct platform_device *pdev) spin_unlock(&list_lock); for (i = dd->pdata->algs_info_size - 1; i >= 0; i--) - for (j = dd->pdata->algs_info[i].registered - 1; j >= 0; j--) + for (j = dd->pdata->algs_info[i].registered - 1; j >= 0; j--) { crypto_unregister_skcipher( &dd->pdata->algs_info[i].algs_list[j]); + dd->pdata->algs_info[i].registered--; + } - for (i = dd->pdata->aead_algs_info->size - 1; i >= 0; i--) { + for (i = dd->pdata->aead_algs_info->registered - 1; i >= 0; i--) { aalg = &dd->pdata->aead_algs_info->algs_list[i]; crypto_unregister_aead(aalg); + dd->pdata->aead_algs_info->registered--; + } crypto_engine_exit(dd->engine); -- 2.17.1 -- Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki