"Blake, Geoff" <blakg...@amazon.com> writes: > Hi Tom, Andres, > Any additional feedback for this patch?
I did some more research and testing: * Using a Mac with the M1 Pro chip (marginally beefier than the M1 I was testing on before), I think I can see some benefit in the test case I proposed upthread. It's marginal though. * On a Raspberry Pi 3B+, there's no outside-the-noise difference. * ISB doesn't exist in pre-V7 ARM, so it seems prudent to restrict the patch to ARM64. I doubt any flavor of ARM32 would be able to benefit anyway. (Googling finds that MariaDB made this same choice not long ago [1].) So what we've got is that there seems to be benefit at high core counts, and it at least doesn't hurt at lower ones. That's good enough for me, so pushed. regards, tom lane [1] https://jira.mariadb.org/browse/MDEV-25807