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
[email protected]
http://driverdev.linuxdriverproject.org/mailman/listinfo/devel

Reply via email to