Crashes with SIGILL on Linux/amd64 i7 CPU

[karalabe]

Caught SIGILL in blst_cgo_init [...]

[karalabe]

Seems my older laptop CPU is too old for for some instructions. This is bad really, the upstream solution is "if it crashes, rebuild with flag xyz". Can we please fix the upstream BLS library to detect and not use unavailable CPU. instructions, or alternatively have this library add the "portable" flags via CGO build constraints? I cannot add the flag downstream into Geth.

[jtraglia]

We cannot add __BLST_PORTABLE__ to the CGO_CFLAGS either. I suggest you make a PR to blst.

[karalabe]

I'm reluctant to accept that suggestion as a viable one.

The issuer was opened on their repo 3 years ago and they don't seem to care: supranational/blst#10

In the issue, it's also written that about 25% of cloud VMs will crash, interestingly, at random since you don't know exactly which CPU you will get (unless you pay enough to have a specifically new gen one).

As for fixing it upstream, with all due respect, it is your dependency - and hence your library - that is broken. It is your task to fix it or work around a broken dependency. My choices are either to use CKZG or to not (and be very vocal about why it is broken so others won't use it either).