> Yeah, how about this ammended patch?
Both this and Greg's look fine.
I guess this is preferred, since the lock hold time is shorter :)
> + spin_lock(&sequence_lock);
> + seq = sequence_num++;
> + spin_unlock(&sequence_lock);
> +
> + envp [i++] = scratch;
> + scratch += sprintf(scratch, "SEQNUM=%ld", seq) + 1;
Nice thinking. It is a shame we need a lock for this, but we don't have
an atomic_inc_and_return().
Robert Love
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/