RE: [PATCH 00/25] Staging: hv: Cleanup vmbus driver code

From: KY Srinivasan
Date: Tue Apr 26 2011 - 21:54:10 EST




> -----Original Message-----
> From: Greg KH [mailto:greg@xxxxxxxxx]
> Sent: Tuesday, April 26, 2011 7:29 PM
> To: KY Srinivasan
> Cc: gregkh@xxxxxxx; linux-kernel@xxxxxxxxxxxxxxx;
> devel@xxxxxxxxxxxxxxxxxxxxxx; virtualization@xxxxxxxxxxxxxx
> Subject: Re: [PATCH 00/25] Staging: hv: Cleanup vmbus driver code
>
> 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?

Greg, this is restriction imposed by the Windows host: you cannot reload the
Vmbus driver without rebooting the guest. If you cannot re-load, what good is it
to be able to unload? Distros that integrate these drivers will load these drivers
automatically on boot and there is not much point in being able to unload this since
most likely the root device will be handled by these drivers. For systems that don't
integrate these drivers; I don't see much point in allowing the driver to be unloaded,
if you cannot reload the driver without rebooting the guest. If and when the Windows
host supports reloading the vmbus driver, we can very easily add this functionality.
The situation currently at best very misleading - you think you can unload the vmbus
driver, only to discover that you have to reboot the guest!

>
> 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.

I will send you a patch for this.

> 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?)

I would prefer that we go through the review process. What is the process for
this review? Is there a time window for people to respond. I am hoping I will be able
to address all the review comments well in advance of the next closing of the tree,
with the hope of taking the vmbus driver out of staging this go around (hope springs
eternal in the human breast ...)!

Regards,

K. Y
>
> thanks,
>
> greg k-h

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/