On Tue, Apr 26, 2011 at 09:19:45AM -0700, K. Y. Srinivasan wrote: > This patch-set addresses some of the bus/driver model cleanup that > Greg sugested over the last couple of days. In this patch-set we > deal with the following issues: > > 1) Cleanup unnecessary state in struct hv_device and > struct hv_driver to be compliant with the Linux > Driver model. > > 2) Cleanup the vmbus_match() function to conform with the > Linux Driver model. > > 3) Cleanup error handling in the vmbus_probe() and > vmbus_child_device_register() functions. Fixed a > bug in the probe failure path as part of this cleanup. > > 4) The Windows host cannot handle the vmbus_driver being > unloaded and subsequently loaded. Cleanup the driver with > this in mind. I've stopped at this patch (well, I applied one more, but you can see that.) I'd like to get some confirmation that this is really what you all want to do here before applying it. If it is, care to resend them with a bit more information about this issue and why you all are making it? Anyway, other than this one, the series looks good. But you should follow-up with some driver structure changes like what Christoph said to do. After that, do you want another round of review of the code, or do you have more things you want to send in (like the name[64] removal?) thanks, greg k-h _______________________________________________ Virtualization mailing list Virtualization@xxxxxxxxxxxxxxxxxxxxxxxxxx https://lists.linux-foundation.org/mailman/listinfo/virtualization
