]> git.ipfire.org Git - thirdparty/kernel/stable.git/commitdiff
irqbypass: Drop superfluous might_sleep() annotations
authorSean Christopherson <seanjc@google.com>
Fri, 16 May 2025 23:07:28 +0000 (16:07 -0700)
committerSean Christopherson <seanjc@google.com>
Fri, 20 Jun 2025 20:52:37 +0000 (13:52 -0700)
Drop superfluous might_sleep() annotations from irqbypass, mutex_lock()
provides all of the necessary tracking.

Reviewed-by: Kevin Tian <kevin.tian@intel.com>
Acked-by: Michael S. Tsirkin <mst@redhat.com>
Reviewed-by: Alex Williamson <alex.williamson@redhat.com>
Link: https://lore.kernel.org/r/20250516230734.2564775-3-seanjc@google.com
Signed-off-by: Sean Christopherson <seanjc@google.com>
virt/lib/irqbypass.c

index 080c706f3b012c717765032f279ca63a6eb535e9..28a4d933569aca7d9fabbc61041a9b08f2b623cf 100644 (file)
@@ -90,8 +90,6 @@ int irq_bypass_register_producer(struct irq_bypass_producer *producer)
        if (!producer->token)
                return -EINVAL;
 
-       might_sleep();
-
        mutex_lock(&lock);
 
        list_for_each_entry(tmp, &producers, node) {
@@ -136,8 +134,6 @@ void irq_bypass_unregister_producer(struct irq_bypass_producer *producer)
        if (!producer->token)
                return;
 
-       might_sleep();
-
        mutex_lock(&lock);
 
        list_for_each_entry(tmp, &producers, node) {
@@ -176,8 +172,6 @@ int irq_bypass_register_consumer(struct irq_bypass_consumer *consumer)
            !consumer->add_producer || !consumer->del_producer)
                return -EINVAL;
 
-       might_sleep();
-
        mutex_lock(&lock);
 
        list_for_each_entry(tmp, &consumers, node) {
@@ -222,8 +216,6 @@ void irq_bypass_unregister_consumer(struct irq_bypass_consumer *consumer)
        if (!consumer->token)
                return;
 
-       might_sleep();
-
        mutex_lock(&lock);
 
        list_for_each_entry(tmp, &consumers, node) {