Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753500AbdHOHvb convert rfc822-to-8bit (ORCPT ); Tue, 15 Aug 2017 03:51:31 -0400 Received: from mx08-00178001.pphosted.com ([91.207.212.93]:37780 "EHLO mx07-00178001.pphosted.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1753232AbdHOHv2 (ORCPT ); Tue, 15 Aug 2017 03:51:28 -0400 From: Fabien DESSENNE To: Corentin Labbe , "herbert@gondor.apana.org.au" , "davem@davemloft.net" CC: "linux-crypto@vger.kernel.org" , "linux-kernel@vger.kernel.org" Subject: RE: [PATCH 2/3] crypto: engine - find request type with cra_type Thread-Topic: [PATCH 2/3] crypto: engine - find request type with cra_type Thread-Index: AQHTFQAQzC1V1ZBg7UOX4kfxy7PVkqKFCCCA Date: Tue, 15 Aug 2017 07:51:14 +0000 Message-ID: References: <20170814131725.9402-1-clabbe.montjoie@gmail.com> <20170814131725.9402-3-clabbe.montjoie@gmail.com> In-Reply-To: <20170814131725.9402-3-clabbe.montjoie@gmail.com> Accept-Language: fr-FR, en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-ms-exchange-transport-fromentityheader: Hosted x-originating-ip: [10.75.127.51] Content-Type: text/plain; charset="iso-8859-1" Content-Transfer-Encoding: 8BIT MIME-Version: 1.0 X-Proofpoint-Virus-Version: vendor=fsecure engine=2.50.10432:,, definitions=2017-08-15_04:,, signatures=0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3375 Lines: 102 Hi Corentin, Since I have just sent a patch to add the support of "aead_request" to crypto engine, I am wondering if your proposed change (checking cra_type instead of crypto_tfm_alg_type) and mine are compatible. It looks like they are (assuming we export crypto_aead_type): can you confirm? BR Fabien. >-----Original Message----- >From: linux-crypto-owner@vger.kernel.org [mailto:linux-crypto- >owner@vger.kernel.org] On Behalf Of Corentin Labbe >Sent: lundi 14 ao?t 2017 15:17 >To: herbert@gondor.apana.org.au; davem@davemloft.net >Cc: linux-crypto@vger.kernel.org; linux-kernel@vger.kernel.org; Corentin Labbe > >Subject: [PATCH 2/3] crypto: engine - find request type with cra_type > >The current method for finding request type is based on crypto_tfm_alg_type. > >But in case of skcipher, it is the same than ablkcipher. >Using cra_type for this work permits to make the distinction between the two. > >Signed-off-by: Corentin Labbe >--- > crypto/crypto_engine.c | 19 ++++++++----------- > 1 file changed, 8 insertions(+), 11 deletions(-) > >diff --git a/crypto/crypto_engine.c b/crypto/crypto_engine.c index >61e7c4e02fd2..74b840749074 100644 >--- a/crypto/crypto_engine.c >+++ b/crypto/crypto_engine.c >@@ -38,7 +38,8 @@ static void crypto_pump_requests(struct crypto_engine >*engine, > struct ablkcipher_request *breq; > unsigned long flags; > bool was_busy = false; >- int ret, rtype; >+ int ret; >+ const struct crypto_type *cratype; > > spin_lock_irqsave(&engine->queue_lock, flags); > >@@ -94,7 +95,7 @@ static void crypto_pump_requests(struct crypto_engine >*engine, > > spin_unlock_irqrestore(&engine->queue_lock, flags); > >- rtype = crypto_tfm_alg_type(engine->cur_req->tfm); >+ cratype = engine->cur_req->tfm->__crt_alg->cra_type; > /* Until here we get the request need to be encrypted successfully */ > if (!was_busy && engine->prepare_crypt_hardware) { > ret = engine->prepare_crypt_hardware(engine); >@@ -104,8 +105,7 @@ static void crypto_pump_requests(struct crypto_engine >*engine, > } > } > >- switch (rtype) { >- case CRYPTO_ALG_TYPE_AHASH: >+ if (cratype == &crypto_ahash_type) { > hreq = ahash_request_cast(engine->cur_req); > if (engine->prepare_hash_request) { > ret = engine->prepare_hash_request(engine, hreq); @@ >-122,7 +122,7 @@ static void crypto_pump_requests(struct crypto_engine >*engine, > goto req_err; > } > return; >- case CRYPTO_ALG_TYPE_ABLKCIPHER: >+ } else if (cratype == &crypto_ablkcipher_type) { > breq = ablkcipher_request_cast(engine->cur_req); > if (engine->prepare_cipher_request) { > ret = engine->prepare_cipher_request(engine, breq); >@@ -139,21 +139,18 @@ static void crypto_pump_requests(struct >crypto_engine *engine, > goto req_err; > } > return; >- default: >+ } else { > dev_err(engine->dev, "failed to prepare request of unknown >type\n"); > return; > } > > req_err: >- switch (rtype) { >- case CRYPTO_ALG_TYPE_AHASH: >+ if (cratype == &crypto_ahash_type) { > hreq = ahash_request_cast(engine->cur_req); > crypto_finalize_hash_request(engine, hreq, ret); >- break; >- case CRYPTO_ALG_TYPE_ABLKCIPHER: >+ } else if (cratype == &crypto_ablkcipher_type) { > breq = ablkcipher_request_cast(engine->cur_req); > crypto_finalize_cipher_request(engine, breq, ret); >- break; > } > return; > >-- >2.13.0