Received-SPF: pass (google.com: domain of linux-crypto+bounces-2262-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.80.249 as permitted sender) client-ip=147.75.80.249;
From: "Elliott, Robert (Servers)" <elliott@hpe.com>
To: Kamlesh Gurudasani <kamlesh@ti.com>, Eric Biggers <ebiggers@kernel.org>
CC: Herbert Xu <herbert@gondor.apana.org.au>,
        "David S. Miller"
	<davem@davemloft.net>,
        Rob Herring <robh+dt@kernel.org>,
        Krzysztof Kozlowski
	<krzysztof.kozlowski+dt@linaro.org>,
        Conor Dooley <conor+dt@kernel.org>, Nishanth Menon <nm@ti.com>,
        Vignesh Raghavendra <vigneshr@ti.com>,
        "Tero
 Kristo" <kristo@kernel.org>,
        Catalin Marinas <catalin.marinas@arm.com>,
        "Will
 Deacon" <will@kernel.org>,
        Maxime Coquelin <mcoquelin.stm32@gmail.com>,
        Alexandre Torgue <alexandre.torgue@foss.st.com>,
        "linux-crypto@vger.kernel.org" <linux-crypto@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-arm-kernel@lists.infradead.org"
	<linux-arm-kernel@lists.infradead.org>,
        "linux-stm32@st-md-mailman.stormreply.com"
	<linux-stm32@st-md-mailman.stormreply.com>
Thread-Topic: [EXTERNAL] Re: [PATCH v2 2/6] crypto: crc64 - add crc64-iso
 framework
Thread-Index: AQHZy8EnMktNYynuQEKGWzL0s+J88K/l+VoAgAm5hACBKF9AsA==
Date: Thu, 22 Feb 2024 21:50:44 +0000
Message-ID: <MW5PR84MB18424EC8DDB4863777D302E2AB562@MW5PR84MB1842.NAMPRD84.PROD.OUTLOOK.COM>
References: <20230719-mcrc-upstream-v2-0-4152b987e4c2@ti.com>
 <20230719-mcrc-upstream-v2-2-4152b987e4c2@ti.com>
 <20230812025520.GE971@sol.localdomain>
 <87jztserrf.fsf@kamlesh.i-did-not-set--mail-host-address--so-tickle-me>
In-Reply-To: <87jztserrf.fsf@kamlesh.i-did-not-set--mail-host-address--so-tickle-me>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?6+r+EqKpGd10WQywf3fx+jgGXwMRQS2KXcBwdybn55SvfhmU77DTZIb67E0H?=
 =?us-ascii?Q?oux75wf68Rb4nY7e3SFTDGZelnhkkBSwPvnT7Pk82cSAkR3fAtwDxPYr0u4o?=
 =?us-ascii?Q?c82JwpkxXVGodeHscuon9GgCI6ytWF4yTXv0Z8qMZCVM8qMc3Qj1FC9gVZX8?=
 =?us-ascii?Q?tv5A1AKGQay4RphCzv48hwzQ3gzmkbI/YfLXTLYBgpF2m2DWwnBCCBQRt2JP?=
 =?us-ascii?Q?7bWaloIQ2iyTUZ29OWU7RhGxlEFdjKtTA2vIWHJB3ZQQWzvK+KQhtPmfhESO?=
 =?us-ascii?Q?vmTO0OY5huGEl02Gk8EpHy09v+MSD3uO9VpotenUX2+az304iZRnCoRzqNSf?=
 =?us-ascii?Q?sXVixwdbXq0bmzSrIpsuO1lDtQQeOdHgn+dHeA4XAjG3ZVpZ3qTqzmYVqWcz?=
 =?us-ascii?Q?c1ChEkMLSJLHZg3z87OevBjlLLyu0gw1jueeexmtTt3AyWAtgjB75jhQq5CG?=
 =?us-ascii?Q?0/fH7su+QlpQ8dTazy0Mi9HHhuzT/OO6rdUI4DRuclc+KtfbTYAq5Cjzi9mI?=
 =?us-ascii?Q?fm/4jl8k/dfe2c8rba3AtPvRw7igCR1gjmf21Rf3kCERMiYwZzwN7tohHOc5?=
 =?us-ascii?Q?722BtMlL8sDVkWH0QHMaJP1hJ5qUIUKIT9ZcCG8hkIvAZvB2gcI0V4dX8MHk?=
 =?us-ascii?Q?UeXOKfBJtROPJvJwPbKmpVjjnuF2yvLUSAvPuNZy2eIlLq/d2wuL1viY8lK6?=
 =?us-ascii?Q?sa8sl6I9Jb4B8rO5RtAd8tFfCJW5qXiU8A8cxnZzGCqV66EsjAiSftiNcJZP?=
 =?us-ascii?Q?SiLSr4PqC3yiSDou4Jnb78buvFikMaJ5w1FzViY3LWChbOafflvHUiFKRCvV?=
 =?us-ascii?Q?GqI5bu5rI90Dhiq/di54jpyGyWDZ5zw752oBkPdHt0dKj7H7YcMCq8rgtS/a?=
 =?us-ascii?Q?NFhaBwFvniBCE8spIo2+sxAUVoQmgVCx0gJZl3NXTg0pEW4znlpi0fQWIXDu?=
 =?us-ascii?Q?F9GQl9r4aCbFzxVzDT7+/knc4Gk/4S7gQqfHg/V4KZ1MPWeIAJn6p4uh95Bk?=
 =?us-ascii?Q?z7O9rBwYIKT4XqiNiE4q06Uu1LVTkgIdhqJXuGtnR8hQRSMPAsDRpmfKOptO?=
 =?us-ascii?Q?QT/9GZ6Hn1SZrP7dJVpsswDKLvL1bIAYdvc8LQMYenvMHfpjLfuY+oiKN1w2?=
 =?us-ascii?Q?9ZgmT/HicB9bej8rJtxXmg+xOdQ7z1P9CLetgKR0XwSsCgep+FJY4u4YdWrK?=
 =?us-ascii?Q?kV8G28HxRoH26tSem6gq3zmx/8FT+b55+g++Eap6c8GTwiJmiu3y7+wYz93r?=
 =?us-ascii?Q?uBYpsK8T36xA3p1Oe+4KnkA9giQfOpEdMQ4HBEcMuAG6BFHR/V3bChiFo84f?=
 =?us-ascii?Q?8dYG36lCE3PXjWqLHjah17Z9warTV6z6a6zXI/agE23YNMcxmvWInUZswUtm?=
 =?us-ascii?Q?mWpR3wLd5GNb1h23fWmn8WuQb0tbonEVrGeIpYZtDM5EdL6PsSBJ1QMUN5dc?=
 =?us-ascii?Q?TgW7CTRsc/966xJBEym+kT1pGuZJnDg6kg9irSJRMvjIZEdK7Jql4jk5cyJb?=
 =?us-ascii?Q?xZO/0U/rVFJYBHqx0nTgMUPzQgkankb2jFTcLgHhjxTU4LPkI33uCNa+VCiq?=
 =?us-ascii?Q?kvzwSxkLrGncNYFgTY8=3D?=
