Skip to content

Integrate InterruptStatsMonitor with KernelMonitor #375

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

Closed

Conversation

abhiShedge
Copy link
Contributor

Summary: Use InterruptStatsMonitor to collect the interrupt stats from procfs. These stats are eventually picked up by the monitoring thread.

Differential Revision: D74770810

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 14, 2025
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D74770810

abhiShedge added a commit to abhiShedge/dynolog that referenced this pull request May 19, 2025
…#375)

Summary:

Use InterruptStatsMonitor to collect the interrupt stats from procfs. These stats are eventually picked up by the monitoring thread.

Differential Revision: D74770810
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D74770810

abhiShedge added a commit to abhiShedge/dynolog that referenced this pull request May 19, 2025
…#375)

Summary:
Pull Request resolved: facebookincubator#375

Use InterruptStatsMonitor to collect the interrupt stats from procfs. These stats are eventually picked up by the monitoring thread.

Differential Revision: D74770810
abhiShedge added a commit to abhiShedge/dynolog that referenced this pull request May 20, 2025
…#375)

Summary:

Use InterruptStatsMonitor to collect the interrupt stats from procfs. These stats are eventually picked up by the monitoring thread.

Reviewed By: homie-homin

Differential Revision: D74770810
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D74770810

abhiShedge added a commit to abhiShedge/dynolog that referenced this pull request May 20, 2025
…#375)

Summary:
Pull Request resolved: facebookincubator#375

Use InterruptStatsMonitor to collect the interrupt stats from procfs. These stats are eventually picked up by the monitoring thread.

Reviewed By: homie-homin

Differential Revision: D74770810
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D74770810

abhiShedge added a commit to abhiShedge/dynolog that referenced this pull request May 21, 2025
…#375)

Summary:
Pull Request resolved: facebookincubator#375

Use InterruptStatsMonitor to collect the interrupt stats from procfs. These stats are eventually picked up by the monitoring thread.

Reviewed By: homie-homin

Differential Revision: D74770810
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D74770810

abhiShedge added a commit to abhiShedge/dynolog that referenced this pull request May 21, 2025
…#375)

Summary:
Pull Request resolved: facebookincubator#375

Use InterruptStatsMonitor to collect the interrupt stats from procfs. These stats are eventually picked up by the monitoring thread.

Reviewed By: homie-homin

Differential Revision: D74770810
abhiShedge added a commit to abhiShedge/dynolog that referenced this pull request Jun 3, 2025
…#375)

Summary:

Use InterruptStatsMonitor to collect the interrupt stats from procfs. These stats are eventually picked up by the monitoring thread.

Reviewed By: homie-homin

Differential Revision: D74770810
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D74770810

abhiShedge added a commit to abhiShedge/dynolog that referenced this pull request Jun 3, 2025
…#375)

Summary:
Pull Request resolved: facebookincubator#375

Use InterruptStatsMonitor to collect the interrupt stats from procfs. These stats are eventually picked up by the monitoring thread.

Reviewed By: homie-homin

Differential Revision: D74770810
abhiShedge added a commit to abhiShedge/dynolog that referenced this pull request Jun 3, 2025
…#375)

Summary:

Use InterruptStatsMonitor to collect the interrupt stats from procfs. These stats are eventually picked up by the monitoring thread.

Reviewed By: homie-homin

Differential Revision: D74770810
abhiShedge added a commit to abhiShedge/dynolog that referenced this pull request Jun 3, 2025
…#375)

Summary:

Use InterruptStatsMonitor to collect the interrupt stats from procfs. These stats are eventually picked up by the monitoring thread.

Reviewed By: homie-homin

Differential Revision: D74770810
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D74770810

1 similar comment
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D74770810

abhiShedge added a commit to abhiShedge/dynolog that referenced this pull request Jun 3, 2025
…#375)

Summary:
Pull Request resolved: facebookincubator#375

Use InterruptStatsMonitor to collect the interrupt stats from procfs. These stats are eventually picked up by the monitoring thread.

Reviewed By: homie-homin

Differential Revision: D74770810
Abhijeet Shankar Shedge and others added 2 commits June 11, 2025 16:44
Summary:
Pull Request resolved: facebookincubator#374

Add unit tests for the newly created interrupt monitor class.

Differential Revision: D74662865

Reviewed By: homie-homin
…#375)

Summary:
Pull Request resolved: facebookincubator#375

Use InterruptStatsMonitor to collect the interrupt stats from procfs. These stats are eventually picked up by the monitoring thread.

Reviewed By: homie-homin

Differential Revision: D74770810
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D74770810

@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 20dd672.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants