On 05.01.21 10:19, Alice Guo (OSS) wrote:
> From: Alice Guo <[email protected]>
> 
> When enabling Jailhouse on the iMX8M platforms, the stage 2 translation
> about IPA->PA includes the address range contains GICD and GICRs, which
> should not be allowed because GICD or GICRs should not be accessed in
> the root cell. In order to solve this problem, remove the physical
> address range contains GICD and GICRs.
> 
> Signed-off-by: Alice Guo <[email protected]>
> ---
>  configs/arm64/imx8mm.c | 2 +-
>  configs/arm64/imx8mn.c | 2 +-
>  configs/arm64/imx8mp.c | 2 +-
>  configs/arm64/imx8mq.c | 2 +-
>  4 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/configs/arm64/imx8mm.c b/configs/arm64/imx8mm.c
> index 88365919..ebc2d294 100644
> --- a/configs/arm64/imx8mm.c
> +++ b/configs/arm64/imx8mm.c
> @@ -107,7 +107,7 @@ struct {
>               /* IO */ {
>                       .phys_start = 0x00000000,
>                       .virt_start = 0x00000000,
> -                     .size =       0x40000000,
> +                     .size =       0x38800000,
>                       .flags = JAILHOUSE_MEM_READ | JAILHOUSE_MEM_WRITE |
>                               JAILHOUSE_MEM_IO,
>               },
> diff --git a/configs/arm64/imx8mn.c b/configs/arm64/imx8mn.c
> index ecd00a90..667221da 100644
> --- a/configs/arm64/imx8mn.c
> +++ b/configs/arm64/imx8mn.c
> @@ -102,7 +102,7 @@ struct {
>               /* IO */ {
>                       .phys_start = 0x00000000,
>                       .virt_start = 0x00000000,
> -                     .size =       0x40000000,
> +                     .size =       0x38800000,
>                       .flags = JAILHOUSE_MEM_READ | JAILHOUSE_MEM_WRITE |
>                               JAILHOUSE_MEM_IO,
>               },
> diff --git a/configs/arm64/imx8mp.c b/configs/arm64/imx8mp.c
> index b870a788..a1e221f9 100644
> --- a/configs/arm64/imx8mp.c
> +++ b/configs/arm64/imx8mp.c
> @@ -103,7 +103,7 @@ struct {
>               /* IO */ {
>                       .phys_start = 0x00000000,
>                       .virt_start = 0x00000000,
> -                     .size =       0x40000000,
> +                     .size =       0x38800000,
>                       .flags = JAILHOUSE_MEM_READ | JAILHOUSE_MEM_WRITE |
>                               JAILHOUSE_MEM_IO,
>               },
> diff --git a/configs/arm64/imx8mq.c b/configs/arm64/imx8mq.c
> index 12afedb4..9f9c6071 100644
> --- a/configs/arm64/imx8mq.c
> +++ b/configs/arm64/imx8mq.c
> @@ -101,7 +101,7 @@ struct {
>               /* MMIO (permissive) */ {
>                       .phys_start = 0x00000000,
>                       .virt_start = 0x00000000,
> -                     .size =       0x40000000,
> +                     .size =       0x38800000,
>                       .flags = JAILHOUSE_MEM_READ | JAILHOUSE_MEM_WRITE |
>                               JAILHOUSE_MEM_IO,
>               },
> 

Thanks, applied.

I will send out patches for the config checker soon which help to find
such issues earlier.

Jan

-- 
Siemens AG, T RDA IOT
Corporate Competence Center Embedded Linux

-- 
You received this message because you are subscribed to the Google Groups 
"Jailhouse" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to [email protected].
To view this discussion on the web visit 
https://groups.google.com/d/msgid/jailhouse-dev/fec3ccfe-8776-74c6-3286-98db19fad8c8%40siemens.com.

Reply via email to