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

RFE: bump minimum required version to 2.2.0 #18

Closed
wants to merge 1 commit into from
Closed

RFE: bump minimum required version to 2.2.0 #18

wants to merge 1 commit into from

Conversation

thorduri
Copy link
Contributor

@thorduri thorduri commented Dec 3, 2016

Set the SCMP_FLTATR_CTL_TSYNC attribute in NewFilter, and error out should
that fail.

Since the threads are hidden from the programmer, it is inherently unsafe
to run without this bit set, as the runtime can and does create threads
very early, which are used in subsequent operations (go routines), meaning
that the filter might not be applied, see discussion in #17.

Signed-off-by: Thordur Bjornsson thorduri@secnorth.net

Set the SCMP_FLTATR_CTL_TSYNC attribute in NewFilter, and error out should
that fail.

Since the threads are hidden from the programmer, it is inherently unsafe
to run without this bit set, as the runtime can and does create threads
very early, which are used in subsequent operations (go routines), meaning
that the filter might not be applied, see discussion in #17.

Signed-off-by: Thordur Bjornsson <thorduri@secnorth.net>
@pcmoore pcmoore changed the title all: Bump minimum required version to 2.2.0. RFE: bump minimum required version to 2.2.0 Dec 16, 2016
@pcmoore pcmoore modified the milestone: v2.4 Dec 16, 2016
@thorduri thorduri mentioned this pull request Jan 5, 2017
@thorduri
Copy link
Contributor Author

thorduri commented Feb 1, 2018

@pcmoore @mheon Worth rebasing and keeping open ?

@mheon
Copy link
Contributor

mheon commented Feb 1, 2018

@thorduri We set TSync by default now, so I think this is obsolete

@mheon mheon closed this Feb 1, 2018
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.

None yet

3 participants