Hi Zide Please don't mention this rule set such as 120D, 87S, etc in the maillist. It may cause potential issues. Thx. Best regards, Junjun Shan OTC -> ACRN hypervisor

Now we have name reuse definitions in hypervisor as following: "enum cpu_state cpu_state" in per_cpu.h, "struct shell_cmd *shell_cmd" in shell_priv.h. MISRAC requires that tag names shall not be reuse

Hi eddie & Xiangyang Thank you very much for your comments. Best regards, Junjun Shan OTC -> hypervisor

Hi eddie We plan to modify the definition to "enum hv_pcpu_state pcpu_state". ACRN type is now used for VCPU, VLAPIC, VPIC definitions which are ACRN specified concept while pcpu_state is common for a

Now we have name reuse definitions in hypervisor as following: "enum cpu_state cpu_state" in per_cpu.h, "struct shell_cmd *shell_cmd" in shell_priv.h. MISRAC requires that tag names shall not be reuse

Hi eddie Now we have name reuse definitions in hypervisor as following: "enum cpu_state cpu_state" in per_cpu.h; "struct shell_cmd *shell_cmd" in shell_priv.h; These definitions violate MISRAC rules "

MISRAC requires that tag names shall not be reused anywhere with in a program. This patch is used to fix reuse violations of enum cpu_state and strcture shell_cmd. Tracked-On: #861 Signed-off-by: Junj

MISRAC requires that tag names shall not be reused anywhere with in a program. This patch is used to fix reuse violations of enum cpu_state and strcture shell_cmd. Tracked-On: #861 Signed-off-by: Junj

Best regards, Junjun Shan OTC -> hypervisor

The printf related functions have been called by other various functions, e.g. get_ptdev_info(), get_ioapic_info, etc. The patch is used to clear up the usage. Signed-off-by: Junjun Shan <junjun.shan@

In hypervisor, all the parameter and return value printf related are unsigned int, this patch is used to fix the function definitions. Signed-off-by: Junjun Shan <junjun.shan@...> --- hypervisor

This patch series try to clear up printf related data struct due to all the int related data are unsigned int in hypervisor. The first patch is used to clear up the definition and the second patch is

MISRAC checks whether a variable is assigned a value not used in all branches of a program. Var value which is unused on all paths can be removed with a consequent improvement in the readability and e

Best regards, Steven Shan OTC -> hypervisor

MISRAC has requirement about implict conversion: actual to formal param. This patch is used to fix part of these violations. 1.Add a new structure seg_desc_vmcs to hold the VMCS field address of segme

MISRAC has requirement about implict conversion: actual to formal param. This patch is used to fix part of these violations. 1.Add a new structure seg_desc_vmcs to hold the VMCS field address of segme

Hi Junjie I'll double check with the suppression table and update the patch. THX. Best regards, Steven Shan OTC -> hypervisor

Hi eddie Got it. I'll confirm this issue and update the patch. Best regards, Steven Shan OTC -> hypervisor

Delete the dead code get_irq_info(), local_get_irq_info() in io_request.c and definition in guest.h. v1->v2 *the dead code used pointers which not checked before use, therefore it causes "pointer not

MISARC has requirement about implicat conversion: actual to formal param.This patch is used to fix these violations. Tracked-On: #861 Signed-off-by: Junjun Shan <junjun.shan@...> --- hypervisor/