Received: by 2002:a05:6a10:f347:0:0:0:0 with SMTP id d7csp2838892pxu; Mon, 14 Dec 2020 12:07:25 -0800 (PST) X-Google-Smtp-Source: ABdhPJxAjaPRP7JK++AhMw3aLtPObDm9vwNMwHdPtD62uPqsq0c9MPzvPpTdWqA//zOwVq7v5oRO X-Received: by 2002:a17:907:41e3:: with SMTP id nb3mr3803971ejb.378.1607976445515; Mon, 14 Dec 2020 12:07:25 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1607976445; cv=none; d=google.com; s=arc-20160816; b=jKcbu12YB3Rn2C/+3AEBzuvcBS9Fd//Ryx8taKYVFd2JxDe3MBsWamsLdddwJ4b1G6 ZVoSfaet+TVfiNJcj0gFdw3L7chbril+IBfsV6aOnIeySs2hCpY89pFok9YGjTTAxgux S/glJG8SUWtwcp6941E6twsqU9DJm4PGBqxoGE1m4IzMJKR479XfgNXJMKcHqjpLMBSj Mm4oOdtmOlaflmDRlu1C09nBk76y3pMXDRCSMG0gABcDy+qzy0YiXHSF27R1Dh3KKvh4 VR99p+ofQXHnM4PcOPVQbrwegQtIN9AQBjfM2T9feP7zJlN/JEVaCdZmUFmB5YqdEpRV BSkQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:message-id:date:subject:cc:to:from :dkim-signature; bh=QT8NIpdXPvB5AxHdzPljCniOubRI5wdzJKZjH5NenqY=; b=I+3xH1ViSmTiNnSQVAEMAtjMdMY2QmkVBj+YE54hdJ/9tjUsXzyi+ylwIj+AmJg3hm aK3YeGM+533h8ZyrvG2XcrSjn9BOUKfLxPexYleFkVVirlbfk5ezKftxUhbHgTliVGYx rUa30pEFwusysEumKx8s26/U+4GVxETfnS9WqgrMhmMg1bVO6hB84jVcrLC5ajNPFwnU MVfSy8UmdDjxGOVM+Erl/o15yfRCrLHEBHWOZNlK+YLfaJWJhOIB+Jg7yF4sbu7pGeDb T93sUibs8sbEyIx8mBUNysgWYtirmqO3OAtaAnYfxy3Mk4QgzkmF94FP/Y4Kn+gegF5s hM2Q== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@baylibre-com.20150623.gappssmtp.com header.s=20150623 header.b=DujLLjbN; 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 Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id b21si12604237ejz.649.2020.12.14.12.07.02; Mon, 14 Dec 2020 12:07:25 -0800 (PST) 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=@baylibre-com.20150623.gappssmtp.com header.s=20150623 header.b=DujLLjbN; 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 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S2502884AbgLNUDk (ORCPT + 99 others); Mon, 14 Dec 2020 15:03:40 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:40988 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S2502903AbgLNUDa (ORCPT ); Mon, 14 Dec 2020 15:03:30 -0500 Received: from mail-pj1-x1044.google.com (mail-pj1-x1044.google.com [IPv6:2607:f8b0:4864:20::1044]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 3CE93C0613D3 for ; Mon, 14 Dec 2020 12:02:50 -0800 (PST) Received: by mail-pj1-x1044.google.com with SMTP id l23so7343725pjg.1 for ; Mon, 14 Dec 2020 12:02:50 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=baylibre-com.20150623.gappssmtp.com; s=20150623; h=from:to:cc:subject:date:message-id; bh=QT8NIpdXPvB5AxHdzPljCniOubRI5wdzJKZjH5NenqY=; b=DujLLjbNY2EqBfSfkapUKZhtAMbzp6keOCUoGrS6UzdoyiRrmkjKWP6Ufrc4ru/3K2 axkWExX+zhOPGOUPZIXVJk/zH6AwlYuYYbyF/y+U1WXH4rCPIgfcBOUyrOgH6DOHKVRD aB83q/S1BUogVWX5nRzlndQxQ4f2F/9dySQGMpsfe2t9nb9hpsx9C2hfiM+Dl29uc+IV Hh/WhUTDAILLkg7PFCmkWyMdi8UU2MQ+N4lPYDze0VMhgsww8DWG4PjN3d4hI9ckMJ4Y bdbxgjiIyqtTSuBkqj1PxtCjofRXWjzP5lmgyQ1ghOumixPFbxYc9MhWVqyP5mdLcUWZ +h2w== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id; bh=QT8NIpdXPvB5AxHdzPljCniOubRI5wdzJKZjH5NenqY=; b=m0KJMJkRk/KlV0TkdI7kUcukFSIc629tmU1BLo0+vIZdydCWeyUyidTihsjCQwnKuF r95ZGvcxj5rCQubJFsKYqLQ4Nij0hFlDaB6b8qMlqEFhToqgV5qtN08jSRSya7ySKMJ2 UarwhiW9I5V6Qi1A7xEQr733LLsL5f4e5f19SzyrqTK1PT1lRw3xXo9UxIf2SC0oER59 1MSl3Fi7Dk4wlthsSxvWjLCKdSYHOjzpgjLJQO/p9f8xS4hlOQpwzG3sRbuOJ4yq0gHq 1F1EKJklH1O0eZlqrY1FNRs2MwPlfQvOHUMK3e5UqD9PFo1LJsh/vGk9H9aXgrgBsLEh cTIg== X-Gm-Message-State: AOAM530ugeiVEu6dMVIyoSft1/1rMCkDM8hgvVFMse49wzgssE+tRH1m Hu/T/ikIx0g9z6rQkJp6Kqw2+A== X-Received: by 2002:a17:90a:7e95:: with SMTP id j21mr26783571pjl.217.1607976169812; Mon, 14 Dec 2020 12:02:49 -0800 (PST) Received: from localhost.localdomain ([163.172.76.58]) by smtp.googlemail.com with ESMTPSA id js9sm22434109pjb.2.2020.12.14.12.02.43 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 14 Dec 2020 12:02:49 -0800 (PST) From: Corentin Labbe To: arnd@arndb.de, davem@davemloft.net, herbert@gondor.apana.org.au, jernej.skrabec@siol.net, mripard@kernel.org, wens@csie.org Cc: linux-arm-kernel@lists.infradead.org, linux-crypto@vger.kernel.org, linux-kernel@vger.kernel.org, linux-sunxi@googlegroups.com, Corentin Labbe Subject: [PATCH v4 0/8] crypto: sun4i-ss: prevent always fallback for ciphers Date: Mon, 14 Dec 2020 20:02:24 +0000 Message-Id: <20201214200232.17357-1-clabbe@baylibre.com> X-Mailer: git-send-email 2.17.1 Precedence: bulk List-ID: X-Mailing-List: linux-crypto@vger.kernel.org Hello For help testing on "crypto: sun4i-ss - Fix sparse endianness markers", I have added "stats" support like other allwinner's crypto drivers. Seeing stats showed a clear problem, the ciphers function were not used at all. This is due to the not-inialized need_fallback which is "init" as true everytime. So basicly, since the patch introduced it, this probem hidden some bugs. This serie fixes all hidden problems, then fix the initialization of "need_fallback" and then add the stats like other allwinner drivers. Regards changes since v3: - patch #2: Rewrite test as suggested by David Laight - patch #7: removed all ifdef CONFIG_CRYPTO_DEV_SUN4I_SS_DEBUG - added kmap patch Changes since v2: - patch #1: move buf/bufo out of function for reducing stack usage - patch #4: use writesl() - patch #6: use IS_ENABLED instead of #ifdef Changes since v1: - patch #4 is sufficient to fix BE problem (removed todo) Corentin Labbe (8): crypto: sun4i-ss: linearize buffers content must be kept crypto: sun4i-ss: checking sg length is not sufficient crypto: sun4i-ss: IV register does not work on A10 and A13 crypto: sun4i-ss: handle BigEndian for cipher crypto: sun4i-ss: initialize need_fallback crypto: sun4i-ss: fix kmap usage crypto: sun4i-ss: enabled stats via debugfs crypto: sun4i-ss: add SPDX header and remove blank lines drivers/crypto/allwinner/Kconfig | 9 + .../allwinner/sun4i-ss/sun4i-ss-cipher.c | 196 +++++++++++------- .../crypto/allwinner/sun4i-ss/sun4i-ss-core.c | 52 +++++ .../crypto/allwinner/sun4i-ss/sun4i-ss-hash.c | 6 + .../crypto/allwinner/sun4i-ss/sun4i-ss-prng.c | 6 + drivers/crypto/allwinner/sun4i-ss/sun4i-ss.h | 8 + 6 files changed, 207 insertions(+), 70 deletions(-) -- 2.26.2