Re: [PATCH 9/9] iosys_map: embed the is_iomem bit into the pointer.

2025-05-26 Thread Thomas Zimmermann
Hi Am 26.05.25 um 09:58 schrieb Dave Airlie: On Mon, 26 May 2025 at 16:39, Thomas Zimmermann wrote: Hi Am 22.05.25 um 17:09 schrieb Lucas De Marchi: On Thu, May 22, 2025 at 04:52:18PM +1000, Dave Airlie wrote: From: Dave Airlie This reduces this struct from 16 to 8 bytes, and it gets embe

Re: [PATCH 9/9] iosys_map: embed the is_iomem bit into the pointer.

2025-05-26 Thread Dave Airlie
On Mon, 26 May 2025 at 16:39, Thomas Zimmermann wrote: > > Hi > > Am 22.05.25 um 17:09 schrieb Lucas De Marchi: > > On Thu, May 22, 2025 at 04:52:18PM +1000, Dave Airlie wrote: > >> From: Dave Airlie > >> > >> This reduces this struct from 16 to 8 bytes, and it gets embedded > >> into a lot of th

Re: [PATCH 9/9] iosys_map: embed the is_iomem bit into the pointer.

2025-05-25 Thread Thomas Zimmermann
Hi Am 22.05.25 um 17:09 schrieb Lucas De Marchi: On Thu, May 22, 2025 at 04:52:18PM +1000, Dave Airlie wrote: From: Dave Airlie This reduces this struct from 16 to 8 bytes, and it gets embedded into a lot of things. Signed-off-by: Dave Airlie Replied too early on cover. Chatting with Mich

Re: [PATCH 9/9] iosys_map: embed the is_iomem bit into the pointer.

2025-05-23 Thread Jocelyn Falempe
On 22/05/2025 23:05, Lucas De Marchi wrote: On Fri, May 23, 2025 at 06:32:43AM +1000, Dave Airlie wrote: On Fri, 23 May 2025 at 01:10, Lucas De Marchi wrote: On Thu, May 22, 2025 at 04:52:18PM +1000, Dave Airlie wrote: >From: Dave Airlie > >This reduces this struct from 16 to 8 bytes, and it

Re: [PATCH 9/9] iosys_map: embed the is_iomem bit into the pointer.

2025-05-22 Thread Lucas De Marchi
On Fri, May 23, 2025 at 06:32:43AM +1000, Dave Airlie wrote: On Fri, 23 May 2025 at 01:10, Lucas De Marchi wrote: On Thu, May 22, 2025 at 04:52:18PM +1000, Dave Airlie wrote: >From: Dave Airlie > >This reduces this struct from 16 to 8 bytes, and it gets embedded >into a lot of things. > >Sign

Re: [PATCH 9/9] iosys_map: embed the is_iomem bit into the pointer.

2025-05-22 Thread Dave Airlie
On Fri, 23 May 2025 at 01:10, Lucas De Marchi wrote: > > On Thu, May 22, 2025 at 04:52:18PM +1000, Dave Airlie wrote: > >From: Dave Airlie > > > >This reduces this struct from 16 to 8 bytes, and it gets embedded > >into a lot of things. > > > >Signed-off-by: Dave Airlie > > Replied too early on

Re: [PATCH 9/9] iosys_map: embed the is_iomem bit into the pointer.

2025-05-22 Thread Lucas De Marchi
On Thu, May 22, 2025 at 04:52:18PM +1000, Dave Airlie wrote: From: Dave Airlie This reduces this struct from 16 to 8 bytes, and it gets embedded into a lot of things. Signed-off-by: Dave Airlie Replied too early on cover. Chatting with Michal Wajdeczko today, this may break things as we the

Re: [PATCH 9/9] iosys_map: embed the is_iomem bit into the pointer.

2025-05-22 Thread Thomas Zimmermann
Hi Am 22.05.25 um 08:52 schrieb Dave Airlie: From: Dave Airlie This reduces this struct from 16 to 8 bytes, and it gets embedded into a lot of things. IIRC this has been discussed before. Makes sense to me. Signed-off-by: Dave Airlie --- include/linux/iosys-map.h | 30 --

Re: [PATCH 9/9] iosys_map: embed the is_iomem bit into the pointer.

2025-05-22 Thread Jani Nikula
On Thu, 22 May 2025, Dave Airlie wrote: > From: Dave Airlie > > This reduces this struct from 16 to 8 bytes, and it gets embedded > into a lot of things. It's an abomination. And i915 has helpers for it. Because of course it does. i915_utils.h contains ptr_mask_bits(), ptr_unmask_bits(), ptr_p

[PATCH 9/9] iosys_map: embed the is_iomem bit into the pointer.

2025-05-21 Thread Dave Airlie
From: Dave Airlie This reduces this struct from 16 to 8 bytes, and it gets embedded into a lot of things. Signed-off-by: Dave Airlie --- include/linux/iosys-map.h | 30 -- 1 file changed, 8 insertions(+), 22 deletions(-) diff --git a/include/linux/iosys-map.h b/inc