Skip to content
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

Fix wrong behavior on s390x #91

Merged
merged 2 commits into from
May 7, 2020
Merged

Fix wrong behavior on s390x #91

merged 2 commits into from
May 7, 2020

Conversation

iii-i
Copy link
Contributor

@iii-i iii-i commented Apr 29, 2020

Fixes #86

Copy link
Collaborator

@lmb lmb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for contributing! I left some nits, I'd appreciate if you could look into them. Other than that I'm happy to merge the PR.

internal/testutils/endianness.go Show resolved Hide resolved
testdata/Makefile Show resolved Hide resolved
asm/instruction_systemz.go Outdated Show resolved Hide resolved
asm/instruction_systemz.go Outdated Show resolved Hide resolved
iii-i added 2 commits May 4, 2020 15:57
Strictly speaking, bitfield order is defined by a respective arch ABI,
however, in practice it always the same as byte order, and LLVM already
makes use of this correlation, therefore do the same in here.

Signed-off-by: Ilya Leoshkevich <[email protected]>
Signed-off-by: Ilya Leoshkevich <[email protected]>
@lmb
Copy link
Collaborator

lmb commented May 7, 2020

Your proposals sound sane, looking forward to your update!

@iii-i
Copy link
Contributor Author

iii-i commented May 7, 2020

I actually pushed one a few days back (3db7828) :-) Could you have a look please?

@lmb
Copy link
Collaborator

lmb commented May 7, 2020

Oh boy, sorry about that! I missed the notification somehow. Thanks for working on this!

@lmb lmb merged commit e7924d2 into cilium:master May 7, 2020
alicefr pushed a commit to alicefr/runc that referenced this pull request May 8, 2020
Update ebpf to include PR cilium/ebpf#91.
The update is needed to fix opencontainers#2316.

Signed-off-by: Alice Frosi <[email protected]>
alicefr pushed a commit to alicefr/runc that referenced this pull request May 8, 2020
Update ebpf to include PR cilium/ebpf#91.
The update is needed to fix opencontainers#2316.

Signed-off-by: Alice Frosi <[email protected]>
alicefr pushed a commit to alicefr/runc that referenced this pull request May 8, 2020
Update ebpf to include PR cilium/ebpf#91.
The update is needed to fix opencontainers#2316.

Signed-off-by: Alice Frosi <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wrong behavior on s390x
2 participants