Content-Type: text/plain; charset="us-ascii"
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: MW5PR84MB1842.NAMPRD84.PROD.OUTLOOK.COM
X-MS-Exchange-CrossTenant-Network-Message-Id: d9e97282-1297-435d-9f47-08dc33f05250
X-MS-Exchange-CrossTenant-originalarrivaltime: 22 Feb 2024 21:50:44.5865
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 105b2061-b669-4b31-92ac-24d304d195dc
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: c8KgAzNQfJMca6vn03G2lziiAFAvpGYy0LS1CFesfAddaa0LsH3ieBNmOBTYxkuS
X-MS-Exchange-Transport-CrossTenantHeadersStamped: LV8PR84MB3436
X-OriginatorOrg: hpe.com
X-Proofpoint-GUID: 8GFB0xbM8BaJFVgYpxyQNvK8HVYkeMNv
X-Proofpoint-ORIG-GUID: 8GFB0xbM8BaJFVgYpxyQNvK8HVYkeMNv
Content-Transfer-Encoding: quoted-printable
X-Proofpoint-UnRewURL: 0 URL was un-rewritten
Precedence: bulk
X-Mailing-List: linux-crypto@vger.kernel.org
List-Id: <linux-crypto.vger.kernel.org>
List-Subscribe: <mailto:linux-crypto+subscribe@vger.kernel.org>
List-Unsubscribe: <mailto:linux-crypto+unsubscribe@vger.kernel.org>
MIME-Version: 1.0
X-HPE-SCL: -1
Subject: RE:  Re: [PATCH v2 2/6] crypto: crc64 - add crc64-iso framework
X-Proofpoint-Virus-Version: vendor=baseguard
 engine=ICAP:2.0.272,Aquarius:18.0.1011,Hydra:6.0.619,FMLib:17.11.176.26
 definitions=2024-02-22_15,2024-02-22_01,2023-05-22_02
X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 priorityscore=1501 mlxscore=0
 adultscore=0 malwarescore=0 phishscore=0 impostorscore=0
 lowpriorityscore=0 mlxlogscore=999 suspectscore=0 spamscore=0
 clxscore=1011 bulkscore=0 classifier=spam adjust=0 reason=mlx scancount=1
 engine=8.12.0-2311290000 definitions=main-2402220169


> -----Original Message-----
> From: Kamlesh Gurudasani <kamlesh@ti.com>
> Sent: Friday, August 18, 2023 2:26 AM
> Subject: Re: [EXTERNAL] Re: [PATCH v2 2/6] crypto: crc64 - add crc64-iso
> framework
>=20
> Eric Biggers <ebiggers@kernel.org> writes:
>=20
> > 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
> https://en.wikipedia.org/wiki/Cyclic_redundancy_check
>=20
> Last entry CRC-64-ISO in the table.
> It is mentioned as crc64-iso and that's the
> only 64-bit CRC standardized by ISO.=20

ECMA-182 (DLT-1 tapes) was contributed to become ISO/IEC 13421 in 1993, so
that was another "64-bit CRC standardized by ISO." Plus, ISO could publish =
new
standards with new CRCs at any time.

> But I do agree that crc64-iso3309 would be more specific, will change it
> to crc64-iso3309 in next revision. Thanks.
>=20
> Regards,
> Kamlesh

ISO-3309:1991 was withdrawn and revised by
ISO/IEC 3309:1993, which was withdrawn and revised by
ISO/IEC 13239:2002, which was confirmed in 2007 and is still current.

Apparently only the 1991 version defined a CRC-64; later versions dropped
that.

Is there really a demand for adding a 23 year old deprecated algorithm to
the kernel?