(In reply to comment #20)
> The size checks should probably be done earlier, to save superfluous function
> calls if they fail. In particular, I suspect that calling RADEONCPGetBuffer()
> but then still failing could cause the stability issues some people on IRC
> encountered with this patch.
>
yeah, I was actually just thinking the same thing.
Created an attachment (id=21387)
updated patch
(In reply to comment #20)
> The size checks should probably be done earlier, to save superfluous function
> calls if they fail. In particular, I suspect that calling RADEONCPGetBuffer()
> but then still failing could cause the stability issues some people on IRC
> encountered with this patch.
>
yeah, I was actually just thinking the same thing.