MIME-Version: 1.0
References: <20250911054220.3784-1-root@ip-172-31-36-228.ec2.internal>
 <CANWCAZYZQw-nzTXbx3Bk332VtY9_D7ksDsuMZ0A-iDZ53yG7Ng@mail.gmail.com>
 <CAK64mnfeWLBRbMfnOsag0vGTDnT84KJzpuei40nG0OHyw4SESw@mail.gmail.com>
 <CANWCAZa1b2rcvoK657SmcKwh2P2cgASQ1D-0JPj5d3LbfaAVgA@mail.gmail.com>
 <CAK64mneN20+sW5WhV+r7hMVo4Rd0z11B6=3L039rWMt1wK3nPg@mail.gmail.com>
 <CANWCAZZuS3sNgLRo8Z4AM=uY4zTmz=dH5D4Z9xV6K0CEuJ8Hdw@mail.gmail.com>
 <CAK64mnejn9AZMYz03e7HX8Uui35PihUuOy=b+iBG=YtRKx0Log@mail.gmail.com>
 <CANWCAZZ_0AQMk1HgHXHX+JaeBfy_4kzwHgTdqMptDA7zM+nm+Q@mail.gmail.com>
 <CAK64mnc6jbehHv5AHc84tVFRJg4zeMiFuvPX9xZkRpq0210MFA@mail.gmail.com>
 <CANWCAZY940P3wGOQAZWMLQL4MQGGyOu7WBjBEcn_gqcrr+NvAw@mail.gmail.com>
 <CAK64mne_oWN9d4mf+0c_5-4Emb9kRXA-OC05OJ4F_1fVqpjzDA@mail.gmail.com>
 <CANWCAZZcKYp+01u1QmkShfXVkUCCdxtJAgHT-61Vw0ALoWj47A@mail.gmail.com>
In-Reply-To: 
 <CANWCAZZcKYp+01u1QmkShfXVkUCCdxtJAgHT-61Vw0ALoWj47A@mail.gmail.com>
From: Andrew Kim <tenistarkim@gmail.com>
Date: Tue, 18 Nov 2025 17:18:36 -0800
Message-ID: 
 <CAK64mne=Q_4VSpJ8f4RQB-yAThd4+i-BRYMvfdGOhvwJQdYoKQ@mail.gmail.com>
Subject: Re: Proposal for enabling auto-vectorization for checksum
 calculations
To: John Naylor <johncnaylorls@gmail.com>
Cc: pgsql-hackers@lists.postgresql.org,
	Oleg Tselebrovskiy <o.tselebrovskiy@postgrespro.ru>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: quoted-printable
Archived-At: 
 <https://www.postgresql.org/message-id/CAK64mne%3DQ_4VSpJ8f4RQB-yAThd4%2Bi-BRYMvfdGOhvwJQdYoKQ%40mail.gmail.com>
Precedence: bulk

On Mon, Nov 17, 2025 at 8:48=E2=80=AFPM John Naylor <johncnaylorls@gmail.co=
m> wrote:
>
> On Tue, Nov 18, 2025 at 10:32=E2=80=AFAM Andrew Kim <tenistarkim@gmail.co=
m> wrote:
>
> > I've double-checked everything after applying the v9 checksum patches
> > and updating pg_filedump accordingly.
> > Following your suggestion, I removed the checksum_impl.h include and
> > the Assert redefinition, keeping only the port/checksum.h include.
> > build compiles cleanly with the new architecture, and pg_filedump
> > functions correctly with the AVX2 optimizations.
> > If you agree with this approach, I'd like to prepare a patch for
> > upstream submission.
>
> Thanks, but there is no action required now. I mentioned some
> observations, and I'm going to experiment a bit further.
>
> Also, it's way too early to be suggesting changes to other projects,
> since we haven't committed anything yet. Even things committed can
> occasionally be reverted before beta 1 anyway.

It makes sense, I fully understand. I will standby and wait for your
next update.

-Andrew Kim

> --
> John Naylor
> Amazon Web Services