Re: [patch 2.4.11-pre5] atomic_dec_and_lock() for alpha

From: Richard Henderson (rth@twiddle.net)
Date: Mon Oct 08 2001 - 12:24:12 EST


On Mon, Oct 08, 2001 at 07:42:57PM +0400, Ivan Kokshaysky wrote:
> + " ret\n"

I am extremely uncomfortable with you returning out of the middle of
an asm statement. What if the compiler decides to allocate a stack
frame for some reason? You'll return without deallocating it.

Please write the whole thing in assembly or avoid the early return.

r~
-
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/



This archive was generated by hypermail 2b29 : Mon Oct 15 2001 - 21:00:19 EST