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.

Sign up for GitHub

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

Jump to bottom

getsockopt (TCP_INFO) #503

Open
aep opened this issue Apr 7, 2022 · 3 comments
Open

getsockopt (TCP_INFO) #503

aep opened this issue Apr 7, 2022 · 3 comments
Labels
feature request P: 3

Comments

@aep
Copy link

aep commented Apr 7, 2022

php is requesting tcpi_sacked from tcp info.
this is a quick fix, as we can just set it to 0 to pretend being linux 2.4

should i PR?
@boryspoplawski
Copy link
Contributor

The networking stack is being completely rewritten, I wouldn't want to introduce new feature and/or change much code in the meantime.

aep added a commit to enclaive/enclaive-docker-nginx-sgx that referenced this issue Apr 7, 2022
@aep
nginx + php-fpm + wordpress
ed11d17 
nginx in enclave:

    - do not fork workers due to gramineproject/gramine#468 (also generally dont)
    - use select instead of epoll due to gramineproject/gramine#489
    - link pcre static due to undetermined dynloader bug

php in enclave:

    - patch away sockopt SO_LISTENQLEN and TCP_INFO due to gramineproject/gramine#503
    - do not fork workers
    - manually force linking libm static, because the ld loader implements glibc versions incorrectly
    - remove apparmor from AC

wordpress in enclave:

    - installer loads fine but needs mysql server to proceed
@aep aep mentioned this issue Apr 7, 2022
Closed
aep added a commit to enclaive/enclaive-docker-nginx-sgx that referenced this issue Apr 8, 2022
@aep
nginx + php-fpm + wordpress
689fc2c 
nginx in enclave:

    - do not fork workers due to gramineproject/gramine#468 (also generally dont)
    - use select instead of epoll due to gramineproject/gramine#489
    - link pcre static due to undetermined dynloader bug

php in enclave:

    - patch away sockopt SO_LISTENQLEN and TCP_INFO due to gramineproject/gramine#503
    - do not fork workers
    - manually force linking libm static, because the ld loader implements glibc versions incorrectly
    - remove apparmor from AC

wordpress in enclave:

    - installer loads fine but needs mysql server to proceed
aep added a commit to enclaive/enclaive-docker-nginx-sgx that referenced this issue Apr 8, 2022
@aep
nginx + php-fpm + wordpress
d517fa9 
nginx in enclave:

    - do not fork workers due to gramineproject/gramine#468 (also generally dont)
    - use select instead of epoll due to gramineproject/gramine#489
    - link pcre static due to undetermined dynloader bug

php in enclave:

    - patch away sockopt SO_LISTENQLEN and TCP_INFO due to gramineproject/gramine#503
    - do not fork workers
    - manually force linking libm static, because the ld loader implements glibc versions incorrectly
    - remove apparmor from AC

wordpress in enclave:

    - installer loads fine but needs mysql server to proceed
aep added a commit to enclaive/enclaive-docker-nginx-sgx that referenced this issue Apr 8, 2022
@aep
nginx + php-fpm + wordpress
bbcdd9b 
nginx in enclave:

    - do not fork workers due to gramineproject/gramine#468 (also generally dont)
    - use select instead of epoll due to gramineproject/gramine#489
    - link pcre static due to undetermined dynloader bug

php in enclave:

    - patch away sockopt SO_LISTENQLEN and TCP_INFO due to gramineproject/gramine#503
    - do not fork workers
    - manually force linking libm static, because the ld loader implements glibc versions incorrectly
    - remove apparmor from AC

wordpress in enclave:

    - installer loads fine but needs mysql server to proceed
@boryspoplawski
Copy link
Contributor

#579 rewrites the socket stack. After it's merged (which hopefully will happen, I would estimate in two weeks), we can start accepting PRs modifying that code.

But I must say, we cannot really pretend we are linux 2.4, we already present some info about versions (and they are higher than that).

@dimakuv
Copy link
Contributor

dimakuv commented Mar 9, 2023

@aep The networking stack was rewritten. If you still need this/want this, please submit a PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
feature request P: 3
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@aep @dimakuv @boryspoplawski