All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH 1/2] drm/nouveau/bar: add noncached ioremap property
@ 2014-06-27 10:28 ` Alexandre Courbot
  0 siblings, 0 replies; 15+ messages in thread
From: Alexandre Courbot @ 2014-06-27 10:28 UTC (permalink / raw)
  To: Ben Skeggs
  Cc: nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
	dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
	linux-tegra-u79uwXL29TY76Z2rM5mHXA,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	gnurou-Re5JQEeQqe8AvxtiuMwx3w, Alexandre Courbot

Some BARs (like GK20A's) do not support being ioremapped write-combined.
Add a boolean property to the BAR structure and handle that case in the
Nouveau BO implementation.

Signed-off-by: Alexandre Courbot <acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
---
 drivers/gpu/drm/nouveau/core/include/subdev/bar.h |  3 +++
 drivers/gpu/drm/nouveau/nouveau_bo.c              | 17 ++++++++++++-----
 2 files changed, 15 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
index 9faa98e67ad8..9002cbb6432b 100644
--- a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
+++ b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
@@ -20,6 +20,9 @@ struct nouveau_bar {
 		    u32 flags, struct nouveau_vma *);
 	void (*unmap)(struct nouveau_bar *, struct nouveau_vma *);
 	void (*flush)(struct nouveau_bar *);
+
+	/* whether the BAR supports to be ioremapped WC or should be uncached */
+	bool iomap_uncached;
 };
 
 static inline struct nouveau_bar *
diff --git a/drivers/gpu/drm/nouveau/nouveau_bo.c b/drivers/gpu/drm/nouveau/nouveau_bo.c
index b6dc85c614be..4db886f9f793 100644
--- a/drivers/gpu/drm/nouveau/nouveau_bo.c
+++ b/drivers/gpu/drm/nouveau/nouveau_bo.c
@@ -500,18 +500,25 @@ nouveau_bo_init_mem_type(struct ttm_bo_device *bdev, uint32_t type,
 		man->default_caching = TTM_PL_FLAG_CACHED;
 		break;
 	case TTM_PL_VRAM:
+		man->flags = TTM_MEMTYPE_FLAG_FIXED |
+			     TTM_MEMTYPE_FLAG_MAPPABLE;
+		man->available_caching = TTM_PL_FLAG_UNCACHED |
+					 TTM_PL_FLAG_WC;
+		man->default_caching = TTM_PL_FLAG_WC;
+
 		if (nv_device(drm->device)->card_type >= NV_50) {
+			/* Some BARs do not support being ioremapped WC */
+			if (nouveau_bar(drm->device)->iomap_uncached) {
+				man->available_caching = TTM_PL_FLAG_UNCACHED;
+				man->default_caching = TTM_PL_FLAG_UNCACHED;
+			}
+
 			man->func = &nouveau_vram_manager;
 			man->io_reserve_fastpath = false;
 			man->use_io_reserve_lru = true;
 		} else {
 			man->func = &ttm_bo_manager_func;
 		}
-		man->flags = TTM_MEMTYPE_FLAG_FIXED |
-			     TTM_MEMTYPE_FLAG_MAPPABLE;
-		man->available_caching = TTM_PL_FLAG_UNCACHED |
-					 TTM_PL_FLAG_WC;
-		man->default_caching = TTM_PL_FLAG_WC;
 		break;
 	case TTM_PL_TT:
 		if (nv_device(drm->device)->card_type >= NV_50)
-- 
2.0.0

^ permalink raw reply related	[flat|nested] 15+ messages in thread
* [PATCH 1/2] drm/nouveau/bar: add noncached ioremap property
@ 2014-06-27 10:28 ` Alexandre Courbot
  0 siblings, 0 replies; 15+ messages in thread
From: Alexandre Courbot @ 2014-06-27 10:28 UTC (permalink / raw)
  To: Ben Skeggs
  Cc: nouveau, dri-devel, linux-tegra, linux-kernel, gnurou, Alexandre Courbot

Some BARs (like GK20A's) do not support being ioremapped write-combined.
Add a boolean property to the BAR structure and handle that case in the
Nouveau BO implementation.

Signed-off-by: Alexandre Courbot <acourbot@nvidia.com>
---
 drivers/gpu/drm/nouveau/core/include/subdev/bar.h |  3 +++
 drivers/gpu/drm/nouveau/nouveau_bo.c              | 17 ++++++++++++-----
 2 files changed, 15 insertions(+), 5 deletions(-)

diff --git a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
index 9faa98e67ad8..9002cbb6432b 100644
--- a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
+++ b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
@@ -20,6 +20,9 @@ struct nouveau_bar {
 		    u32 flags, struct nouveau_vma *);
 	void (*unmap)(struct nouveau_bar *, struct nouveau_vma *);
 	void (*flush)(struct nouveau_bar *);
+
+	/* whether the BAR supports to be ioremapped WC or should be uncached */
+	bool iomap_uncached;
 };
 
 static inline struct nouveau_bar *
