Deadlock when detaching network interface
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Ubuntu Cloud Archive |
Fix Released
|
High
|
Unassigned | ||
Mitaka |
Fix Released
|
High
|
Unassigned | ||
Ocata |
Fix Released
|
High
|
Unassigned | ||
Pike |
Fix Released
|
High
|
Unassigned | ||
Queens |
Fix Released
|
High
|
Unassigned | ||
Rocky |
Won't Fix
|
High
|
Unassigned | ||
Stein |
Fix Released
|
High
|
Unassigned | ||
qemu (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
Xenial |
Fix Released
|
Undecided
|
Heitor Alves de Siqueira | ||
Bionic |
Fix Released
|
Undecided
|
Unassigned | ||
Cosmic |
Fix Released
|
Undecided
|
Unassigned | ||
Disco |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
[Impact]
Qemu guests hang indefinitely
[Description]
When running a Qemu guest with VirtIO network interfaces, detaching an interface that's currently being used can result in a deadlock. The guest instance will hang and become unresponsive to commands, and the only option is to kill -9 the instance.
The reason for this is a dealock between a monitor and an RCU thread, which will fight over the BQL (qemu_global_mutex) and the critical RCU section locks. The monitor thread will acquire the BQL for detaching the network interface, and fire up a helper thread to deal with detaching the network adapter. That new thread needs to wait on the RCU thread to complete the deletion, but the RCU thread wants the BQL to commit its transactions.
This bug is already fixed upstream (73c6e4013b4c rcu: completely disable pthread_atfork callbacks as soon as possible) and included for other series (see below), so we don't need to backport it to Bionic onwards.
Upstream commit: https:/
$ git describe --contains 73c6e4013b4c
v2.10.0-rc2~1^2~8
$ rmadison qemu
===> qemu | 1:2.5+dfsg-
qemu | 1:2.11+
qemu | 1:2.12+
qemu | 1:3.1+dfsg-2ubuntu2 | disco/universe | amd64, ...
[Test Case]
Being a racing condition, this is a tricky bug to reproduce consistently. We've had reports of users running into this with OpenStack deployments and Windows Server guests, and the scenario is usually like this:
1) Deploy a 16vCPU Windows Server 2012 R2 guest with a virtio network interface
2) Stress the network interface with e.g. Windows HLK test suite or similar
3) Repeatedly attach/detach the network adapter that's in use
It usually takes more than ~4000 attach/detach cycles to trigger the bug.
[Regression Potential]
Regressions for this might arise from the fact that the fix changes RCU lock code. Since this patch has been upstream and in other series for a while, it's unlikely that it would regressions in RCU code specifically. Other code that makes use of the RCU locks (MMIO and some monitor events) will be thoroughly tested for any regressions with use-case scenarios and scripted runs.
Changed in qemu: | |
assignee: | nobody → Heitor R. Alves de Siqueira (halves) |
status: | New → Confirmed |
status: | Confirmed → Fix Released |
Changed in qemu (Ubuntu): | |
assignee: | nobody → Heitor R. Alves de Siqueira (halves) |
status: | New → Confirmed |
Changed in qemu (Ubuntu Cosmic): | |
assignee: | nobody → Heitor R. Alves de Siqueira (halves) |
Changed in qemu (Ubuntu Bionic): | |
assignee: | nobody → Heitor R. Alves de Siqueira (halves) |
Changed in qemu (Ubuntu Xenial): | |
assignee: | nobody → Heitor R. Alves de Siqueira (halves) |
Changed in qemu: | |
assignee: | Heitor R. Alves de Siqueira (halves) → nobody |
Changed in qemu (Ubuntu Disco): | |
status: | Confirmed → Fix Released |
Changed in qemu (Ubuntu Cosmic): | |
status: | New → Fix Released |
Changed in qemu (Ubuntu Bionic): | |
status: | New → Fix Released |
Changed in qemu (Ubuntu Xenial): | |
status: | New → Confirmed |
Changed in cloud-archive: | |
status: | New → Confirmed |
tags: | added: sts-sponsor |
Changed in qemu (Ubuntu Disco): | |
assignee: | Heitor R. Alves de Siqueira (halves) → nobody |
Changed in qemu (Ubuntu Cosmic): | |
assignee: | Heitor R. Alves de Siqueira (halves) → nobody |
Changed in qemu (Ubuntu Bionic): | |
assignee: | Heitor R. Alves de Siqueira (halves) → nobody |
tags: | added: sts-sponsor-ddstreet |
no longer affects: | qemu |
tags: | removed: sts-sponsor sts-sponsor-ddstreet |
Patch v2:
Added missing DEP3 info and corrected pkg version