-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
s390x support #990
Comments
Could you please paste the content of |
Thanks @Lomanic for considering s390x support. Here are the contents from /proc/cpuinfo and lscpu from a s390x m/c:
lscpu
1 cpu vm:
lscpu
|
@Lomanic Can you please tel me if we have any update on this. |
@Lomanic Any update on this , please? |
See #1004 (comment), I added some mocks for #881 in https://github.com/Lomanic/gopsutil/tree/issue881, which will also enable this issue #990 to be handled without regressions for untouched platforms (386 and amd64 namely). I have to get back to it. |
@Lomanic could you please let us know if support to s390x arch is added? |
Currently there are two tests are failing on s390x.
Test_Connections looks like indeed a bug, since the code assumes little endian. Lines 724 to 726 in 7a094df
|
I have opened #1322. Could you test by using that PR? (sorry, I don't have a s390x) |
@shirou Could you please share exact steps that I can follow to build and test on s390x? |
I think you can just run |
@zhsj got below result : $ go test ./...
|
(perhaps cpuinfo is still problem. so re-opened this PR) |
#1414 fixes this issue. |
@Nayana-ibm I added this support so that we could get the DataDog agent running on our IBM Z. |
I was building/executing tests for vault. There are some tests makes use of routines from https://github.com/shirou/gopsutil/blob/master/cpu/cpu_linux.go#L116 to read /proc/cpuinfo
I could build vault from source however thetwo tests are failing for s390x architecture ( with an error: Expected non-nil CPU value) as this arch's /proc/cpuinfo has different contents than intel x86.
Is there any plan to support s390x arch for https://github.com/shirou/gopsutil ?
Please let me know.
The text was updated successfully, but these errors were encountered: