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 5C5B4C41513 for ; Sat, 12 Aug 2023 02:55:53 +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=4V4ZYlVpdiZdA6SN76MJxBxtE1UdP6oXuMwT3qkRlU4=; b=tKn7bsum4z6FPc 82huxOzH958MazG7mt9a5lvfm09/rfizFeOIpu1X95k4OGU0nOeqmZg6UpzEb2aHsn6t0C6j3haKJ /xPygrOwIfseIZnoq/7EWCR4qIeUv8Esu0kqlYZP530cbNB1/4ny96VnZIWwgjfT63dFn2VhFa64O QRZ1jC918b9B9c5Xj19ZkTnAcp4QW9Pokgrpfko7s76/CqJygsNUbEPg+1d26VUQhvA9T9HHeuTl+ 1IwAG49cGOtUlxHhDzDeJeR8b1lx2lMezR2YHOtaQ6bhu9yoZ1H4SahGcQCyJJM1ip+UVmfXSSwTa oXq4EfwxlI2/1gNdU9+A==; Received: from localhost ([::1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.96 #2 (Red Hat Linux)) id 1qUemk-00CWBj-0C; Sat, 12 Aug 2023 02:55:26 +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 1qUemh-00CWAg-19 for linux-arm-kernel@lists.infradead.org; Sat, 12 Aug 2023 02:55:24 +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 C818061311; Sat, 12 Aug 2023 02:55:22 +0000 (UTC) Received: by smtp.kernel.org (Postfix) with ESMTPSA id A8815C433C7; Sat, 12 Aug 2023 02:55:21 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1691808922; bh=8xhMi8cCxfgXnMrPb750nSrQBcgTHJS5NlxPxURCcW8=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=qDtF4baCkmTYqMhnD6jEUxmemiA2ZKrOgocYikKT2OcRX4Rkymkte02ADgwxauQIb b687ULIO7ZD8Uxqeb/UM1lR3ahx2gCrv/V0ANfZJrDwanBUfu9hNbZUOvopmBoUrSz VASQEYNoZb7LbAjB1MO5NvarJzr+BxM3TWWAmx60GxD9lNuAp5rFBgWT6pWti/d1RX KkTAJoh8aXOvA8d30tYlpkOWgBgn8Gp1RpQJ9ufhEsNNMLYtuY7xEMJm19/r4j1u/k vuGTm3ihEgbffmvmvKxUoGjFeDedH3F1+HcavQRT+KG179e/pCatY7gm8ETRGxEosS b6Os7KV70F0vQ== Date: Fri, 11 Aug 2023 19:55:20 -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: <20230812025520.GE971@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-20230811_195523_433440_196C7323 X-CRM114-Status: GOOD ( 13.16 ) 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/include/linux/crc64.h b/include/linux/crc64.h > index 70202da51c2c..10b792080374 100644 > --- a/include/linux/crc64.h > +++ b/include/linux/crc64.h > @@ -8,11 +8,15 @@ > #include > > #define CRC64_ROCKSOFT_STRING "crc64-rocksoft" > +#define CRC64_ISO_STRING "crc64-iso" > > u64 __pure crc64_be(u64 crc, const void *p, size_t len); > u64 __pure crc64_iso_generic(u64 crc, const void *p, size_t len); > u64 __pure crc64_rocksoft_generic(u64 crc, const void *p, size_t len); > > +u64 crc64_iso(const unsigned char *buffer, size_t len); > +u64 crc64_iso_update(u64 crc, const unsigned char *buffer, size_t len); > + > u64 crc64_rocksoft(const unsigned char *buffer, size_t len); > u64 crc64_rocksoft_update(u64 crc, const unsigned char *buffer, size_t len); Is "crc64-iso" clear enough, or should it be "crc64-iso3309"? There are thousands of ISO standards. Different CRC variants are specified by different ISO standards. Is this particular variant indeed commonly referred to as simply the "ISO" CRC-64? Even if it's currently the case that all other CRCs in ISO standards are different widths than 64 bits (which may be unlikely?), I'm not sure we should count on no CRC-64 variant ever being standardized by ISO. - Eric _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel