* possible event_space corruption with ipp_get_event()?
@ 2014-11-26 17:05 Rob Clark
2014-11-27 8:32 ` Andrzej Hajda
0 siblings, 1 reply; 2+ messages in thread
From: Rob Clark @ 2014-11-26 17:05 UTC (permalink / raw)
To: daeinki; +Cc: dri-devel@lists.freedesktop.org
Hey Inki,
Could you or one of the exynos folks have a look at ipp_get_event()?
That handling of file_priv->event_space looks rather suspect. I don't
see where event_space is decremented in the first place. I'm not sure
if that is just broken, or if there is something subtle going on.. but
I guess it probably either needs fixing or a comment.
BR,
-R
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: possible event_space corruption with ipp_get_event()?
2014-11-26 17:05 possible event_space corruption with ipp_get_event()? Rob Clark
@ 2014-11-27 8:32 ` Andrzej Hajda
0 siblings, 0 replies; 2+ messages in thread
From: Andrzej Hajda @ 2014-11-27 8:32 UTC (permalink / raw)
To: Rob Clark, daeinki; +Cc: dri-devel@lists.freedesktop.org
On 11/26/2014 06:05 PM, Rob Clark wrote:
> Hey Inki,
>
> Could you or one of the exynos folks have a look at ipp_get_event()?
> That handling of file_priv->event_space looks rather suspect. I don't
> see where event_space is decremented in the first place. I'm not sure
> if that is just broken, or if there is something subtle going on.. but
> I guess it probably either needs fixing or a comment.
It is just broken. I have looked at it some time ago and I saw more
things to fix. I plan to prepare another ipp fix/improve patchset after
finishing other tasks.
Regards
Andrzej
>
> BR,
> -R
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel
>
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/dri-devel
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2014-11-27 8:32 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2014-11-26 17:05 possible event_space corruption with ipp_get_event()? Rob Clark
2014-11-27 8:32 ` Andrzej Hajda
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.