diff mbox series

osstest: install irqbalance

Message ID 20220412112859.73330-1-roger.pau@citrix.com (mailing list archive)
State New, archived
Headers show
Series osstest: install irqbalance | expand

Commit Message

Roger Pau Monné April 12, 2022, 11:28 a.m. UTC 
Or else all interrupts will get bound to (v)CPU 0.

This doesn't cause issues on small boxes, but boxes with a non-trivial
amount of CPUs can struggle without interrupts being balanced across
available vCPUs, as the number of vCPUs offered to dom0 matches the
number of physical CPUs.

For example sabro boxes (Xeon Silver 4114 x 2 sockets) would sometimes
report timeouts which seem to be solved by using irqbalance in dom0.

irqbalance is also available on Arm, so install unconditionally.

Reported-by: Jan Beulich <jbeulich@suse.com>
Signed-off-by: Roger Pau Monné <roger.pau@citrix.com>
---
 ts-host-install | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Ian Jackson April 12, 2022, 2:19 p.m. UTC | #1 
Roger Pau Monne writes ("[PATCH] osstest: install irqbalance"):
> Or else all interrupts will get bound to (v)CPU 0.
> 
> This doesn't cause issues on small boxes, but boxes with a non-trivial
> amount of CPUs can struggle without interrupts being balanced across
> available vCPUs, as the number of vCPUs offered to dom0 matches the
> number of physical CPUs.

Wow.  This was missing all this time!  I wonder if it was in the
default install in some previous Debian release...

Acked-by: Ian Jackson <ijackson@chiark.greenend.org.uk>
diff mbox series

Patch

diff --git a/ts-host-install b/ts-host-install
index 276c6af8..f79a1beb 100755
--- a/ts-host-install
+++ b/ts-host-install
@@ -139,7 +139,7 @@  END
 
     target_cmd_root($ho, "chmod 2775 /root");
 
-    target_install_packages($ho, qw(ed));
+    target_install_packages($ho, qw(ed irqbalance));
     if ($ho->{Arch} =~ m/^(?:i386|amd64)$/) {
 	# We don't necessarily know the CPU vendor, but the Debian
 	# packaged microcode doesn't mind us installing both.