Skip to content

Conversation

@harp-intel
Copy link
Contributor

This pull request enhances the clarity and usability of the NIC table in the report by adding detailed descriptions to several technical fields. These descriptions help users better understand the purpose and impact of each setting, particularly for interrupt handling and coalescing parameters.

Improvements to NIC table field documentation:

  • Added descriptive Description fields to IRQBalance, Adaptive RX, Adaptive TX, rx-usecs, and tx-usecs in the nicTableValues function within internal/report/table_defs.go, explaining their roles and effects on system performance and network latency.… and coalescing settings

… and coalescing settings

Signed-off-by: Harper, Jason M <jason.m.harper@intel.com>
@harp-intel harp-intel requested a review from Copilot October 18, 2025 14:40
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR enhances the documentation of the NIC table in the report by adding descriptive text to five technical fields related to interrupt handling and coalescing. These descriptions help users understand how each setting affects system performance and network latency.

Key changes:

  • Added description for IRQBalance explaining its role in distributing interrupts across cores
  • Added descriptions for Adaptive RX/TX explaining dynamic coalescing adjustments
  • Added descriptions for rx-usecs and tx-usecs explaining the latency vs. CPU usage tradeoff

@harp-intel harp-intel merged commit 3da8ae9 into main Oct 18, 2025
3 checks passed
@harp-intel harp-intel deleted the nicvaluedesc branch October 18, 2025 14:45
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.

2 participants