On Tue, Dec 11, 2012 at 11:07:17AM -0800, K. Y. Srinivasan wrote: > -probe_error1: > +probe_error2: > kthread_stop(dm_device.thread); > > -probe_error0: > +probe_error1: > vmbus_close(dev->channel); > +probe_error0: > + kfree(send_buffer); If you used labels that describe what the code does instead of GW-BASIC labels then you wouldn't need to rename them. err_stop_thread: kthread_stop(dm_device.thread); err_close_vmbus: vmbus_close(dev->channel); err_free_send_buf: kfree(send_buffer); regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/devel