drm/nouveau/bar: implement bar1 teardown
authorBen Skeggs <bskeggs@redhat.com>
Tue, 31 Oct 2017 17:56:19 +0000 (03:56 +1000)
committerBen Skeggs <bskeggs@redhat.com>
Thu, 2 Nov 2017 03:32:18 +0000 (13:32 +1000)
Will prevent spurious MMU fault interrupts if something decides to touch
BAR1 after we've unloaded the driver.

Signed-off-by: Ben Skeggs <bskeggs@redhat.com>
drivers/gpu/drm/nouveau/nvkm/subdev/bar/base.c
drivers/gpu/drm/nouveau/nvkm/subdev/bar/g84.c
drivers/gpu/drm/nouveau/nvkm/subdev/bar/gf100.c
drivers/gpu/drm/nouveau/nvkm/subdev/bar/gk20a.c
drivers/gpu/drm/nouveau/nvkm/subdev/bar/nv50.c
drivers/gpu/drm/nouveau/nvkm/subdev/bar/priv.h

index adaf91ee8326e126eea248ba9e23aa4d88666f17..0fee5e0a090e8a2dff7ba663df101f557f191e6e 100644 (file)
@@ -45,6 +45,14 @@ nvkm_bar_umap(struct nvkm_bar *bar, u64 size, int type, struct nvkm_vma *vma)
        return bar->func->umap(bar, size, type, vma);
 }
 
+static int
+nvkm_bar_fini(struct nvkm_subdev *subdev, bool suspend)
+{
+       struct nvkm_bar *bar = nvkm_bar(subdev);
+       bar->func->bar1.fini(bar);
+       return 0;
+}
+
 static int
 nvkm_bar_init(struct nvkm_subdev *subdev)
 {
@@ -74,6 +82,7 @@ nvkm_bar = {
        .dtor = nvkm_bar_dtor,
        .oneinit = nvkm_bar_oneinit,
        .init = nvkm_bar_init,
+       .fini = nvkm_bar_fini,
 };
 
 void
index dee1cd9987c2dc0d22046f78315da52d8a507011..0b63f224fa26f6c00f7426c03e74ac59d685f47f 100644 (file)
@@ -45,6 +45,7 @@ g84_bar_func = {
        .oneinit = nv50_bar_oneinit,
        .init = nv50_bar_init,
        .bar1.init = nv50_bar_bar1_init,
+       .bar1.fini = nv50_bar_bar1_fini,
        .bar1.wait = nv50_bar_bar1_wait,
        .kmap = nv50_bar_kmap,
        .umap = nv50_bar_umap,
index fc3d771cef508ffe1b314c3cca6698a8bb37f4fa..fb57c0175e575656c43410aa6910e35f738cd246 100644 (file)
@@ -49,6 +49,12 @@ gf100_bar_bar1_wait(struct nvkm_bar *base)
        nvkm_bar_flush(base);
 }
 
+void
+gf100_bar_bar1_fini(struct nvkm_bar *bar)
+{
+       nvkm_mask(bar->subdev.device, 0x001704, 0x80000000, 0x00000000);
+}
+
 void
 gf100_bar_bar1_init(struct nvkm_bar *base)
 {
@@ -186,6 +192,7 @@ gf100_bar_func = {
        .oneinit = gf100_bar_oneinit,
        .init = gf100_bar_init,
        .bar1.init = gf100_bar_bar1_init,
+       .bar1.fini = gf100_bar_bar1_fini,
        .bar1.wait = gf100_bar_bar1_wait,
        .kmap = gf100_bar_kmap,
        .umap = gf100_bar_umap,
index 86aca93e76e804c4639af40a1c6f445624cb6276..ab4664b4b2b2382229cd16efae4e9457bbccdd95 100644 (file)
@@ -26,6 +26,7 @@ gk20a_bar_func = {
        .dtor = gf100_bar_dtor,
        .oneinit = gf100_bar_oneinit,
        .bar1.init = gf100_bar_bar1_init,
+       .bar1.fini = gf100_bar_bar1_fini,
        .bar1.wait = gf100_bar_bar1_wait,
        .umap = gf100_bar_umap,
        .flush = g84_bar_flush,
index 8cb8e721a5152174c995af1a8bac3ccfb0aa3e96..c9776121a076d20efcf9a18cc42e32e0a8a0b3ec 100644 (file)
@@ -62,6 +62,12 @@ nv50_bar_bar1_wait(struct nvkm_bar *base)
        nvkm_bar_flush(base);
 }
 
+void
+nv50_bar_bar1_fini(struct nvkm_bar *bar)
+{
+       nvkm_wr32(bar->subdev.device, 0x001708, 0x00000000);
+}
+
 void
 nv50_bar_bar1_init(struct nvkm_bar *base)
 {
@@ -208,6 +214,7 @@ nv50_bar_func = {
        .oneinit = nv50_bar_oneinit,
        .init = nv50_bar_init,
        .bar1.init = nv50_bar_bar1_init,
+       .bar1.fini = nv50_bar_bar1_fini,
        .bar1.wait = nv50_bar_bar1_wait,
        .kmap = nv50_bar_kmap,
        .umap = nv50_bar_umap,
index 8b508373769f8746acc6b42847563c36fd423702..d130aab01aca7280adf43b9c86712d2d85aa8284 100644 (file)
@@ -13,6 +13,7 @@ struct nvkm_bar_func {
 
        struct {
                void (*init)(struct nvkm_bar *);
+               void (*fini)(struct nvkm_bar *);
                void (*wait)(struct nvkm_bar *);
        } bar1;
 
@@ -21,5 +22,9 @@ struct nvkm_bar_func {
        void (*flush)(struct nvkm_bar *);
 };
 
+void nv50_bar_bar1_fini(struct nvkm_bar *);
+
 void g84_bar_flush(struct nvkm_bar *);
+
+void gf100_bar_bar1_fini(struct nvkm_bar *);
 #endif