All of lore.kernel.org
 help / color / mirror / Atom feed
* 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.