acrn-dev@lists.projectacrn.org

Home Messages Hashtags Subgroups

Messages

Between

and

Re: [PATCH] hv: ept: fix MISRA-C violations

Acked-by: Eddie Dong <eddie.dong@...> Is this kind of issue checked by current CI system? This code was merged very soon, and I don't know why it leaked the check..

By Eddie Dong · #16489 ·

Re: [PATCH 10/10] HV: remove multiple exit/return in routines in the file of vlapic.c

By Eddie Dong · #16488 ·

Re: [PATCH 09/10] HV: Remove goto statement in guest.c

This seems to be a logic change.... Can we split the logic change and MISRAC cleanup patches?

By Eddie Dong · #16487 ·

[PATCH] hv: ept: fix MISRA-C violations

This patch fixes the MISRA-C violations in arch/x86/ept.c * add local variable if the input parameter passed by value is changed in function definition * remove the duplicated semi

By Shiqing Gao · #16486 ·

[PATCH V3 4/4] hv: virq: refine acrn_handle_pending_request() has more than one exit point

The MISRA-C Standards suggests procedures to be single exit Signed-off-by: Tao Yuhong <yuhong.tao@...> --- hypervisor/arch/x86/virq.c | 100 ++++++++++++++++++------------------- 1 file

By Tao, Yuhong · #16485 ·

[PATCH V3 3/4] hv: virq: refine acrn_handle_pending_request() use goto instruction

The MISRA-C Standards suggests not to use 'goto' instruction. Signed-off-by: Tao Yuhong <yuhong.tao@...> --- hypervisor/arch/x86/virq.c | 184 +++++++++++++++++++++---------------- 1 file

By Tao, Yuhong · #16484 ·

[PATCH V3 2/4] hv: virq: refine vcpu_inject_hi_exception()

The MISRA-C Standards suggests use brackets to clarify the precedence order of logical conjunctions. Acked-by: Eddie Dong <eddie.dong@...> Signed-off-by: Tao Yuhong <yuhong.tao@...> ---

By Tao, Yuhong · #16483 ·

[PATCH V3 1/4] hv: virq: refine vcpu_inject_vlapic_int() has more than one exit point

The MISRA-C Standards suggests procedures to be single exit. Acked-by: Eddie Dong <eddie.dong@...> Signed-off-by: Tao Yuhong <yuhong.tao@...> --- hypervisor/arch/x86/virq.c | 47

By Tao, Yuhong · #16482 ·

[PATCH V3 0/4] virq code refine for MISRA-C Standards

changes in V2: 1.keep the original original code in semantics in vcpu_inject_vlapic_int() 2.Split "hv: virq: refine acrn_handle_pending_request()" in to 2, one for multi-return and another for

By Tao, Yuhong · #16481 ·

Re: [PATCH 08/10] HV: trival changes to meet MISRA

Same with previous comments

By Eddie Dong · #16480 ·

Re: [PATCH V2 4/4] hv: virq: refine acrn_handle_pending_request() use goto instruction

Hi, Eddie

By Tao, Yuhong · #16479 ·

Re: [PATCH 08/10] HV: trival changes to meet MISRA

This is good, but we need to add in header file side for public API.

By Eddie Dong · #16478 ·

Re: [PATCH 07/10] HV: check if the returned target_vcpu pointer is NULL

By Eddie Dong · #16477 ·