xref: /qemu/accel/accel-blocker.c (revision 195801d7)
1bd688fc9SEmanuele Giuseppe Esposito /*
2bd688fc9SEmanuele Giuseppe Esposito  * Lock to inhibit accelerator ioctls
3bd688fc9SEmanuele Giuseppe Esposito  *
4bd688fc9SEmanuele Giuseppe Esposito  * Copyright (c) 2022 Red Hat Inc.
5bd688fc9SEmanuele Giuseppe Esposito  *
6bd688fc9SEmanuele Giuseppe Esposito  * Author: Emanuele Giuseppe Esposito       <eesposit@redhat.com>
7bd688fc9SEmanuele Giuseppe Esposito  *
8bd688fc9SEmanuele Giuseppe Esposito  * Permission is hereby granted, free of charge, to any person obtaining a copy
9bd688fc9SEmanuele Giuseppe Esposito  * of this software and associated documentation files (the "Software"), to deal
10bd688fc9SEmanuele Giuseppe Esposito  * in the Software without restriction, including without limitation the rights
11bd688fc9SEmanuele Giuseppe Esposito  * to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
12bd688fc9SEmanuele Giuseppe Esposito  * copies of the Software, and to permit persons to whom the Software is
13bd688fc9SEmanuele Giuseppe Esposito  * furnished to do so, subject to the following conditions:
14bd688fc9SEmanuele Giuseppe Esposito  *
15bd688fc9SEmanuele Giuseppe Esposito  * The above copyright notice and this permission notice shall be included in
16bd688fc9SEmanuele Giuseppe Esposito  * all copies or substantial portions of the Software.
17bd688fc9SEmanuele Giuseppe Esposito  *
18bd688fc9SEmanuele Giuseppe Esposito  * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
19bd688fc9SEmanuele Giuseppe Esposito  * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
20bd688fc9SEmanuele Giuseppe Esposito  * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL
21bd688fc9SEmanuele Giuseppe Esposito  * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
22bd688fc9SEmanuele Giuseppe Esposito  * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
23bd688fc9SEmanuele Giuseppe Esposito  * OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
24bd688fc9SEmanuele Giuseppe Esposito  * THE SOFTWARE.
25bd688fc9SEmanuele Giuseppe Esposito  */
26bd688fc9SEmanuele Giuseppe Esposito 
27bd688fc9SEmanuele Giuseppe Esposito #include "qemu/osdep.h"
28bd688fc9SEmanuele Giuseppe Esposito #include "qemu/thread.h"
29bd688fc9SEmanuele Giuseppe Esposito #include "qemu/main-loop.h"
30bd688fc9SEmanuele Giuseppe Esposito #include "hw/core/cpu.h"
31bd688fc9SEmanuele Giuseppe Esposito #include "sysemu/accel-blocker.h"
32bd688fc9SEmanuele Giuseppe Esposito 
33bd688fc9SEmanuele Giuseppe Esposito static QemuLockCnt accel_in_ioctl_lock;
34bd688fc9SEmanuele Giuseppe Esposito static QemuEvent accel_in_ioctl_event;
35bd688fc9SEmanuele Giuseppe Esposito 
accel_blocker_init(void)36bd688fc9SEmanuele Giuseppe Esposito void accel_blocker_init(void)
37bd688fc9SEmanuele Giuseppe Esposito {
38bd688fc9SEmanuele Giuseppe Esposito     qemu_lockcnt_init(&accel_in_ioctl_lock);
39bd688fc9SEmanuele Giuseppe Esposito     qemu_event_init(&accel_in_ioctl_event, false);
40bd688fc9SEmanuele Giuseppe Esposito }
41bd688fc9SEmanuele Giuseppe Esposito 
accel_ioctl_begin(void)42bd688fc9SEmanuele Giuseppe Esposito void accel_ioctl_begin(void)
43bd688fc9SEmanuele Giuseppe Esposito {
44*195801d7SStefan Hajnoczi     if (likely(bql_locked())) {
45bd688fc9SEmanuele Giuseppe Esposito         return;
46bd688fc9SEmanuele Giuseppe Esposito     }
47bd688fc9SEmanuele Giuseppe Esposito 
48bd688fc9SEmanuele Giuseppe Esposito     /* block if lock is taken in kvm_ioctl_inhibit_begin() */
49bd688fc9SEmanuele Giuseppe Esposito     qemu_lockcnt_inc(&accel_in_ioctl_lock);
50bd688fc9SEmanuele Giuseppe Esposito }
51bd688fc9SEmanuele Giuseppe Esposito 
accel_ioctl_end(void)52bd688fc9SEmanuele Giuseppe Esposito void accel_ioctl_end(void)
53bd688fc9SEmanuele Giuseppe Esposito {
54*195801d7SStefan Hajnoczi     if (likely(bql_locked())) {
55bd688fc9SEmanuele Giuseppe Esposito         return;
56bd688fc9SEmanuele Giuseppe Esposito     }
57bd688fc9SEmanuele Giuseppe Esposito 
58bd688fc9SEmanuele Giuseppe Esposito     qemu_lockcnt_dec(&accel_in_ioctl_lock);
59bd688fc9SEmanuele Giuseppe Esposito     /* change event to SET. If event was BUSY, wake up all waiters */
60bd688fc9SEmanuele Giuseppe Esposito     qemu_event_set(&accel_in_ioctl_event);
61bd688fc9SEmanuele Giuseppe Esposito }
62bd688fc9SEmanuele Giuseppe Esposito 
accel_cpu_ioctl_begin(CPUState * cpu)63bd688fc9SEmanuele Giuseppe Esposito void accel_cpu_ioctl_begin(CPUState *cpu)
64bd688fc9SEmanuele Giuseppe Esposito {
65*195801d7SStefan Hajnoczi     if (unlikely(bql_locked())) {
66bd688fc9SEmanuele Giuseppe Esposito         return;
67bd688fc9SEmanuele Giuseppe Esposito     }
68bd688fc9SEmanuele Giuseppe Esposito 
69bd688fc9SEmanuele Giuseppe Esposito     /* block if lock is taken in kvm_ioctl_inhibit_begin() */
70bd688fc9SEmanuele Giuseppe Esposito     qemu_lockcnt_inc(&cpu->in_ioctl_lock);
71bd688fc9SEmanuele Giuseppe Esposito }
72bd688fc9SEmanuele Giuseppe Esposito 
accel_cpu_ioctl_end(CPUState * cpu)73bd688fc9SEmanuele Giuseppe Esposito void accel_cpu_ioctl_end(CPUState *cpu)
74bd688fc9SEmanuele Giuseppe Esposito {
75*195801d7SStefan Hajnoczi     if (unlikely(bql_locked())) {
76bd688fc9SEmanuele Giuseppe Esposito         return;
77bd688fc9SEmanuele Giuseppe Esposito     }
78bd688fc9SEmanuele Giuseppe Esposito 
79bd688fc9SEmanuele Giuseppe Esposito     qemu_lockcnt_dec(&cpu->in_ioctl_lock);
80bd688fc9SEmanuele Giuseppe Esposito     /* change event to SET. If event was BUSY, wake up all waiters */
81bd688fc9SEmanuele Giuseppe Esposito     qemu_event_set(&accel_in_ioctl_event);
82bd688fc9SEmanuele Giuseppe Esposito }
83bd688fc9SEmanuele Giuseppe Esposito 
accel_has_to_wait(void)84bd688fc9SEmanuele Giuseppe Esposito static bool accel_has_to_wait(void)
85bd688fc9SEmanuele Giuseppe Esposito {
86bd688fc9SEmanuele Giuseppe Esposito     CPUState *cpu;
87bd688fc9SEmanuele Giuseppe Esposito     bool needs_to_wait = false;
88bd688fc9SEmanuele Giuseppe Esposito 
89bd688fc9SEmanuele Giuseppe Esposito     CPU_FOREACH(cpu) {
90bd688fc9SEmanuele Giuseppe Esposito         if (qemu_lockcnt_count(&cpu->in_ioctl_lock)) {
91bd688fc9SEmanuele Giuseppe Esposito             /* exit the ioctl, if vcpu is running it */
92bd688fc9SEmanuele Giuseppe Esposito             qemu_cpu_kick(cpu);
93bd688fc9SEmanuele Giuseppe Esposito             needs_to_wait = true;
94bd688fc9SEmanuele Giuseppe Esposito         }
95bd688fc9SEmanuele Giuseppe Esposito     }
96bd688fc9SEmanuele Giuseppe Esposito 
97bd688fc9SEmanuele Giuseppe Esposito     return needs_to_wait || qemu_lockcnt_count(&accel_in_ioctl_lock);
98bd688fc9SEmanuele Giuseppe Esposito }
99bd688fc9SEmanuele Giuseppe Esposito 
accel_ioctl_inhibit_begin(void)100bd688fc9SEmanuele Giuseppe Esposito void accel_ioctl_inhibit_begin(void)
101bd688fc9SEmanuele Giuseppe Esposito {
102bd688fc9SEmanuele Giuseppe Esposito     CPUState *cpu;
103bd688fc9SEmanuele Giuseppe Esposito 
104bd688fc9SEmanuele Giuseppe Esposito     /*
105bd688fc9SEmanuele Giuseppe Esposito      * We allow to inhibit only when holding the BQL, so we can identify
106bd688fc9SEmanuele Giuseppe Esposito      * when an inhibitor wants to issue an ioctl easily.
107bd688fc9SEmanuele Giuseppe Esposito      */
108*195801d7SStefan Hajnoczi     g_assert(bql_locked());
109bd688fc9SEmanuele Giuseppe Esposito 
110bd688fc9SEmanuele Giuseppe Esposito     /* Block further invocations of the ioctls outside the BQL.  */
111bd688fc9SEmanuele Giuseppe Esposito     CPU_FOREACH(cpu) {
112bd688fc9SEmanuele Giuseppe Esposito         qemu_lockcnt_lock(&cpu->in_ioctl_lock);
113bd688fc9SEmanuele Giuseppe Esposito     }
114bd688fc9SEmanuele Giuseppe Esposito     qemu_lockcnt_lock(&accel_in_ioctl_lock);
115bd688fc9SEmanuele Giuseppe Esposito 
116bd688fc9SEmanuele Giuseppe Esposito     /* Keep waiting until there are running ioctls */
117bd688fc9SEmanuele Giuseppe Esposito     while (true) {
118bd688fc9SEmanuele Giuseppe Esposito 
119bd688fc9SEmanuele Giuseppe Esposito         /* Reset event to FREE. */
120bd688fc9SEmanuele Giuseppe Esposito         qemu_event_reset(&accel_in_ioctl_event);
121bd688fc9SEmanuele Giuseppe Esposito 
122bd688fc9SEmanuele Giuseppe Esposito         if (accel_has_to_wait()) {
123bd688fc9SEmanuele Giuseppe Esposito             /*
124bd688fc9SEmanuele Giuseppe Esposito              * If event is still FREE, and there are ioctls still in progress,
125bd688fc9SEmanuele Giuseppe Esposito              * wait.
126bd688fc9SEmanuele Giuseppe Esposito              *
127bd688fc9SEmanuele Giuseppe Esposito              *  If an ioctl finishes before qemu_event_wait(), it will change
128bd688fc9SEmanuele Giuseppe Esposito              * the event state to SET. This will prevent qemu_event_wait() from
129bd688fc9SEmanuele Giuseppe Esposito              * blocking, but it's not a problem because if other ioctls are
130bd688fc9SEmanuele Giuseppe Esposito              * still running the loop will iterate once more and reset the event
131bd688fc9SEmanuele Giuseppe Esposito              * status to FREE so that it can wait properly.
132bd688fc9SEmanuele Giuseppe Esposito              *
133bd688fc9SEmanuele Giuseppe Esposito              * If an ioctls finishes while qemu_event_wait() is blocking, then
134bd688fc9SEmanuele Giuseppe Esposito              * it will be waken up, but also here the while loop makes sure
135bd688fc9SEmanuele Giuseppe Esposito              * to re-enter the wait if there are other running ioctls.
136bd688fc9SEmanuele Giuseppe Esposito              */
137bd688fc9SEmanuele Giuseppe Esposito             qemu_event_wait(&accel_in_ioctl_event);
138bd688fc9SEmanuele Giuseppe Esposito         } else {
139bd688fc9SEmanuele Giuseppe Esposito             /* No ioctl is running */
140bd688fc9SEmanuele Giuseppe Esposito             return;
141bd688fc9SEmanuele Giuseppe Esposito         }
142bd688fc9SEmanuele Giuseppe Esposito     }
143bd688fc9SEmanuele Giuseppe Esposito }
144bd688fc9SEmanuele Giuseppe Esposito 
accel_ioctl_inhibit_end(void)145bd688fc9SEmanuele Giuseppe Esposito void accel_ioctl_inhibit_end(void)
146bd688fc9SEmanuele Giuseppe Esposito {
147bd688fc9SEmanuele Giuseppe Esposito     CPUState *cpu;
148bd688fc9SEmanuele Giuseppe Esposito 
149bd688fc9SEmanuele Giuseppe Esposito     qemu_lockcnt_unlock(&accel_in_ioctl_lock);
150bd688fc9SEmanuele Giuseppe Esposito     CPU_FOREACH(cpu) {
151bd688fc9SEmanuele Giuseppe Esposito         qemu_lockcnt_unlock(&cpu->in_ioctl_lock);
152bd688fc9SEmanuele Giuseppe Esposito     }
153bd688fc9SEmanuele Giuseppe Esposito }
154bd688fc9SEmanuele Giuseppe Esposito 
155