Module Name:    src
Committed By:   riastradh
Date:           Wed Mar 16 23:32:52 UTC 2022

Modified Files:
        src/sys/external/bsd/drm2/dist/drm/i915: i915_active.c

Log Message:
i915: Avoid freeing anything under a spin lock.


To generate a diff of this commit:
cvs rdiff -u -r1.13 -r1.14 \
    src/sys/external/bsd/drm2/dist/drm/i915/i915_active.c

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.

Modified files:

Index: src/sys/external/bsd/drm2/dist/drm/i915/i915_active.c
diff -u src/sys/external/bsd/drm2/dist/drm/i915/i915_active.c:1.13 src/sys/external/bsd/drm2/dist/drm/i915/i915_active.c:1.14
--- src/sys/external/bsd/drm2/dist/drm/i915/i915_active.c:1.13	Sun Feb 27 14:18:42 2022
+++ src/sys/external/bsd/drm2/dist/drm/i915/i915_active.c	Wed Mar 16 23:32:52 2022
@@ -1,4 +1,4 @@
-/*	$NetBSD: i915_active.c,v 1.13 2022/02/27 14:18:42 riastradh Exp $	*/
+/*	$NetBSD: i915_active.c,v 1.14 2022/03/16 23:32:52 riastradh Exp $	*/
 
 /*
  * SPDX-License-Identifier: MIT
@@ -7,7 +7,7 @@
  */
 
 #include <sys/cdefs.h>
-__KERNEL_RCSID(0, "$NetBSD: i915_active.c,v 1.13 2022/02/27 14:18:42 riastradh Exp $");
+__KERNEL_RCSID(0, "$NetBSD: i915_active.c,v 1.14 2022/03/16 23:32:52 riastradh Exp $");
 
 #include <linux/debugobjects.h>
 
@@ -301,7 +301,6 @@ active_instance(struct i915_active *ref,
 	node = rb_tree_find_node(&ref->tree.rbr_tree, &idx);
 	if (node) {
 		KASSERT(node->timeline == idx);
-		kmem_cache_free(global.slab_cache, prealloc);
 		goto out;
 	}
 #else
@@ -324,6 +323,7 @@ active_instance(struct i915_active *ref,
 #endif
 
 	node = prealloc;
+	prealloc = NULL;
 	__i915_active_fence_init(&node->base, NULL, node_retire);
 	node->ref = ref;
 	node->timeline = idx;
@@ -341,6 +341,11 @@ out:
 	ref->cache = node;
 	spin_unlock_irq(&ref->tree_lock);
 
+#ifdef __NetBSD__
+	if (prealloc)
+		kmem_cache_free(global.slab_cache, prealloc);
+#endif
+
 	BUILD_BUG_ON(offsetof(typeof(*node), base));
 	return &node->base;
 }

Reply via email to