hv: schedule: minor fix about the return type of need_offline
ACRN Coding guidelines requires type conversion shall be explicity. However, there's no need for this case since we could return bool directly. Tracked-On: #1842 Signed-off-by: Li, Fei1 <fei1.li@intel.com> Acked-by: Eddie Dong <eddie.dong@intel.com>
This commit is contained in:
parent
e69b3dcf67
commit
0eb0854858
|
@ -88,7 +88,7 @@ void default_idle(__unused struct sched_object *obj)
|
||||||
while (1) {
|
while (1) {
|
||||||
if (need_reschedule(pcpu_id)) {
|
if (need_reschedule(pcpu_id)) {
|
||||||
schedule();
|
schedule();
|
||||||
} else if (need_offline(pcpu_id) != 0) {
|
} else if (need_offline(pcpu_id)) {
|
||||||
cpu_dead();
|
cpu_dead();
|
||||||
} else if (need_shutdown_vm(pcpu_id)) {
|
} else if (need_shutdown_vm(pcpu_id)) {
|
||||||
shutdown_vm_from_idle(pcpu_id);
|
shutdown_vm_from_idle(pcpu_id);
|
||||||
|
|
|
@ -136,7 +136,7 @@ void make_pcpu_offline(uint16_t pcpu_id)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
int32_t need_offline(uint16_t pcpu_id)
|
bool need_offline(uint16_t pcpu_id)
|
||||||
{
|
{
|
||||||
struct sched_context *ctx = &per_cpu(sched_ctx, pcpu_id);
|
struct sched_context *ctx = &per_cpu(sched_ctx, pcpu_id);
|
||||||
|
|
||||||
|
|
|
@ -49,7 +49,7 @@ void remove_from_cpu_runqueue(struct sched_object *obj);
|
||||||
void make_reschedule_request(uint16_t pcpu_id, uint16_t delmode);
|
void make_reschedule_request(uint16_t pcpu_id, uint16_t delmode);
|
||||||
bool need_reschedule(uint16_t pcpu_id);
|
bool need_reschedule(uint16_t pcpu_id);
|
||||||
void make_pcpu_offline(uint16_t pcpu_id);
|
void make_pcpu_offline(uint16_t pcpu_id);
|
||||||
int32_t need_offline(uint16_t pcpu_id);
|
bool need_offline(uint16_t pcpu_id);
|
||||||
void make_shutdown_vm_request(uint16_t pcpu_id);
|
void make_shutdown_vm_request(uint16_t pcpu_id);
|
||||||
bool need_shutdown_vm(uint16_t pcpu_id);
|
bool need_shutdown_vm(uint16_t pcpu_id);
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue