From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id 6BC13EB64DD for ; Fri, 11 Aug 2023 04:24:59 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20210309; h=Sender: Content-Transfer-Encoding:Content-Type:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:Cc:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=ig7l3e2MXorRv7QfkS6PxBjPzp3uEQbzYOEJ9PYSG4s=; b=q1LhWKFG06J2UM f+tiR7494OEbRYMHbek//SEYso9QDyRhBvrxpzdirq6IxC/ZWEpbUQc+jbrsJ0hljAkH3xC10j7fd iesyX0fnssODeptfA4yw9jWXgQczUeMG0OaC2Hai2Y/bSU0uefEwDruPQbRH8tjyfSHuZMJRPnq2p c/LLR00PJ0q6dR9XHNDiZxxiBc/crdVP9ihkrt1SSLgsCMKpnAYeVR4mYmliCkAg8sjC45tfuwk+J E91YG3idivBEidPX92j1TZq1w+IsiaTeinvhMjdqOlpgdq1bfae9N8qywIswqK56l13+t5EVbpv6/ nQ6+qvy47rcj42ORKQXw==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qUJhQ-009Mam-23; Fri, 11 Aug 2023 04:24:32 +0000 Received: from dfw.source.kernel.org ([2604:1380:4641:c500::1]) by bombadil.infradead.org with esmtps (Exim 4.96 #2 (Red Hat Linux)) id 1qUJhN-009Ma5-0k for linux-arm-kernel@lists.infradead.org; Fri, 11 Aug 2023 04:24:30 +0000 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by dfw.source.kernel.org (Postfix) with ESMTPS id DABC96146E; Fri, 11 Aug 2023 04:24:25 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id C2C27C433C7; Fri, 11 Aug 2023 04:24:24 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1691727865; bh=tFTch+Vt3hmXVV5W6NxEILrRRxKFsve4GjiYPL3bFxI=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=tDwSTMykeWxpIGLeOu3IiYkMhslf3W0y9Wd7xmQudpL+OJX6C2GX3kZkJzcIPO0AQ Bh9FzaJP6YfG6qeTg3ulmo0/+YSn8TQ4CJSlv44kV7jKQVjZnpxZoEbawJ83Ns+sD+ s+eSVrjCTmW+8Ais4F+9lsn1QSbgeRvh9HMLNdNXu3osjyDo2T4OPY7GMsIFmNE9V6 OBaHWJ6EEBKvhFqyHprATLQeyixBG6wAYb8j9pnQvm2UOhjiUGH519JqOlBKseeRjm tpu7AufenVwScWolHFut7isQfj5tTE2TGkA7Bj4iwJ4gQoufZ5ZFcX1NnKnxJYoH50 zk8bxUwLkhfbQ== Date: Thu, 10 Aug 2023 21:24:23 -0700 From: Eric Biggers To: Kamlesh Gurudasani Cc: Herbert Xu , "David S. Miller" , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Nishanth Menon , Vignesh Raghavendra , Tero Kristo , Catalin Marinas , Will Deacon , Maxime Coquelin , Alexandre Torgue , linux-crypto@vger.kernel.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-stm32@st-md-mailman.stormreply.com Subject: Re: [PATCH v2 2/6] crypto: crc64 - add crc64-iso framework Message-ID: <20230811042423.GA1295@sol.localdomain> References: <20230719-mcrc-upstream-v2-0-4152b987e4c2@ti.com> <20230719-mcrc-upstream-v2-2-4152b987e4c2@ti.com> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: <20230719-mcrc-upstream-v2-2-4152b987e4c2@ti.com> X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20230810_212429_375905_0BDEAC35 X-CRM114-Status: GOOD ( 14.63 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Fri, Aug 11, 2023 at 12:58:49AM +0530, Kamlesh Gurudasani wrote: > diff --git a/lib/crc64-iso.c b/lib/crc64-iso.c > new file mode 100644 > index 000000000000..d6e803124fa0 [...] > +u64 crc64_iso_update(u64 crc, const unsigned char *buffer, size_t len) > +{ > + struct { > + struct shash_desc shash; > + u64 crc; > + } desc; > + int err; > + > + if (static_branch_unlikely(&crc64_iso_fallback)) > + return crc64_iso_generic(crc, buffer, len); > + > + rcu_read_lock(); > + desc.shash.tfm = rcu_dereference(crc64_iso_tfm); > + desc.crc = crc; > + err = crypto_shash_update(&desc.shash, buffer, len); > + rcu_read_unlock(); > + > + WARN_ON_ONCE(err); > + > + return desc.crc; > +} > +EXPORT_SYMBOL_GPL(crc64_iso_update); > + > +u64 crc64_iso(const unsigned char *buffer, size_t len) > +{ > + return crc64_iso_update(0, buffer, len); > +} > +EXPORT_SYMBOL_GPL(crc64_iso); These functions are never called. Why are you trying to add unused code to the kernel? - Eric _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel