Re: [PATCH] drm/radeon/kms/ni: fix packet2 handling for VM IB parser

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Thu, Jan 12, 2012 at 03:42:37PM -0500, alexdeucher@xxxxxxxxx wrote:
> From: Alex Deucher <alexander.deucher@xxxxxxx>
> 
> Packet2 is only one dword.
> 
> Signed-off-by: Alex Deucher <alexander.deucher@xxxxxxx>
Reviewed-by: Jerome Glisse <jglisse@xxxxxxxxxx>
> ---
>  drivers/gpu/drm/radeon/evergreen_cs.c |    3 ++-
>  1 files changed, 2 insertions(+), 1 deletions(-)
> 
> diff --git a/drivers/gpu/drm/radeon/evergreen_cs.c b/drivers/gpu/drm/radeon/evergreen_cs.c
> index f7442e6..8e8cd85 100644
> --- a/drivers/gpu/drm/radeon/evergreen_cs.c
> +++ b/drivers/gpu/drm/radeon/evergreen_cs.c
> @@ -1793,10 +1793,12 @@ int evergreen_ib_parse(struct radeon_device *rdev, struct radeon_ib *ib)
>  			ret = -EINVAL;
>  			break;
>  		case PACKET_TYPE2:
> +			idx += 1;
>  			break;
>  		case PACKET_TYPE3:
>  			pkt.opcode = CP_PACKET3_GET_OPCODE(ib->ptr[idx]);
>  			ret = evergreen_vm_packet3_check(rdev, ib->ptr, &pkt);
> +			idx += pkt.count + 2;
>  			break;
>  		default:
>  			dev_err(rdev->dev, "Unknown packet type %d !\n", pkt.type);
> @@ -1805,7 +1807,6 @@ int evergreen_ib_parse(struct radeon_device *rdev, struct radeon_ib *ib)
>  		}
>  		if (ret)
>  			break;
> -		idx += pkt.count + 2;
>  	} while (idx < ib->length_dw);
>  
>  	return ret;
> -- 
> 1.7.3.4
> 
> _______________________________________________
> dri-devel mailing list
> dri-devel@xxxxxxxxxxxxxxxxxxxxx
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
_______________________________________________
dri-devel mailing list
dri-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/dri-devel


[Index of Archives]     [Linux DRI Users]     [Linux Intel Graphics]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux