Illegal instruction fix

Torbjörn Granlund tg at gmplib.org
Wed Jan 6 00:26:31 UTC 2021


William Blanke <bill at chia.net> writes:

  At Chia, we are using GMP inside Python binary wheels. This means we
  compile GMP on one machine and then run this code on different machines,
  possibly with different CPUs.

  When GMP 6.2 was released, we encountered illegal instruction crashes on
  Intel CPUs older than Haswell. We traced this to the LZCNT instruction
  being run on these older CPUs even though fat binary was chosen on the post
  Haswell compile machine. Unfortunately, this still happens with GMP 6.2.1

  To fix this we have created the patch below. It chooses whether to use the
  LZCNT, MULX, and TZCNT instructions based on CPU support via runtime CPUID
  checks for LZCNT, BMI2, and BMI1 capabilities respectively.

  We would like to submit this for upstream inclusion in future versions of
  GMP. We hope that it helps and we thank you for all your efforts!

We don't do this type of runtime detection for single instructions, as
the choosing takes way more time than any basic old code sequence.

Now, it is somewhat surprising thet rep;bsr fails.  The rep should be
ignored for old processors.  OK, so maybe not.

Then the correct approach is to suppress that variant for fat builds
and fall back to the less-defined plain bsr.

-- 
Torbjörn
Please encrypt, key id 0xC8601622


More information about the gmp-bugs mailing list