On 2014-08-04 02:28, Inki Dae wrote:
> Oops, sorry. I didn't check v2.
No problem :)
It would be good if you could also check the libdrm patches that I've
resent some while ago.
With best wishes,
Tobias
On 2014? 08? 03? 18:52, Tobias Jakobi wrote:
> Inki Dae wrote:
>> It seems that you have no build test because above line incurs build
>> error. Anyway I fixed it up.
> That's why I have sent v2 of the patch.
>
Oops, sorry. I didn't check v2.
Thanks,
Inki Dae
> With best wishes,
> Tobias
>
>
2014-07-23 23:57 GMT+09:00 Tobias Jakobi :
> Both exynos_g2d_set_cmdlist_ioctl and exynos_g2d_exec_ioctl don't check
> if the G2D was succesfully probe. If that is not the case, then g2d_priv
> is just NULL and extracting 'dev' from it in the next step is going to
> produce a kernel oops.
>
> Add
Inki Dae wrote:
> It seems that you have no build test because above line incurs build
> error. Anyway I fixed it up.
That's why I have sent v2 of the patch.
With best wishes,
Tobias
Both exynos_g2d_set_cmdlist_ioctl and exynos_g2d_exec_ioctl don't check
if the G2D was succesfully probe. If that is not the case, then g2d_priv
is just NULL and extracting 'dev' from it in the next step is going to
produce a kernel oops.
Add proper checks and return ENODEV if the G2D is not