Re: [PATCH] libxl: arm: Allow grant mappings for backends running on Dom0

2023-05-11 Thread Viresh Kumar
On 09-05-23, 16:05, Anthony PERARD wrote: > I guess the way virtio devices are implemented in libxl suggest to me > that the are just Xen PV devices. So I guess some documentation in the > tree would be useful, maybe some comments in libxl_virtio.c. Our use case is just PV devices, not sure if som

Re: [PATCH] libxl: arm: Allow grant mappings for backends running on Dom0

2023-05-09 Thread Anthony PERARD
On Fri, May 05, 2023 at 03:08:35PM +0530, Viresh Kumar wrote: > Hi Anthony, > > On 02-05-23, 15:44, Anthony PERARD wrote: > > > diff --git a/tools/libs/light/libxl_virtio.c > > > b/tools/libs/light/libxl_virtio.c > > > index faada49e184e..e1f15344ef97 100644 > > > --- a/tools/libs/light/libxl_vir

Re: [PATCH] libxl: arm: Allow grant mappings for backends running on Dom0

2023-05-08 Thread Juergen Gross
On 08.05.23 06:09, Viresh Kumar wrote: On 05-05-23, 16:11, Oleksandr Tyshchenko wrote: I was going to propose an idea, but I have just realized that you already voiced it here [1] )) So what you proposed there sounds reasonable to me. I will just rephrase it according to my understanding: We p

Re: [PATCH] libxl: arm: Allow grant mappings for backends running on Dom0

2023-05-07 Thread Viresh Kumar
On 05-05-23, 16:11, Oleksandr Tyshchenko wrote: > I was going to propose an idea, but I have just realized that you already > voiced it here [1] )) > So what you proposed there sounds reasonable to me. > > I will just rephrase it according to my understanding: > > We probably need to consider tra

Re: [PATCH] libxl: arm: Allow grant mappings for backends running on Dom0

2023-05-05 Thread Oleksandr Tyshchenko
Hello Viresh [sorry for the possible format issues] On Fri, May 5, 2023 at 9:19 AM Viresh Kumar wrote: > On 05-04-23, 05:12, Viresh Kumar wrote: > > On 04-04-23, 21:16, Oleksandr Tyshchenko wrote: > > > ok, probably makes sense > > > > While testing both foreign and grant mappings I stumbled up

Re: [PATCH] libxl: arm: Allow grant mappings for backends running on Dom0

2023-05-05 Thread Viresh Kumar
Hi Anthony, On 02-05-23, 15:44, Anthony PERARD wrote: > > diff --git a/tools/libs/light/libxl_virtio.c > > b/tools/libs/light/libxl_virtio.c > > index faada49e184e..e1f15344ef97 100644 > > --- a/tools/libs/light/libxl_virtio.c > > +++ b/tools/libs/light/libxl_virtio.c > > @@ -48,11 +48,13 @@ stat

Re: [PATCH] libxl: arm: Allow grant mappings for backends running on Dom0

2023-05-04 Thread Viresh Kumar
On 05-04-23, 05:12, Viresh Kumar wrote: > On 04-04-23, 21:16, Oleksandr Tyshchenko wrote: > > ok, probably makes sense > > While testing both foreign and grant mappings I stumbled upon another > related problem. How do I control the creation of iommu node from > guest configuration file, irrespect

Re: [PATCH] libxl: arm: Allow grant mappings for backends running on Dom0

2023-05-02 Thread Anthony PERARD
On Thu, Mar 30, 2023 at 02:13:08PM +0530, Viresh Kumar wrote: > diff --git a/docs/man/xl.cfg.5.pod.in b/docs/man/xl.cfg.5.pod.in > index 10f37990be57..4879f136aab8 100644 > --- a/docs/man/xl.cfg.5.pod.in > +++ b/docs/man/xl.cfg.5.pod.in > @@ -1616,6 +1616,10 @@ properties in the Device Tree, the ty

Re: [PATCH] libxl: arm: Allow grant mappings for backends running on Dom0

2023-04-04 Thread Viresh Kumar
On 04-04-23, 21:16, Oleksandr Tyshchenko wrote: > ok, probably makes sense While testing both foreign and grant mappings I stumbled upon another related problem. How do I control the creation of iommu node from guest configuration file, irrespective of the domain backend is running at ? This is wh

Re: [PATCH] libxl: arm: Allow grant mappings for backends running on Dom0

2023-04-04 Thread Oleksandr Tyshchenko
On 30.03.23 11:43, Viresh Kumar wrote: Hello Viresh Currently, we add grant mapping related device tree properties if the backend domain is not Dom0. While Dom0 is privileged and can do foreign mapping for the entire guest memory, it is still okay for Dom0 to access guest's memory via grant

[PATCH] libxl: arm: Allow grant mappings for backends running on Dom0

2023-03-30 Thread Viresh Kumar
Currently, we add grant mapping related device tree properties if the backend domain is not Dom0. While Dom0 is privileged and can do foreign mapping for the entire guest memory, it is still okay for Dom0 to access guest's memory via grant mappings and hence map only what is required. This commit