blob: 8ff3b60ecd23ecb25f3390558c2f0f5ae2e1a393 [file] [log] [blame]
Andrew Scull18834872018-10-12 11:48:09 +01001/*
2 * Copyright 2018 Google LLC
3 *
4 * Licensed under the Apache License, Version 2.0 (the "License");
5 * you may not use this file except in compliance with the License.
6 * You may obtain a copy of the License at
7 *
8 * https://www.apache.org/licenses/LICENSE-2.0
9 *
10 * Unless required by applicable law or agreed to in writing, software
11 * distributed under the License is distributed on an "AS IS" BASIS,
12 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13 * See the License for the specific language governing permissions and
14 * limitations under the License.
15 */
16
Andrew Scull18c78fc2018-08-20 12:57:41 +010017#include "hf/api.h"
Wedson Almeida Filho3fcbcff2018-07-10 23:53:39 +010018
Andrew Scull13652af2018-09-17 14:49:08 +010019#include <assert.h>
20
Andrew Walbran318f5732018-11-20 16:23:42 +000021#include "hf/arch/cpu.h"
22
23#include "hf/dlog.h"
Andrew Scull18c78fc2018-08-20 12:57:41 +010024#include "hf/std.h"
25#include "hf/vm.h"
26
Andrew Scullf35a5c92018-08-07 18:09:46 +010027#include "vmapi/hf/call.h"
Wedson Almeida Filho3fcbcff2018-07-10 23:53:39 +010028
Wedson Almeida Filhoba641ef2018-12-03 04:19:44 +000029/*
30 * To eliminate the risk of deadlocks, we define a partial order for the
31 * acquisition of locks held concurrently by the same physical CPU. Our current
32 * ordering requirements are as follows:
33 *
34 * vm::lock -> vcpu::lock
35 */
36
Andrew Scullaa039b32018-10-04 15:02:26 +010037static_assert(HF_MAILBOX_SIZE == PAGE_SIZE,
Andrew Scull13652af2018-09-17 14:49:08 +010038 "Currently, a page is mapped for the send and receive buffers so "
39 "the maximum request is the size of a page.");
40
Wedson Almeida Filho9ed8da52018-12-17 16:09:11 +000041static struct mpool api_page_pool;
Wedson Almeida Filho22d5eaa2018-12-16 00:38:49 +000042
43/**
Wedson Almeida Filho81568c42019-01-04 13:33:02 +000044 * Initialises the API page pool by taking ownership of the contents of the
45 * given page pool.
Wedson Almeida Filho22d5eaa2018-12-16 00:38:49 +000046 */
47void api_init(struct mpool *ppool)
48{
Wedson Almeida Filho9ed8da52018-12-17 16:09:11 +000049 mpool_init_from(&api_page_pool, ppool);
Wedson Almeida Filho22d5eaa2018-12-16 00:38:49 +000050}
51
Wedson Almeida Filho3fcbcff2018-07-10 23:53:39 +010052/**
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +010053 * Switches the physical CPU back to the corresponding vcpu of the primary VM.
Andrew Scullaa039b32018-10-04 15:02:26 +010054 *
55 * This triggers the scheduling logic to run. Run in the context of secondary VM
56 * to cause HF_VCPU_RUN to return and the primary VM to regain control of the
57 * cpu.
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +010058 */
Wedson Almeida Filho00df6c72018-10-18 11:19:24 +010059static struct vcpu *api_switch_to_primary(struct vcpu *current,
Wedson Almeida Filhoba641ef2018-12-03 04:19:44 +000060 struct hf_vcpu_run_return primary_ret,
61 enum vcpu_state secondary_state)
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +010062{
Andrew Scull19503262018-09-20 14:48:39 +010063 struct vm *primary = vm_get(HF_PRIMARY_VM_ID);
Wedson Almeida Filho00df6c72018-10-18 11:19:24 +010064 struct vcpu *next = &primary->vcpus[cpu_index(current->cpu)];
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +010065
Wedson Almeida Filho00df6c72018-10-18 11:19:24 +010066 /* Set the return value for the primary VM's call to HF_VCPU_RUN. */
Andrew Scull6d2db332018-10-10 15:28:17 +010067 arch_regs_set_retval(&next->regs,
68 hf_vcpu_run_return_encode(primary_ret));
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +010069
Wedson Almeida Filhoba641ef2018-12-03 04:19:44 +000070 /* Mark the current vcpu as waiting. */
71 sl_lock(&current->lock);
72 current->state = secondary_state;
73 sl_unlock(&current->lock);
74
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +010075 return next;
76}
77
78/**
Andrew Scull33fecd32019-01-08 14:48:27 +000079 * Returns to the primary vm and signals that the vcpu still has work to do so.
80 */
81struct vcpu *api_preempt(struct vcpu *current)
82{
83 struct hf_vcpu_run_return ret = {
84 .code = HF_VCPU_RUN_PREEMPTED,
85 };
86
87 return api_switch_to_primary(current, ret, vcpu_state_ready);
88}
89
90/**
91 * Returns to the primary vm to allow this cpu to be used for other tasks as the
92 * vcpu does not have work to do at this moment. The current vcpu is marked as
93 * ready to be scheduled again.
Andrew Scullaa039b32018-10-04 15:02:26 +010094 */
Wedson Almeida Filho00df6c72018-10-18 11:19:24 +010095struct vcpu *api_yield(struct vcpu *current)
Andrew Scullaa039b32018-10-04 15:02:26 +010096{
Andrew Scull6d2db332018-10-10 15:28:17 +010097 struct hf_vcpu_run_return ret = {
98 .code = HF_VCPU_RUN_YIELD,
99 };
Wedson Almeida Filho81568c42019-01-04 13:33:02 +0000100
Wedson Almeida Filhoba641ef2018-12-03 04:19:44 +0000101 return api_switch_to_primary(current, ret, vcpu_state_ready);
Andrew Scullaa039b32018-10-04 15:02:26 +0100102}
103
104/**
105 * Puts the current vcpu in wait for interrupt mode, and returns to the primary
106 * vm.
107 */
Wedson Almeida Filho00df6c72018-10-18 11:19:24 +0100108struct vcpu *api_wait_for_interrupt(struct vcpu *current)
Andrew Scullaa039b32018-10-04 15:02:26 +0100109{
Andrew Scull6d2db332018-10-10 15:28:17 +0100110 struct hf_vcpu_run_return ret = {
111 .code = HF_VCPU_RUN_WAIT_FOR_INTERRUPT,
112 };
Wedson Almeida Filho81568c42019-01-04 13:33:02 +0000113
Wedson Almeida Filhoba641ef2018-12-03 04:19:44 +0000114 return api_switch_to_primary(current, ret,
115 vcpu_state_blocked_interrupt);
Andrew Scullaa039b32018-10-04 15:02:26 +0100116}
117
118/**
Andrew Scull55c4d8b2018-12-18 18:50:18 +0000119 * Returns the ID of the VM.
120 */
121int64_t api_vm_get_id(const struct vcpu *current)
122{
123 return current->vm->id;
124}
125
126/**
Wedson Almeida Filho3fcbcff2018-07-10 23:53:39 +0100127 * Returns the number of VMs configured to run.
128 */
Andrew Scullc0e569a2018-10-02 18:05:21 +0100129int64_t api_vm_get_count(void)
Wedson Almeida Filho3fcbcff2018-07-10 23:53:39 +0100130{
Andrew Scull19503262018-09-20 14:48:39 +0100131 return vm_get_count();
Wedson Almeida Filho3fcbcff2018-07-10 23:53:39 +0100132}
133
134/**
135 * Returns the number of vcpus configured in the given VM.
136 */
Wedson Almeida Filho00df6c72018-10-18 11:19:24 +0100137int64_t api_vcpu_get_count(uint32_t vm_id, const struct vcpu *current)
Wedson Almeida Filho3fcbcff2018-07-10 23:53:39 +0100138{
Andrew Scull19503262018-09-20 14:48:39 +0100139 struct vm *vm;
140
141 /* Only the primary VM needs to know about vcpus for scheduling. */
Wedson Almeida Filho00df6c72018-10-18 11:19:24 +0100142 if (current->vm->id != HF_PRIMARY_VM_ID) {
Wedson Almeida Filho3fcbcff2018-07-10 23:53:39 +0100143 return -1;
Andrew Scull7364a8e2018-07-19 15:39:29 +0100144 }
Wedson Almeida Filho3fcbcff2018-07-10 23:53:39 +0100145
Andrew Scull19503262018-09-20 14:48:39 +0100146 vm = vm_get(vm_id);
147 if (vm == NULL) {
148 return -1;
149 }
150
151 return vm->vcpu_count;
Wedson Almeida Filho3fcbcff2018-07-10 23:53:39 +0100152}
153
154/**
Wedson Almeida Filho03306112018-11-26 00:08:03 +0000155 * This function is called by the architecture-specific context switching
156 * function to indicate that register state for the given vcpu has been saved
157 * and can therefore be used by other pcpus.
158 */
159void api_regs_state_saved(struct vcpu *vcpu)
160{
161 sl_lock(&vcpu->lock);
162 vcpu->regs_available = true;
163 sl_unlock(&vcpu->lock);
164}
165
166/**
167 * Prepares the vcpu to run by updating its state and fetching whether a return
168 * value needs to be forced onto the vCPU.
169 */
170static bool api_vcpu_prepare_run(const struct vcpu *current, struct vcpu *vcpu,
171 struct retval_state *vcpu_retval)
172{
173 bool ret;
174
175 sl_lock(&vcpu->lock);
176 if (vcpu->state != vcpu_state_ready) {
177 ret = false;
178 goto out;
179 }
180
181 vcpu->cpu = current->cpu;
182 vcpu->state = vcpu_state_running;
183
184 /* Fetch return value to inject into vCPU if there is one. */
185 *vcpu_retval = vcpu->retval;
186 if (vcpu_retval->force) {
187 vcpu->retval.force = false;
188 }
189
190 /*
191 * Wait until the registers become available. Care must be taken when
Wedson Almeida Filho81568c42019-01-04 13:33:02 +0000192 * looping on this: it shouldn't be done while holding other locks to
193 * avoid deadlocks.
Wedson Almeida Filho03306112018-11-26 00:08:03 +0000194 */
195 while (!vcpu->regs_available) {
196 sl_unlock(&vcpu->lock);
197 sl_lock(&vcpu->lock);
198 }
199
200 /*
201 * Mark the registers as unavailable now that we're about to reflect
202 * them onto the real registers. This will also prevent another physical
203 * CPU from trying to read these registers.
204 */
205 vcpu->regs_available = false;
206
207 ret = true;
208
209out:
210 sl_unlock(&vcpu->lock);
211 return ret;
212}
213
214/**
Wedson Almeida Filho3fcbcff2018-07-10 23:53:39 +0100215 * Runs the given vcpu of the given vm.
216 */
Andrew Scull6d2db332018-10-10 15:28:17 +0100217struct hf_vcpu_run_return api_vcpu_run(uint32_t vm_id, uint32_t vcpu_idx,
Wedson Almeida Filho00df6c72018-10-18 11:19:24 +0100218 const struct vcpu *current,
Andrew Scull6d2db332018-10-10 15:28:17 +0100219 struct vcpu **next)
Wedson Almeida Filho3fcbcff2018-07-10 23:53:39 +0100220{
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100221 struct vm *vm;
Wedson Almeida Filho3fcbcff2018-07-10 23:53:39 +0100222 struct vcpu *vcpu;
Wedson Almeida Filho03306112018-11-26 00:08:03 +0000223 struct retval_state vcpu_retval;
Andrew Scull6d2db332018-10-10 15:28:17 +0100224 struct hf_vcpu_run_return ret = {
225 .code = HF_VCPU_RUN_WAIT_FOR_INTERRUPT,
226 };
Wedson Almeida Filho3fcbcff2018-07-10 23:53:39 +0100227
228 /* Only the primary VM can switch vcpus. */
Wedson Almeida Filho00df6c72018-10-18 11:19:24 +0100229 if (current->vm->id != HF_PRIMARY_VM_ID) {
Andrew Scull6d2db332018-10-10 15:28:17 +0100230 goto out;
Andrew Scull7364a8e2018-07-19 15:39:29 +0100231 }
Wedson Almeida Filho3fcbcff2018-07-10 23:53:39 +0100232
Andrew Scull19503262018-09-20 14:48:39 +0100233 /* Only secondary VM vcpus can be run. */
234 if (vm_id == HF_PRIMARY_VM_ID) {
Andrew Scull6d2db332018-10-10 15:28:17 +0100235 goto out;
Andrew Scull7364a8e2018-07-19 15:39:29 +0100236 }
Wedson Almeida Filho3fcbcff2018-07-10 23:53:39 +0100237
Andrew Scull19503262018-09-20 14:48:39 +0100238 /* The requested VM must exist. */
239 vm = vm_get(vm_id);
240 if (vm == NULL) {
Andrew Scull6d2db332018-10-10 15:28:17 +0100241 goto out;
Andrew Scull19503262018-09-20 14:48:39 +0100242 }
243
244 /* The requested vcpu must exist. */
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100245 if (vcpu_idx >= vm->vcpu_count) {
Andrew Scull6d2db332018-10-10 15:28:17 +0100246 goto out;
Andrew Scull7364a8e2018-07-19 15:39:29 +0100247 }
Wedson Almeida Filho3fcbcff2018-07-10 23:53:39 +0100248
Wedson Almeida Filho03306112018-11-26 00:08:03 +0000249 /* Update state if allowed. */
Andrew Scullf3d45592018-09-20 14:30:22 +0100250 vcpu = &vm->vcpus[vcpu_idx];
Wedson Almeida Filho03306112018-11-26 00:08:03 +0000251 if (!api_vcpu_prepare_run(current, vcpu, &vcpu_retval)) {
Andrew Scull6d2db332018-10-10 15:28:17 +0100252 ret.code = HF_VCPU_RUN_WAIT_FOR_INTERRUPT;
Wedson Almeida Filho03306112018-11-26 00:08:03 +0000253 goto out;
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100254 }
Wedson Almeida Filho03306112018-11-26 00:08:03 +0000255
Andrew Scull33fecd32019-01-08 14:48:27 +0000256 /* Switch to the vcpu. */
Wedson Almeida Filho03306112018-11-26 00:08:03 +0000257 *next = vcpu;
Wedson Almeida Filho03306112018-11-26 00:08:03 +0000258
Andrew Scull33fecd32019-01-08 14:48:27 +0000259 /*
260 * Set a placeholder return code to the scheduler. This will be
261 * overwritten when the switch back to the primary occurs.
262 */
263 ret.code = HF_VCPU_RUN_PREEMPTED;
264
265 /* Update return value for the next vcpu if one was injected. */
Wedson Almeida Filho03306112018-11-26 00:08:03 +0000266 if (vcpu_retval.force) {
267 arch_regs_set_retval(&vcpu->regs, vcpu_retval.value);
268 }
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100269
Andrew Scull6d2db332018-10-10 15:28:17 +0100270out:
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100271 return ret;
Wedson Almeida Filho3fcbcff2018-07-10 23:53:39 +0100272}
273
274/**
Andrew Scull81e85092018-12-12 12:56:20 +0000275 * Check that the mode indicates memory that is valid, owned and exclusive.
276 */
277bool static api_mode_valid_owned_and_exclusive(int mode)
278{
279 return (mode & (MM_MODE_INVALID | MM_MODE_UNOWNED | MM_MODE_SHARED)) ==
280 0;
281}
282
283/**
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100284 * Configures the VM to send/receive data through the specified pages. The pages
285 * must not be shared.
286 */
Wedson Almeida Filho00df6c72018-10-18 11:19:24 +0100287int64_t api_vm_configure(ipaddr_t send, ipaddr_t recv,
288 const struct vcpu *current)
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100289{
Wedson Almeida Filho00df6c72018-10-18 11:19:24 +0100290 struct vm *vm = current->vm;
Andrew Scull80871322018-08-06 12:04:09 +0100291 paddr_t pa_send_begin;
292 paddr_t pa_send_end;
293 paddr_t pa_recv_begin;
294 paddr_t pa_recv_end;
Andrew Scull220e6212018-12-21 18:09:00 +0000295 int orig_send_mode;
296 int orig_recv_mode;
297 struct mpool local_page_pool;
Andrew Scullc0e569a2018-10-02 18:05:21 +0100298 int64_t ret;
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100299
300 /* Fail if addresses are not page-aligned. */
Andrew Scull265ada92018-07-30 15:19:01 +0100301 if ((ipa_addr(send) & (PAGE_SIZE - 1)) ||
302 (ipa_addr(recv) & (PAGE_SIZE - 1))) {
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100303 return -1;
304 }
305
Andrew Scullc2eb6a32018-12-13 16:54:24 +0000306 /* Convert to physical addresses. */
307 pa_send_begin = pa_from_ipa(send);
308 pa_send_end = pa_add(pa_send_begin, PAGE_SIZE);
309
310 pa_recv_begin = pa_from_ipa(recv);
311 pa_recv_end = pa_add(pa_recv_begin, PAGE_SIZE);
312
Andrew Scullc9ccb3f2018-08-13 15:27:12 +0100313 /* Fail if the same page is used for the send and receive pages. */
314 if (pa_addr(pa_send_begin) == pa_addr(pa_recv_begin)) {
Andrew Scull220e6212018-12-21 18:09:00 +0000315 return -1;
316 }
317
318 sl_lock(&vm->lock);
319
320 /* We only allow these to be setup once. */
321 if (vm->mailbox.send || vm->mailbox.recv) {
322 goto fail;
323 }
324
325 /*
326 * Ensure the pages are valid, owned and exclusive to the VM and that
327 * the VM has the required access to the memory.
328 */
329 if (!mm_vm_get_mode(&vm->ptable, send, ipa_add(send, PAGE_SIZE),
330 &orig_send_mode) ||
331 !api_mode_valid_owned_and_exclusive(orig_send_mode) ||
332 (orig_send_mode & MM_MODE_R) == 0 ||
333 (orig_send_mode & MM_MODE_W) == 0) {
334 goto fail;
335 }
336
337 if (!mm_vm_get_mode(&vm->ptable, recv, ipa_add(recv, PAGE_SIZE),
338 &orig_recv_mode) ||
339 !api_mode_valid_owned_and_exclusive(orig_recv_mode) ||
340 (orig_recv_mode & MM_MODE_R) == 0) {
341 goto fail;
342 }
343
344 /*
345 * Create a local pool so any freed memory can't be used by another
346 * thread. This is to ensure the original mapping can be restored if any
347 * stage of the process fails.
348 */
349 mpool_init_with_fallback(&local_page_pool, &api_page_pool);
350
351 /* Take memory ownership away from the VM and mark as shared. */
352 if (!mm_vm_identity_map(
353 &vm->ptable, pa_send_begin, pa_send_end,
354 MM_MODE_UNOWNED | MM_MODE_SHARED | MM_MODE_R | MM_MODE_W,
355 NULL, &local_page_pool)) {
356 goto fail_free_pool;
357 }
358
359 if (!mm_vm_identity_map(&vm->ptable, pa_recv_begin, pa_recv_end,
360 MM_MODE_UNOWNED | MM_MODE_SHARED | MM_MODE_R,
361 NULL, &local_page_pool)) {
362 /* TODO: partial defrag of failed range. */
363 /* Recover any memory consumed in failed mapping. */
Andrew Scullda3df7f2019-01-05 17:49:27 +0000364 mm_vm_defrag(&vm->ptable, &local_page_pool);
Andrew Scull220e6212018-12-21 18:09:00 +0000365 goto fail_undo_send;
Andrew Scullc9ccb3f2018-08-13 15:27:12 +0100366 }
367
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100368 /* Map the send page as read-only in the hypervisor address space. */
Wedson Almeida Filho22d5eaa2018-12-16 00:38:49 +0000369 vm->mailbox.send = mm_identity_map(pa_send_begin, pa_send_end,
Andrew Scull220e6212018-12-21 18:09:00 +0000370 MM_MODE_R, &local_page_pool);
Andrew Scullaa039b32018-10-04 15:02:26 +0100371 if (!vm->mailbox.send) {
Andrew Scull220e6212018-12-21 18:09:00 +0000372 /* TODO: partial defrag of failed range. */
373 /* Recover any memory consumed in failed mapping. */
374 mm_defrag(&local_page_pool);
375 goto fail_undo_send_and_recv;
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100376 }
377
378 /*
379 * Map the receive page as writable in the hypervisor address space. On
380 * failure, unmap the send page before returning.
381 */
Wedson Almeida Filho22d5eaa2018-12-16 00:38:49 +0000382 vm->mailbox.recv = mm_identity_map(pa_recv_begin, pa_recv_end,
Andrew Scull220e6212018-12-21 18:09:00 +0000383 MM_MODE_W, &local_page_pool);
Andrew Scullaa039b32018-10-04 15:02:26 +0100384 if (!vm->mailbox.recv) {
Andrew Scull220e6212018-12-21 18:09:00 +0000385 /* TODO: partial defrag of failed range. */
386 /* Recover any memory consumed in failed mapping. */
387 mm_defrag(&local_page_pool);
388 goto fail_undo_all;
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100389 }
390
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100391 /* TODO: Notify any waiters. */
392
393 ret = 0;
Andrew Scull220e6212018-12-21 18:09:00 +0000394 goto exit;
395
396 /*
397 * The following mappings will not require more memory than is available
398 * in the local pool.
399 */
400fail_undo_all:
401 vm->mailbox.send = NULL;
Andrew Scullda241972019-01-05 18:17:48 +0000402 mm_unmap(pa_send_begin, pa_send_end, &local_page_pool);
Andrew Scull220e6212018-12-21 18:09:00 +0000403
404fail_undo_send_and_recv:
405 mm_vm_identity_map(&vm->ptable, pa_recv_begin, pa_recv_end,
406 orig_recv_mode, NULL, &local_page_pool);
407
408fail_undo_send:
409 mm_vm_identity_map(&vm->ptable, pa_send_begin, pa_send_end,
410 orig_send_mode, NULL, &local_page_pool);
411
412fail_free_pool:
413 mpool_fini(&local_page_pool);
414
415fail:
416 ret = -1;
417
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100418exit:
419 sl_unlock(&vm->lock);
420
421 return ret;
422}
423
424/**
Andrew Scullaa039b32018-10-04 15:02:26 +0100425 * Copies data from the sender's send buffer to the recipient's receive buffer
426 * and notifies the recipient.
Wedson Almeida Filho17c997f2019-01-09 18:50:09 +0000427 *
428 * If the recipient's receive buffer is busy, it can optionally register the
429 * caller to be notified when the recipient's receive buffer becomes available.
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100430 */
Wedson Almeida Filho17c997f2019-01-09 18:50:09 +0000431int64_t api_mailbox_send(uint32_t vm_id, size_t size, bool notify,
432 struct vcpu *current, struct vcpu **next)
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100433{
Wedson Almeida Filho00df6c72018-10-18 11:19:24 +0100434 struct vm *from = current->vm;
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100435 struct vm *to;
436 const void *from_buf;
Andrew Scullaa039b32018-10-04 15:02:26 +0100437 uint16_t vcpu;
Andrew Scullc0e569a2018-10-02 18:05:21 +0100438 int64_t ret;
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100439
Wedson Almeida Filho17c997f2019-01-09 18:50:09 +0000440 (void)notify;
441
Andrew Scullaa039b32018-10-04 15:02:26 +0100442 /* Limit the size of transfer. */
443 if (size > HF_MAILBOX_SIZE) {
Andrew Scull19503262018-09-20 14:48:39 +0100444 return -1;
445 }
446
447 /* Disallow reflexive requests as this suggests an error in the VM. */
448 if (vm_id == from->id) {
449 return -1;
450 }
451
452 /* Ensure the target VM exists. */
453 to = vm_get(vm_id);
454 if (to == NULL) {
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100455 return -1;
456 }
457
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100458 /*
459 * Check that the sender has configured its send buffer. It is safe to
460 * use from_buf after releasing the lock because the buffer cannot be
461 * modified once it's configured.
462 */
463 sl_lock(&from->lock);
Andrew Scullaa039b32018-10-04 15:02:26 +0100464 from_buf = from->mailbox.send;
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100465 sl_unlock(&from->lock);
Andrew Scullaa039b32018-10-04 15:02:26 +0100466 if (from_buf == NULL) {
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100467 return -1;
468 }
469
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100470 sl_lock(&to->lock);
471
Andrew Scullaa039b32018-10-04 15:02:26 +0100472 if (to->mailbox.state != mailbox_state_empty ||
473 to->mailbox.recv == NULL) {
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100474 /* Fail if the target isn't currently ready to receive data. */
475 ret = -1;
Andrew Scullaa039b32018-10-04 15:02:26 +0100476 goto out;
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100477 }
478
Andrew Scullaa039b32018-10-04 15:02:26 +0100479 /* Copy data. */
480 memcpy(to->mailbox.recv, from_buf, size);
481 to->mailbox.recv_bytes = size;
482 to->mailbox.recv_from_id = from->id;
483 to->mailbox.state = mailbox_state_read;
484
485 /* Messages for the primary VM are delivered directly. */
486 if (to->id == HF_PRIMARY_VM_ID) {
Wedson Almeida Filho80eb4a32018-11-30 17:11:15 +0000487 struct hf_vcpu_run_return primary_ret = {
488 .code = HF_VCPU_RUN_MESSAGE,
489 .message.size = size,
490 };
Wedson Almeida Filho81568c42019-01-04 13:33:02 +0000491
Wedson Almeida Filhoba641ef2018-12-03 04:19:44 +0000492 *next = api_switch_to_primary(current, primary_ret,
493 vcpu_state_ready);
Andrew Scullaa039b32018-10-04 15:02:26 +0100494 ret = 0;
495 goto out;
496 }
497
498 /*
499 * Try to find a vcpu to handle the message and tell the scheduler to
500 * run it.
501 */
502 if (to->mailbox.recv_waiter == NULL) {
503 /*
504 * The scheduler must choose a vcpu to interrupt so it can
505 * handle the message.
506 */
507 to->mailbox.state = mailbox_state_received;
508 vcpu = HF_INVALID_VCPU;
509 } else {
510 struct vcpu *to_vcpu = to->mailbox.recv_waiter;
511
512 /*
Wedson Almeida Filho80eb4a32018-11-30 17:11:15 +0000513 * Take target vcpu out of waiter list and mark it as ready to
514 * run again.
Andrew Scullaa039b32018-10-04 15:02:26 +0100515 */
516 sl_lock(&to_vcpu->lock);
517 to->mailbox.recv_waiter = to_vcpu->mailbox_next;
518 to_vcpu->state = vcpu_state_ready;
519
520 /* Return from HF_MAILBOX_RECEIVE. */
Wedson Almeida Filho03306112018-11-26 00:08:03 +0000521 to_vcpu->retval.force = true;
522 to_vcpu->retval.value = hf_mailbox_receive_return_encode(
523 (struct hf_mailbox_receive_return){
524 .vm_id = to->mailbox.recv_from_id,
525 .size = size,
526 });
Andrew Scullaa039b32018-10-04 15:02:26 +0100527
528 sl_unlock(&to_vcpu->lock);
529
530 vcpu = to_vcpu - to->vcpus;
531 }
532
533 /* Return to the primary VM directly or with a switch. */
Wedson Almeida Filho80eb4a32018-11-30 17:11:15 +0000534 if (from->id == HF_PRIMARY_VM_ID) {
535 ret = vcpu;
536 } else {
537 struct hf_vcpu_run_return primary_ret = {
538 .code = HF_VCPU_RUN_WAKE_UP,
539 .wake_up.vm_id = to->id,
540 .wake_up.vcpu = vcpu,
541 };
Wedson Almeida Filho81568c42019-01-04 13:33:02 +0000542
Wedson Almeida Filhoba641ef2018-12-03 04:19:44 +0000543 *next = api_switch_to_primary(current, primary_ret,
544 vcpu_state_ready);
Wedson Almeida Filho80eb4a32018-11-30 17:11:15 +0000545 ret = 0;
546 }
Andrew Scullaa039b32018-10-04 15:02:26 +0100547
548out:
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100549 sl_unlock(&to->lock);
550
Wedson Almeida Filho80eb4a32018-11-30 17:11:15 +0000551 return ret;
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100552}
553
554/**
Andrew Scullaa039b32018-10-04 15:02:26 +0100555 * Receives a message from the mailbox. If one isn't available, this function
556 * can optionally block the caller until one becomes available.
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100557 *
Andrew Scullaa039b32018-10-04 15:02:26 +0100558 * No new messages can be received until the mailbox has been cleared.
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100559 */
Andrew Scull6d2db332018-10-10 15:28:17 +0100560struct hf_mailbox_receive_return api_mailbox_receive(bool block,
Wedson Almeida Filho00df6c72018-10-18 11:19:24 +0100561 struct vcpu *current,
Andrew Scull6d2db332018-10-10 15:28:17 +0100562 struct vcpu **next)
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100563{
Wedson Almeida Filho00df6c72018-10-18 11:19:24 +0100564 struct vm *vm = current->vm;
Andrew Scull6d2db332018-10-10 15:28:17 +0100565 struct hf_mailbox_receive_return ret = {
566 .vm_id = HF_INVALID_VM_ID,
567 };
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100568
Andrew Scullaa039b32018-10-04 15:02:26 +0100569 /*
570 * The primary VM will receive messages as a status code from running
571 * vcpus and must not call this function.
572 */
Andrew Scull19503262018-09-20 14:48:39 +0100573 if (vm->id == HF_PRIMARY_VM_ID) {
Andrew Scull6d2db332018-10-10 15:28:17 +0100574 return ret;
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100575 }
576
577 sl_lock(&vm->lock);
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100578
Andrew Scullaa039b32018-10-04 15:02:26 +0100579 /* Return pending messages without blocking. */
580 if (vm->mailbox.state == mailbox_state_received) {
581 vm->mailbox.state = mailbox_state_read;
Andrew Scull6d2db332018-10-10 15:28:17 +0100582 ret.vm_id = vm->mailbox.recv_from_id;
583 ret.size = vm->mailbox.recv_bytes;
Andrew Scullaa039b32018-10-04 15:02:26 +0100584 goto out;
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100585 }
Andrew Scullaa039b32018-10-04 15:02:26 +0100586
587 /* No pending message so fail if not allowed to block. */
588 if (!block) {
Andrew Scullaa039b32018-10-04 15:02:26 +0100589 goto out;
590 }
591
Wedson Almeida Filho00df6c72018-10-18 11:19:24 +0100592 sl_lock(&current->lock);
Andrew Scullaa039b32018-10-04 15:02:26 +0100593
594 /* Push vcpu into waiter list. */
Wedson Almeida Filho00df6c72018-10-18 11:19:24 +0100595 current->mailbox_next = vm->mailbox.recv_waiter;
596 vm->mailbox.recv_waiter = current;
597 sl_unlock(&current->lock);
Andrew Scullaa039b32018-10-04 15:02:26 +0100598
599 /* Switch back to primary vm to block. */
Andrew Walbranb4816552018-12-05 17:35:42 +0000600 {
601 struct hf_vcpu_run_return run_return = {
602 .code = HF_VCPU_RUN_WAIT_FOR_INTERRUPT,
603 };
Wedson Almeida Filho81568c42019-01-04 13:33:02 +0000604
Andrew Walbranb4816552018-12-05 17:35:42 +0000605 *next = api_switch_to_primary(current, run_return,
606 vcpu_state_blocked_mailbox);
607 }
Andrew Scullaa039b32018-10-04 15:02:26 +0100608out:
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100609 sl_unlock(&vm->lock);
610
611 return ret;
612}
613
614/**
Andrew Scullaa039b32018-10-04 15:02:26 +0100615 * Clears the caller's mailbox so that a new message can be received. The caller
616 * must have copied out all data they wish to preserve as new messages will
617 * overwrite the old and will arrive asynchronously.
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100618 */
Wedson Almeida Filho00df6c72018-10-18 11:19:24 +0100619int64_t api_mailbox_clear(const struct vcpu *current)
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100620{
Wedson Almeida Filho00df6c72018-10-18 11:19:24 +0100621 struct vm *vm = current->vm;
Andrew Scullc0e569a2018-10-02 18:05:21 +0100622 int64_t ret;
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100623
624 sl_lock(&vm->lock);
Andrew Scullaa039b32018-10-04 15:02:26 +0100625 if (vm->mailbox.state == mailbox_state_read) {
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100626 ret = 0;
Andrew Scullaa039b32018-10-04 15:02:26 +0100627 vm->mailbox.state = mailbox_state_empty;
628 } else {
629 ret = -1;
Wedson Almeida Filho2f94ec12018-07-26 16:00:48 +0100630 }
631 sl_unlock(&vm->lock);
632
633 if (ret == 0) {
634 /* TODO: Notify waiters, if any. */
635 }
636
637 return ret;
Wedson Almeida Filho3fcbcff2018-07-10 23:53:39 +0100638}
Andrew Walbran318f5732018-11-20 16:23:42 +0000639
640/**
641 * Enables or disables a given interrupt ID for the calling vCPU.
642 *
643 * Returns 0 on success, or -1 if the intid is invalid.
644 */
Wedson Almeida Filhoc559d132019-01-09 19:33:40 +0000645int64_t api_interrupt_enable(uint32_t intid, bool enable, struct vcpu *current)
Andrew Walbran318f5732018-11-20 16:23:42 +0000646{
647 uint32_t intid_index = intid / INTERRUPT_REGISTER_BITS;
648 uint32_t intid_mask = 1u << (intid % INTERRUPT_REGISTER_BITS);
Wedson Almeida Filho81568c42019-01-04 13:33:02 +0000649
Andrew Walbran318f5732018-11-20 16:23:42 +0000650 if (intid >= HF_NUM_INTIDS) {
651 return -1;
652 }
653
654 sl_lock(&current->lock);
655 if (enable) {
Andrew Walbran3d84a262018-12-13 14:41:19 +0000656 /*
657 * If it is pending and was not enabled before, increment the
658 * count.
659 */
660 if (current->interrupts.interrupt_pending[intid_index] &
661 ~current->interrupts.interrupt_enabled[intid_index] &
662 intid_mask) {
663 current->interrupts.enabled_and_pending_count++;
664 }
Andrew Walbran318f5732018-11-20 16:23:42 +0000665 current->interrupts.interrupt_enabled[intid_index] |=
666 intid_mask;
Andrew Walbran318f5732018-11-20 16:23:42 +0000667 } else {
Andrew Walbran3d84a262018-12-13 14:41:19 +0000668 /*
669 * If it is pending and was enabled before, decrement the count.
670 */
671 if (current->interrupts.interrupt_pending[intid_index] &
672 current->interrupts.interrupt_enabled[intid_index] &
673 intid_mask) {
674 current->interrupts.enabled_and_pending_count--;
675 }
Andrew Walbran318f5732018-11-20 16:23:42 +0000676 current->interrupts.interrupt_enabled[intid_index] &=
677 ~intid_mask;
678 }
679
680 sl_unlock(&current->lock);
681 return 0;
682}
683
684/**
685 * Returns the ID of the next pending interrupt for the calling vCPU, and
686 * acknowledges it (i.e. marks it as no longer pending). Returns
687 * HF_INVALID_INTID if there are no pending interrupts.
688 */
Wedson Almeida Filhoc559d132019-01-09 19:33:40 +0000689uint32_t api_interrupt_get(struct vcpu *current)
Andrew Walbran318f5732018-11-20 16:23:42 +0000690{
691 uint8_t i;
692 uint32_t first_interrupt = HF_INVALID_INTID;
Andrew Walbran318f5732018-11-20 16:23:42 +0000693
694 /*
695 * Find the first enabled and pending interrupt ID, return it, and
696 * deactivate it.
697 */
698 sl_lock(&current->lock);
699 for (i = 0; i < HF_NUM_INTIDS / INTERRUPT_REGISTER_BITS; ++i) {
700 uint32_t enabled_and_pending =
701 current->interrupts.interrupt_enabled[i] &
702 current->interrupts.interrupt_pending[i];
Wedson Almeida Filho81568c42019-01-04 13:33:02 +0000703
Andrew Walbran318f5732018-11-20 16:23:42 +0000704 if (enabled_and_pending != 0) {
Andrew Walbran3d84a262018-12-13 14:41:19 +0000705 uint8_t bit_index = ctz(enabled_and_pending);
706 /*
707 * Mark it as no longer pending and decrement the count.
708 */
709 current->interrupts.interrupt_pending[i] &=
710 ~(1u << bit_index);
711 current->interrupts.enabled_and_pending_count--;
712 first_interrupt =
713 i * INTERRUPT_REGISTER_BITS + bit_index;
Andrew Walbran318f5732018-11-20 16:23:42 +0000714 break;
715 }
716 }
Andrew Walbran318f5732018-11-20 16:23:42 +0000717
718 sl_unlock(&current->lock);
719 return first_interrupt;
720}
721
722/**
Andrew Walbran4cf217a2018-12-14 15:24:50 +0000723 * Returns whether the current vCPU is allowed to inject an interrupt into the
Andrew Walbran318f5732018-11-20 16:23:42 +0000724 * given VM and vCPU.
725 */
726static inline bool is_injection_allowed(uint32_t target_vm_id,
727 struct vcpu *current)
728{
729 uint32_t current_vm_id = current->vm->id;
Wedson Almeida Filho81568c42019-01-04 13:33:02 +0000730
Andrew Walbran318f5732018-11-20 16:23:42 +0000731 /*
732 * The primary VM is allowed to inject interrupts into any VM. Secondary
733 * VMs are only allowed to inject interrupts into their own vCPUs.
734 */
735 return current_vm_id == HF_PRIMARY_VM_ID ||
736 current_vm_id == target_vm_id;
737}
738
739/**
740 * Injects a virtual interrupt of the given ID into the given target vCPU.
741 * This doesn't cause the vCPU to actually be run immediately; it will be taken
742 * when the vCPU is next run, which is up to the scheduler.
743 *
Andrew Walbran3d84a262018-12-13 14:41:19 +0000744 * Returns:
745 * - -1 on failure because the target VM or vCPU doesn't exist, the interrupt
746 * ID is invalid, or the current VM is not allowed to inject interrupts to
747 * the target VM.
748 * - 0 on success if no further action is needed.
749 * - 1 if it was called by the primary VM and the primary VM now needs to wake
750 * up or kick the target vCPU.
Andrew Walbran318f5732018-11-20 16:23:42 +0000751 */
Wedson Almeida Filhoc559d132019-01-09 19:33:40 +0000752int64_t api_interrupt_inject(uint32_t target_vm_id, uint32_t target_vcpu_idx,
Andrew Walbran318f5732018-11-20 16:23:42 +0000753 uint32_t intid, struct vcpu *current,
754 struct vcpu **next)
755{
756 uint32_t intid_index = intid / INTERRUPT_REGISTER_BITS;
757 uint32_t intid_mask = 1u << (intid % INTERRUPT_REGISTER_BITS);
758 struct vcpu *target_vcpu;
759 struct vm *target_vm = vm_get(target_vm_id);
Andrew Walbran69520dc2018-12-06 11:39:38 +0000760 bool need_vm_lock;
Andrew Walbran3d84a262018-12-13 14:41:19 +0000761 int64_t ret = 0;
Andrew Walbran318f5732018-11-20 16:23:42 +0000762
763 if (intid >= HF_NUM_INTIDS) {
764 return -1;
765 }
Wedson Almeida Filho81568c42019-01-04 13:33:02 +0000766
Andrew Walbran318f5732018-11-20 16:23:42 +0000767 if (target_vm == NULL) {
768 return -1;
769 }
Wedson Almeida Filho81568c42019-01-04 13:33:02 +0000770
Andrew Walbran318f5732018-11-20 16:23:42 +0000771 if (target_vcpu_idx >= target_vm->vcpu_count) {
772 /* The requested vcpu must exist. */
773 return -1;
774 }
Wedson Almeida Filho81568c42019-01-04 13:33:02 +0000775
Andrew Walbran318f5732018-11-20 16:23:42 +0000776 if (!is_injection_allowed(target_vm_id, current)) {
777 return -1;
778 }
Wedson Almeida Filho81568c42019-01-04 13:33:02 +0000779
Andrew Walbran318f5732018-11-20 16:23:42 +0000780 target_vcpu = &target_vm->vcpus[target_vcpu_idx];
781
782 dlog("Injecting IRQ %d for VM %d VCPU %d from VM %d VCPU %d\n", intid,
783 target_vm_id, target_vcpu_idx, current->vm->id, current->cpu->id);
784
785 sl_lock(&target_vcpu->lock);
Andrew Walbran69520dc2018-12-06 11:39:38 +0000786 /*
787 * If we need the target_vm lock we need to release the target_vcpu lock
788 * first to maintain the correct order of locks. In-between releasing
789 * and acquiring it again the state of the vCPU could change in such a
790 * way that we don't actually need to touch the target_vm after all, but
791 * that's alright: we'll take the target_vm lock anyway, but it's safe,
792 * just perhaps a little slow in this unusual case. The reverse is not
793 * possible: if need_vm_lock is false, we don't release the target_vcpu
794 * lock until we are done, so nothing should change in such as way that
795 * we need the VM lock after all.
796 */
Andrew Walbran3d84a262018-12-13 14:41:19 +0000797 need_vm_lock =
798 (target_vcpu->interrupts.interrupt_enabled[intid_index] &
799 ~target_vcpu->interrupts.interrupt_pending[intid_index] &
800 intid_mask) &&
801 target_vcpu->state == vcpu_state_blocked_mailbox;
Andrew Walbran69520dc2018-12-06 11:39:38 +0000802 if (need_vm_lock) {
803 sl_unlock(&target_vcpu->lock);
804 sl_lock(&target_vm->lock);
805 sl_lock(&target_vcpu->lock);
806 }
Andrew Walbran318f5732018-11-20 16:23:42 +0000807
Andrew Walbran3d84a262018-12-13 14:41:19 +0000808 /*
809 * We only need to change state and (maybe) trigger a virtual IRQ if it
810 * is enabled and was not previously pending. Otherwise we can skip
811 * everything except setting the pending bit.
812 *
813 * If you change this logic make sure to update the need_vm_lock logic
814 * above to match.
815 */
816 if (!(target_vcpu->interrupts.interrupt_enabled[intid_index] &
817 ~target_vcpu->interrupts.interrupt_pending[intid_index] &
818 intid_mask)) {
819 goto out;
820 }
821
822 /* Increment the count. */
823 target_vcpu->interrupts.enabled_and_pending_count++;
Andrew Walbran318f5732018-11-20 16:23:42 +0000824
Andrew Walbran69520dc2018-12-06 11:39:38 +0000825 /*
Andrew Walbran3d84a262018-12-13 14:41:19 +0000826 * Only need to update state if there was not already an
827 * interrupt enabled and pending.
Andrew Walbran69520dc2018-12-06 11:39:38 +0000828 */
Andrew Walbran3d84a262018-12-13 14:41:19 +0000829 if (target_vcpu->interrupts.enabled_and_pending_count != 1) {
830 goto out;
831 }
Andrew Walbran318f5732018-11-20 16:23:42 +0000832
Andrew Walbran3d84a262018-12-13 14:41:19 +0000833 if (target_vcpu->state == vcpu_state_blocked_interrupt) {
834 target_vcpu->state = vcpu_state_ready;
835 } else if (target_vcpu->state == vcpu_state_blocked_mailbox) {
836 /*
837 * If you change this logic make sure to update the need_vm_lock
838 * logic above to match.
839 */
840 target_vcpu->state = vcpu_state_ready;
Andrew Walbran69520dc2018-12-06 11:39:38 +0000841
Andrew Walbran3d84a262018-12-13 14:41:19 +0000842 /* Take target vCPU out of mailbox recv_waiter list. */
843 /*
844 * TODO: Consider using a doubly-linked list for
845 * the receive waiter list to avoid the linear
846 * search here.
847 */
848 struct vcpu **previous_next_pointer =
849 &target_vm->mailbox.recv_waiter;
850 while (*previous_next_pointer != NULL &&
851 *previous_next_pointer != target_vcpu) {
Andrew Walbran69520dc2018-12-06 11:39:38 +0000852 /*
Andrew Walbran3d84a262018-12-13 14:41:19 +0000853 * TODO(qwandor): Do we need to lock the vCPUs somehow
854 * while we walk the linked list, or is the VM lock
855 * enough?
Andrew Walbran69520dc2018-12-06 11:39:38 +0000856 */
Andrew Walbran3d84a262018-12-13 14:41:19 +0000857 previous_next_pointer =
858 &(*previous_next_pointer)->mailbox_next;
Andrew Walbran318f5732018-11-20 16:23:42 +0000859 }
Wedson Almeida Filho81568c42019-01-04 13:33:02 +0000860
Andrew Walbran3d84a262018-12-13 14:41:19 +0000861 if (*previous_next_pointer == NULL) {
862 dlog("Target VCPU state is vcpu_state_blocked_mailbox "
863 "but is not in VM mailbox waiter list. This "
864 "should never happen.\n");
865 } else {
866 *previous_next_pointer = target_vcpu->mailbox_next;
Andrew Walbran318f5732018-11-20 16:23:42 +0000867 }
868 }
869
Andrew Walbran3d84a262018-12-13 14:41:19 +0000870 if (current->vm->id == HF_PRIMARY_VM_ID) {
871 /*
872 * If the call came from the primary VM, let it know that it
873 * should run or kick the target vCPU.
874 */
875 ret = 1;
876 } else if (current != target_vcpu) {
877 /*
878 * Switch to the primary so that it can switch to the target, or
879 * kick it if it is already running on a different physical CPU.
880 */
881 struct hf_vcpu_run_return ret = {
882 .code = HF_VCPU_RUN_WAKE_UP,
883 .wake_up.vm_id = target_vm_id,
884 .wake_up.vcpu = target_vcpu_idx,
885 };
Wedson Almeida Filho81568c42019-01-04 13:33:02 +0000886
Andrew Walbran3d84a262018-12-13 14:41:19 +0000887 *next = api_switch_to_primary(current, ret, vcpu_state_ready);
888 }
889
890out:
891 /* Either way, make it pending. */
892 target_vcpu->interrupts.interrupt_pending[intid_index] |= intid_mask;
893
Andrew Walbran318f5732018-11-20 16:23:42 +0000894 sl_unlock(&target_vcpu->lock);
Andrew Walbran69520dc2018-12-06 11:39:38 +0000895 if (need_vm_lock) {
896 sl_unlock(&target_vm->lock);
897 }
Andrew Walbran318f5732018-11-20 16:23:42 +0000898
Andrew Walbran3d84a262018-12-13 14:41:19 +0000899 return ret;
Andrew Walbran318f5732018-11-20 16:23:42 +0000900}