From nobody Sat Feb 04 15:55:48 2023 X-Original-To: freebsd-ppc@mlmmj.nyi.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mlmmj.nyi.freebsd.org (Postfix) with ESMTP id 4P8HDm3zyYz3nXc2 for ; Sat, 4 Feb 2023 15:55:52 +0000 (UTC) (envelope-from pfg@freebsd.org) Received: from sonic317-32.consmr.mail.ne1.yahoo.com (sonic317-32.consmr.mail.ne1.yahoo.com [66.163.184.43]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (Client did not present a certificate) by mx1.freebsd.org (Postfix) with ESMTPS id 4P8HDl2nlHz41HV for ; Sat, 4 Feb 2023 15:55:51 +0000 (UTC) (envelope-from pfg@freebsd.org) Authentication-Results: mx1.freebsd.org; dkim=pass header.d=yahoo.com header.s=s2048 header.b=MkggXBn5; spf=softfail (mx1.freebsd.org: 66.163.184.43 is neither permitted nor denied by domain of pfg@freebsd.org) smtp.mailfrom=pfg@freebsd.org; dmarc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=yahoo.com; s=s2048; t=1675526149; bh=CIKvYP6/9GcTVJYIUGCIe63sjJaGuW2Lo/1VgnnBd/k=; h=Date:From:To:In-Reply-To:References:Subject:From:Subject:Reply-To; b=MkggXBn5Iypm/rcMxhvPUZoGV2HUuvhMsMulH8gsa6DnafZV6krdhXo4e11KN94rgImrN4Lu36o/G9Q1az/qaCEqsePEMx/F970c/hAlFNf/tEJ+MYIl085k3gO8MnoRVGTc1LcpTDZk2L6IEmRLWPore6j9hdUW0zQEClZrVHnIQFVyhqNGOtHHUuOvwn7/Bc6V/LmQEg57FgsJ6VcK1ff3WXNfLevOQ97PHWGJx/B6JEgHv1ecKFMsZDg5KvzKjrq34MmUxLgffTQocLdta6oSECh43m8e/wedb7IQXpp+4+V2Ors7o0osNq83o8BVNWaScZALROGR8Zn6MESZQg== X-SONIC-DKIM-SIGN: v=1; a=rsa-sha256; c=relaxed/relaxed; d=yahoo.com; s=s2048; t=1675526149; bh=ZmcM15IaCVGo6Pgh4AiaEX08O/spYkaZMxHN4WH/0jc=; h=X-Sonic-MF:Date:From:To:Subject:From:Subject; b=jhbCV36/5z3pXfLBIRzC/Sef8mQfFTdDqLyUL1xGCgdu3KaQmuWyi2USMpNJL+zvcPHuT/mH5WbRBU8VWwKAA5XFmbfKAwS94oDvCFzzeLeGG5A5RhQmP9Mv3PMlTUXikEkK2wgzMUNduWViWT+V+A8kIvD19yRUUUq02E/yjvBuIHivkxBb4UBz4sarkxZecGCmYTH1+0p4f+bkWkuRitv7RV4+YPBVHg2nTPfqmTS3jcb3znbSsG2FuTiDOm+RSJrWg7yB1SHMD5rI8CM8KSteCkadrBZ17EX1iG5y2RGOj8yZb/tCL+0tOgsQ532Yoe2QDcRTGghH4ytBK/Fnow== X-YMail-OSG: oX8NtAwVM1k0aOF62XwvqkybcVRFHhtHrCplA1gLd9UQ6VHR9fvCWRn5LXKk2cW M_PIXc.kFVTbZffkU30kqHcA1DWsuLJo9fOmLVbjxACSWAKSahs4ZPBL4k.cPoyM13WEGUUt5dMI MU3BpREJpxs4cUu6W6UuH_V.8bOlMGHhXYVkgWIOkaTSuSF.gCsSL47uVYcbgVNFy4TZ4LH8n_Ng yqzVxuKGOHV9BxBCvF21hDjAjiGSDmjmvaB6HBW0H3Wcl2NKiWL3ZeHSdfepYh6FbUj9YrFGjq9i ctdQMA.gN4oPYkWsTVOUUkir6PJfz1ZQEcI4xNMXLClp4ssF.6zxJeHtIYPOicdIJ9NWn4WzNKYj On3Qg6aA6jqg.Aso3qE0_oKOGisHh_EqEEOxtVQycgxMwwntfLqzU7vVN5ApOQBnaRk9zbtXX0jM JxPfvSqnwmN7OwQwo7vKsZCu7IroOAEq8sgCpB1Pu0XRvKTXqLalaeWoLpgWomxu9J9wiCkuBUSU HkkKVLEscwKRMA8s2kmHKpp1_sJtT5ly0q3TYA.J..kQcuA_W6cz1ZuQ5vdSo.Lfn5zbE_uaAwuH 4fV4bMcggq0TBOI5.Qcih_akokpYoGUs4YaIosSB7vs7l0mI8fXO_DgRkGXGU67tDET1XxVT.Qcf nfR0_jZdcBYAYxJgcyePzNSOtBZgnRYE9FbiLW5YbsInSnEtxRxaplJWTqq2kelxwbI7gSSYt47W nzlgB2Ms8IYGAzNWdUOXu33qE5mVmzoW0QQqujHDC2nLB_MbEHM65XNJD.MNlkiqU.paiqEZPw8z U9mpwBnWzANfbqc9Zv3wfTAcJAWbZwWcIWO1SYAYIKFve.H_EJcGEQ4gNUjW.jrfymZ8js0eotdK lb9dESjXOC3gcryau4TfIMi9x5tq2FZa7s1koUWQCIoosoBuFtk3Th6f7bzgLL3_n1wy9GkANhBN 8wBZrvXbH_nLNksFvdQMD8mvU6yN77CzgiebRibaH.v6JhgefqONZQZ1WxyxskSxDen8SpS4C_Dp lQMspkzRMf.grFqQkrh4KMRzBE0sz4RaE8K1thMbX13zcCENnnKfD1R0HDkXgoUxqmYeTH9xsZ9P 1X9VMObQw1sjob7Jqk1DgMPGA4e2GEJ.ImAU2JesrEhOVrfcZLF8W8wtL4QpJSoTFf6kK68ZcnSn BDSj1aLCSrMubCTTZOFmmxCRuPl90lWpZSodjI7s.SXs2Znbl3eZd8g67clr718qMrbd5IyprIXi HfeXTfbjdSLUXlnX7.l0lGCRLNkVci09IGXL1eksX7QnlSIbaGElX_X4WYjel8ZxAsQwW3rZwXuq Rhv.qoTljr014gzpPnP4GmysqZHZi2.dv1EBoBIySjpnkfPRjH6Xm4H8l8tW.Py2idRINTxELoKu c_O8cfYrdQa86welSdFWYtEUPYhylqIUg8at_PqrS5tlIU07A7KNK9eGm3msc9N0s3b4u92G5uc4 81DG_6rbJOZL3wEIqRGSEN3M22J2d7xYJ5Xl9e2k9cFqRW_FXzWCWiYwu7dTgClOGkEdTRw3o.tC uEkEJbT1vrZAcgr2biv7X_qLxz2FJz36mSvOZgksowEumgOOd18WPMtPoJltjsCdxem.z6U1yHrR MYo1HKrRmlIVmRUcqQOCrm13b8jtfRIHd5xUeuaTR6LNp1ONKBdBUF8g3UGYXKUNfIyr5loTfRaO BE7YxHfnPsniXqxL9licEA85X6BX9Xlf0bSxoTM2jY.jKXaENqABJ8eqNOMm0rZbpLXpxOYOEipz rv10SeS0GBkoX.sqYOFVSLUzPyfJGSDwz0qdZB1uZKWg_z5keOO9QFAVYVWgu6AwfCm2IFUZYfDR .AFGKhPUjmfJLEwMGDhb361muF5.YgnzGN4I7C48mfBPw8ADbXrTtErl0rQn353.Hct4qo0R9QgU DSOAn3jTI3VlB2rr5XENJ_RCL0WAfXjirbWj4q70Ev.Y.0NXhhhn7rhTOA6SCT0BcVXMph3MzPiJ L.yRnzwbUB865RiIbWW5tp0HsdjcEZ6CcVOitz7bCvuHgcVWpaYFmXvdgK0MOoVRFZGEtUaxtYFU zGl41i_N43RVO6qrvCC_nPMdaoMduGF73K8mpqqwP0fxZpdwO.SU7Bb8T7YvTA4oYTSC__PZVcSX wTB45j0TzSxOl9Qq_fSaLYtoQJUs8kfdScPtHGURYbokA92QupAjNb1Y- X-Sonic-MF: Received: from sonic.gate.mail.ne1.yahoo.com by sonic317.consmr.mail.ne1.yahoo.com with HTTP; Sat, 4 Feb 2023 15:55:49 +0000 Date: Sat, 4 Feb 2023 15:55:48 +0000 (UTC) From: Pedro Giffuni To: "freebsd-ppc@FreeBSD.org" Message-ID: <1967210382.221190.1675526148958@mail.yahoo.com> In-Reply-To: <202302030714.3137EVRQ088519@gitrepo.freebsd.org> References: <202302030714.3137EVRQ088519@gitrepo.freebsd.org> Subject: Does hastd work on Big Endian? Fw: git: 6998572a74a9 - main - hastd: use zlib's crc32 implementation. List-Id: Porting FreeBSD to the PowerPC List-Archive: https://lists.freebsd.org/archives/freebsd-ppc List-Help: List-Post: List-Subscribe: List-Unsubscribe: Sender: owner-freebsd-ppc@freebsd.org X-BeenThere: freebsd-ppc@freebsd.org MIME-Version: 1.0 Content-Type: multipart/alternative; boundary="----=_Part_221189_1075028453.1675526148956" X-Mailer: WebService/1.1.21161 YMailNorrin X-Spamd-Result: default: False [-3.30 / 15.00]; NEURAL_HAM_LONG(-1.00)[-1.000]; NEURAL_HAM_MEDIUM(-1.00)[-1.000]; NEURAL_HAM_SHORT(-1.00)[-1.000]; R_DKIM_ALLOW(-0.20)[yahoo.com:s=s2048]; MIME_GOOD(-0.10)[multipart/alternative,text/plain]; FROM_EQ_ENVFROM(0.00)[]; MLMMJ_DEST(0.00)[freebsd-ppc@FreeBSD.org]; TO_DN_EQ_ADDR_ALL(0.00)[]; MIME_TRACE(0.00)[0:+,1:+,2:~]; RCVD_TLS_LAST(0.00)[]; RCVD_IN_DNSWL_NONE(0.00)[66.163.184.43:from]; RCVD_COUNT_TWO(0.00)[2]; ASN(0.00)[asn:36646, ipnet:66.163.184.0/21, country:US]; ARC_NA(0.00)[]; RCPT_COUNT_ONE(0.00)[1]; SUBJECT_HAS_QUESTION(0.00)[]; FROM_HAS_DN(0.00)[]; FREEFALL_USER(0.00)[pfg]; DKIM_TRACE(0.00)[yahoo.com:+]; DWL_DNSWL_NONE(0.00)[yahoo.com:dkim]; DMARC_NA(0.00)[freebsd.org]; TO_MATCH_ENVRCPT_ALL(0.00)[]; R_SPF_SOFTFAIL(0.00)[~all]; TO_DOM_EQ_FROM_DOM(0.00)[] X-Rspamd-Queue-Id: 4P8HDl2nlHz41HV X-Spamd-Bar: --- X-ThisMailContainsUnwantedMimeParts: N ------=_Part_221189_1075028453.1675526148956 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: quoted-printable Hi; Not really concerned about the commit here .. but about the last comment. C= hecksums may need adjustments for endianness. Cheers, Pedro.=C2=A0 ----- Forwarded Message ----- From: Xin LI To: "src= -committers@freebsd.org" ; "dev-commits-src-all= @freebsd.org" ; "dev-commits-src-main@free= bsd.org" Sent: Friday, February 3, 2023 a= t 02:14:34=E2=80=AFAM GMT-5Subject: git: 6998572a74a9 - main - hastd: use z= lib's crc32 implementation. The branch main has been updated by delphij: URL: https://cgit.FreeBSD.org/src/commit/?id=3D6998572a74a98721781ecd5b6829= 435259f9825a commit 6998572a74a98721781ecd5b6829435259f9825a Author:=C2=A0 =C2=A0 Xin LI AuthorDate: 2023-02-03 07:14:21 +0000 Commit:=C2=A0 =C2=A0 Xin LI CommitDate: 2023-02-03 07:14:21 +0000 =C2=A0 =C2=A0 hastd: use zlib's crc32 implementation. =C2=A0 =C2=A0=20 =C2=A0 =C2=A0 Reviewed by:=C2=A0 =C2=A0 pjd =C2=A0 =C2=A0 MFC after:=C2=A0 =C2=A0 =C2=A0 2 weeks =C2=A0 =C2=A0 Differential Revision: https://reviews.freebsd.org/D35767 --- ......diff --git a/sbin/hastd/hast_checksum.c b/sbin/hastd/hast_check= sum.c index f7ef6a64b4e1..dccba3752cb1 100644 --- a/sbin/hastd/hast_checksum.c +++ b/sbin/hastd/hast_checksum.c @@ -33,10 +33,10 @@ __FBSDID("$FreeBSD$"); #include #include =20 -#include #include #include #include +#include #include =20 #include "hast_checksum.h" @@ -49,7 +49,9 @@ hast_crc32_checksum(const unsigned char *data, size_t siz= e, { =C2=A0=C2=A0=C2=A0 uint32_t crc; =20 -=C2=A0=C2=A0=C2=A0 crc =3D crc32(data, size); +=C2=A0=C2=A0=C2=A0 crc =3D crc32(0L, Z_NULL, 0); +=C2=A0=C2=A0=C2=A0 crc =3D crc32(crc, data, size); + =C2=A0=C2=A0=C2=A0 /* XXXPJD: Do we have to use htole32() on crc first? */ =C2=A0=C2=A0=C2=A0 bcopy(&crc, hash, sizeof(crc)); =C2=A0=C2=A0=C2=A0 *hsizep =3D sizeof(crc); =20 ------=_Part_221189_1075028453.1675526148956 Content-Type: text/html; charset=UTF-8 Content-Transfer-Encoding: quoted-printable
Hi;

Not re= ally concerned about the commit here .. but about the last comment. Checksu= ms may need adjustments for endianness.

Cheers,

Pedro.
 
<= div>
=20
----- Forwarded Message -----
From: Xin LI <delphij@freebsd.org>
<= div>To: "src-committers@freebsd.org" <src-committers@freebsd.org&= gt;; "dev-commits-src-all@freebsd.org" <dev-commits-src-all@freebsd.org&= gt;; "dev-commits-src-main@freebsd.org" <dev-commits-src-main@freebsd.or= g>
Sent: Friday, February 3, 2023 at 02:14:34=E2=80=AFA= M GMT-5
Subject: git: 6998572a74a9 - main - hastd: use zli= b's crc32 implementation.

The branch main has been updated by d= elphij:


commit 6998572a74a98721781ecd5b682943= 5259f9825a
Author:    Xin LI <delphi= j@FreeBSD.org>
AuthorDate: 2023-02-03 07:1= 4:21 +0000
Commit:    Xin LI <delphi= j@FreeBSD.org>
CommitDate: 2023-02-03 07:1= 4:21 +0000

  &nbs= p; hastd: use zlib's crc32 implementation.
 =  
    Reviewed by:    = pjd
    MFC after:      = 2 weeks
    Differential Revision: https://reviews.freebsd.org/D35767
---
=
...
<= snip>
...
diff --git a/sbin/hastd/hast_checksum.c b/sbin/hastd/hast_checksum.c
index f7ef6a64b4e1..dccba3752cb1 100644
--- a/sbin/hastd/hast_checksum.c
+= ++ b/sbin/hastd/hast_checksum.c
@@ -33,10 +33,10 = @@ __FBSDID("$FreeBSD$");
#include <string.h&= gt;
#include <strings.h>

-#include <crc32.h>
#include <hast.h>
#include = <nv.h>
#include <sha256.h>
=
+#include <zlib.h>
#inclu= de <pjdlog.h>

= #include "hast_checksum.h"
@@ -49,7 +49,9 @@ hast= _crc32_checksum(const unsigned char *data, size_t size,
{
    uint32_t crc;
=

-    crc = =3D crc32(data, size);
+    crc = =3D crc32(0L, Z_NULL, 0);
+    crc= =3D crc32(crc, data, size);
+
    /* XXXPJD: Do we have to use htole32() on crc = first? */
    bcopy(&crc, has= h, sizeof(crc));
    *hsizep =3D = sizeof(crc);
------=_Part_221189_1075028453.1675526148956--