|
[PATCH v6 4/4] hv: refine the define of *PIO_IDX
OK. I have drop this patch and send others to PR.
OK. I have drop this patch and send others to PR.
|
By
chenli.wei
· #36035
·
|
|
[PATCH v6 3/4] hv: move the MAX_VUART_NUM_PER_VM to offline tool
I am fine with this. Once you get +1 for the misc patches, please PR.
I am fine with this. Once you get +1 for the misc patches, please PR.
|
By
Eddie Dong
· #36034
·
|
|
[PATCH v6 4/4] hv: refine the define of *PIO_IDX
Aaa, If this is the case, we have to drop this patch.
Aaa, If this is the case, we have to drop this patch.
|
By
Eddie Dong
· #36033
·
|
|
[PATCH v6 4/4] hv: refine the define of *PIO_IDX
By
Eddie Dong
· #36032
·
|
|
[PATCH v6 4/4] hv: refine the define of *PIO_IDX
It seems we should confirm this patch again. I will pause this patch and send others of this series to PR. >
It seems we should confirm this patch again. I will pause this patch and send others of this series to PR. >
|
By
chenli.wei
· #36031
·
|
|
[PATCH v6 2/4] misc: define the MAX_VUART_NUM_PER_VM by offline tool
OK, I will change it. >
By
chenli.wei
· #36030
·
|
|
[PATCH v6 4/4] hv: refine the define of *PIO_IDX
Chenli Wei <chenli.wei@...> writes: A reminder: using enumeration constants as integers is prohibited by ACRN coding guideline. In this case the enumeration above is not exhaustive (due to the v
Chenli Wei <chenli.wei@...> writes: A reminder: using enumeration constants as integers is prohibited by ACRN coding guideline. In this case the enumeration above is not exhaustive (due to the v
|
By
Junjie Mao
· #36029
·
|
|
[PATCH v6 2/4] misc: define the MAX_VUART_NUM_PER_VM by offline tool
Chenli Wei <chenli.wei@...> writes: Reviewed-by: Junjie Mao <junjie.mao@...> One minor comment below. Why do you need `concat` here? A single ',' should do the trick.
Chenli Wei <chenli.wei@...> writes: Reviewed-by: Junjie Mao <junjie.mao@...> One minor comment below. Why do you need `concat` here? A single ',' should do the trick.
|
By
Junjie Mao
· #36028
·
|
|
[PATCH v6 1/4] misc: add vUART channel for s5 feature
Chenli Wei <chenli.wei@...> writes: As a general guideline, please append "Reviewed-by" or "Acked-by" after "Signed-off-by".
Chenli Wei <chenli.wei@...> writes: As a general guideline, please append "Reviewed-by" or "Acked-by" after "Signed-off-by".
|
By
Junjie Mao
· #36027
·
|
|
[PATCH v6 4/4] hv: refine the define of *PIO_IDX
The current code define *PIO_IDX by macro, it's not a good style and the UART PIO was in the middle of these macro which should move to the end to conveniently modify. This patch change these macro to
The current code define *PIO_IDX by macro, it's not a good style and the UART PIO was in the middle of these macro which should move to the end to conveniently modify. This patch change these macro to
|
By
chenli.wei
· #36026
·
|
|
[PATCH v6 3/4] hv: move the MAX_VUART_NUM_PER_VM to offline tool
Current code limit the MAX vUART number to 8 which is not enough for Service VM which should config S5 UART for each user VM. We could count how many vUARTs we need by offline tool, so remove the defi
Current code limit the MAX vUART number to 8 which is not enough for Service VM which should config S5 UART for each user VM. We could count how many vUARTs we need by offline tool, so remove the defi
|
By
chenli.wei
· #36025
·
|
|
[PATCH v6 2/4] misc: define the MAX_VUART_NUM_PER_VM by offline tool
The current code define MAX_VUART_NUM_PER_VM for all VM and it's a hard code which can't match our vUART logic which need dynamic setting. This patch move the define of MAX_VUART_NUM_PER_VM to offline
The current code define MAX_VUART_NUM_PER_VM for all VM and it's a hard code which can't match our vUART logic which need dynamic setting. This patch move the define of MAX_VUART_NUM_PER_VM to offline
|
By
chenli.wei
· #36024
·
|
|
[PATCH v6 1/4] misc: add vUART channel for s5 feature
"S5" is one of the ACPI sleep states which means the function to shut down the VMs. On ACRN, the User VM must be shut down before powering off the Service VM, so we need a vUART channel to communicate
"S5" is one of the ACPI sleep states which means the function to shut down the VMs. On ACRN, the User VM must be shut down before powering off the Service VM, so we need a vUART channel to communicate
|
By
chenli.wei
· #36023
·
|
|
[PATCH v6 0/4] Add vUART channel for s5 feature
"S5" is one of the ACPI sleep states which means the function to shut down the VMs. On ACRN, the User VM must be shut down before powering off the Service VM, so we need a vUART channel to communicate
"S5" is one of the ACPI sleep states which means the function to shut down the VMs. On ACRN, the User VM must be shut down before powering off the Service VM, so we need a vUART channel to communicate
|
By
chenli.wei
· #36022
·
|
|
[PATCH v5 2/4] misc: define the MAX_VUART_NUM_PER_VM by offline tool
OK, I will change the logic of get MAX number. >
OK, I will change the logic of get MAX number. >
|
By
chenli.wei
· #36021
·
|
|
[PATCH v5 2/4] misc: define the MAX_VUART_NUM_PER_VM by offline tool
Chenli Wei <chenli.wei@...> writes: There is no guarantee that the service VM has the most vUART controllers. Also there can be cases where the service VM is absent. Thus, you need to collect th
Chenli Wei <chenli.wei@...> writes: There is no guarantee that the service VM has the most vUART controllers. Also there can be cases where the service VM is absent. Thus, you need to collect th
|
By
Junjie Mao
· #36020
·
|
|
[PATCH v5 1/4] misc: add vUART channel for s5 feature
Chenli Wei <chenli.wei@...> writes: Reviewed-by: Junjie Mao <junjie.mao@...>
Chenli Wei <chenli.wei@...> writes: Reviewed-by: Junjie Mao <junjie.mao@...>
|
By
Junjie Mao
· #36019
·
|
|
[PATCH v5 4/4] hv: refine the define of *PIO_IDX
The current code define *PIO_IDX by macro, it's not a good style and the UART PIO was in the middle of these macro which should move to the end to conveniently modify. This patch change these macro to
The current code define *PIO_IDX by macro, it's not a good style and the UART PIO was in the middle of these macro which should move to the end to conveniently modify. This patch change these macro to
|
By
chenli.wei
· #36018
·
|
|
[PATCH v5 3/4] hv: move the MAX_VUART_NUM_PER_VM to offline tool
Current code limit the MAX vUART number to 8 which is not enough for Service VM which should config S5 UART for each user VM. We could count how many vUARTs we need by offline tool, so remove the defi
Current code limit the MAX vUART number to 8 which is not enough for Service VM which should config S5 UART for each user VM. We could count how many vUARTs we need by offline tool, so remove the defi
|
By
chenli.wei
· #36017
·
|
|
[PATCH v5 2/4] misc: define the MAX_VUART_NUM_PER_VM by offline tool
The current code define MAX_VUART_NUM_PER_VM for all VM and it's a hard code which can't match our vUART logic which need dynamic setting. This patch move the define of MAX_VUART_NUM_PER_VM to offline
The current code define MAX_VUART_NUM_PER_VM for all VM and it's a hard code which can't match our vUART logic which need dynamic setting. This patch move the define of MAX_VUART_NUM_PER_VM to offline
|
By
chenli.wei
· #36016
·
|