Sign Up
Log In
Log In
or
Sign Up
Places
All Projects
Status Monitor
Collapse sidebar
openSUSE:Evergreen:11.4
xen.548
23765-x86-irq-vector-leak.patch
Overview
Repositories
Revisions
Requests
Users
Attributes
Meta
File 23765-x86-irq-vector-leak.patch of Package xen.548
# HG changeset patch # User Andrew Cooper <andrew.cooper3@citrix.com> # Date 1313226868 -3600 # Node ID 68b903bb1b01b2a6ef9c6e8ead3be3c1c2208341 # Parent 67b883402736ef1746cd6654da4c898f70f40723 x86: IRQ fix incorrect logic in __clear_irq_vector In the old code, tmp_mask is the cpu_and of cfg->cpu_mask and cpu_online_map. However, in the usual case of moving an IRQ from one PCPU to another because the scheduler decides its a good idea, cfg->cpu_mask and cfg->old_cpu_mask do not intersect. This causes the old cpu vector_irq table to keep the irq reference when it shouldn't. This leads to a resource leak if a domain is shut down wile an irq has a move pending, which results in Xen's create_irq() eventually failing with -ENOSPC when all vector_irq tables are full of stale references. Signed-off-by: Andrew Cooper <andrew.cooper3@citrix.com> Index: xen-4.0.3-testing/xen/arch/x86/irq.c =================================================================== --- xen-4.0.3-testing.orig/xen/arch/x86/irq.c +++ xen-4.0.3-testing/xen/arch/x86/irq.c @@ -193,6 +193,7 @@ static void __clear_irq_vector(int irq) if (likely(!cfg->move_in_progress)) return; + cpus_and(tmp_mask, cfg->old_domain, cpu_online_map); for_each_cpu_mask(cpu, tmp_mask) { for (vector = FIRST_DYNAMIC_VECTOR; vector <= LAST_DYNAMIC_VECTOR; vector++) {
Locations
Projects
Search
Status Monitor
Help
OpenBuildService.org
Documentation
API Documentation
Code of Conduct
Contact
Support
@OBShq
Terms
openSUSE Build Service is sponsored by
The Open Build Service is an
openSUSE project
.
Sign Up
Log In
Places
Places
All Projects
Status Monitor