[Nouveau] [PATCH 3/4] nouveau: move nouveau_map_block() call to nouveau_addmap()

Pekka J Enberg penberg at cs.helsinki.fi
Sun Mar 9 09:36:45 PDT 2008


From: Pekka Enberg <penberg at cs.helsinki.fi>

Move nouveau_map_block() call from nouveau_mem_alloc() to nouveau_addmap() to
simplify error handling.

Signed-off-by: Pekka Enberg <penberg at cs.helsinki.fi>
---
 shared-core/nouveau_mem.c |   14 +++++++-------
 1 file changed, 7 insertions(+), 7 deletions(-)

Index: drm/shared-core/nouveau_mem.c
===================================================================
--- drm.orig/shared-core/nouveau_mem.c
+++ drm/shared-core/nouveau_mem.c
@@ -532,8 +532,13 @@ static int nouveau_map_block(struct drm_
 static int nouveau_addmap(struct drm_device *dev, struct mem_block *block,
 			  unsigned long offset, enum drm_map_type type)
 {
-	return drm_addmap(dev, block->start, block->size, type, 0,
-			  &block->map);
+	int err;
+
+	err = drm_addmap(dev, block->start, block->size, type, 0, &block->map);
+	if (err)
+		return err;
+
+	return nouveau_map_block(dev, block);
 }
 
 static int nouveau_addmap_pci(struct drm_device *dev, struct mem_block *block)
@@ -646,11 +651,6 @@ alloc_ok:
 			nouveau_mem_free_block(block);
 			return NULL;
 		}
-
-		if (!nouveau_map_block(dev, block)) {
-			nouveau_mem_free_block(block);
-			return NULL;
-		}
 	}
 
 	DRM_DEBUG("allocated %lld bytes at 0x%llx type=0x%08x\n", block->size, block->start, block->flags);


More information about the Nouveau mailing list