Re: [PATCH i-g-t 08/11] lib/intel_compute: Relocate input and output objects

["Thomas Hellström" <thomas.hellstrom@linux.intel.com>]

On Tue, 2025-03-11 at 16:21 +0100, Francois Dugast wrote:
> Move those objects after other objects and add space between them to
> make it possible to increase their dimension without overlapping
> between input and output.
> 
> Signed-off-by: Francois Dugast <francois.dugast@intel.com>
Reviewed-by: Thomas Hellström <thomas.hellstrom@linux.intel.com>

> ---
>  lib/intel_compute.c | 20 ++++++++++----------
>  1 file changed, 10 insertions(+), 10 deletions(-)
> 
> diff --git a/lib/intel_compute.c b/lib/intel_compute.c
> index 364388e20..7a2e04b10 100644
> --- a/lib/intel_compute.c
> +++ b/lib/intel_compute.c
> @@ -37,20 +37,20 @@
>  #define ADDR_SYNC			0x010000ULL
>  #define ADDR_SYNC2			0x020000ULL
>  #define ADDR_BATCH			0x100000ULL
> -#define ADDR_INPUT			0x200000ULL
> -#define ADDR_OUTPUT			0x300000ULL
> -#define ADDR_SURFACE_STATE_BASE		0x400000ULL
> -#define ADDR_DYNAMIC_STATE_BASE		0x500000ULL
> -#define ADDR_INDIRECT_OBJECT_BASE	0x600000ULL
> +#define ADDR_INPUT			0x40000000ULL
> +#define ADDR_OUTPUT			0x80000000ULL
> +#define ADDR_SURFACE_STATE_BASE		0x200000ULL
> +#define ADDR_DYNAMIC_STATE_BASE		0x300000ULL
> +#define ADDR_INDIRECT_OBJECT_BASE	0x400000ULL
>  #define ADDR_BINDING_TABLE		(ADDR_SURFACE_STATE_BASE +
> OFFSET_BINDING_TABLE)
> -#define OFFSET_INDIRECT_DATA_START	0x5D0000ULL
> -#define OFFSET_KERNEL			0x5E0000ULL
> +#define OFFSET_INDIRECT_DATA_START	0x3D0000ULL
> +#define OFFSET_KERNEL			0x3E0000ULL
>  
> -#define ADDR_GENERAL_STATE_BASE		0x8000000ULL
> -#define ADDR_INSTRUCTION_STATE_BASE	0xa000000ULL
> +#define ADDR_GENERAL_STATE_BASE		0x6000000ULL
> +#define ADDR_INSTRUCTION_STATE_BASE	0x8000000ULL
>  #define OFFSET_BINDING_TABLE		0x10000
>  
> -#define XE2_ADDR_STATE_CONTEXT_DATA_BASE	0xb000000ULL
> +#define XE2_ADDR_STATE_CONTEXT_DATA_BASE	0x9000000ULL
>  #define OFFSET_STATE_SIP			0xFFFF0000
>  
>  #define
> USER_FENCE_VALUE			0xdeadbeefdeadbeefull