xref: /openbmc/qemu/accel/accel-blocker.c (revision f774a677507966222624a9b2859f06ede7608100)
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"
28*51483f6cSPeter Maydell #include "qemu/lockcnt.h"
29bd688fc9SEmanuele Giuseppe Esposito #include "qemu/thread.h"
30bd688fc9SEmanuele Giuseppe Esposito #include "qemu/main-loop.h"
31bd688fc9SEmanuele Giuseppe Esposito #include "hw/core/cpu.h"
32bd688fc9SEmanuele Giuseppe Esposito #include "sysemu/accel-blocker.h"
33bd688fc9SEmanuele Giuseppe Esposito 
34bd688fc9SEmanuele Giuseppe Esposito static QemuLockCnt accel_in_ioctl_lock;
35bd688fc9SEmanuele Giuseppe Esposito static QemuEvent accel_in_ioctl_event;
36bd688fc9SEmanuele Giuseppe Esposito 
accel_blocker_init(void)37bd688fc9SEmanuele Giuseppe Esposito void accel_blocker_init(void)
38bd688fc9SEmanuele Giuseppe Esposito {
39bd688fc9SEmanuele Giuseppe Esposito     qemu_lockcnt_init(&accel_in_ioctl_lock);
40bd688fc9SEmanuele Giuseppe Esposito     qemu_event_init(&accel_in_ioctl_event, false);
41bd688fc9SEmanuele Giuseppe Esposito }
42bd688fc9SEmanuele Giuseppe Esposito 
accel_ioctl_begin(void)43bd688fc9SEmanuele Giuseppe Esposito void accel_ioctl_begin(void)
44bd688fc9SEmanuele Giuseppe Esposito {
45195801d7SStefan Hajnoczi     if (likely(bql_locked())) {
46bd688fc9SEmanuele Giuseppe Esposito         return;
47bd688fc9SEmanuele Giuseppe Esposito     }
48bd688fc9SEmanuele Giuseppe Esposito 
49bd688fc9SEmanuele Giuseppe Esposito     /* block if lock is taken in kvm_ioctl_inhibit_begin() */
50bd688fc9SEmanuele Giuseppe Esposito     qemu_lockcnt_inc(&accel_in_ioctl_lock);
51bd688fc9SEmanuele Giuseppe Esposito }
52bd688fc9SEmanuele Giuseppe Esposito 
accel_ioctl_end(void)53bd688fc9SEmanuele Giuseppe Esposito void accel_ioctl_end(void)
54bd688fc9SEmanuele Giuseppe Esposito {
55195801d7SStefan Hajnoczi     if (likely(bql_locked())) {
56bd688fc9SEmanuele Giuseppe Esposito         return;
57bd688fc9SEmanuele Giuseppe Esposito     }
58bd688fc9SEmanuele Giuseppe Esposito 
59bd688fc9SEmanuele Giuseppe Esposito     qemu_lockcnt_dec(&accel_in_ioctl_lock);
60bd688fc9SEmanuele Giuseppe Esposito     /* change event to SET. If event was BUSY, wake up all waiters */
61bd688fc9SEmanuele Giuseppe Esposito     qemu_event_set(&accel_in_ioctl_event);
62bd688fc9SEmanuele Giuseppe Esposito }
63bd688fc9SEmanuele Giuseppe Esposito 
accel_cpu_ioctl_begin(CPUState * cpu)64bd688fc9SEmanuele Giuseppe Esposito void accel_cpu_ioctl_begin(CPUState *cpu)
65bd688fc9SEmanuele Giuseppe Esposito {
66195801d7SStefan Hajnoczi     if (unlikely(bql_locked())) {
67bd688fc9SEmanuele Giuseppe Esposito         return;
68bd688fc9SEmanuele Giuseppe Esposito     }
69bd688fc9SEmanuele Giuseppe Esposito 
70bd688fc9SEmanuele Giuseppe Esposito     /* block if lock is taken in kvm_ioctl_inhibit_begin() */
71bd688fc9SEmanuele Giuseppe Esposito     qemu_lockcnt_inc(&cpu->in_ioctl_lock);
72bd688fc9SEmanuele Giuseppe Esposito }
73bd688fc9SEmanuele Giuseppe Esposito 
accel_cpu_ioctl_end(CPUState * cpu)74bd688fc9SEmanuele Giuseppe Esposito void accel_cpu_ioctl_end(CPUState *cpu)
75bd688fc9SEmanuele Giuseppe Esposito {
76195801d7SStefan Hajnoczi     if (unlikely(bql_locked())) {
77bd688fc9SEmanuele Giuseppe Esposito         return;
78bd688fc9SEmanuele Giuseppe Esposito     }
79bd688fc9SEmanuele Giuseppe Esposito 
80bd688fc9SEmanuele Giuseppe Esposito     qemu_lockcnt_dec(&cpu->in_ioctl_lock);
81bd688fc9SEmanuele Giuseppe Esposito     /* change event to SET. If event was BUSY, wake up all waiters */
82bd688fc9SEmanuele Giuseppe Esposito     qemu_event_set(&accel_in_ioctl_event);
83bd688fc9SEmanuele Giuseppe Esposito }
84bd688fc9SEmanuele Giuseppe Esposito 
accel_has_to_wait(void)85bd688fc9SEmanuele Giuseppe Esposito static bool accel_has_to_wait(void)
86bd688fc9SEmanuele Giuseppe Esposito {
87bd688fc9SEmanuele Giuseppe Esposito     CPUState *cpu;
88bd688fc9SEmanuele Giuseppe Esposito     bool needs_to_wait = false;
89bd688fc9SEmanuele Giuseppe Esposito 
90bd688fc9SEmanuele Giuseppe Esposito     CPU_FOREACH(cpu) {
91bd688fc9SEmanuele Giuseppe Esposito         if (qemu_lockcnt_count(&cpu->in_ioctl_lock)) {
92bd688fc9SEmanuele Giuseppe Esposito             /* exit the ioctl, if vcpu is running it */
93bd688fc9SEmanuele Giuseppe Esposito             qemu_cpu_kick(cpu);
94bd688fc9SEmanuele Giuseppe Esposito             needs_to_wait = true;
95bd688fc9SEmanuele Giuseppe Esposito         }
96bd688fc9SEmanuele Giuseppe Esposito     }
97bd688fc9SEmanuele Giuseppe Esposito 
98bd688fc9SEmanuele Giuseppe Esposito     return needs_to_wait || qemu_lockcnt_count(&accel_in_ioctl_lock);
99bd688fc9SEmanuele Giuseppe Esposito }
100bd688fc9SEmanuele Giuseppe Esposito 
accel_ioctl_inhibit_begin(void)101bd688fc9SEmanuele Giuseppe Esposito void accel_ioctl_inhibit_begin(void)
102bd688fc9SEmanuele Giuseppe Esposito {
103bd688fc9SEmanuele Giuseppe Esposito     CPUState *cpu;
104bd688fc9SEmanuele Giuseppe Esposito 
105bd688fc9SEmanuele Giuseppe Esposito     /*
106bd688fc9SEmanuele Giuseppe Esposito      * We allow to inhibit only when holding the BQL, so we can identify
107bd688fc9SEmanuele Giuseppe Esposito      * when an inhibitor wants to issue an ioctl easily.
108bd688fc9SEmanuele Giuseppe Esposito      */
109195801d7SStefan Hajnoczi     g_assert(bql_locked());
110bd688fc9SEmanuele Giuseppe Esposito 
111bd688fc9SEmanuele Giuseppe Esposito     /* Block further invocations of the ioctls outside the BQL.  */
112bd688fc9SEmanuele Giuseppe Esposito     CPU_FOREACH(cpu) {
113bd688fc9SEmanuele Giuseppe Esposito         qemu_lockcnt_lock(&cpu->in_ioctl_lock);
114bd688fc9SEmanuele Giuseppe Esposito     }
115bd688fc9SEmanuele Giuseppe Esposito     qemu_lockcnt_lock(&accel_in_ioctl_lock);
116bd688fc9SEmanuele Giuseppe Esposito 
117bd688fc9SEmanuele Giuseppe Esposito     /* Keep waiting until there are running ioctls */
118bd688fc9SEmanuele Giuseppe Esposito     while (true) {
119bd688fc9SEmanuele Giuseppe Esposito 
120bd688fc9SEmanuele Giuseppe Esposito         /* Reset event to FREE. */
121bd688fc9SEmanuele Giuseppe Esposito         qemu_event_reset(&accel_in_ioctl_event);
122bd688fc9SEmanuele Giuseppe Esposito 
123bd688fc9SEmanuele Giuseppe Esposito         if (accel_has_to_wait()) {
124bd688fc9SEmanuele Giuseppe Esposito             /*
125bd688fc9SEmanuele Giuseppe Esposito              * If event is still FREE, and there are ioctls still in progress,
126bd688fc9SEmanuele Giuseppe Esposito              * wait.
127bd688fc9SEmanuele Giuseppe Esposito              *
128bd688fc9SEmanuele Giuseppe Esposito              *  If an ioctl finishes before qemu_event_wait(), it will change
129bd688fc9SEmanuele Giuseppe Esposito              * the event state to SET. This will prevent qemu_event_wait() from
130bd688fc9SEmanuele Giuseppe Esposito              * blocking, but it's not a problem because if other ioctls are
131bd688fc9SEmanuele Giuseppe Esposito              * still running the loop will iterate once more and reset the event
132bd688fc9SEmanuele Giuseppe Esposito              * status to FREE so that it can wait properly.
133bd688fc9SEmanuele Giuseppe Esposito              *
134bd688fc9SEmanuele Giuseppe Esposito              * If an ioctls finishes while qemu_event_wait() is blocking, then
135bd688fc9SEmanuele Giuseppe Esposito              * it will be waken up, but also here the while loop makes sure
136bd688fc9SEmanuele Giuseppe Esposito              * to re-enter the wait if there are other running ioctls.
137bd688fc9SEmanuele Giuseppe Esposito              */
138bd688fc9SEmanuele Giuseppe Esposito             qemu_event_wait(&accel_in_ioctl_event);
139bd688fc9SEmanuele Giuseppe Esposito         } else {
140bd688fc9SEmanuele Giuseppe Esposito             /* No ioctl is running */
141bd688fc9SEmanuele Giuseppe Esposito             return;
142bd688fc9SEmanuele Giuseppe Esposito         }
143bd688fc9SEmanuele Giuseppe Esposito     }
144bd688fc9SEmanuele Giuseppe Esposito }
145bd688fc9SEmanuele Giuseppe Esposito 
accel_ioctl_inhibit_end(void)146bd688fc9SEmanuele Giuseppe Esposito void accel_ioctl_inhibit_end(void)
147bd688fc9SEmanuele Giuseppe Esposito {
148bd688fc9SEmanuele Giuseppe Esposito     CPUState *cpu;
149bd688fc9SEmanuele Giuseppe Esposito 
150bd688fc9SEmanuele Giuseppe Esposito     qemu_lockcnt_unlock(&accel_in_ioctl_lock);
151bd688fc9SEmanuele Giuseppe Esposito     CPU_FOREACH(cpu) {
152bd688fc9SEmanuele Giuseppe Esposito         qemu_lockcnt_unlock(&cpu->in_ioctl_lock);
153bd688fc9SEmanuele Giuseppe Esposito     }
154bd688fc9SEmanuele Giuseppe Esposito }
155bd688fc9SEmanuele Giuseppe Esposito 
156