On 05.01.21 11:19, Alice Guo (OSS) wrote:
> From: Peng Fan <[email protected]>
> 
> The inmate linux should have root shared flag set for ivshmem region
> 
> Signed-off-by: Peng Fan <[email protected]>
> ---
>  configs/arm64/imx8mm-linux-demo.c | 16 +++++++++-------
>  1 file changed, 9 insertions(+), 7 deletions(-)
> 
> diff --git a/configs/arm64/imx8mm-linux-demo.c 
> b/configs/arm64/imx8mm-linux-demo.c
> index a3e85c0a..196740cf 100644
> --- a/configs/arm64/imx8mm-linux-demo.c
> +++ b/configs/arm64/imx8mm-linux-demo.c
> @@ -47,34 +47,36 @@ struct {
>                       .phys_start = 0xbba00000,
>                       .virt_start = 0xbba00000,
>                       .size = 0x1000,
> -                     .flags = JAILHOUSE_MEM_READ ,
> +                     .flags = JAILHOUSE_MEM_READ | JAILHOUSE_MEM_ROOTSHARED,
>               },
>               {
>                       .phys_start = 0xbba01000,
>                       .virt_start = 0xbba01000,
>                       .size = 0x9000,
> -                     .flags = JAILHOUSE_MEM_READ | JAILHOUSE_MEM_WRITE ,
> +                     .flags = JAILHOUSE_MEM_READ | JAILHOUSE_MEM_WRITE |
> +                             JAILHOUSE_MEM_ROOTSHARED,
>               },
>               {
>                       .phys_start = 0xbba0a000,
>                       .virt_start = 0xbba0a000,
>                       .size = 0x2000,
> -                     .flags = JAILHOUSE_MEM_READ | JAILHOUSE_MEM_WRITE ,
> +                     .flags = JAILHOUSE_MEM_READ | JAILHOUSE_MEM_ROOTSHARED,
>               },
>               {
>                       .phys_start = 0xbba0c000,
>                       .virt_start = 0xbba0c000,
>                       .size = 0x2000,
> -                     .flags = JAILHOUSE_MEM_READ,
> +                     .flags = JAILHOUSE_MEM_READ | JAILHOUSE_MEM_ROOTSHARED,
>               },
>               {
>                       .phys_start = 0xbba0e000,
>                       .virt_start = 0xbba0e000,
>                       .size = 0x2000,
> -                     .flags = JAILHOUSE_MEM_READ,
> +                     .flags = JAILHOUSE_MEM_READ | JAILHOUSE_MEM_WRITE |
> +                             JAILHOUSE_MEM_ROOTSHARED,
>               },
>               /* IVSHMEM shared memory regions for 00:01.0 (networking) */
> -             JAILHOUSE_SHMEM_NET_REGIONS(0xbbb00000, 0),
> +             JAILHOUSE_SHMEM_NET_REGIONS(0xbbb00000, 1),
>               /* UART2 earlycon */ {
>                       .phys_start = 0x30890000,
>                       .virt_start = 0x30890000,
> @@ -143,7 +145,7 @@ struct {
>       .pci_devices = {
>               { /* IVSHMEM 00:00.0 (demo) */
>                       .type = JAILHOUSE_PCI_TYPE_IVSHMEM,
> -                     .domain = 1,
> +                     .domain = 0,
>                       .bdf = 0 << 3,
>                       .bar_mask = JAILHOUSE_IVSHMEM_BAR_MASK_INTX,
>                       .shmem_regions_start = 0,
> 

Thanks, applied.

We should add consistency tests to "jailhouse config check" for ivshmem
as well...

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/e72cc5a5-c637-4584-21f8-887740ead880%40siemens.com.

Reply via email to