diff --git a/drivers/gpu/drm/nouveau/nouveau_bo.c b/drivers/gpu/drm/nouveau/nouveau_bo.c
index b6dc85c614be..4db886f9f793 100644
--- a/drivers/gpu/drm/nouveau/nouveau_bo.c
+++ b/drivers/gpu/drm/nouveau/nouveau_bo.c
@@ -500,18 +500,25 @@ nouveau_bo_init_mem_type(struct ttm_bo_device *bdev, uint32_t type,
 		man->default_caching = TTM_PL_FLAG_CACHED;
 		break;
 	case TTM_PL_VRAM:
+		man->flags = TTM_MEMTYPE_FLAG_FIXED |
+			     TTM_MEMTYPE_FLAG_MAPPABLE;
+		man->available_caching = TTM_PL_FLAG_UNCACHED |
+					 TTM_PL_FLAG_WC;
+		man->default_caching = TTM_PL_FLAG_WC;
+
 		if (nv_device(drm->device)->card_type >= NV_50) {
+			/* Some BARs do not support being ioremapped WC */
+			if (nouveau_bar(drm->device)->iomap_uncached) {
+				man->available_caching = TTM_PL_FLAG_UNCACHED;
+				man->default_caching = TTM_PL_FLAG_UNCACHED;
+			}
+
 			man->func = &nouveau_vram_manager;
 			man->io_reserve_fastpath = false;
 			man->use_io_reserve_lru = true;
 		} else {
 			man->func = &ttm_bo_manager_func;
 		}
-		man->flags = TTM_MEMTYPE_FLAG_FIXED |
-			     TTM_MEMTYPE_FLAG_MAPPABLE;
-		man->available_caching = TTM_PL_FLAG_UNCACHED |
-					 TTM_PL_FLAG_WC;
-		man->default_caching = TTM_PL_FLAG_WC;
 		break;
 	case TTM_PL_TT:
 		if (nv_device(drm->device)->card_type >= NV_50)
-- 
2.0.0


^ permalink raw reply related	[flat|nested] 15+ messages in thread
* [PATCH 2/2] drm/gk20a: add BAR instance
  2014-06-27 10:28 ` Alexandre Courbot
@ 2014-06-27 10:28     ` Alexandre Courbot
  -1 siblings, 0 replies; 15+ messages in thread
From: Alexandre Courbot @ 2014-06-27 10:28 UTC (permalink / raw)
  To: Ben Skeggs
  Cc: nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
	linux-tegra-u79uwXL29TY76Z2rM5mHXA

GK20A's BAR is functionally identical to NVC0's, but do not support
being ioremapped write-combined. Create a BAR instance for GK20A that
reflect that state.

Signed-off-by: Alexandre Courbot <acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
---
 drivers/gpu/drm/nouveau/Makefile                  |  1 +
 drivers/gpu/drm/nouveau/core/engine/device/nve0.c |  2 +-
 drivers/gpu/drm/nouveau/core/include/subdev/bar.h |  1 +
 drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c   | 54 +++++++++++++++++++++++
 drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c    |  6 +--
 drivers/gpu/drm/nouveau/core/subdev/bar/priv.h    |  6 +++
 6 files changed, 66 insertions(+), 4 deletions(-)
 create mode 100644 drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c

diff --git a/drivers/gpu/drm/nouveau/Makefile b/drivers/gpu/drm/nouveau/Makefile
index 8b307e143632..11d9561d67c1 100644
--- a/drivers/gpu/drm/nouveau/Makefile
+++ b/drivers/gpu/drm/nouveau/Makefile
@@ -26,6 +26,7 @@ nouveau-y += core/core/subdev.o
 nouveau-y += core/subdev/bar/base.o
 nouveau-y += core/subdev/bar/nv50.o
 nouveau-y += core/subdev/bar/nvc0.o
+nouveau-y += core/subdev/bar/gk20a.o
 nouveau-y += core/subdev/bios/base.o
 nouveau-y += core/subdev/bios/bit.o
 nouveau-y += core/subdev/bios/boost.o
diff --git a/drivers/gpu/drm/nouveau/core/engine/device/nve0.c b/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
index 2d1e97d4264f..a2b9ccc48f66 100644
--- a/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
+++ b/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
@@ -165,7 +165,7 @@ nve0_identify(struct nouveau_device *device)
 		device->oclass[NVDEV_SUBDEV_IBUS   ] = &gk20a_ibus_oclass;
 		device->oclass[NVDEV_SUBDEV_INSTMEM] = nv50_instmem_oclass;
 		device->oclass[NVDEV_SUBDEV_VM     ] = &nvc0_vmmgr_oclass;
-		device->oclass[NVDEV_SUBDEV_BAR    ] = &nvc0_bar_oclass;
+		device->oclass[NVDEV_SUBDEV_BAR    ] = &gk20a_bar_oclass;
 		device->oclass[NVDEV_ENGINE_DMAOBJ ] = &nvd0_dmaeng_oclass;
 		device->oclass[NVDEV_ENGINE_FIFO   ] =  gk20a_fifo_oclass;
 		device->oclass[NVDEV_ENGINE_SW     ] =  nvc0_software_oclass;
diff --git a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
index 9002cbb6432b..be037fac534c 100644
--- a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
+++ b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
@@ -33,5 +33,6 @@ nouveau_bar(void *obj)
 
 extern struct nouveau_oclass nv50_bar_oclass;
 extern struct nouveau_oclass nvc0_bar_oclass;
+extern struct nouveau_oclass gk20a_bar_oclass;
 
 #endif
diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c b/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
new file mode 100644
index 000000000000..cc7c3db211d1
--- /dev/null
+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
@@ -0,0 +1,54 @@
+/*
+ * Copyright (c) 2014, NVIDIA CORPORATION. All rights reserved.
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
+ * DEALINGS IN THE SOFTWARE.
+ */
+
+#include <subdev/bar.h>
+
+#include "priv.h"
+
+int
+gk20a_bar_ctor(struct nouveau_object *parent, struct nouveau_object *engine,
+	       struct nouveau_oclass *oclass, void *data, u32 size,
+	       struct nouveau_object **pobject)
+{
+	struct nouveau_bar *bar;
+	int ret;
+
+	ret = nvc0_bar_ctor(parent, engine, oclass, data, size, pobject);
+	if (ret)
+		return ret;
+
+	bar = *pobject;
+	bar->iomap_uncached = true;
+
+	return 0;
+}
+
+struct nouveau_oclass
+gk20a_bar_oclass = {
+	.handle = NV_SUBDEV(BAR, 0xea),
+	.ofuncs = &(struct nouveau_ofuncs) {
+		.ctor = gk20a_bar_ctor,
+		.dtor = nvc0_bar_dtor,
+		.init = nvc0_bar_init,
+		.fini = _nouveau_bar_fini,
+	},
+};
diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c b/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
index ca8139b9ab27..0a44459844e3 100644
--- a/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
@@ -133,7 +133,7 @@ nvc0_bar_init_vm(struct nvc0_bar_priv *priv, struct nvc0_bar_priv_vm *bar_vm,
 	return 0;
 }
 
-static int
+int
 nvc0_bar_ctor(struct nouveau_object *parent, struct nouveau_object *engine,
 	      struct nouveau_oclass *oclass, void *data, u32 size,
 	      struct nouveau_object **pobject)
@@ -169,7 +169,7 @@ nvc0_bar_ctor(struct nouveau_object *parent, struct nouveau_object *engine,
 	return 0;
 }
 
-static void
+void
 nvc0_bar_dtor(struct nouveau_object *object)
 {
 	struct nvc0_bar_priv *priv = (void *)object;
@@ -188,7 +188,7 @@ nvc0_bar_dtor(struct nouveau_object *object)
 	nouveau_bar_destroy(&priv->base);
 }
 
-static int
+int
 nvc0_bar_init(struct nouveau_object *object)
 {
 	struct nvc0_bar_priv *priv = (void *)object;
diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h b/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
index ffad8f337ead..3ee8b1476d00 100644
--- a/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
@@ -23,4 +23,10 @@ int  nouveau_bar_alloc(struct nouveau_bar *, struct nouveau_object *,
 
 void nv84_bar_flush(struct nouveau_bar *);
 
+int nvc0_bar_ctor(struct nouveau_object *, struct nouveau_object *,
+		  struct nouveau_oclass *, void *, u32,
+		  struct nouveau_object **);
+void nvc0_bar_dtor(struct nouveau_object *);
+int nvc0_bar_init(struct nouveau_object *);
+
 #endif
-- 
2.0.0

^ permalink raw reply related	[flat|nested] 15+ messages in thread
* [PATCH 2/2] drm/gk20a: add BAR instance
@ 2014-06-27 10:28     ` Alexandre Courbot
  0 siblings, 0 replies; 15+ messages in thread
From: Alexandre Courbot @ 2014-06-27 10:28 UTC (permalink / raw)
  To: Ben Skeggs
  Cc: nouveau, dri-devel, linux-tegra, linux-kernel, gnurou, Alexandre Courbot

GK20A's BAR is functionally identical to NVC0's, but do not support
being ioremapped write-combined. Create a BAR instance for GK20A that
reflect that state.

Signed-off-by: Alexandre Courbot <acourbot@nvidia.com>
---
 drivers/gpu/drm/nouveau/Makefile                  |  1 +
 drivers/gpu/drm/nouveau/core/engine/device/nve0.c |  2 +-
 drivers/gpu/drm/nouveau/core/include/subdev/bar.h |  1 +
 drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c   | 54 +++++++++++++++++++++++
 drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c    |  6 +--
 drivers/gpu/drm/nouveau/core/subdev/bar/priv.h    |  6 +++
 6 files changed, 66 insertions(+), 4 deletions(-)
 create mode 100644 drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c

diff --git a/drivers/gpu/drm/nouveau/Makefile b/drivers/gpu/drm/nouveau/Makefile
index 8b307e143632..11d9561d67c1 100644
--- a/drivers/gpu/drm/nouveau/Makefile
+++ b/drivers/gpu/drm/nouveau/Makefile
@@ -26,6 +26,7 @@ nouveau-y += core/core/subdev.o
 nouveau-y += core/subdev/bar/base.o
 nouveau-y += core/subdev/bar/nv50.o
 nouveau-y += core/subdev/bar/nvc0.o
+nouveau-y += core/subdev/bar/gk20a.o
 nouveau-y += core/subdev/bios/base.o
 nouveau-y += core/subdev/bios/bit.o
 nouveau-y += core/subdev/bios/boost.o
diff --git a/drivers/gpu/drm/nouveau/core/engine/device/nve0.c b/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
index 2d1e97d4264f..a2b9ccc48f66 100644
--- a/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
+++ b/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
@@ -165,7 +165,7 @@ nve0_identify(struct nouveau_device *device)
 		device->oclass[NVDEV_SUBDEV_IBUS   ] = &gk20a_ibus_oclass;
 		device->oclass[NVDEV_SUBDEV_INSTMEM] = nv50_instmem_oclass;
 		device->oclass[NVDEV_SUBDEV_VM     ] = &nvc0_vmmgr_oclass;
-		device->oclass[NVDEV_SUBDEV_BAR    ] = &nvc0_bar_oclass;
+		device->oclass[NVDEV_SUBDEV_BAR    ] = &gk20a_bar_oclass;
 		device->oclass[NVDEV_ENGINE_DMAOBJ ] = &nvd0_dmaeng_oclass;
 		device->oclass[NVDEV_ENGINE_FIFO   ] =  gk20a_fifo_oclass;
 		device->oclass[NVDEV_ENGINE_SW     ] =  nvc0_software_oclass;
diff --git a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
index 9002cbb6432b..be037fac534c 100644
--- a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
+++ b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
@@ -33,5 +33,6 @@ nouveau_bar(void *obj)
 
 extern struct nouveau_oclass nv50_bar_oclass;
 extern struct nouveau_oclass nvc0_bar_oclass;
+extern struct nouveau_oclass gk20a_bar_oclass;
 
 #endif
diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c b/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
new file mode 100644
index 000000000000..cc7c3db211d1
--- /dev/null
+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
@@ -0,0 +1,54 @@
+/*
+ * Copyright (c) 2014, NVIDIA CORPORATION. All rights reserved.
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
+ * DEALINGS IN THE SOFTWARE.
+ */
+
+#include <subdev/bar.h>
+
+#include "priv.h"
+
+int
+gk20a_bar_ctor(struct nouveau_object *parent, struct nouveau_object *engine,
+	       struct nouveau_oclass *oclass, void *data, u32 size,
+	       struct nouveau_object **pobject)
+{
+	struct nouveau_bar *bar;
+	int ret;
+
+	ret = nvc0_bar_ctor(parent, engine, oclass, data, size, pobject);
+	if (ret)
+		return ret;
+
+	bar = *pobject;
+	bar->iomap_uncached = true;
+
+	return 0;
+}
+
+struct nouveau_oclass
+gk20a_bar_oclass = {
+	.handle = NV_SUBDEV(BAR, 0xea),
+	.ofuncs = &(struct nouveau_ofuncs) {
+		.ctor = gk20a_bar_ctor,
+		.dtor = nvc0_bar_dtor,
+		.init = nvc0_bar_init,
+		.fini = _nouveau_bar_fini,
+	},
+};
diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c b/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
index ca8139b9ab27..0a44459844e3 100644
--- a/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
@@ -133,7 +133,7 @@ nvc0_bar_init_vm(struct nvc0_bar_priv *priv, struct nvc0_bar_priv_vm *bar_vm,
 	return 0;
 }
 
-static int
+int
 nvc0_bar_ctor(struct nouveau_object *parent, struct nouveau_object *engine,
 	      struct nouveau_oclass *oclass, void *data, u32 size,
 	      struct nouveau_object **pobject)
@@ -169,7 +169,7 @@ nvc0_bar_ctor(struct nouveau_object *parent, struct nouveau_object *engine,
 	return 0;
 }
 
-static void
+void
 nvc0_bar_dtor(struct nouveau_object *object)
 {
 	struct nvc0_bar_priv *priv = (void *)object;
@@ -188,7 +188,7 @@ nvc0_bar_dtor(struct nouveau_object *object)
 	nouveau_bar_destroy(&priv->base);
 }
 
-static int
+int
 nvc0_bar_init(struct nouveau_object *object)
 {
 	struct nvc0_bar_priv *priv = (void *)object;
diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h b/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
index ffad8f337ead..3ee8b1476d00 100644
--- a/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
@@ -23,4 +23,10 @@ int  nouveau_bar_alloc(struct nouveau_bar *, struct nouveau_object *,
 
 void nv84_bar_flush(struct nouveau_bar *);
 
+int nvc0_bar_ctor(struct nouveau_object *, struct nouveau_object *,
+		  struct nouveau_oclass *, void *, u32,
+		  struct nouveau_object **);
+void nvc0_bar_dtor(struct nouveau_object *);
+int nvc0_bar_init(struct nouveau_object *);
+
 #endif
-- 
2.0.0


^ permalink raw reply related	[flat|nested] 15+ messages in thread
* [PATCH v2] drm/gk20a: add BAR instance
  2014-06-27 10:28     ` Alexandre Courbot
@ 2014-06-27 11:36         ` Alexandre Courbot
  -1 siblings, 0 replies; 15+ messages in thread
From: Alexandre Courbot @ 2014-06-27 11:36 UTC (permalink / raw)
  To: Ben Skeggs
  Cc: nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
	linux-tegra-u79uwXL29TY76Z2rM5mHXA

GK20A's BAR is functionally identical to NVC0's, but do not support
being ioremapped write-combined. Create a BAR instance for GK20A that
reflect that state.

Signed-off-by: Alexandre Courbot <acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
---
Changes since v1:
- Fix compilation warning due to missing cast

Patch 1 of the series was ok and thus has not been resent.

 drivers/gpu/drm/nouveau/Makefile                  |  1 +
 drivers/gpu/drm/nouveau/core/engine/device/nve0.c |  2 +-
 drivers/gpu/drm/nouveau/core/include/subdev/bar.h |  1 +
 drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c   | 54 +++++++++++++++++++++++
 drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c    |  6 +--
 drivers/gpu/drm/nouveau/core/subdev/bar/priv.h    |  6 +++
 6 files changed, 66 insertions(+), 4 deletions(-)
 create mode 100644 drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c

diff --git a/drivers/gpu/drm/nouveau/Makefile b/drivers/gpu/drm/nouveau/Makefile
index 8b307e143632..11d9561d67c1 100644
--- a/drivers/gpu/drm/nouveau/Makefile
+++ b/drivers/gpu/drm/nouveau/Makefile
@@ -26,6 +26,7 @@ nouveau-y += core/core/subdev.o
 nouveau-y += core/subdev/bar/base.o
 nouveau-y += core/subdev/bar/nv50.o
 nouveau-y += core/subdev/bar/nvc0.o
+nouveau-y += core/subdev/bar/gk20a.o
 nouveau-y += core/subdev/bios/base.o
 nouveau-y += core/subdev/bios/bit.o
 nouveau-y += core/subdev/bios/boost.o
diff --git a/drivers/gpu/drm/nouveau/core/engine/device/nve0.c b/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
index 2d1e97d4264f..a2b9ccc48f66 100644
--- a/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
+++ b/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
@@ -165,7 +165,7 @@ nve0_identify(struct nouveau_device *device)
 		device->oclass[NVDEV_SUBDEV_IBUS   ] = &gk20a_ibus_oclass;
 		device->oclass[NVDEV_SUBDEV_INSTMEM] = nv50_instmem_oclass;
 		device->oclass[NVDEV_SUBDEV_VM     ] = &nvc0_vmmgr_oclass;
-		device->oclass[NVDEV_SUBDEV_BAR    ] = &nvc0_bar_oclass;
+		device->oclass[NVDEV_SUBDEV_BAR    ] = &gk20a_bar_oclass;
 		device->oclass[NVDEV_ENGINE_DMAOBJ ] = &nvd0_dmaeng_oclass;
 		device->oclass[NVDEV_ENGINE_FIFO   ] =  gk20a_fifo_oclass;
 		device->oclass[NVDEV_ENGINE_SW     ] =  nvc0_software_oclass;
diff --git a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
index 9002cbb6432b..be037fac534c 100644
--- a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
+++ b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
@@ -33,5 +33,6 @@ nouveau_bar(void *obj)
 
 extern struct nouveau_oclass nv50_bar_oclass;
 extern struct nouveau_oclass nvc0_bar_oclass;
+extern struct nouveau_oclass gk20a_bar_oclass;
 
 #endif
diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c b/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
new file mode 100644
index 000000000000..bf877af9d3bd
--- /dev/null
+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
@@ -0,0 +1,54 @@
+/*
+ * Copyright (c) 2014, NVIDIA CORPORATION. All rights reserved.
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
+ * DEALINGS IN THE SOFTWARE.
+ */
+
+#include <subdev/bar.h>
+
+#include "priv.h"
+
+int
+gk20a_bar_ctor(struct nouveau_object *parent, struct nouveau_object *engine,
+	       struct nouveau_oclass *oclass, void *data, u32 size,
+	       struct nouveau_object **pobject)
+{
+	struct nouveau_bar *bar;
+	int ret;
+
+	ret = nvc0_bar_ctor(parent, engine, oclass, data, size, pobject);
+	if (ret)
+		return ret;
+
+	bar = (struct nouveau_bar *)*pobject;
+	bar->iomap_uncached = true;
+
+	return 0;
+}
+
+struct nouveau_oclass
+gk20a_bar_oclass = {
+	.handle = NV_SUBDEV(BAR, 0xea),
+	.ofuncs = &(struct nouveau_ofuncs) {
+		.ctor = gk20a_bar_ctor,
+		.dtor = nvc0_bar_dtor,
+		.init = nvc0_bar_init,
+		.fini = _nouveau_bar_fini,
+	},
+};
diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c b/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
index ca8139b9ab27..0a44459844e3 100644
--- a/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
@@ -133,7 +133,7 @@ nvc0_bar_init_vm(struct nvc0_bar_priv *priv, struct nvc0_bar_priv_vm *bar_vm,
 	return 0;
 }
 
-static int
+int
 nvc0_bar_ctor(struct nouveau_object *parent, struct nouveau_object *engine,
 	      struct nouveau_oclass *oclass, void *data, u32 size,
 	      struct nouveau_object **pobject)
@@ -169,7 +169,7 @@ nvc0_bar_ctor(struct nouveau_object *parent, struct nouveau_object *engine,
 	return 0;
 }
 
-static void
+void
 nvc0_bar_dtor(struct nouveau_object *object)
 {
 	struct nvc0_bar_priv *priv = (void *)object;
@@ -188,7 +188,7 @@ nvc0_bar_dtor(struct nouveau_object *object)
 	nouveau_bar_destroy(&priv->base);
 }
 
-static int
+int
 nvc0_bar_init(struct nouveau_object *object)
 {
 	struct nvc0_bar_priv *priv = (void *)object;
diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h b/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
index ffad8f337ead..3ee8b1476d00 100644
--- a/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
@@ -23,4 +23,10 @@ int  nouveau_bar_alloc(struct nouveau_bar *, struct nouveau_object *,
 
 void nv84_bar_flush(struct nouveau_bar *);
 
+int nvc0_bar_ctor(struct nouveau_object *, struct nouveau_object *,
+		  struct nouveau_oclass *, void *, u32,
+		  struct nouveau_object **);
+void nvc0_bar_dtor(struct nouveau_object *);
+int nvc0_bar_init(struct nouveau_object *);
+
 #endif
-- 
2.0.0

^ permalink raw reply related	[flat|nested] 15+ messages in thread
* [PATCH v2] drm/gk20a: add BAR instance
@ 2014-06-27 11:36         ` Alexandre Courbot
  0 siblings, 0 replies; 15+ messages in thread
From: Alexandre Courbot @ 2014-06-27 11:36 UTC (permalink / raw)
  To: Ben Skeggs
  Cc: nouveau, dri-devel, linux-tegra, linux-kernel, gnurou, Alexandre Courbot

GK20A's BAR is functionally identical to NVC0's, but do not support
being ioremapped write-combined. Create a BAR instance for GK20A that
reflect that state.

Signed-off-by: Alexandre Courbot <acourbot@nvidia.com>
---
Changes since v1:
- Fix compilation warning due to missing cast

Patch 1 of the series was ok and thus has not been resent.

 drivers/gpu/drm/nouveau/Makefile                  |  1 +
 drivers/gpu/drm/nouveau/core/engine/device/nve0.c |  2 +-
 drivers/gpu/drm/nouveau/core/include/subdev/bar.h |  1 +
 drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c   | 54 +++++++++++++++++++++++
 drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c    |  6 +--
 drivers/gpu/drm/nouveau/core/subdev/bar/priv.h    |  6 +++
 6 files changed, 66 insertions(+), 4 deletions(-)
 create mode 100644 drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c

diff --git a/drivers/gpu/drm/nouveau/Makefile b/drivers/gpu/drm/nouveau/Makefile
index 8b307e143632..11d9561d67c1 100644
--- a/drivers/gpu/drm/nouveau/Makefile
+++ b/drivers/gpu/drm/nouveau/Makefile
@@ -26,6 +26,7 @@ nouveau-y += core/core/subdev.o
 nouveau-y += core/subdev/bar/base.o
 nouveau-y += core/subdev/bar/nv50.o
 nouveau-y += core/subdev/bar/nvc0.o
+nouveau-y += core/subdev/bar/gk20a.o
 nouveau-y += core/subdev/bios/base.o
 nouveau-y += core/subdev/bios/bit.o
 nouveau-y += core/subdev/bios/boost.o
diff --git a/drivers/gpu/drm/nouveau/core/engine/device/nve0.c b/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
index 2d1e97d4264f..a2b9ccc48f66 100644
--- a/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
+++ b/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
@@ -165,7 +165,7 @@ nve0_identify(struct nouveau_device *device)
 		device->oclass[NVDEV_SUBDEV_IBUS   ] = &gk20a_ibus_oclass;
 		device->oclass[NVDEV_SUBDEV_INSTMEM] = nv50_instmem_oclass;
 		device->oclass[NVDEV_SUBDEV_VM     ] = &nvc0_vmmgr_oclass;
-		device->oclass[NVDEV_SUBDEV_BAR    ] = &nvc0_bar_oclass;
+		device->oclass[NVDEV_SUBDEV_BAR    ] = &gk20a_bar_oclass;
 		device->oclass[NVDEV_ENGINE_DMAOBJ ] = &nvd0_dmaeng_oclass;
 		device->oclass[NVDEV_ENGINE_FIFO   ] =  gk20a_fifo_oclass;
 		device->oclass[NVDEV_ENGINE_SW     ] =  nvc0_software_oclass;
diff --git a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
index 9002cbb6432b..be037fac534c 100644
--- a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
+++ b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
@@ -33,5 +33,6 @@ nouveau_bar(void *obj)
 
 extern struct nouveau_oclass nv50_bar_oclass;
 extern struct nouveau_oclass nvc0_bar_oclass;
+extern struct nouveau_oclass gk20a_bar_oclass;
 
 #endif
diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c b/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
new file mode 100644
index 000000000000..bf877af9d3bd
--- /dev/null
+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
@@ -0,0 +1,54 @@
+/*
+ * Copyright (c) 2014, NVIDIA CORPORATION. All rights reserved.
+ *
+ * Permission is hereby granted, free of charge, to any person obtaining a
+ * copy of this software and associated documentation files (the "Software"),
+ * to deal in the Software without restriction, including without limitation
+ * the rights to use, copy, modify, merge, publish, distribute, sublicense,
+ * and/or sell copies of the Software, and to permit persons to whom the
+ * Software is furnished to do so, subject to the following conditions:
+ *
+ * The above copyright notice and this permission notice shall be included in
+ * all copies or substantial portions of the Software.
+ *
+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT SHALL
+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
+ * DEALINGS IN THE SOFTWARE.
+ */
+
+#include <subdev/bar.h>
+
+#include "priv.h"
+
+int
+gk20a_bar_ctor(struct nouveau_object *parent, struct nouveau_object *engine,
+	       struct nouveau_oclass *oclass, void *data, u32 size,
+	       struct nouveau_object **pobject)
+{
+	struct nouveau_bar *bar;
+	int ret;
+
+	ret = nvc0_bar_ctor(parent, engine, oclass, data, size, pobject);
+	if (ret)
+		return ret;
+
+	bar = (struct nouveau_bar *)*pobject;
+	bar->iomap_uncached = true;
+
+	return 0;
+}
+
+struct nouveau_oclass
+gk20a_bar_oclass = {
+	.handle = NV_SUBDEV(BAR, 0xea),
+	.ofuncs = &(struct nouveau_ofuncs) {
+		.ctor = gk20a_bar_ctor,
+		.dtor = nvc0_bar_dtor,
+		.init = nvc0_bar_init,
+		.fini = _nouveau_bar_fini,
+	},
+};
diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c b/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
index ca8139b9ab27..0a44459844e3 100644
--- a/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
@@ -133,7 +133,7 @@ nvc0_bar_init_vm(struct nvc0_bar_priv *priv, struct nvc0_bar_priv_vm *bar_vm,
 	return 0;
 }
 
-static int
+int
 nvc0_bar_ctor(struct nouveau_object *parent, struct nouveau_object *engine,
 	      struct nouveau_oclass *oclass, void *data, u32 size,
 	      struct nouveau_object **pobject)
@@ -169,7 +169,7 @@ nvc0_bar_ctor(struct nouveau_object *parent, struct nouveau_object *engine,
 	return 0;
 }
 
-static void
+void
 nvc0_bar_dtor(struct nouveau_object *object)
 {
 	struct nvc0_bar_priv *priv = (void *)object;
@@ -188,7 +188,7 @@ nvc0_bar_dtor(struct nouveau_object *object)
 	nouveau_bar_destroy(&priv->base);
 }
 
-static int
+int
 nvc0_bar_init(struct nouveau_object *object)
 {
 	struct nvc0_bar_priv *priv = (void *)object;
diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h b/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
index ffad8f337ead..3ee8b1476d00 100644
--- a/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
@@ -23,4 +23,10 @@ int  nouveau_bar_alloc(struct nouveau_bar *, struct nouveau_object *,
 
 void nv84_bar_flush(struct nouveau_bar *);
 
+int nvc0_bar_ctor(struct nouveau_object *, struct nouveau_object *,
+		  struct nouveau_oclass *, void *, u32,
+		  struct nouveau_object **);
+void nvc0_bar_dtor(struct nouveau_object *);
+int nvc0_bar_init(struct nouveau_object *);
+
 #endif
-- 
2.0.0


^ permalink raw reply related	[flat|nested] 15+ messages in thread
* Re: [PATCH v2] drm/gk20a: add BAR instance
  2014-06-27 11:36         ` Alexandre Courbot
@ 2014-06-27 18:51             ` Ken Adams
  -1 siblings, 0 replies; 15+ messages in thread
From: Ken Adams @ 2014-06-27 18:51 UTC (permalink / raw)
  To: Alex Courbot, Ben Skeggs
  Cc: nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
	dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
	linux-tegra-u79uwXL29TY76Z2rM5mHXA,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	gnurou-Re5JQEeQqe8AvxtiuMwx3w

quick note re: tegra and gpu bars...

to this point we've explicitly avoided providing user-mode mappings due to
power management issues, etc.
looks to me like this would allow such mappings.  is that the case?  are
there any paths which would require such mappings to function properly?

thanks
---
ken

p.s.: hello :)

On 6/27/14 7:36 AM, "Alex Courbot" <acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> wrote:

>GK20A's BAR is functionally identical to NVC0's, but do not support
>being ioremapped write-combined. Create a BAR instance for GK20A that
>reflect that state.
>
>Signed-off-by: Alexandre Courbot <acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>---
>Changes since v1:
>- Fix compilation warning due to missing cast
>
>Patch 1 of the series was ok and thus has not been resent.
>
> drivers/gpu/drm/nouveau/Makefile                  |  1 +
> drivers/gpu/drm/nouveau/core/engine/device/nve0.c |  2 +-
> drivers/gpu/drm/nouveau/core/include/subdev/bar.h |  1 +
> drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c   | 54
>+++++++++++++++++++++++
> drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c    |  6 +--
> drivers/gpu/drm/nouveau/core/subdev/bar/priv.h    |  6 +++
> 6 files changed, 66 insertions(+), 4 deletions(-)
> create mode 100644 drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>
>diff --git a/drivers/gpu/drm/nouveau/Makefile
>b/drivers/gpu/drm/nouveau/Makefile
>index 8b307e143632..11d9561d67c1 100644
>--- a/drivers/gpu/drm/nouveau/Makefile
>+++ b/drivers/gpu/drm/nouveau/Makefile
>@@ -26,6 +26,7 @@ nouveau-y += core/core/subdev.o
> nouveau-y += core/subdev/bar/base.o
> nouveau-y += core/subdev/bar/nv50.o
> nouveau-y += core/subdev/bar/nvc0.o
>+nouveau-y += core/subdev/bar/gk20a.o
> nouveau-y += core/subdev/bios/base.o
> nouveau-y += core/subdev/bios/bit.o
> nouveau-y += core/subdev/bios/boost.o
>diff --git a/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>b/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>index 2d1e97d4264f..a2b9ccc48f66 100644
>--- a/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>+++ b/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>@@ -165,7 +165,7 @@ nve0_identify(struct nouveau_device *device)
> 		device->oclass[NVDEV_SUBDEV_IBUS   ] = &gk20a_ibus_oclass;
> 		device->oclass[NVDEV_SUBDEV_INSTMEM] = nv50_instmem_oclass;
> 		device->oclass[NVDEV_SUBDEV_VM     ] = &nvc0_vmmgr_oclass;
>-		device->oclass[NVDEV_SUBDEV_BAR    ] = &nvc0_bar_oclass;
>+		device->oclass[NVDEV_SUBDEV_BAR    ] = &gk20a_bar_oclass;
> 		device->oclass[NVDEV_ENGINE_DMAOBJ ] = &nvd0_dmaeng_oclass;
> 		device->oclass[NVDEV_ENGINE_FIFO   ] =  gk20a_fifo_oclass;
> 		device->oclass[NVDEV_ENGINE_SW     ] =  nvc0_software_oclass;
>diff --git a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>index 9002cbb6432b..be037fac534c 100644
>--- a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>+++ b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>@@ -33,5 +33,6 @@ nouveau_bar(void *obj)
> 
> extern struct nouveau_oclass nv50_bar_oclass;
> extern struct nouveau_oclass nvc0_bar_oclass;
>+extern struct nouveau_oclass gk20a_bar_oclass;
> 
> #endif
>diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>b/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>new file mode 100644
>index 000000000000..bf877af9d3bd
>--- /dev/null
>+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>@@ -0,0 +1,54 @@
>+/*
>+ * Copyright (c) 2014, NVIDIA CORPORATION. All rights reserved.
>+ *
>+ * Permission is hereby granted, free of charge, to any person obtaining
>a
>+ * copy of this software and associated documentation files (the
>"Software"),
>+ * to deal in the Software without restriction, including without
>limitation
>+ * the rights to use, copy, modify, merge, publish, distribute,
>sublicense,
>+ * and/or sell copies of the Software, and to permit persons to whom the
>+ * Software is furnished to do so, subject to the following conditions:
>+ *
>+ * The above copyright notice and this permission notice shall be
>included in
>+ * all copies or substantial portions of the Software.
>+ *
>+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
>EXPRESS OR
>+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
>MERCHANTABILITY,
>+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT
>SHALL
>+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR
>OTHER
>+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
>ARISING
>+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
>+ * DEALINGS IN THE SOFTWARE.
>+ */
>+
>+#include <subdev/bar.h>
>+
>+#include "priv.h"
>+
>+int
>+gk20a_bar_ctor(struct nouveau_object *parent, struct nouveau_object
>*engine,
>+	       struct nouveau_oclass *oclass, void *data, u32 size,
>+	       struct nouveau_object **pobject)
>+{
>+	struct nouveau_bar *bar;
>+	int ret;
>+
>+	ret = nvc0_bar_ctor(parent, engine, oclass, data, size, pobject);
>+	if (ret)
>+		return ret;
>+
>+	bar = (struct nouveau_bar *)*pobject;
>+	bar->iomap_uncached = true;
>+
>+	return 0;
>+}
>+
>+struct nouveau_oclass
>+gk20a_bar_oclass = {
>+	.handle = NV_SUBDEV(BAR, 0xea),
>+	.ofuncs = &(struct nouveau_ofuncs) {
>+		.ctor = gk20a_bar_ctor,
>+		.dtor = nvc0_bar_dtor,
>+		.init = nvc0_bar_init,
>+		.fini = _nouveau_bar_fini,
>+	},
>+};
>diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>b/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>index ca8139b9ab27..0a44459844e3 100644
>--- a/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>@@ -133,7 +133,7 @@ nvc0_bar_init_vm(struct nvc0_bar_priv *priv, struct
>nvc0_bar_priv_vm *bar_vm,
> 	return 0;
> }
> 
>-static int
>+int
> nvc0_bar_ctor(struct nouveau_object *parent, struct nouveau_object
>*engine,
> 	      struct nouveau_oclass *oclass, void *data, u32 size,
> 	      struct nouveau_object **pobject)
>@@ -169,7 +169,7 @@ nvc0_bar_ctor(struct nouveau_object *parent, struct
>nouveau_object *engine,
> 	return 0;
> }
> 
>-static void
>+void
> nvc0_bar_dtor(struct nouveau_object *object)
> {
> 	struct nvc0_bar_priv *priv = (void *)object;
>@@ -188,7 +188,7 @@ nvc0_bar_dtor(struct nouveau_object *object)
> 	nouveau_bar_destroy(&priv->base);
> }
> 
>-static int
>+int
> nvc0_bar_init(struct nouveau_object *object)
> {
> 	struct nvc0_bar_priv *priv = (void *)object;
>diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>b/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>index ffad8f337ead..3ee8b1476d00 100644
>--- a/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>@@ -23,4 +23,10 @@ int  nouveau_bar_alloc(struct nouveau_bar *, struct
>nouveau_object *,
> 
> void nv84_bar_flush(struct nouveau_bar *);
> 
>+int nvc0_bar_ctor(struct nouveau_object *, struct nouveau_object *,
>+		  struct nouveau_oclass *, void *, u32,
>+		  struct nouveau_object **);
>+void nvc0_bar_dtor(struct nouveau_object *);
>+int nvc0_bar_init(struct nouveau_object *);
>+
> #endif
>-- 
>2.0.0
>
>--
>To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
>the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
>More majordomo info at  http://vger.kernel.org/majordomo-info.html

^ permalink raw reply	[flat|nested] 15+ messages in thread
* Re: [PATCH v2] drm/gk20a: add BAR instance
@ 2014-06-27 18:51             ` Ken Adams
  0 siblings, 0 replies; 15+ messages in thread
From: Ken Adams @ 2014-06-27 18:51 UTC (permalink / raw)
  To: Alex Courbot, Ben Skeggs
  Cc: nouveau, dri-devel, linux-tegra, linux-kernel, gnurou

quick note re: tegra and gpu bars...

to this point we've explicitly avoided providing user-mode mappings due to
power management issues, etc.
looks to me like this would allow such mappings.  is that the case?  are
there any paths which would require such mappings to function properly?

thanks
---
ken

p.s.: hello :)

On 6/27/14 7:36 AM, "Alex Courbot" <acourbot@nvidia.com> wrote:

>GK20A's BAR is functionally identical to NVC0's, but do not support
>being ioremapped write-combined. Create a BAR instance for GK20A that
>reflect that state.
>
>Signed-off-by: Alexandre Courbot <acourbot@nvidia.com>
>---
>Changes since v1:
>- Fix compilation warning due to missing cast
>
>Patch 1 of the series was ok and thus has not been resent.
>
> drivers/gpu/drm/nouveau/Makefile                  |  1 +
> drivers/gpu/drm/nouveau/core/engine/device/nve0.c |  2 +-
> drivers/gpu/drm/nouveau/core/include/subdev/bar.h |  1 +
> drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c   | 54
>+++++++++++++++++++++++
> drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c    |  6 +--
> drivers/gpu/drm/nouveau/core/subdev/bar/priv.h    |  6 +++
> 6 files changed, 66 insertions(+), 4 deletions(-)
> create mode 100644 drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>
>diff --git a/drivers/gpu/drm/nouveau/Makefile
>b/drivers/gpu/drm/nouveau/Makefile
>index 8b307e143632..11d9561d67c1 100644
>--- a/drivers/gpu/drm/nouveau/Makefile
>+++ b/drivers/gpu/drm/nouveau/Makefile
>@@ -26,6 +26,7 @@ nouveau-y += core/core/subdev.o
> nouveau-y += core/subdev/bar/base.o
> nouveau-y += core/subdev/bar/nv50.o
> nouveau-y += core/subdev/bar/nvc0.o
>+nouveau-y += core/subdev/bar/gk20a.o
> nouveau-y += core/subdev/bios/base.o
> nouveau-y += core/subdev/bios/bit.o
> nouveau-y += core/subdev/bios/boost.o
>diff --git a/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>b/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>index 2d1e97d4264f..a2b9ccc48f66 100644
>--- a/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>+++ b/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>@@ -165,7 +165,7 @@ nve0_identify(struct nouveau_device *device)
> 		device->oclass[NVDEV_SUBDEV_IBUS   ] = &gk20a_ibus_oclass;
> 		device->oclass[NVDEV_SUBDEV_INSTMEM] = nv50_instmem_oclass;
> 		device->oclass[NVDEV_SUBDEV_VM     ] = &nvc0_vmmgr_oclass;
>-		device->oclass[NVDEV_SUBDEV_BAR    ] = &nvc0_bar_oclass;
>+		device->oclass[NVDEV_SUBDEV_BAR    ] = &gk20a_bar_oclass;
> 		device->oclass[NVDEV_ENGINE_DMAOBJ ] = &nvd0_dmaeng_oclass;
> 		device->oclass[NVDEV_ENGINE_FIFO   ] =  gk20a_fifo_oclass;
> 		device->oclass[NVDEV_ENGINE_SW     ] =  nvc0_software_oclass;
>diff --git a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>index 9002cbb6432b..be037fac534c 100644
>--- a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>+++ b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>@@ -33,5 +33,6 @@ nouveau_bar(void *obj)
> 
> extern struct nouveau_oclass nv50_bar_oclass;
> extern struct nouveau_oclass nvc0_bar_oclass;
>+extern struct nouveau_oclass gk20a_bar_oclass;
> 
> #endif
>diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>b/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>new file mode 100644
>index 000000000000..bf877af9d3bd
>--- /dev/null
>+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>@@ -0,0 +1,54 @@
>+/*
>+ * Copyright (c) 2014, NVIDIA CORPORATION. All rights reserved.
>+ *
>+ * Permission is hereby granted, free of charge, to any person obtaining
>a
>+ * copy of this software and associated documentation files (the
>"Software"),
>+ * to deal in the Software without restriction, including without
>limitation
>+ * the rights to use, copy, modify, merge, publish, distribute,
>sublicense,
>+ * and/or sell copies of the Software, and to permit persons to whom the
>+ * Software is furnished to do so, subject to the following conditions:
>+ *
>+ * The above copyright notice and this permission notice shall be
>included in
>+ * all copies or substantial portions of the Software.
>+ *
>+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
>EXPRESS OR
>+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
>MERCHANTABILITY,
>+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT
>SHALL
>+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR
>OTHER
>+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
>ARISING
>+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
>+ * DEALINGS IN THE SOFTWARE.
>+ */
>+
>+#include <subdev/bar.h>
>+
>+#include "priv.h"
>+
>+int
>+gk20a_bar_ctor(struct nouveau_object *parent, struct nouveau_object
>*engine,
>+	       struct nouveau_oclass *oclass, void *data, u32 size,
>+	       struct nouveau_object **pobject)
>+{
>+	struct nouveau_bar *bar;
>+	int ret;
>+
>+	ret = nvc0_bar_ctor(parent, engine, oclass, data, size, pobject);
>+	if (ret)
>+		return ret;
>+
>+	bar = (struct nouveau_bar *)*pobject;
>+	bar->iomap_uncached = true;
>+
>+	return 0;
>+}
>+
>+struct nouveau_oclass
>+gk20a_bar_oclass = {
>+	.handle = NV_SUBDEV(BAR, 0xea),
>+	.ofuncs = &(struct nouveau_ofuncs) {
>+		.ctor = gk20a_bar_ctor,
>+		.dtor = nvc0_bar_dtor,
>+		.init = nvc0_bar_init,
>+		.fini = _nouveau_bar_fini,
>+	},
>+};
>diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>b/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>index ca8139b9ab27..0a44459844e3 100644
>--- a/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>@@ -133,7 +133,7 @@ nvc0_bar_init_vm(struct nvc0_bar_priv *priv, struct
>nvc0_bar_priv_vm *bar_vm,
> 	return 0;
> }
> 
>-static int
>+int
> nvc0_bar_ctor(struct nouveau_object *parent, struct nouveau_object
>*engine,
> 	      struct nouveau_oclass *oclass, void *data, u32 size,
> 	      struct nouveau_object **pobject)
>@@ -169,7 +169,7 @@ nvc0_bar_ctor(struct nouveau_object *parent, struct
>nouveau_object *engine,
> 	return 0;
> }
> 
>-static void
>+void
> nvc0_bar_dtor(struct nouveau_object *object)
> {
> 	struct nvc0_bar_priv *priv = (void *)object;
>@@ -188,7 +188,7 @@ nvc0_bar_dtor(struct nouveau_object *object)
> 	nouveau_bar_destroy(&priv->base);
> }
> 
>-static int
>+int
> nvc0_bar_init(struct nouveau_object *object)
> {
> 	struct nvc0_bar_priv *priv = (void *)object;
>diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>b/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>index ffad8f337ead..3ee8b1476d00 100644
>--- a/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>@@ -23,4 +23,10 @@ int  nouveau_bar_alloc(struct nouveau_bar *, struct
>nouveau_object *,
> 
> void nv84_bar_flush(struct nouveau_bar *);
> 
>+int nvc0_bar_ctor(struct nouveau_object *, struct nouveau_object *,
>+		  struct nouveau_oclass *, void *, u32,
>+		  struct nouveau_object **);
>+void nvc0_bar_dtor(struct nouveau_object *);
>+int nvc0_bar_init(struct nouveau_object *);
>+
> #endif
>-- 
>2.0.0
>
>--
>To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
>the body of a message to majordomo@vger.kernel.org
>More majordomo info at  http://vger.kernel.org/majordomo-info.html


^ permalink raw reply	[flat|nested] 15+ messages in thread
* Re: [PATCH v2] drm/gk20a: add BAR instance
  2014-06-27 18:51             ` Ken Adams
@ 2014-06-28  0:56                 ` Ben Skeggs
  -1 siblings, 0 replies; 15+ messages in thread
From: Ben Skeggs @ 2014-06-28  0:56 UTC (permalink / raw)
  To: Ken Adams
  Cc: Alex Courbot, Ben Skeggs, linux-tegra-u79uwXL29TY76Z2rM5mHXA,
	nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
	gnurou-Re5JQEeQqe8AvxtiuMwx3w

On Sat, Jun 28, 2014 at 4:51 AM, Ken Adams <KAdams-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> wrote:
> quick note re: tegra and gpu bars...
>
> to this point we've explicitly avoided providing user-mode mappings due to
> power management issues, etc.
> looks to me like this would allow such mappings.  is that the case?  are
> there any paths which would require such mappings to function properly?
What power management issues are you worried about in particular?  We
have these concerns on discrete cards too, when doing things like
changing vram frequencies.  TTM is able to kick out all userspace
mappings, and clients will then block in the fault handler until it's
safe - if they touch the mappings.

Ben.

>
> thanks
> ---
> ken
>
> p.s.: hello :)
>
> On 6/27/14 7:36 AM, "Alex Courbot" <acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> wrote:
>
>>GK20A's BAR is functionally identical to NVC0's, but do not support
>>being ioremapped write-combined. Create a BAR instance for GK20A that
>>reflect that state.
>>
>>Signed-off-by: Alexandre Courbot <acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>>---
>>Changes since v1:
>>- Fix compilation warning due to missing cast
>>
>>Patch 1 of the series was ok and thus has not been resent.
>>
>> drivers/gpu/drm/nouveau/Makefile                  |  1 +
>> drivers/gpu/drm/nouveau/core/engine/device/nve0.c |  2 +-
>> drivers/gpu/drm/nouveau/core/include/subdev/bar.h |  1 +
>> drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c   | 54
>>+++++++++++++++++++++++
>> drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c    |  6 +--
>> drivers/gpu/drm/nouveau/core/subdev/bar/priv.h    |  6 +++
>> 6 files changed, 66 insertions(+), 4 deletions(-)
>> create mode 100644 drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>>
>>diff --git a/drivers/gpu/drm/nouveau/Makefile
>>b/drivers/gpu/drm/nouveau/Makefile
>>index 8b307e143632..11d9561d67c1 100644
>>--- a/drivers/gpu/drm/nouveau/Makefile
>>+++ b/drivers/gpu/drm/nouveau/Makefile
>>@@ -26,6 +26,7 @@ nouveau-y += core/core/subdev.o
>> nouveau-y += core/subdev/bar/base.o
>> nouveau-y += core/subdev/bar/nv50.o
>> nouveau-y += core/subdev/bar/nvc0.o
>>+nouveau-y += core/subdev/bar/gk20a.o
>> nouveau-y += core/subdev/bios/base.o
>> nouveau-y += core/subdev/bios/bit.o
>> nouveau-y += core/subdev/bios/boost.o
>>diff --git a/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>>b/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>>index 2d1e97d4264f..a2b9ccc48f66 100644
>>--- a/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>>+++ b/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>>@@ -165,7 +165,7 @@ nve0_identify(struct nouveau_device *device)
>>               device->oclass[NVDEV_SUBDEV_IBUS   ] = &gk20a_ibus_oclass;
>>               device->oclass[NVDEV_SUBDEV_INSTMEM] = nv50_instmem_oclass;
>>               device->oclass[NVDEV_SUBDEV_VM     ] = &nvc0_vmmgr_oclass;
>>-              device->oclass[NVDEV_SUBDEV_BAR    ] = &nvc0_bar_oclass;
>>+              device->oclass[NVDEV_SUBDEV_BAR    ] = &gk20a_bar_oclass;
>>               device->oclass[NVDEV_ENGINE_DMAOBJ ] = &nvd0_dmaeng_oclass;
>>               device->oclass[NVDEV_ENGINE_FIFO   ] =  gk20a_fifo_oclass;
>>               device->oclass[NVDEV_ENGINE_SW     ] =  nvc0_software_oclass;
>>diff --git a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>>b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>>index 9002cbb6432b..be037fac534c 100644
>>--- a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>>+++ b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>>@@ -33,5 +33,6 @@ nouveau_bar(void *obj)
>>
>> extern struct nouveau_oclass nv50_bar_oclass;
>> extern struct nouveau_oclass nvc0_bar_oclass;
>>+extern struct nouveau_oclass gk20a_bar_oclass;
>>
>> #endif
>>diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>>b/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>>new file mode 100644
>>index 000000000000..bf877af9d3bd
>>--- /dev/null
>>+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>>@@ -0,0 +1,54 @@
>>+/*
>>+ * Copyright (c) 2014, NVIDIA CORPORATION. All rights reserved.
>>+ *
>>+ * Permission is hereby granted, free of charge, to any person obtaining
>>a
>>+ * copy of this software and associated documentation files (the
>>"Software"),
>>+ * to deal in the Software without restriction, including without
>>limitation
>>+ * the rights to use, copy, modify, merge, publish, distribute,
>>sublicense,
>>+ * and/or sell copies of the Software, and to permit persons to whom the
>>+ * Software is furnished to do so, subject to the following conditions:
>>+ *
>>+ * The above copyright notice and this permission notice shall be
>>included in
>>+ * all copies or substantial portions of the Software.
>>+ *
>>+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
>>EXPRESS OR
>>+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
>>MERCHANTABILITY,
>>+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT
>>SHALL
>>+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR
>>OTHER
>>+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
>>ARISING
>>+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
>>+ * DEALINGS IN THE SOFTWARE.
>>+ */
>>+
>>+#include <subdev/bar.h>
>>+
>>+#include "priv.h"
>>+
>>+int
>>+gk20a_bar_ctor(struct nouveau_object *parent, struct nouveau_object
>>*engine,
>>+             struct nouveau_oclass *oclass, void *data, u32 size,
>>+             struct nouveau_object **pobject)
>>+{
>>+      struct nouveau_bar *bar;
>>+      int ret;
>>+
>>+      ret = nvc0_bar_ctor(parent, engine, oclass, data, size, pobject);
>>+      if (ret)
>>+              return ret;
>>+
>>+      bar = (struct nouveau_bar *)*pobject;
>>+      bar->iomap_uncached = true;
>>+
>>+      return 0;
>>+}
>>+
>>+struct nouveau_oclass
>>+gk20a_bar_oclass = {
>>+      .handle = NV_SUBDEV(BAR, 0xea),
>>+      .ofuncs = &(struct nouveau_ofuncs) {
>>+              .ctor = gk20a_bar_ctor,
>>+              .dtor = nvc0_bar_dtor,
>>+              .init = nvc0_bar_init,
>>+              .fini = _nouveau_bar_fini,
>>+      },
>>+};
>>diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>>b/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>>index ca8139b9ab27..0a44459844e3 100644
>>--- a/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>>+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>>@@ -133,7 +133,7 @@ nvc0_bar_init_vm(struct nvc0_bar_priv *priv, struct
>>nvc0_bar_priv_vm *bar_vm,
>>       return 0;
>> }
>>
>>-static int
>>+int
>> nvc0_bar_ctor(struct nouveau_object *parent, struct nouveau_object
>>*engine,
>>             struct nouveau_oclass *oclass, void *data, u32 size,
>>             struct nouveau_object **pobject)
>>@@ -169,7 +169,7 @@ nvc0_bar_ctor(struct nouveau_object *parent, struct
>>nouveau_object *engine,
>>       return 0;
>> }
>>
>>-static void
>>+void
>> nvc0_bar_dtor(struct nouveau_object *object)
>> {
>>       struct nvc0_bar_priv *priv = (void *)object;
>>@@ -188,7 +188,7 @@ nvc0_bar_dtor(struct nouveau_object *object)
>>       nouveau_bar_destroy(&priv->base);
>> }
>>
>>-static int
>>+int
>> nvc0_bar_init(struct nouveau_object *object)
>> {
>>       struct nvc0_bar_priv *priv = (void *)object;
>>diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>>b/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>>index ffad8f337ead..3ee8b1476d00 100644
>>--- a/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>>+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>>@@ -23,4 +23,10 @@ int  nouveau_bar_alloc(struct nouveau_bar *, struct
>>nouveau_object *,
>>
>> void nv84_bar_flush(struct nouveau_bar *);
>>
>>+int nvc0_bar_ctor(struct nouveau_object *, struct nouveau_object *,
>>+                struct nouveau_oclass *, void *, u32,
>>+                struct nouveau_object **);
>>+void nvc0_bar_dtor(struct nouveau_object *);
>>+int nvc0_bar_init(struct nouveau_object *);
>>+
>> #endif
>>--
>>2.0.0
>>
>>--
>>To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
>>the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
>>More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
> _______________________________________________
> dri-devel mailing list
> dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 15+ messages in thread
* Re: [PATCH v2] drm/gk20a: add BAR instance
@ 2014-06-28  0:56                 ` Ben Skeggs
  0 siblings, 0 replies; 15+ messages in thread
From: Ben Skeggs @ 2014-06-28  0:56 UTC (permalink / raw)
  To: Ken Adams
  Cc: Alex Courbot, Ben Skeggs, linux-tegra, nouveau, linux-kernel,
	dri-devel, gnurou

On Sat, Jun 28, 2014 at 4:51 AM, Ken Adams <KAdams@nvidia.com> wrote:
> quick note re: tegra and gpu bars...
>
> to this point we've explicitly avoided providing user-mode mappings due to
> power management issues, etc.
> looks to me like this would allow such mappings.  is that the case?  are
> there any paths which would require such mappings to function properly?
What power management issues are you worried about in particular?  We
have these concerns on discrete cards too, when doing things like
changing vram frequencies.  TTM is able to kick out all userspace
mappings, and clients will then block in the fault handler until it's
safe - if they touch the mappings.

Ben.

>
> thanks
> ---
> ken
>
> p.s.: hello :)
>
> On 6/27/14 7:36 AM, "Alex Courbot" <acourbot@nvidia.com> wrote:
>
>>GK20A's BAR is functionally identical to NVC0's, but do not support
>>being ioremapped write-combined. Create a BAR instance for GK20A that
>>reflect that state.
>>
>>Signed-off-by: Alexandre Courbot <acourbot@nvidia.com>
>>---
>>Changes since v1:
>>- Fix compilation warning due to missing cast
>>
>>Patch 1 of the series was ok and thus has not been resent.
>>
>> drivers/gpu/drm/nouveau/Makefile                  |  1 +
>> drivers/gpu/drm/nouveau/core/engine/device/nve0.c |  2 +-
>> drivers/gpu/drm/nouveau/core/include/subdev/bar.h |  1 +
>> drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c   | 54
>>+++++++++++++++++++++++
>> drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c    |  6 +--
>> drivers/gpu/drm/nouveau/core/subdev/bar/priv.h    |  6 +++
>> 6 files changed, 66 insertions(+), 4 deletions(-)
>> create mode 100644 drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>>
>>diff --git a/drivers/gpu/drm/nouveau/Makefile
>>b/drivers/gpu/drm/nouveau/Makefile
>>index 8b307e143632..11d9561d67c1 100644
>>--- a/drivers/gpu/drm/nouveau/Makefile
>>+++ b/drivers/gpu/drm/nouveau/Makefile
>>@@ -26,6 +26,7 @@ nouveau-y += core/core/subdev.o
>> nouveau-y += core/subdev/bar/base.o
>> nouveau-y += core/subdev/bar/nv50.o
>> nouveau-y += core/subdev/bar/nvc0.o
>>+nouveau-y += core/subdev/bar/gk20a.o
>> nouveau-y += core/subdev/bios/base.o
>> nouveau-y += core/subdev/bios/bit.o
>> nouveau-y += core/subdev/bios/boost.o
>>diff --git a/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>>b/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>>index 2d1e97d4264f..a2b9ccc48f66 100644
>>--- a/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>>+++ b/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>>@@ -165,7 +165,7 @@ nve0_identify(struct nouveau_device *device)
>>               device->oclass[NVDEV_SUBDEV_IBUS   ] = &gk20a_ibus_oclass;
>>               device->oclass[NVDEV_SUBDEV_INSTMEM] = nv50_instmem_oclass;
>>               device->oclass[NVDEV_SUBDEV_VM     ] = &nvc0_vmmgr_oclass;
>>-              device->oclass[NVDEV_SUBDEV_BAR    ] = &nvc0_bar_oclass;
>>+              device->oclass[NVDEV_SUBDEV_BAR    ] = &gk20a_bar_oclass;
>>               device->oclass[NVDEV_ENGINE_DMAOBJ ] = &nvd0_dmaeng_oclass;
>>               device->oclass[NVDEV_ENGINE_FIFO   ] =  gk20a_fifo_oclass;
>>               device->oclass[NVDEV_ENGINE_SW     ] =  nvc0_software_oclass;
>>diff --git a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>>b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>>index 9002cbb6432b..be037fac534c 100644
>>--- a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>>+++ b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>>@@ -33,5 +33,6 @@ nouveau_bar(void *obj)
>>
>> extern struct nouveau_oclass nv50_bar_oclass;
>> extern struct nouveau_oclass nvc0_bar_oclass;
>>+extern struct nouveau_oclass gk20a_bar_oclass;
>>
>> #endif
>>diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>>b/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>>new file mode 100644
>>index 000000000000..bf877af9d3bd
>>--- /dev/null
>>+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>>@@ -0,0 +1,54 @@
>>+/*
>>+ * Copyright (c) 2014, NVIDIA CORPORATION. All rights reserved.
>>+ *
>>+ * Permission is hereby granted, free of charge, to any person obtaining
>>a
>>+ * copy of this software and associated documentation files (the
>>"Software"),
>>+ * to deal in the Software without restriction, including without
>>limitation
>>+ * the rights to use, copy, modify, merge, publish, distribute,
>>sublicense,
>>+ * and/or sell copies of the Software, and to permit persons to whom the
>>+ * Software is furnished to do so, subject to the following conditions:
>>+ *
>>+ * The above copyright notice and this permission notice shall be
>>included in
>>+ * all copies or substantial portions of the Software.
>>+ *
>>+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
>>EXPRESS OR
>>+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
>>MERCHANTABILITY,
>>+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT
>>SHALL
>>+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR
>>OTHER
>>+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
>>ARISING
>>+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
>>+ * DEALINGS IN THE SOFTWARE.
>>+ */
>>+
>>+#include <subdev/bar.h>
>>+
>>+#include "priv.h"
>>+
>>+int
>>+gk20a_bar_ctor(struct nouveau_object *parent, struct nouveau_object
>>*engine,
>>+             struct nouveau_oclass *oclass, void *data, u32 size,
>>+             struct nouveau_object **pobject)
>>+{
>>+      struct nouveau_bar *bar;
>>+      int ret;
>>+
>>+      ret = nvc0_bar_ctor(parent, engine, oclass, data, size, pobject);
>>+      if (ret)
>>+              return ret;
>>+
>>+      bar = (struct nouveau_bar *)*pobject;
>>+      bar->iomap_uncached = true;
>>+
>>+      return 0;
>>+}
>>+
>>+struct nouveau_oclass
>>+gk20a_bar_oclass = {
>>+      .handle = NV_SUBDEV(BAR, 0xea),
>>+      .ofuncs = &(struct nouveau_ofuncs) {
>>+              .ctor = gk20a_bar_ctor,
>>+              .dtor = nvc0_bar_dtor,
>>+              .init = nvc0_bar_init,
>>+              .fini = _nouveau_bar_fini,
>>+      },
>>+};
>>diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>>b/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>>index ca8139b9ab27..0a44459844e3 100644
>>--- a/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>>+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>>@@ -133,7 +133,7 @@ nvc0_bar_init_vm(struct nvc0_bar_priv *priv, struct
>>nvc0_bar_priv_vm *bar_vm,
>>       return 0;
>> }
>>
>>-static int
>>+int
>> nvc0_bar_ctor(struct nouveau_object *parent, struct nouveau_object
>>*engine,
>>             struct nouveau_oclass *oclass, void *data, u32 size,
>>             struct nouveau_object **pobject)
>>@@ -169,7 +169,7 @@ nvc0_bar_ctor(struct nouveau_object *parent, struct
>>nouveau_object *engine,
>>       return 0;
>> }
>>
>>-static void
>>+void
>> nvc0_bar_dtor(struct nouveau_object *object)
>> {
>>       struct nvc0_bar_priv *priv = (void *)object;
>>@@ -188,7 +188,7 @@ nvc0_bar_dtor(struct nouveau_object *object)
>>       nouveau_bar_destroy(&priv->base);
>> }
>>
>>-static int
>>+int
>> nvc0_bar_init(struct nouveau_object *object)
>> {
>>       struct nvc0_bar_priv *priv = (void *)object;
>>diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>>b/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>>index ffad8f337ead..3ee8b1476d00 100644
>>--- a/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>>+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>>@@ -23,4 +23,10 @@ int  nouveau_bar_alloc(struct nouveau_bar *, struct
>>nouveau_object *,
>>
>> void nv84_bar_flush(struct nouveau_bar *);
>>
>>+int nvc0_bar_ctor(struct nouveau_object *, struct nouveau_object *,
>>+                struct nouveau_oclass *, void *, u32,
>>+                struct nouveau_object **);
>>+void nvc0_bar_dtor(struct nouveau_object *);
>>+int nvc0_bar_init(struct nouveau_object *);
>>+
>> #endif
>>--
>>2.0.0
>>
>>--
>>To unsubscribe from this list: send the line "unsubscribe linux-tegra" in
>>the body of a message to majordomo@vger.kernel.org
>>More majordomo info at  http://vger.kernel.org/majordomo-info.html
>
> _______________________________________________
> dri-devel mailing list
> dri-devel@lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 15+ messages in thread
* Re: [PATCH v2] drm/gk20a: add BAR instance
  2014-06-28  0:56                 ` Ben Skeggs
  (?)
@ 2014-06-28  1:41                 ` Ken Adams
       [not found]                   ` <CFD3932D.8F2A2%kadams-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
  -1 siblings, 1 reply; 15+ messages in thread
From: Ken Adams @ 2014-06-28  1:41 UTC (permalink / raw)
  To: Ben Skeggs
  Cc: Alex Courbot, Ben Skeggs, linux-tegra, nouveau, linux-kernel,
	dri-devel, gnurou


On 6/27/14 8:56 PM, "Ben Skeggs" <skeggsb@gmail.com> wrote:

>On Sat, Jun 28, 2014 at 4:51 AM, Ken Adams <KAdams@nvidia.com> wrote:
>> quick note re: tegra and gpu bars...
>>
>> to this point we've explicitly avoided providing user-mode mappings due
>>to
>> power management issues, etc.
>> looks to me like this would allow such mappings.  is that the case?  are
>> there any paths which would require such mappings to function properly?

>What power management issues are you worried about in particular?  We
>have these concerns on discrete cards too, when doing things like
>changing vram frequencies.  TTM is able to kick out all userspace
>mappings, and clients will then block in the fault handler until it's
>safe - if they touch the mappings.
>
>Ben.


hi ben,

primarily it's the access problem you mentioned.  managing those mappings,
and kicking them out at best adds to the latency to take down power/detach
busii and the like.

and, generally, there are very few (if any) cases where there isn't a
better way to manipulate the pixels than with the cpu :) i understand
there are plenty of paths i don't know about hereŠ and so i asked.

it's a solvable problem, of course.  but especially in the mobile world it
can pop up unexpectedly.  typically on someone's perf/power/stress tests :)

---
ken





>
>>
>> thanks
>> ---
>> ken
>>
>> p.s.: hello :)
>>
>> On 6/27/14 7:36 AM, "Alex Courbot" <acourbot@nvidia.com> wrote:
>>
>>>GK20A's BAR is functionally identical to NVC0's, but do not support
>>>being ioremapped write-combined. Create a BAR instance for GK20A that
>>>reflect that state.
>>>
>>>Signed-off-by: Alexandre Courbot <acourbot@nvidia.com>
>>>---
>>>Changes since v1:
>>>- Fix compilation warning due to missing cast
>>>
>>>Patch 1 of the series was ok and thus has not been resent.
>>>
>>> drivers/gpu/drm/nouveau/Makefile                  |  1 +
>>> drivers/gpu/drm/nouveau/core/engine/device/nve0.c |  2 +-
>>> drivers/gpu/drm/nouveau/core/include/subdev/bar.h |  1 +
>>> drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c   | 54
>>>+++++++++++++++++++++++
>>> drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c    |  6 +--
>>> drivers/gpu/drm/nouveau/core/subdev/bar/priv.h    |  6 +++
>>> 6 files changed, 66 insertions(+), 4 deletions(-)
>>> create mode 100644 drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>>>
>>>diff --git a/drivers/gpu/drm/nouveau/Makefile
>>>b/drivers/gpu/drm/nouveau/Makefile
>>>index 8b307e143632..11d9561d67c1 100644
>>>--- a/drivers/gpu/drm/nouveau/Makefile
>>>+++ b/drivers/gpu/drm/nouveau/Makefile
>>>@@ -26,6 +26,7 @@ nouveau-y += core/core/subdev.o
>>> nouveau-y += core/subdev/bar/base.o
>>> nouveau-y += core/subdev/bar/nv50.o
>>> nouveau-y += core/subdev/bar/nvc0.o
>>>+nouveau-y += core/subdev/bar/gk20a.o
>>> nouveau-y += core/subdev/bios/base.o
>>> nouveau-y += core/subdev/bios/bit.o
>>> nouveau-y += core/subdev/bios/boost.o
>>>diff --git a/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>>>b/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>>>index 2d1e97d4264f..a2b9ccc48f66 100644
>>>--- a/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>>>+++ b/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>>>@@ -165,7 +165,7 @@ nve0_identify(struct nouveau_device *device)
>>>               device->oclass[NVDEV_SUBDEV_IBUS   ] =
>>>&gk20a_ibus_oclass;
>>>               device->oclass[NVDEV_SUBDEV_INSTMEM] =
>>>nv50_instmem_oclass;
>>>               device->oclass[NVDEV_SUBDEV_VM     ] =
>>>&nvc0_vmmgr_oclass;
>>>-              device->oclass[NVDEV_SUBDEV_BAR    ] = &nvc0_bar_oclass;
>>>+              device->oclass[NVDEV_SUBDEV_BAR    ] = &gk20a_bar_oclass;
>>>               device->oclass[NVDEV_ENGINE_DMAOBJ ] =
>>>&nvd0_dmaeng_oclass;
>>>               device->oclass[NVDEV_ENGINE_FIFO   ] =
>>>gk20a_fifo_oclass;
>>>               device->oclass[NVDEV_ENGINE_SW     ] =
>>>nvc0_software_oclass;
>>>diff --git a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>>>b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>>>index 9002cbb6432b..be037fac534c 100644
>>>--- a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>>>+++ b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>>>@@ -33,5 +33,6 @@ nouveau_bar(void *obj)
>>>
>>> extern struct nouveau_oclass nv50_bar_oclass;
>>> extern struct nouveau_oclass nvc0_bar_oclass;
>>>+extern struct nouveau_oclass gk20a_bar_oclass;
>>>
>>> #endif
>>>diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>>>b/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>>>new file mode 100644
>>>index 000000000000..bf877af9d3bd
>>>--- /dev/null
>>>+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>>>@@ -0,0 +1,54 @@
>>>+/*
>>>+ * Copyright (c) 2014, NVIDIA CORPORATION. All rights reserved.
>>>+ *
>>>+ * Permission is hereby granted, free of charge, to any person
>>>obtaining
>>>a
>>>+ * copy of this software and associated documentation files (the
>>>"Software"),
>>>+ * to deal in the Software without restriction, including without
>>>limitation
>>>+ * the rights to use, copy, modify, merge, publish, distribute,
>>>sublicense,
>>>+ * and/or sell copies of the Software, and to permit persons to whom
>>>the
>>>+ * Software is furnished to do so, subject to the following conditions:
>>>+ *
>>>+ * The above copyright notice and this permission notice shall be
>>>included in
>>>+ * all copies or substantial portions of the Software.
>>>+ *
>>>+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
>>>EXPRESS OR
>>>+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
>>>MERCHANTABILITY,
>>>+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT
>>>SHALL
>>>+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR
>>>OTHER
>>>+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
>>>ARISING
>>>+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
>>>+ * DEALINGS IN THE SOFTWARE.
>>>+ */
>>>+
>>>+#include <subdev/bar.h>
>>>+
>>>+#include "priv.h"
>>>+
>>>+int
>>>+gk20a_bar_ctor(struct nouveau_object *parent, struct nouveau_object
>>>*engine,
>>>+             struct nouveau_oclass *oclass, void *data, u32 size,
>>>+             struct nouveau_object **pobject)
>>>+{
>>>+      struct nouveau_bar *bar;
>>>+      int ret;
>>>+
>>>+      ret = nvc0_bar_ctor(parent, engine, oclass, data, size, pobject);
>>>+      if (ret)
>>>+              return ret;
>>>+
>>>+      bar = (struct nouveau_bar *)*pobject;
>>>+      bar->iomap_uncached = true;
>>>+
>>>+      return 0;
>>>+}
>>>+
>>>+struct nouveau_oclass
>>>+gk20a_bar_oclass = {
>>>+      .handle = NV_SUBDEV(BAR, 0xea),
>>>+      .ofuncs = &(struct nouveau_ofuncs) {
>>>+              .ctor = gk20a_bar_ctor,
>>>+              .dtor = nvc0_bar_dtor,
>>>+              .init = nvc0_bar_init,
>>>+              .fini = _nouveau_bar_fini,
>>>+      },
>>>+};
>>>diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>>>b/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>>>index ca8139b9ab27..0a44459844e3 100644
>>>--- a/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>>>+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>>>@@ -133,7 +133,7 @@ nvc0_bar_init_vm(struct nvc0_bar_priv *priv, struct
>>>nvc0_bar_priv_vm *bar_vm,
>>>       return 0;
>>> }
>>>
>>>-static int
>>>+int
>>> nvc0_bar_ctor(struct nouveau_object *parent, struct nouveau_object
>>>*engine,
>>>             struct nouveau_oclass *oclass, void *data, u32 size,
>>>             struct nouveau_object **pobject)
>>>@@ -169,7 +169,7 @@ nvc0_bar_ctor(struct nouveau_object *parent, struct
>>>nouveau_object *engine,
>>>       return 0;
>>> }
>>>
>>>-static void
>>>+void
>>> nvc0_bar_dtor(struct nouveau_object *object)
>>> {
>>>       struct nvc0_bar_priv *priv = (void *)object;
>>>@@ -188,7 +188,7 @@ nvc0_bar_dtor(struct nouveau_object *object)
>>>       nouveau_bar_destroy(&priv->base);
>>> }
>>>
>>>-static int
>>>+int
>>> nvc0_bar_init(struct nouveau_object *object)
>>> {
>>>       struct nvc0_bar_priv *priv = (void *)object;
>>>diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>>>b/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>>>index ffad8f337ead..3ee8b1476d00 100644
>>>--- a/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>>>+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>>>@@ -23,4 +23,10 @@ int  nouveau_bar_alloc(struct nouveau_bar *, struct
>>>nouveau_object *,
>>>
>>> void nv84_bar_flush(struct nouveau_bar *);
>>>
>>>+int nvc0_bar_ctor(struct nouveau_object *, struct nouveau_object *,
>>>+                struct nouveau_oclass *, void *, u32,
>>>+                struct nouveau_object **);
>>>+void nvc0_bar_dtor(struct nouveau_object *);
>>>+int nvc0_bar_init(struct nouveau_object *);
>>>+
>>> #endif
>>>--
>>>2.0.0
>>>
>>>--
>>>To unsubscribe from this list: send the line "unsubscribe linux-tegra"
>>>in
>>>the body of a message to majordomo@vger.kernel.org
>>>More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>
>> _______________________________________________
>> dri-devel mailing list
>> dri-devel@lists.freedesktop.org
>> http://lists.freedesktop.org/mailman/listinfo/dri-devel

^ permalink raw reply	[flat|nested] 15+ messages in thread
* Re: [PATCH v2] drm/gk20a: add BAR instance
  2014-06-28  1:41                 ` Ken Adams
@ 2014-06-28 13:10                       ` Ben Skeggs
  0 siblings, 0 replies; 15+ messages in thread
From: Ben Skeggs @ 2014-06-28 13:10 UTC (permalink / raw)
  To: Ken Adams
  Cc: Alex Courbot, Ben Skeggs, linux-tegra-u79uwXL29TY76Z2rM5mHXA,
	nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
	linux-kernel-u79uwXL29TY76Z2rM5mHXA,
	dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
	gnurou-Re5JQEeQqe8AvxtiuMwx3w

On Sat, Jun 28, 2014 at 11:41 AM, Ken Adams <KAdams-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> wrote:
>
> On 6/27/14 8:56 PM, "Ben Skeggs" <skeggsb-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org> wrote:
>
>>On Sat, Jun 28, 2014 at 4:51 AM, Ken Adams <KAdams-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> wrote:
>>> quick note re: tegra and gpu bars...
>>>
>>> to this point we've explicitly avoided providing user-mode mappings due
>>>to
>>> power management issues, etc.
>>> looks to me like this would allow such mappings.  is that the case?  are
>>> there any paths which would require such mappings to function properly?
>
>>What power management issues are you worried about in particular?  We
>>have these concerns on discrete cards too, when doing things like
>>changing vram frequencies.  TTM is able to kick out all userspace
>>mappings, and clients will then block in the fault handler until it's
>>safe - if they touch the mappings.
>>
>>Ben.
>
>
> hi ben,
>
> primarily it's the access problem you mentioned.  managing those mappings,
> and kicking them out at best adds to the latency to take down power/detach
> busii and the like.
>
> and, generally, there are very few (if any) cases where there isn't a
> better way to manipulate the pixels than with the cpu :) i understand
> there are plenty of paths i don't know about hereŠ and so i asked.
I actually don't believe our userspace (on nv50 and up at least)
touches them directly either in most (all?) cases.  It's definitely
something one wants to avoid if at all possible, that's for sure; but
the possibility is there to have/use the CPU mappings.

>
> it's a solvable problem, of course.  but especially in the mobile world it
> can pop up unexpectedly.  typically on someone's perf/power/stress tests :)
Unless they're actually used by userspace, I don't *think* this should
be an issue.  It's something we should probably address if it is.

Ben.

>
> ---
> ken
>
>
>
>
>
>>
>>>
>>> thanks
>>> ---
>>> ken
>>>
>>> p.s.: hello :)
>>>
>>> On 6/27/14 7:36 AM, "Alex Courbot" <acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> wrote:
>>>
>>>>GK20A's BAR is functionally identical to NVC0's, but do not support
>>>>being ioremapped write-combined. Create a BAR instance for GK20A that
>>>>reflect that state.
>>>>
>>>>Signed-off-by: Alexandre Courbot <acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
>>>>---
>>>>Changes since v1:
>>>>- Fix compilation warning due to missing cast
>>>>
>>>>Patch 1 of the series was ok and thus has not been resent.
>>>>
>>>> drivers/gpu/drm/nouveau/Makefile                  |  1 +
>>>> drivers/gpu/drm/nouveau/core/engine/device/nve0.c |  2 +-
>>>> drivers/gpu/drm/nouveau/core/include/subdev/bar.h |  1 +
>>>> drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c   | 54
>>>>+++++++++++++++++++++++
>>>> drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c    |  6 +--
>>>> drivers/gpu/drm/nouveau/core/subdev/bar/priv.h    |  6 +++
>>>> 6 files changed, 66 insertions(+), 4 deletions(-)
>>>> create mode 100644 drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>>>>
>>>>diff --git a/drivers/gpu/drm/nouveau/Makefile
>>>>b/drivers/gpu/drm/nouveau/Makefile
>>>>index 8b307e143632..11d9561d67c1 100644
>>>>--- a/drivers/gpu/drm/nouveau/Makefile
>>>>+++ b/drivers/gpu/drm/nouveau/Makefile
>>>>@@ -26,6 +26,7 @@ nouveau-y += core/core/subdev.o
>>>> nouveau-y += core/subdev/bar/base.o
>>>> nouveau-y += core/subdev/bar/nv50.o
>>>> nouveau-y += core/subdev/bar/nvc0.o
>>>>+nouveau-y += core/subdev/bar/gk20a.o
>>>> nouveau-y += core/subdev/bios/base.o
>>>> nouveau-y += core/subdev/bios/bit.o
>>>> nouveau-y += core/subdev/bios/boost.o
>>>>diff --git a/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>>>>b/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>>>>index 2d1e97d4264f..a2b9ccc48f66 100644
>>>>--- a/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>>>>+++ b/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>>>>@@ -165,7 +165,7 @@ nve0_identify(struct nouveau_device *device)
>>>>               device->oclass[NVDEV_SUBDEV_IBUS   ] =
>>>>&gk20a_ibus_oclass;
>>>>               device->oclass[NVDEV_SUBDEV_INSTMEM] =
>>>>nv50_instmem_oclass;
>>>>               device->oclass[NVDEV_SUBDEV_VM     ] =
>>>>&nvc0_vmmgr_oclass;
>>>>-              device->oclass[NVDEV_SUBDEV_BAR    ] = &nvc0_bar_oclass;
>>>>+              device->oclass[NVDEV_SUBDEV_BAR    ] = &gk20a_bar_oclass;
>>>>               device->oclass[NVDEV_ENGINE_DMAOBJ ] =
>>>>&nvd0_dmaeng_oclass;
>>>>               device->oclass[NVDEV_ENGINE_FIFO   ] =
>>>>gk20a_fifo_oclass;
>>>>               device->oclass[NVDEV_ENGINE_SW     ] =
>>>>nvc0_software_oclass;
>>>>diff --git a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>>>>b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>>>>index 9002cbb6432b..be037fac534c 100644
>>>>--- a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>>>>+++ b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>>>>@@ -33,5 +33,6 @@ nouveau_bar(void *obj)
>>>>
>>>> extern struct nouveau_oclass nv50_bar_oclass;
>>>> extern struct nouveau_oclass nvc0_bar_oclass;
>>>>+extern struct nouveau_oclass gk20a_bar_oclass;
>>>>
>>>> #endif
>>>>diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>>>>b/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>>>>new file mode 100644
>>>>index 000000000000..bf877af9d3bd
>>>>--- /dev/null
>>>>+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>>>>@@ -0,0 +1,54 @@
>>>>+/*
>>>>+ * Copyright (c) 2014, NVIDIA CORPORATION. All rights reserved.
>>>>+ *
>>>>+ * Permission is hereby granted, free of charge, to any person
>>>>obtaining
>>>>a
>>>>+ * copy of this software and associated documentation files (the
>>>>"Software"),
>>>>+ * to deal in the Software without restriction, including without
>>>>limitation
>>>>+ * the rights to use, copy, modify, merge, publish, distribute,
>>>>sublicense,
>>>>+ * and/or sell copies of the Software, and to permit persons to whom
>>>>the
>>>>+ * Software is furnished to do so, subject to the following conditions:
>>>>+ *
>>>>+ * The above copyright notice and this permission notice shall be
>>>>included in
>>>>+ * all copies or substantial portions of the Software.
>>>>+ *
>>>>+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
>>>>EXPRESS OR
>>>>+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
>>>>MERCHANTABILITY,
>>>>+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT
>>>>SHALL
>>>>+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR
>>>>OTHER
>>>>+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
>>>>ARISING
>>>>+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
>>>>+ * DEALINGS IN THE SOFTWARE.
>>>>+ */
>>>>+
>>>>+#include <subdev/bar.h>
>>>>+
>>>>+#include "priv.h"
>>>>+
>>>>+int
>>>>+gk20a_bar_ctor(struct nouveau_object *parent, struct nouveau_object
>>>>*engine,
>>>>+             struct nouveau_oclass *oclass, void *data, u32 size,
>>>>+             struct nouveau_object **pobject)
>>>>+{
>>>>+      struct nouveau_bar *bar;
>>>>+      int ret;
>>>>+
>>>>+      ret = nvc0_bar_ctor(parent, engine, oclass, data, size, pobject);
>>>>+      if (ret)
>>>>+              return ret;
>>>>+
>>>>+      bar = (struct nouveau_bar *)*pobject;
>>>>+      bar->iomap_uncached = true;
>>>>+
>>>>+      return 0;
>>>>+}
>>>>+
>>>>+struct nouveau_oclass
>>>>+gk20a_bar_oclass = {
>>>>+      .handle = NV_SUBDEV(BAR, 0xea),
>>>>+      .ofuncs = &(struct nouveau_ofuncs) {
>>>>+              .ctor = gk20a_bar_ctor,
>>>>+              .dtor = nvc0_bar_dtor,
>>>>+              .init = nvc0_bar_init,
>>>>+              .fini = _nouveau_bar_fini,
>>>>+      },
>>>>+};
>>>>diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>>>>b/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>>>>index ca8139b9ab27..0a44459844e3 100644
>>>>--- a/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>>>>+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>>>>@@ -133,7 +133,7 @@ nvc0_bar_init_vm(struct nvc0_bar_priv *priv, struct
>>>>nvc0_bar_priv_vm *bar_vm,
>>>>       return 0;
>>>> }
>>>>
>>>>-static int
>>>>+int
>>>> nvc0_bar_ctor(struct nouveau_object *parent, struct nouveau_object
>>>>*engine,
>>>>             struct nouveau_oclass *oclass, void *data, u32 size,
>>>>             struct nouveau_object **pobject)
>>>>@@ -169,7 +169,7 @@ nvc0_bar_ctor(struct nouveau_object *parent, struct
>>>>nouveau_object *engine,
>>>>       return 0;
>>>> }
>>>>
>>>>-static void
>>>>+void
>>>> nvc0_bar_dtor(struct nouveau_object *object)
>>>> {
>>>>       struct nvc0_bar_priv *priv = (void *)object;
>>>>@@ -188,7 +188,7 @@ nvc0_bar_dtor(struct nouveau_object *object)
>>>>       nouveau_bar_destroy(&priv->base);
>>>> }
>>>>
>>>>-static int
>>>>+int
>>>> nvc0_bar_init(struct nouveau_object *object)
>>>> {
>>>>       struct nvc0_bar_priv *priv = (void *)object;
>>>>diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>>>>b/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>>>>index ffad8f337ead..3ee8b1476d00 100644
>>>>--- a/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>>>>+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>>>>@@ -23,4 +23,10 @@ int  nouveau_bar_alloc(struct nouveau_bar *, struct
>>>>nouveau_object *,
>>>>
>>>> void nv84_bar_flush(struct nouveau_bar *);
>>>>
>>>>+int nvc0_bar_ctor(struct nouveau_object *, struct nouveau_object *,
>>>>+                struct nouveau_oclass *, void *, u32,
>>>>+                struct nouveau_object **);
>>>>+void nvc0_bar_dtor(struct nouveau_object *);
>>>>+int nvc0_bar_init(struct nouveau_object *);
>>>>+
>>>> #endif
>>>>--
>>>>2.0.0
>>>>
>>>>--
>>>>To unsubscribe from this list: send the line "unsubscribe linux-tegra"
>>>>in
>>>>the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org
>>>>More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>>
>>> _______________________________________________
>>> dri-devel mailing list
>>> dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
>>> http://lists.freedesktop.org/mailman/listinfo/dri-devel
>

^ permalink raw reply	[flat|nested] 15+ messages in thread
* Re: [PATCH v2] drm/gk20a: add BAR instance
@ 2014-06-28 13:10                       ` Ben Skeggs
  0 siblings, 0 replies; 15+ messages in thread
From: Ben Skeggs @ 2014-06-28 13:10 UTC (permalink / raw)
  To: Ken Adams
  Cc: Alex Courbot, Ben Skeggs, linux-tegra, nouveau, linux-kernel,
	dri-devel, gnurou

On Sat, Jun 28, 2014 at 11:41 AM, Ken Adams <KAdams@nvidia.com> wrote:
>
> On 6/27/14 8:56 PM, "Ben Skeggs" <skeggsb@gmail.com> wrote:
>
>>On Sat, Jun 28, 2014 at 4:51 AM, Ken Adams <KAdams@nvidia.com> wrote:
>>> quick note re: tegra and gpu bars...
>>>
>>> to this point we've explicitly avoided providing user-mode mappings due
>>>to
>>> power management issues, etc.
>>> looks to me like this would allow such mappings.  is that the case?  are
>>> there any paths which would require such mappings to function properly?
>
>>What power management issues are you worried about in particular?  We
>>have these concerns on discrete cards too, when doing things like
>>changing vram frequencies.  TTM is able to kick out all userspace
>>mappings, and clients will then block in the fault handler until it's
>>safe - if they touch the mappings.
>>
>>Ben.
>
>
> hi ben,
>
> primarily it's the access problem you mentioned.  managing those mappings,
> and kicking them out at best adds to the latency to take down power/detach
> busii and the like.
>
> and, generally, there are very few (if any) cases where there isn't a
> better way to manipulate the pixels than with the cpu :) i understand
> there are plenty of paths i don't know about hereŠ and so i asked.
I actually don't believe our userspace (on nv50 and up at least)
touches them directly either in most (all?) cases.  It's definitely
something one wants to avoid if at all possible, that's for sure; but
the possibility is there to have/use the CPU mappings.

>
> it's a solvable problem, of course.  but especially in the mobile world it
> can pop up unexpectedly.  typically on someone's perf/power/stress tests :)
Unless they're actually used by userspace, I don't *think* this should
be an issue.  It's something we should probably address if it is.

Ben.

>
> ---
> ken
>
>
>
>
>
>>
>>>
>>> thanks
>>> ---
>>> ken
>>>
>>> p.s.: hello :)
>>>
>>> On 6/27/14 7:36 AM, "Alex Courbot" <acourbot@nvidia.com> wrote:
>>>
>>>>GK20A's BAR is functionally identical to NVC0's, but do not support
>>>>being ioremapped write-combined. Create a BAR instance for GK20A that
>>>>reflect that state.
>>>>
>>>>Signed-off-by: Alexandre Courbot <acourbot@nvidia.com>
>>>>---
>>>>Changes since v1:
>>>>- Fix compilation warning due to missing cast
>>>>
>>>>Patch 1 of the series was ok and thus has not been resent.
>>>>
>>>> drivers/gpu/drm/nouveau/Makefile                  |  1 +
>>>> drivers/gpu/drm/nouveau/core/engine/device/nve0.c |  2 +-
>>>> drivers/gpu/drm/nouveau/core/include/subdev/bar.h |  1 +
>>>> drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c   | 54
>>>>+++++++++++++++++++++++
>>>> drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c    |  6 +--
>>>> drivers/gpu/drm/nouveau/core/subdev/bar/priv.h    |  6 +++
>>>> 6 files changed, 66 insertions(+), 4 deletions(-)
>>>> create mode 100644 drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>>>>
>>>>diff --git a/drivers/gpu/drm/nouveau/Makefile
>>>>b/drivers/gpu/drm/nouveau/Makefile
>>>>index 8b307e143632..11d9561d67c1 100644
>>>>--- a/drivers/gpu/drm/nouveau/Makefile
>>>>+++ b/drivers/gpu/drm/nouveau/Makefile
>>>>@@ -26,6 +26,7 @@ nouveau-y += core/core/subdev.o
>>>> nouveau-y += core/subdev/bar/base.o
>>>> nouveau-y += core/subdev/bar/nv50.o
>>>> nouveau-y += core/subdev/bar/nvc0.o
>>>>+nouveau-y += core/subdev/bar/gk20a.o
>>>> nouveau-y += core/subdev/bios/base.o
>>>> nouveau-y += core/subdev/bios/bit.o
>>>> nouveau-y += core/subdev/bios/boost.o
>>>>diff --git a/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>>>>b/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>>>>index 2d1e97d4264f..a2b9ccc48f66 100644
>>>>--- a/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>>>>+++ b/drivers/gpu/drm/nouveau/core/engine/device/nve0.c
>>>>@@ -165,7 +165,7 @@ nve0_identify(struct nouveau_device *device)
>>>>               device->oclass[NVDEV_SUBDEV_IBUS   ] =
>>>>&gk20a_ibus_oclass;
>>>>               device->oclass[NVDEV_SUBDEV_INSTMEM] =
>>>>nv50_instmem_oclass;
>>>>               device->oclass[NVDEV_SUBDEV_VM     ] =
>>>>&nvc0_vmmgr_oclass;
>>>>-              device->oclass[NVDEV_SUBDEV_BAR    ] = &nvc0_bar_oclass;
>>>>+              device->oclass[NVDEV_SUBDEV_BAR    ] = &gk20a_bar_oclass;
>>>>               device->oclass[NVDEV_ENGINE_DMAOBJ ] =
>>>>&nvd0_dmaeng_oclass;
>>>>               device->oclass[NVDEV_ENGINE_FIFO   ] =
>>>>gk20a_fifo_oclass;
>>>>               device->oclass[NVDEV_ENGINE_SW     ] =
>>>>nvc0_software_oclass;
>>>>diff --git a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>>>>b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>>>>index 9002cbb6432b..be037fac534c 100644
>>>>--- a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>>>>+++ b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
>>>>@@ -33,5 +33,6 @@ nouveau_bar(void *obj)
>>>>
>>>> extern struct nouveau_oclass nv50_bar_oclass;
>>>> extern struct nouveau_oclass nvc0_bar_oclass;
>>>>+extern struct nouveau_oclass gk20a_bar_oclass;
>>>>
>>>> #endif
>>>>diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>>>>b/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>>>>new file mode 100644
>>>>index 000000000000..bf877af9d3bd
>>>>--- /dev/null
>>>>+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/gk20a.c
>>>>@@ -0,0 +1,54 @@
>>>>+/*
>>>>+ * Copyright (c) 2014, NVIDIA CORPORATION. All rights reserved.
>>>>+ *
>>>>+ * Permission is hereby granted, free of charge, to any person
>>>>obtaining
>>>>a
>>>>+ * copy of this software and associated documentation files (the
>>>>"Software"),
>>>>+ * to deal in the Software without restriction, including without
>>>>limitation
>>>>+ * the rights to use, copy, modify, merge, publish, distribute,
>>>>sublicense,
>>>>+ * and/or sell copies of the Software, and to permit persons to whom
>>>>the
>>>>+ * Software is furnished to do so, subject to the following conditions:
>>>>+ *
>>>>+ * The above copyright notice and this permission notice shall be
>>>>included in
>>>>+ * all copies or substantial portions of the Software.
>>>>+ *
>>>>+ * THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND,
>>>>EXPRESS OR
>>>>+ * IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF
>>>>MERCHANTABILITY,
>>>>+ * FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT.  IN NO EVENT
>>>>SHALL
>>>>+ * THE AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR
>>>>OTHER
>>>>+ * LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE,
>>>>ARISING
>>>>+ * FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
>>>>+ * DEALINGS IN THE SOFTWARE.
>>>>+ */
>>>>+
>>>>+#include <subdev/bar.h>
>>>>+
>>>>+#include "priv.h"
>>>>+
>>>>+int
>>>>+gk20a_bar_ctor(struct nouveau_object *parent, struct nouveau_object
>>>>*engine,
>>>>+             struct nouveau_oclass *oclass, void *data, u32 size,
>>>>+             struct nouveau_object **pobject)
>>>>+{
>>>>+      struct nouveau_bar *bar;
>>>>+      int ret;
>>>>+
>>>>+      ret = nvc0_bar_ctor(parent, engine, oclass, data, size, pobject);
>>>>+      if (ret)
>>>>+              return ret;
>>>>+
>>>>+      bar = (struct nouveau_bar *)*pobject;
>>>>+      bar->iomap_uncached = true;
>>>>+
>>>>+      return 0;
>>>>+}
>>>>+
>>>>+struct nouveau_oclass
>>>>+gk20a_bar_oclass = {
>>>>+      .handle = NV_SUBDEV(BAR, 0xea),
>>>>+      .ofuncs = &(struct nouveau_ofuncs) {
>>>>+              .ctor = gk20a_bar_ctor,
>>>>+              .dtor = nvc0_bar_dtor,
>>>>+              .init = nvc0_bar_init,
>>>>+              .fini = _nouveau_bar_fini,
>>>>+      },
>>>>+};
>>>>diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>>>>b/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>>>>index ca8139b9ab27..0a44459844e3 100644
>>>>--- a/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>>>>+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/nvc0.c
>>>>@@ -133,7 +133,7 @@ nvc0_bar_init_vm(struct nvc0_bar_priv *priv, struct
>>>>nvc0_bar_priv_vm *bar_vm,
>>>>       return 0;
>>>> }
>>>>
>>>>-static int
>>>>+int
>>>> nvc0_bar_ctor(struct nouveau_object *parent, struct nouveau_object
>>>>*engine,
>>>>             struct nouveau_oclass *oclass, void *data, u32 size,
>>>>             struct nouveau_object **pobject)
>>>>@@ -169,7 +169,7 @@ nvc0_bar_ctor(struct nouveau_object *parent, struct
>>>>nouveau_object *engine,
>>>>       return 0;
>>>> }
>>>>
>>>>-static void
>>>>+void
>>>> nvc0_bar_dtor(struct nouveau_object *object)
>>>> {
>>>>       struct nvc0_bar_priv *priv = (void *)object;
>>>>@@ -188,7 +188,7 @@ nvc0_bar_dtor(struct nouveau_object *object)
>>>>       nouveau_bar_destroy(&priv->base);
>>>> }
>>>>
>>>>-static int
>>>>+int
>>>> nvc0_bar_init(struct nouveau_object *object)
>>>> {
>>>>       struct nvc0_bar_priv *priv = (void *)object;
>>>>diff --git a/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>>>>b/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>>>>index ffad8f337ead..3ee8b1476d00 100644
>>>>--- a/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>>>>+++ b/drivers/gpu/drm/nouveau/core/subdev/bar/priv.h
>>>>@@ -23,4 +23,10 @@ int  nouveau_bar_alloc(struct nouveau_bar *, struct
>>>>nouveau_object *,
>>>>
>>>> void nv84_bar_flush(struct nouveau_bar *);
>>>>
>>>>+int nvc0_bar_ctor(struct nouveau_object *, struct nouveau_object *,
>>>>+                struct nouveau_oclass *, void *, u32,
>>>>+                struct nouveau_object **);
>>>>+void nvc0_bar_dtor(struct nouveau_object *);
>>>>+int nvc0_bar_init(struct nouveau_object *);
>>>>+
>>>> #endif
>>>>--
>>>>2.0.0
>>>>
>>>>--
>>>>To unsubscribe from this list: send the line "unsubscribe linux-tegra"
>>>>in
>>>>the body of a message to majordomo@vger.kernel.org
>>>>More majordomo info at  http://vger.kernel.org/majordomo-info.html
>>>
>>> _______________________________________________
>>> dri-devel mailing list
>>> dri-devel@lists.freedesktop.org
>>> http://lists.freedesktop.org/mailman/listinfo/dri-devel
>

^ permalink raw reply	[flat|nested] 15+ messages in thread
* Re: [PATCH 1/2] drm/nouveau/bar: add noncached ioremap property
  2014-06-27 10:28 ` Alexandre Courbot
@ 2014-07-08  7:41     ` Alexandre Courbot
  -1 siblings, 0 replies; 15+ messages in thread
From: Alexandre Courbot @ 2014-07-08  7:41 UTC (permalink / raw)
  To: Alexandre Courbot
  Cc: linux-tegra-u79uwXL29TY76Z2rM5mHXA,
	nouveau-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW, Ben Skeggs,
	dri-devel-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW,
	Linux Kernel Mailing List

Ping Ben, how do these two patches look like?

On Fri, Jun 27, 2014 at 7:28 PM, Alexandre Courbot <acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org> wrote:
> Some BARs (like GK20A's) do not support being ioremapped write-combined.
> Add a boolean property to the BAR structure and handle that case in the
> Nouveau BO implementation.
>
> Signed-off-by: Alexandre Courbot <acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
> ---
>  drivers/gpu/drm/nouveau/core/include/subdev/bar.h |  3 +++
>  drivers/gpu/drm/nouveau/nouveau_bo.c              | 17 ++++++++++++-----
>  2 files changed, 15 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
> index 9faa98e67ad8..9002cbb6432b 100644
> --- a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
> +++ b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
> @@ -20,6 +20,9 @@ struct nouveau_bar {
>                     u32 flags, struct nouveau_vma *);
>         void (*unmap)(struct nouveau_bar *, struct nouveau_vma *);
>         void (*flush)(struct nouveau_bar *);
> +
> +       /* whether the BAR supports to be ioremapped WC or should be uncached */
> +       bool iomap_uncached;
>  };
>
>  static inline struct nouveau_bar *
> diff --git a/drivers/gpu/drm/nouveau/nouveau_bo.c b/drivers/gpu/drm/nouveau/nouveau_bo.c
> index b6dc85c614be..4db886f9f793 100644
> --- a/drivers/gpu/drm/nouveau/nouveau_bo.c
> +++ b/drivers/gpu/drm/nouveau/nouveau_bo.c
> @@ -500,18 +500,25 @@ nouveau_bo_init_mem_type(struct ttm_bo_device *bdev, uint32_t type,
>                 man->default_caching = TTM_PL_FLAG_CACHED;
>                 break;
>         case TTM_PL_VRAM:
> +               man->flags = TTM_MEMTYPE_FLAG_FIXED |
> +                            TTM_MEMTYPE_FLAG_MAPPABLE;
> +               man->available_caching = TTM_PL_FLAG_UNCACHED |
> +                                        TTM_PL_FLAG_WC;
> +               man->default_caching = TTM_PL_FLAG_WC;
> +
>                 if (nv_device(drm->device)->card_type >= NV_50) {
> +                       /* Some BARs do not support being ioremapped WC */
> +                       if (nouveau_bar(drm->device)->iomap_uncached) {
> +                               man->available_caching = TTM_PL_FLAG_UNCACHED;
> +                               man->default_caching = TTM_PL_FLAG_UNCACHED;
> +                       }
> +
>                         man->func = &nouveau_vram_manager;
>                         man->io_reserve_fastpath = false;
>                         man->use_io_reserve_lru = true;
>                 } else {
>                         man->func = &ttm_bo_manager_func;
>                 }
> -               man->flags = TTM_MEMTYPE_FLAG_FIXED |
> -                            TTM_MEMTYPE_FLAG_MAPPABLE;
> -               man->available_caching = TTM_PL_FLAG_UNCACHED |
> -                                        TTM_PL_FLAG_WC;
> -               man->default_caching = TTM_PL_FLAG_WC;
>                 break;
>         case TTM_PL_TT:
>                 if (nv_device(drm->device)->card_type >= NV_50)
> --
> 2.0.0
>

^ permalink raw reply	[flat|nested] 15+ messages in thread
* Re: [PATCH 1/2] drm/nouveau/bar: add noncached ioremap property
@ 2014-07-08  7:41     ` Alexandre Courbot
  0 siblings, 0 replies; 15+ messages in thread
From: Alexandre Courbot @ 2014-07-08  7:41 UTC (permalink / raw)
  To: Alexandre Courbot
  Cc: Ben Skeggs, nouveau, dri-devel, linux-tegra, Linux Kernel Mailing List

Ping Ben, how do these two patches look like?

On Fri, Jun 27, 2014 at 7:28 PM, Alexandre Courbot <acourbot@nvidia.com> wrote:
> Some BARs (like GK20A's) do not support being ioremapped write-combined.
> Add a boolean property to the BAR structure and handle that case in the
> Nouveau BO implementation.
>
> Signed-off-by: Alexandre Courbot <acourbot@nvidia.com>
> ---
>  drivers/gpu/drm/nouveau/core/include/subdev/bar.h |  3 +++
>  drivers/gpu/drm/nouveau/nouveau_bo.c              | 17 ++++++++++++-----
>  2 files changed, 15 insertions(+), 5 deletions(-)
>
> diff --git a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
> index 9faa98e67ad8..9002cbb6432b 100644
> --- a/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
> +++ b/drivers/gpu/drm/nouveau/core/include/subdev/bar.h
> @@ -20,6 +20,9 @@ struct nouveau_bar {
>                     u32 flags, struct nouveau_vma *);
>         void (*unmap)(struct nouveau_bar *, struct nouveau_vma *);
>         void (*flush)(struct nouveau_bar *);
> +
> +       /* whether the BAR supports to be ioremapped WC or should be uncached */
> +       bool iomap_uncached;
>  };
>
>  static inline struct nouveau_bar *
> diff --git a/drivers/gpu/drm/nouveau/nouveau_bo.c b/drivers/gpu/drm/nouveau/nouveau_bo.c
> index b6dc85c614be..4db886f9f793 100644
> --- a/drivers/gpu/drm/nouveau/nouveau_bo.c
> +++ b/drivers/gpu/drm/nouveau/nouveau_bo.c
> @@ -500,18 +500,25 @@ nouveau_bo_init_mem_type(struct ttm_bo_device *bdev, uint32_t type,
>                 man->default_caching = TTM_PL_FLAG_CACHED;
>                 break;
>         case TTM_PL_VRAM:
> +               man->flags = TTM_MEMTYPE_FLAG_FIXED |
> +                            TTM_MEMTYPE_FLAG_MAPPABLE;
> +               man->available_caching = TTM_PL_FLAG_UNCACHED |
> +                                        TTM_PL_FLAG_WC;
> +               man->default_caching = TTM_PL_FLAG_WC;
> +
>                 if (nv_device(drm->device)->card_type >= NV_50) {
> +                       /* Some BARs do not support being ioremapped WC */
> +                       if (nouveau_bar(drm->device)->iomap_uncached) {
> +                               man->available_caching = TTM_PL_FLAG_UNCACHED;
> +                               man->default_caching = TTM_PL_FLAG_UNCACHED;
> +                       }
> +
>                         man->func = &nouveau_vram_manager;
>                         man->io_reserve_fastpath = false;
>                         man->use_io_reserve_lru = true;
>                 } else {
>                         man->func = &ttm_bo_manager_func;
>                 }
> -               man->flags = TTM_MEMTYPE_FLAG_FIXED |
> -                            TTM_MEMTYPE_FLAG_MAPPABLE;
> -               man->available_caching = TTM_PL_FLAG_UNCACHED |
> -                                        TTM_PL_FLAG_WC;
> -               man->default_caching = TTM_PL_FLAG_WC;
>                 break;
>         case TTM_PL_TT:
>                 if (nv_device(drm->device)->card_type >= NV_50)
> --
> 2.0.0
>

^ permalink raw reply	[flat|nested] 15+ messages in thread
end of thread, other threads:[~2014-07-08  7:41 UTC | newest]

Thread overview: 15+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-06-27 10:28 [PATCH 1/2] drm/nouveau/bar: add noncached ioremap property Alexandre Courbot
2014-06-27 10:28 ` Alexandre Courbot
     [not found] ` <1403864931-4663-1-git-send-email-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2014-06-27 10:28   ` [PATCH 2/2] drm/gk20a: add BAR instance Alexandre Courbot
2014-06-27 10:28     ` Alexandre Courbot
     [not found]     ` <1403864931-4663-2-git-send-email-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2014-06-27 11:36       ` [PATCH v2] " Alexandre Courbot
2014-06-27 11:36         ` Alexandre Courbot
     [not found]         ` <1403869014-16209-1-git-send-email-acourbot-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2014-06-27 18:51           ` Ken Adams
2014-06-27 18:51             ` Ken Adams
     [not found]             ` <CFD332E4.8F1A4%kadams-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2014-06-28  0:56               ` Ben Skeggs
2014-06-28  0:56                 ` Ben Skeggs
2014-06-28  1:41                 ` Ken Adams
     [not found]                   ` <CFD3932D.8F2A2%kadams-DDmLM1+adcrQT0dZR+AlfA@public.gmane.org>
2014-06-28 13:10                     ` Ben Skeggs
2014-06-28 13:10                       ` Ben Skeggs
2014-07-08  7:41   ` [PATCH 1/2] drm/nouveau/bar: add noncached ioremap property Alexandre Courbot
2014-07-08  7:41     ` Alexandre Courbot

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.