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

add effective user context manipulation routines #1127

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

PetrShumilov
Copy link
Contributor

Goals:

  • Remove useless rt_sigprocmask syscall invocation during user context manipulation
  • Add platform-independent API for user context-related things

Related issues: KPHP-1944

@PetrShumilov PetrShumilov added enhancement New feature or request optimization Memory comsumption / CPU speedup runtime Feature related to runtime labels Oct 23, 2024
@PetrShumilov PetrShumilov added this to the next milestone Oct 23, 2024
@PetrShumilov PetrShumilov self-assigned this Oct 23, 2024
@PetrShumilov PetrShumilov force-pushed the pshumilov/remove_rt_sigprocmask branch 25 times, most recently from a130fb4 to 868512e Compare October 28, 2024 10:37
@PetrShumilov PetrShumilov force-pushed the pshumilov/remove_rt_sigprocmask branch from 868512e to 8af5e3f Compare October 28, 2024 10:52
@PetrShumilov PetrShumilov marked this pull request as ready for review October 28, 2024 11:11
@PetrShumilov PetrShumilov force-pushed the pshumilov/remove_rt_sigprocmask branch from 381fabd to 010fd7e Compare October 29, 2024 13:12
server/php-runner.cpp Show resolved Hide resolved
stack_end = reinterpret_cast<char *>(get_context_stack_ptr_portable(exit_context)) + get_context_stack_size_portable(exit_context);

#if defined(__linux__) && defined(__x86_64__)
// The PhpScript::error may be produced in process of signal handling. The default behavior on Linux-based platforms
Copy link
Contributor

Choose a reason for hiding this comment

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

We also have own implementation for darwin aarch64. Do we need same logic for this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I will check it. Previously, we also had own implementation for darwin aarch64 but no needed a special handling for this case. I may be a result of different signal subsystems.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Checked, thanks for notice, fixed for darwin aarch64

Signed-off-by: Petr Shumilov <[email protected]>
Signed-off-by: Petr Shumilov <[email protected]>
Signed-off-by: Petr Shumilov <[email protected]>
Signed-off-by: Petr Shumilov <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request optimization Memory comsumption / CPU speedup runtime Feature related to runtime
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants