[igt-dev] [PATCH i-g-t] lib/intel_compute: Fix Incorrect array size for XEHPC platforms

[Zbigniew Kempczyński]

On Thu, Sep 21, 2023 at 02:16:32AM +0530, janga.rahul.kumar at intel.com wrote:
> From: Janga Rahul Kumar <janga.rahul.kumar at intel.com>
> 
> Correct the STATE_BASE_ADDRESS Instruction data size.
> 
> Signed-off-by: Janga Rahul Kumar <janga.rahul.kumar at intel.com>
> Cc: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>
> ---
>  lib/intel_compute.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/lib/intel_compute.c b/lib/intel_compute.c
> index 4f6716d23..0c30f39c1 100644
> --- a/lib/intel_compute.c
> +++ b/lib/intel_compute.c
> @@ -1030,7 +1030,7 @@ static void xehpc_compute_exec(int fd, const unsigned char *kernel,
>  			       unsigned int size)
>  {
>  #define XEHPC_BO_DICT_ENTRIES 6
> -	struct bo_dict_entry bo_dict[XEHP_BO_DICT_ENTRIES] = {
> +	struct bo_dict_entry bo_dict[XEHPC_BO_DICT_ENTRIES] = {

Eh, copy paste issue. Thanks for spotting this. Luckily for me
XEHP_BO_DICT_ENTRIES >= XEHP_BO_DICT_ENTRIES so nothing wrong happened.

Go ahead and merge:

Reviewed-by: Zbigniew Kempczyński <zbigniew.kempczynski at intel.com>

--
Zbigniew

>  		{ .addr = XEHP_ADDR_INSTRUCTION_STATE_BASE + OFFSET_KERNEL,
>  		  .name = "instr state base"},
>  		{ .addr = XEHP_ADDR_GENERAL_STATE_BASE + OFFSET_INDIRECT_DATA_START,
> -- 
> 2.25.1
>