提交 e28f639e 编写于 作者: D Dave Airlie

Merge branch 'drm-nouveau-fixes-3.8' of...

Merge branch 'drm-nouveau-fixes-3.8' of git://anongit.freedesktop.org/git/nouveau/linux-2.6 into drm-next

Fixes for one major lockdep warning, one oops reported by a few people, and
fix for a long hang on some gpu engines.

* 'drm-nouveau-fixes-3.8' of git://anongit.freedesktop.org/git/nouveau/linux-2.6:
  drm/nouveau: add lockdep annotations
  drm/nv50/fb: Fix nullptr-deref on IGPs
  drm/nouveau: use different register to wait for secret scrubber
...@@ -73,8 +73,11 @@ _nouveau_falcon_init(struct nouveau_object *object) ...@@ -73,8 +73,11 @@ _nouveau_falcon_init(struct nouveau_object *object)
nv_debug(falcon, "data limit: %d\n", falcon->data.limit); nv_debug(falcon, "data limit: %d\n", falcon->data.limit);
/* wait for 'uc halted' to be signalled before continuing */ /* wait for 'uc halted' to be signalled before continuing */
if (falcon->secret) { if (falcon->secret && falcon->version < 4) {
nv_wait(falcon, 0x008, 0x00000010, 0x00000010); if (!falcon->version)
nv_wait(falcon, 0x008, 0x00000010, 0x00000010);
else
nv_wait(falcon, 0x180, 0x80000000, 0);
nv_wo32(falcon, 0x004, 0x00000010); nv_wo32(falcon, 0x004, 0x00000010);
} }
......
...@@ -99,7 +99,7 @@ nouveau_subdev_create_(struct nouveau_object *parent, ...@@ -99,7 +99,7 @@ nouveau_subdev_create_(struct nouveau_object *parent,
if (ret) if (ret)
return ret; return ret;
mutex_init(&subdev->mutex); __mutex_init(&subdev->mutex, subname, &oclass->lock_class_key);
subdev->name = subname; subdev->name = subname;
if (parent) { if (parent) {
......
...@@ -50,10 +50,13 @@ int nouveau_object_fini(struct nouveau_object *, bool suspend); ...@@ -50,10 +50,13 @@ int nouveau_object_fini(struct nouveau_object *, bool suspend);
extern struct nouveau_ofuncs nouveau_object_ofuncs; extern struct nouveau_ofuncs nouveau_object_ofuncs;
/* Don't allocate dynamically, because lockdep needs lock_class_keys to be in
* ".data". */
struct nouveau_oclass { struct nouveau_oclass {
u32 handle; u32 handle;
struct nouveau_ofuncs *ofuncs; struct nouveau_ofuncs * const ofuncs;
struct nouveau_omthds *omthds; struct nouveau_omthds * const omthds;
struct lock_class_key lock_class_key;
}; };
#define nv_oclass(o) nv_object(o)->oclass #define nv_oclass(o) nv_object(o)->oclass
......
...@@ -86,8 +86,8 @@ nouveau_fb_preinit(struct nouveau_fb *pfb) ...@@ -86,8 +86,8 @@ nouveau_fb_preinit(struct nouveau_fb *pfb)
return ret; return ret;
} }
if (!nouveau_mm_initialised(&pfb->tags) && tags) { if (!nouveau_mm_initialised(&pfb->tags)) {
ret = nouveau_mm_init(&pfb->tags, 0, ++tags, 1); ret = nouveau_mm_init(&pfb->tags, 0, tags ? ++tags : 0, 1);
if (ret) if (ret)
return ret; return ret;
} }
......
...@@ -99,7 +99,7 @@ nv50_fb_vram_init(struct nouveau_fb *pfb) ...@@ -99,7 +99,7 @@ nv50_fb_vram_init(struct nouveau_fb *pfb)
struct nouveau_bios *bios = nouveau_bios(device); struct nouveau_bios *bios = nouveau_bios(device);
const u32 rsvd_head = ( 256 * 1024) >> 12; /* vga memory */ const u32 rsvd_head = ( 256 * 1024) >> 12; /* vga memory */
const u32 rsvd_tail = (1024 * 1024) >> 12; /* vbios etc */ const u32 rsvd_tail = (1024 * 1024) >> 12; /* vbios etc */
u32 size; u32 size, tags = 0;
int ret; int ret;
pfb->ram.size = nv_rd32(pfb, 0x10020c); pfb->ram.size = nv_rd32(pfb, 0x10020c);
...@@ -140,10 +140,11 @@ nv50_fb_vram_init(struct nouveau_fb *pfb) ...@@ -140,10 +140,11 @@ nv50_fb_vram_init(struct nouveau_fb *pfb)
return ret; return ret;
pfb->ram.ranks = (nv_rd32(pfb, 0x100200) & 0x4) ? 2 : 1; pfb->ram.ranks = (nv_rd32(pfb, 0x100200) & 0x4) ? 2 : 1;
tags = nv_rd32(pfb, 0x100320);
break; break;
} }
return nv_rd32(pfb, 0x100320); return tags;
} }
static int static int
......
...@@ -245,6 +245,8 @@ static int nouveau_drm_probe(struct pci_dev *pdev, ...@@ -245,6 +245,8 @@ static int nouveau_drm_probe(struct pci_dev *pdev,
return 0; return 0;
} }
static struct lock_class_key drm_client_lock_class_key;
static int static int
nouveau_drm_load(struct drm_device *dev, unsigned long flags) nouveau_drm_load(struct drm_device *dev, unsigned long flags)
{ {
...@@ -256,6 +258,7 @@ nouveau_drm_load(struct drm_device *dev, unsigned long flags) ...@@ -256,6 +258,7 @@ nouveau_drm_load(struct drm_device *dev, unsigned long flags)
ret = nouveau_cli_create(pdev, "DRM", sizeof(*drm), (void**)&drm); ret = nouveau_cli_create(pdev, "DRM", sizeof(*drm), (void**)&drm);
if (ret) if (ret)
return ret; return ret;
lockdep_set_class(&drm->client.mutex, &drm_client_lock_class_key);
dev->dev_private = drm; dev->dev_private = drm;
drm->dev = dev; drm->dev = dev;
......
Markdown is supported
0% .
You are about to add 0 people to the discussion. Proceed with caution.
先完成此消息的编辑!
想要评论请 注册