Skip to content

seccomp: remove modify_ldt#22580

Closed
jessfraz wants to merge 1 commit intomoby:masterfrom
jessfraz:remove_modify_ldt
Closed

seccomp: remove modify_ldt#22580
jessfraz wants to merge 1 commit intomoby:masterfrom
jessfraz:remove_modify_ldt

Conversation

@jessfraz
Copy link
Contributor

@jessfraz jessfraz commented May 8, 2016

Signed-off-by: Jess Frazelle <[email protected]>
@jessfraz
Copy link
Contributor Author

jessfraz commented May 8, 2016

hmm janky got a new font, but still hates me

@justincormack
Copy link
Contributor

For reference this was added because Go uses it on i386 binaries see #19218 #19201

However it has recently been removed from Go in golang/go@0f86d1e with a note that it is not available on some newer kernels, and has to be explicitly made available on others, and replaced by the modern set_thread_area. Issue is golang/go#14795. It does not seem this commit is in a release yet though.

There is also an unconfirmed suggestion that Wine uses it (allegedly even for 64 bit code) https://lkml.org/lkml/2015/7/21/759 would like to confirm if this the case because we have a lot of Steam users it seems based on other issues.

So a bit unsure, would like to block it, but I would be happier if there was a Go release that worked, as no 32 bit Go programs will run at all, and people do use 32 bit mode a bit.

@jessfraz
Copy link
Contributor Author

jessfraz commented May 8, 2016

Ah I have a wine container I can test, and that's super interesting about
go

On Sunday, May 8, 2016, Justin Cormack [email protected] wrote:

For reference this was added because Go uses it on i386 binaries see
#19218 #19218 #19201
#19201

However it has recently been removed from Go in golang/go@0f86d1e
golang/go@0f86d1e
with a note that it is not available on some newer kernels, and has to be
explicitly made available on others, and replaced by the modern
set_thread_area. Issue is golang/go#14795
golang/go#14795. It does not seem this commit
is in a release yet though.

There is also an unconfirmed suggestion that Wine uses it (allegedly even
for 64 bit code) https://lkml.org/lkml/2015/7/21/759 would like to
confirm if this the case because we have a lot of Steam users it seems
based on other issues.

So a bit unsure, would like to block it, but I would be happier if there
was a Go release that worked, as no 32 bit Go programs will run at all, and
people do use 32 bit mode a bit.


You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#22580 (comment)

Jessie Frazelle
4096R / D4C4 DD60 0D66 F65A 8EFC 511E 18F3 685C 0022 BFF3
pgp.mit.edu http://pgp.mit.edu/pks/lookup?op=get&search=0x18F3685C0022BFF3

@jessfraz
Copy link
Contributor Author

jessfraz commented May 8, 2016

I think we can hold off for now, but just for future reference I will try some things, I have a steam container as well ;)

@jessfraz
Copy link
Contributor Author

jessfraz commented May 9, 2016

I'm going to close as it's not realistic at this point in time, but will give updates on this thread after further testing so we have it for the future :)

@jessfraz jessfraz closed this May 9, 2016
@justincormack
Copy link
Contributor

cool, it would be good to work out when we can reasonably remove it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants