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

FS-10367: Granted FreeSwitch permission to manage system scheduler #7

Closed
wants to merge 1 commit into from

Conversation

sergey-safarov
Copy link
Contributor

Granted FreeSwitch permission to manage system scheduler when started as regular user

@sergey-safarov
Copy link
Contributor Author

DoTo
Later need manage FreeSwitch root permission at start.

@mjerris
Copy link
Collaborator

mjerris commented Sep 25, 2019

is it possible to address this with something like systemd instead?

@mjerris
Copy link
Collaborator

mjerris commented Sep 25, 2019

DoTo
Later need manage FreeSwitch root permission at start.

what does this mean?

@sergey-safarov
Copy link
Contributor Author

is it possible to address this with something like systemd instead?

you can use this systemd unit params

  • Nice=-19
  • CPUSchedulingPolicy=fifo
  • CPUSchedulingPriority=1
  • CapabilityBoundingSet=CAP_SYS_ADMIN

But I not sure that permission for manage threads scheduler and priority is provider to FreeSwitch process.
Need write some test module that provide functions to start thread with given scheduler and priority.

More details

@sergey-safarov
Copy link
Contributor Author

what does this mean?

That my note to remember that want create one more PR after this resolved.

@mjerris
Copy link
Collaborator

mjerris commented Sep 25, 2019

please let me know if you find a systemd way to manage this.

@sergey-safarov
Copy link
Contributor Author

I can find way if added FS commands like

utils thread start sheduler ${sheduler_var} priority ${priority_var} nice {nice_var}

If FreeSwitch able start thread, then report

Thread started with ${sheduler_var} priority ${priority_var} nice {nice_var}

If FreeSwitch cannot start thread, then report

FreeSwitch cannot start thread with ${sheduler_var} priority ${priority_var} nice {nice_var}

@mjerris
Copy link
Collaborator

mjerris commented Sep 25, 2019

this seems useful for testing but it does not answer the question. we need to validate if there is a way to do this

@sergey-safarov
Copy link
Contributor Author

I will able validate if freeswitch will have validation tool.

This was referenced Oct 11, 2019
@anshkatriya anshkatriya mentioned this pull request Nov 27, 2019
@sergey-safarov sergey-safarov deleted the FS-10367 branch December 12, 2019 03:34
mjerris pushed a commit that referenced this pull request Jan 9, 2020
[core] support partial speech events
@nwaycn nwaycn mentioned this pull request Mar 28, 2020
@sergey-safarov sergey-safarov restored the FS-10367 branch May 1, 2020 07:46
@sergey-safarov sergey-safarov reopened this May 1, 2020
evanowen pushed a commit to inteliquent/freeswitch-1 that referenced this pull request May 5, 2020
@mjerris mjerris mentioned this pull request Aug 21, 2020
@Len-PGH
Copy link
Collaborator

Len-PGH commented Mar 13, 2023

This PR is older than 6 months and requires some attention to be merged, so we’re closing this issue as inactive. If you still want this submission accepted, please make sure to update your code to the latest version (this includes FreeSWITCH, sofia-sip, libks, and any other dependencies your code touches), double check that you’re following the submission guidelines, and make any revisions/updates suggested by the team.

We’re doing a PR Review Day event on March 21st. You’re welcome to make the necessary changes or join with any questions. We'll have team members on hand to provide immediate feedback, work through any issues that come up, and to commit changes that pass review during the event.

We hope to see you there! If you have any questions or feedback about this event or just want some more information, you can drop in to our Community Slack.

Note: There will be a second Review Day in April if you cannot make this time.

@Len-PGH Len-PGH closed this Mar 13, 2023
@xxmcf xxmcf mentioned this pull request Mar 22, 2023
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.

None yet

3 participants