From: Greg KH on
On Wed, Apr 28, 2010 at 10:11:02PM +0000, Hank Janssen wrote:
>
> From: Hank Janssen <hjanssen(a)microsoft.com>
>
> Addition of new driver for Hyper-V called hv_utils.
> This driver is intended to support things like KVP, Timesync, Heartbeat etc.

Does this duplicate functionality in KY's clocksource driver? Or is it
a separate thing?

> This first release has support for graceful shutdown.
> e.g. Select shutdown from the Hyper-V main admin screen and the
> Linux VM will do a graceful shutdown.
>
> Signed-off-by: Hank Janssen <hjanssen(a)microsoft.com>
> Signed-off-by: Haiyang Zhang <haiyangz(a)microsoft.com>
> Signed-off-by: Greg Kroah-Hartman <gregkh(a)suse.de>

I did not sign off on this, I've never even seen this patch before, why
would you add my name? That is not ok at all!

greg k-h
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Hank Janssen on

> And Greg spoke these words......
>>On Wed, Apr 28, 2010 at 10:11:02PM +0000, Hank Janssen wrote:
>>
>> Addition of new driver for Hyper-V called hv_utils.
>> This driver is intended to support things like KVP, Timesync, Heartbeat etc.
>
>Does this duplicate functionality in KY's clocksource driver? Or is it
>a separate thing?

This is a separate thing, timesync registers itself with the vmbus so it receives
a time sync message from Hyper-V on cold boot and restore from saved state.
With this we set the clock on the current VM. (Human readable time basically)

What KY submitted it a driver that connects to the Hyper-V stable timesource.
There are no messages from Hyper-V for that, you have to read that directly
from the hypervisor page.

>> Signed-off-by: Hank Janssen <hjanssen(a)microsoft.com>
>> Signed-off-by: Haiyang Zhang <haiyangz(a)microsoft.com>
>> Signed-off-by: Greg Kroah-Hartman <gregkh(a)suse.de>
>
>I did not sign off on this, I've never even seen this patch before, why
>would you add my name? That is not ok at all!

I got ahead of myself. Just trying to make your workload easier :)

I will resend right away with the line removed.

Hank.

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