Skip to content

Commit

Permalink
drm/nouveau: init vblank requests list
Browse files Browse the repository at this point in the history
Fixes kernel panic when vblank interrupt triggers before first sync to
vblank request.

(Besides init, remove some relevant leftovers from vblank rework)

Reported-by: Ortwin Glück <odi@odi.ch>
Signed-off-by: Marcin Slusarz <marcin.slusarz@gmail.com>
Cc: stable@vger.kernel.org [3.5]
Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
  • Loading branch information
Marcin Slusarz authored and Ben Skeggs committed Jul 26, 2012
1 parent 0ade74b commit 7158554
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 6 deletions.
2 changes: 0 additions & 2 deletions drivers/gpu/drm/nouveau/nouveau_drv.h
Original file line number Diff line number Diff line change
Expand Up @@ -689,8 +689,6 @@ struct drm_nouveau_private {
void (*irq_handler[32])(struct drm_device *);
bool msi_enabled;

struct list_head vbl_waiting;

struct {
struct drm_global_reference mem_global_ref;
struct ttm_bo_global_ref bo_global_ref;
Expand Down
4 changes: 0 additions & 4 deletions drivers/gpu/drm/nouveau/nouveau_irq.c
Original file line number Diff line number Diff line change
Expand Up @@ -41,12 +41,8 @@
void
nouveau_irq_preinstall(struct drm_device *dev)
{
struct drm_nouveau_private *dev_priv = dev->dev_private;

/* Master disable */
nv_wr32(dev, NV03_PMC_INTR_EN_0, 0);

INIT_LIST_HEAD(&dev_priv->vbl_waiting);
}

int
Expand Down
1 change: 1 addition & 0 deletions drivers/gpu/drm/nouveau/nouveau_software.h
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,7 @@ static inline void
nouveau_software_context_new(struct nouveau_software_chan *pch)
{
INIT_LIST_HEAD(&pch->flip);
INIT_LIST_HEAD(&pch->vblank.list);
}

static inline void
Expand Down

0 comments on commit 7158554

Please sign in to comment.