[Cbe-oss-dev] [PATCH] powerpc/spufs: explain conditional decrement of aff_sched_count

Andre Detsch adetsch at br.ibm.com
Tue Oct 21 02:27:07 EST 2008


This patch adds a comment to clarify why atomic_dec_if_positive is being used
to decrement gang's aff_sched_count on SPU context unbind.

Signed-off-by: Andre Detsch <adetsch at br.ibm.com>

Index: spufs/arch/powerpc/platforms/cell/spufs/sched.c
===================================================================
--- spufs.orig/arch/powerpc/platforms/cell/spufs/sched.c
+++ spufs/arch/powerpc/platforms/cell/spufs/sched.c
@@ -437,6 +437,11 @@ static void spu_unbind_context(struct sp
 		atomic_dec(&cbe_spu_info[spu->node].reserved_spus);

 	if (ctx->gang)
+		/*
+		 * If ctx->gang->aff_sched_count is positive, SPU affinity is
+		 * being considered in this gang. Using atomic_dec_if_positive
+		 * allow us to skip an explicit check for affinity in this gang
+		 */
 		atomic_dec_if_positive(&ctx->gang->aff_sched_count);

 	spu_switch_notify(spu, NULL);



More information about the cbe-oss-dev mailing list