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 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
