Re: [XEN v1 2/9] xen/arm: Define translate_dt_address_size() for the translation between u64 and paddr_t

2023-01-04 Thread Stefano Stabellini
On Fri, 23 Dec 2022, Julien Grall wrote: > On 23/12/2022 10:01, Ayan Kumar Halder wrote: > > Hi Julien/Stefano, > > > > I want to make sure I understand correctly. > > > > On 22/12/2022 23:20, Stefano Stabellini wrote: > > > On Sat, 17 Dec 2022, Julien Grall wrote: > > > > On 17/12/2022 00:46,

Re: [XEN v1 2/9] xen/arm: Define translate_dt_address_size() for the translation between u64 and paddr_t

2022-12-23 Thread Julien Grall
On 23/12/2022 10:01, Ayan Kumar Halder wrote: Hi Julien/Stefano, I want to make sure I understand correctly. On 22/12/2022 23:20, Stefano Stabellini wrote: On Sat, 17 Dec 2022, Julien Grall wrote: On 17/12/2022 00:46, Stefano Stabellini wrote: On Fri, 16 Dec 2022, Julien Grall wrote: Hi

Re: [XEN v1 2/9] xen/arm: Define translate_dt_address_size() for the translation between u64 and paddr_t

2022-12-23 Thread Ayan Kumar Halder
Hi Julien/Stefano, I want to make sure I understand correctly. On 22/12/2022 23:20, Stefano Stabellini wrote: On Sat, 17 Dec 2022, Julien Grall wrote: On 17/12/2022 00:46, Stefano Stabellini wrote: On Fri, 16 Dec 2022, Julien Grall wrote: Hi Ayan, On 15/12/2022 19:32, Ayan Kumar Halder

Re: [XEN v1 2/9] xen/arm: Define translate_dt_address_size() for the translation between u64 and paddr_t

2022-12-22 Thread Stefano Stabellini
On Sat, 17 Dec 2022, Julien Grall wrote: > On 17/12/2022 00:46, Stefano Stabellini wrote: > > On Fri, 16 Dec 2022, Julien Grall wrote: > > > Hi Ayan, > > > > > > On 15/12/2022 19:32, Ayan Kumar Halder wrote: > > > > paddr_t may be u64 or u32 depending of the type of architecture. > > > > Thus,

Re: [XEN v1 2/9] xen/arm: Define translate_dt_address_size() for the translation between u64 and paddr_t

2022-12-17 Thread Julien Grall
Hi, On 17/12/2022 00:46, Stefano Stabellini wrote: On Fri, 16 Dec 2022, Julien Grall wrote: Hi Ayan, On 15/12/2022 19:32, Ayan Kumar Halder wrote: paddr_t may be u64 or u32 depending of the type of architecture. Thus, while translating between u64 and paddr_t, one should check that the

Re: [XEN v1 2/9] xen/arm: Define translate_dt_address_size() for the translation between u64 and paddr_t

2022-12-16 Thread Stefano Stabellini
On Fri, 16 Dec 2022, Julien Grall wrote: > Hi Ayan, > > On 15/12/2022 19:32, Ayan Kumar Halder wrote: > > paddr_t may be u64 or u32 depending of the type of architecture. > > Thus, while translating between u64 and paddr_t, one should check that the > > truncated bits are 0. If not, then raise an

Re: [XEN v1 2/9] xen/arm: Define translate_dt_address_size() for the translation between u64 and paddr_t

2022-12-16 Thread Julien Grall
Hi Ayan, On 15/12/2022 19:32, Ayan Kumar Halder wrote: paddr_t may be u64 or u32 depending of the type of architecture. Thus, while translating between u64 and paddr_t, one should check that the truncated bits are 0. If not, then raise an appropriate error. I am not entirely convinced this

[XEN v1 2/9] xen/arm: Define translate_dt_address_size() for the translation between u64 and paddr_t

2022-12-15 Thread Ayan Kumar Halder
paddr_t may be u64 or u32 depending of the type of architecture. Thus, while translating between u64 and paddr_t, one should check that the truncated bits are 0. If not, then raise an appropriate error. Signed-off-by: Ayan Kumar Halder --- xen/arch/arm/include/asm/platform.h | 26