github/acrn-hypervisor
1
0
Fork 0
mirror of https://github.com/projectacrn/acrn-hypervisor.git synced 2025-06-23 14:07:42 +00:00
Code Issues Projects Releases Wiki Activity

hv:fix return value violation for vioapic_get_rte

Browse Source 
Change this API to void type
Add pre-condition, state the input parameters are not NULL.

Signed-off-by: Mingqiang Chi <mingqiang.chi@intel.com>
Acked-by: Eddie Dong <eddie.dong@intel.com>
This commit is contained in:
Mingqiang Chi 2018-07-31 17:03:18 +08:00 committed by lijinxia
parent cd3a62f89b
commit ae8836d960
2 changed files with 7 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
hypervisor/arch/x86/guest/vioapic.c
View File

@ -629,17 +629,16 @@ int vioapic_mmio_access_handler(struct vcpu *vcpu, struct io_request *io_req,
return ret; return ret;
} }
bool vioapic_get_rte(struct vm *vm, uint8_t pin, union ioapic_rte *rte) /**
* @pre vm->arch_vm.virt_ioapic != NULL
* @pre rte != NULL
*/
void vioapic_get_rte(struct vm *vm, uint8_t pin, union ioapic_rte *rte)
{ {
struct vioapic *vioapic; struct vioapic *vioapic;
vioapic = vm_ioapic(vm); vioapic = vm_ioapic(vm);
if ((vioapic != NULL) && (rte != NULL)) {
*rte = vioapic->rtbl[pin]; *rte = vioapic->rtbl[pin];
return true;
} else {
return false;
}
} }
#ifdef HV_DEBUG #ifdef HV_DEBUG

2
hypervisor/include/arch/x86/guest/vioapic.h
View File

@ -51,7 +51,7 @@ void vioapic_mmio_read(struct vm *vm, uint64_t gpa, uint32_t *rval);
uint8_t vioapic_pincount(struct vm *vm); uint8_t vioapic_pincount(struct vm *vm);
void vioapic_process_eoi(struct vm *vm, uint32_t vector); void vioapic_process_eoi(struct vm *vm, uint32_t vector);
bool vioapic_get_rte(struct vm *vm, uint8_t pin, union ioapic_rte *rte); void vioapic_get_rte(struct vm *vm, uint8_t pin, union ioapic_rte *rte);
int vioapic_mmio_access_handler(struct vcpu *vcpu, int vioapic_mmio_access_handler(struct vcpu *vcpu,
struct io_request *io_req, __unused void *handler_private_data); struct io_request *io_req, __unused void *handler_private_data);