From: <bug...@bu...> - 2009-03-22 13:55:39
|
http://bugzilla.kernel.org/show_bug.cgi?id=12899 ------- Comment #2 from hel...@se... 2009-03-22 06:38 ------- Are you sure this is sane? I mean, i915_gem_cleanup_hws sets dev_priv->hw_status_page=NULL without any locks held (that I am aware of, at least), and the irq handler is accessing the value without any locks either. The check for dev_priv->hw_status_page!=0 added into the irq handler IMHO just "shortens" the race window to the few instructions between performing the test and the subsequent READ_BREADCRUMB and thus "hides" the problem better. Is there any guard between i915_gem_cleanup_hws and the irq handler racing which I am missing so far? -- Configure bugmail: http://bugzilla.kernel.org/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug, or are watching the assignee. |