Andy, On Mon, Dec 15, 2014 at 03:47:04PM +0000, Andy Whitcroft wrote: > On Sun, Dec 14, 2014 at 11:59:19PM -0800, Jeremiah Mahler wrote: [...] > > The calculation appears identical to my reading, the original form was: > > next_gpadl_handle = atomic_read(&vmbus_connection.next_gpadl_handle); > atomic_inc(&vmbus_connection.next_gpadl_handle); > > or: > > y = x; > x++; > > so y == x' (x before incrementing) > > the new code is: > > next_gpadl_handle = (atomic_inc_return(&vmbus_connection.next_gpadl_handle) > - 1); > > or: > > y = ++x - 1; > > Also making y = x' (x before incrementing) > > -apw
Ah, you are right. The increment before/after messed me up. Thanks for clearing that up for me :-) -- - Jeremiah Mahler -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
