lkml.org 
[lkml]   [2011]   [Apr]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    /
    Date
    From
    SubjectRe: [PATCH 08/25] Staging: hv: vmbus_driver cannot be unloaded; cleanup accordingly
    On Wed, Apr 27, 2011 at 02:31:18AM +0000, KY Srinivasan wrote:
    >
    >
    > > -----Original Message-----
    > > From: Greg KH [mailto:greg@kroah.com]
    > > Sent: Tuesday, April 26, 2011 6:46 PM
    > > To: KY Srinivasan
    > > Cc: gregkh@suse.de; linux-kernel@vger.kernel.org;
    > > devel@linuxdriverproject.org; virtualization@lists.osdl.org; Haiyang Zhang;
    > > Abhishek Kane (Mindtree Consulting PVT LTD)
    > > Subject: Re: [PATCH 08/25] Staging: hv: vmbus_driver cannot be unloaded;
    > > cleanup accordingly
    > >
    > > On Tue, Apr 26, 2011 at 09:20:25AM -0700, K. Y. Srinivasan wrote:
    > > > The vmbus driver cannot be unloaded; the windows host does not
    > > > permit this. Cleanup accordingly.
    > >
    > > Woah, you just prevented this driver from ever being able to be
    > > unloaded.
    >
    > It was never unloadable; while the driver defined an exit routine,
    > there were couple of issues unloading the vmbus driver:
    >
    > 1) All guest resources given to the host could not be recovered.

    Is this a problem in the Linux side? If so, that could easily be fixed.

    > 2) Windows host would not permit reloading the driver without
    > rebooting the guest.

    That's a different issue, and one that I am very surprised to hear.
    That kind of invalidates ever being able to update the driver in a guest
    for a long-running system that you want to migrate and not reboot. That
    sounds like a major bug in hyper-v, don't you agree?

    > All I did was acknowledge the current state and cleanup
    > accordingly. This is not unique to Hyper-V; for what it is worth,
    > the Xen platform_pci driver which is equivalent to the vmbus driver
    > is also not unlodable (the last time I checked).

    Why isn't that allowed to be unloaded? What happens if it does?

    I would like to see the following be possible from Linux:
    - running Linux guest on hyperv
    - need to migrate to a newer version of hyper-v
    - pause long-running userspace processes.
    - unload hyperv modules
    - migrate guest to newer hyperv version (possible different host
    machine)
    - load newer hyperv modules
    - resume long-running guest processes

    If this isn't possible due to hyper-v bugs, then I guess we need to be
    able to live with it, but we had better advertise it pretty well as I
    know people will want to be able to do the above sequence for their
    guest instances.

    If so, can you expand this patch to say more in the changelog entry, and
    resend the remaining patches that I didn't apply as they are now gone
    from my pending-patch queue.

    > > That's not a "cleanup" that's a major change in how things work. I'm
    > > sure, if you want to continue down this line, there are more things you
    > > can remove from the code, right?
    > >
    > > What is the real issue here? What happens if you unload the bus? What
    > > goes wrong? Can it be fixed?
    >
    > This needs to be fixed on the host side. I have notified them of the issue.

    Ok, so if this is going to be fixed, why do we need to prevent this from
    ever being possible to have happen on our side?

    thanks,

    greg k-h


    \
     
     \ /
      Last update: 2011-04-28 02:51    [W:4.430 / U:0.432 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site