-
Notifications
You must be signed in to change notification settings - Fork 2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
can't create ZSTDMT_CCtx if number of cores > 128 #760
Comments
rgdoliveira
added a commit
to rgdoliveira/aports
that referenced
this issue
Jul 13, 2017
some zstd tests are breaking in ppc64le because it defines the max number of supported cores as 128 (ZSTDMT_NBTHREADS_MAX 128) but the Alpine ppc64le machine has 160 cores. I am increasing the max core value to 256 for now and opened a bug[1] in zstd project. [1] facebook/zstd#760
rgdoliveira
added a commit
to rgdoliveira/aports
that referenced
this issue
Jul 13, 2017
some zstd tests are breaking in ppc64le because it defines the max number of supported cores as 128 (ZSTDMT_NBTHREADS_MAX 128) but the Alpine ppc64le machine has 160 cores. I am increasing the max core value to 256 for now and opened a bug[1] in zstd project. [1] facebook/zstd#760
That's a good point @rgdoliveira ! |
Cyan4973
added a commit
that referenced
this issue
Jul 13, 2017
…DS_MAX (#760) Previously, ZSTDMT would refuse to create the compressor. Also : increased ZSTDMT_NBTHREADS_MAX to 256, updated doc, and added relevant test
I modified the version in |
algitbot
pushed a commit
to alpinelinux/aports
that referenced
this issue
Jul 13, 2017
some zstd tests are breaking in ppc64le because it defines the max number of supported cores as 128 (ZSTDMT_NBTHREADS_MAX 128) but the Alpine ppc64le machine has 160 cores. I am increasing the max core value to 256 for now and opened a bug[1] in zstd project. [1] facebook/zstd#760
thanks @Cyan4973, As this issue is fixed in dev branch and I guess it will be part of a future zstd release, I'm closing this issue. |
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
I'm compiling zstd in a Powerpc machine (ppc64le) and it builds fine, but when I try to run the test suite it fails with the following error:
"zstd: error 30 : allocation error : can't create ZSTDMT_CCtx"
zstd defines the max number of cores as 128 (ZSTDMT_NBTHREADS_MAX 128) but the Power machine I'm using has 160 cores and the test break.
In this case I guess the execution should continue and use the max number of cores instead of breaking it.
The text was updated successfully, but these errors were encountered: