RE: [PATCH v3] Tools: hv: vssdaemon: ignore the EBUSY on multiple freezing the same partition

2014-10-07 Thread Dexuan Cui
-Original Message- From: devel [mailto:driverdev-devel-boun...@linuxdriverproject.org] On Behalf Of Dexuan Cui Sent: Friday, September 26, 2014 12:52 PM To: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- de...@linuxdriverproject.org; o...@aepfle.de

[PATCH v3 RESEND] Tools: hv: vssdaemon: ignore the EBUSY on multiple freezing the same partition

2014-10-21 Thread Dexuan Cui
fails. Signed-off-by: Dexuan Cui de...@microsoft.com Reviewed-by: K. Y. Srinivasan k...@microsoft.com --- v2: Add errno = 0; before the ioctl() (Unnecessary and removed now since we remove syslog() in vss_do_freeze() in v3) v3: Remove the unsafe syslog() in vss_do_freeze(): that could write

RE: [PATCH v3 RESEND] Tools: hv: vssdaemon: ignore the EBUSY on multiple freezing the same partition

2014-11-03 Thread Dexuan Cui
-Original Message- From: devel [mailto:driverdev-devel-boun...@linuxdriverproject.org] On Behalf Of Dexuan Cui Sent: Tuesday, October 21, 2014 17:56 PM To: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- de...@linuxdriverproject.org; o...@aepfle.de

RE: [PATCH 0/3] Tools: hv: vssdaemon: freeze/thaw logic improvement for the failure case

2014-11-07 Thread Dexuan Cui
-Original Message- From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] Sent: Saturday, November 8, 2014 1:09 AM To: KY Srinivasan; Haiyang Zhang; Greg Kroah-Hartman Cc: de...@linuxdriverproject.org; linux-ker...@vger.kernel.org; Dexuan Cui Subject: [PATCH 0/3] Tools: hv: vssdaemon

RE: [PATCH v2 1/2] Tools: hv: vssdaemon: report freeze errors

2014-11-10 Thread Dexuan Cui
-Original Message- From: Vitaly Kuznetsov Sent: Tuesday, November 11, 2014 0:37 AM To: KY Srinivasan; Haiyang Zhang; Greg Kroah-Hartman Cc: de...@linuxdriverproject.org; linux-ker...@vger.kernel.org; Dexuan Cui Subject: [PATCH v2 1/2] Tools: hv: vssdaemon: report freeze errors

RE: [PATCH v2 2/2] Tools: hv: vssdaemon: skip all filesystems mounted readonly

2014-11-10 Thread Dexuan Cui
-Original Message- From: Vitaly Kuznetsov Sent: Tuesday, November 11, 2014 0:37 AM To: KY Srinivasan; Haiyang Zhang; Greg Kroah-Hartman Cc: de...@linuxdriverproject.org; linux-ker...@vger.kernel.org; Dexuan Cui Subject: [PATCH v2 2/2] Tools: hv: vssdaemon: skip all filesystems

[PATCH] hv: hv_fcopy: drop the obsolete message on transfer failure

2014-11-11 Thread Dexuan Cui
-by: Dexuan Cui de...@microsoft.com --- drivers/hv/hv_fcopy.c | 9 + 1 file changed, 9 insertions(+) diff --git a/drivers/hv/hv_fcopy.c b/drivers/hv/hv_fcopy.c index 23b2ce2..177122a 100644 --- a/drivers/hv/hv_fcopy.c +++ b/drivers/hv/hv_fcopy.c @@ -86,6 +86,15 @@ static void fcopy_work_func

[PATCH] tools: hv: ignore ENOBUFS in the KVP daemon

2014-11-19 Thread Dexuan Cui
it will succeed next time. Cc: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Dexuan Cui de...@microsoft.com --- tools/hv/hv_kvp_daemon.c | 7 +++ 1 file changed, 7 insertions(+) diff --git a/tools/hv/hv_kvp_daemon.c b/tools/hv/hv_kvp_daemon.c index 22b0764..9f4b303 100644 --- a/tools/hv

RE: [PATCH] tools: hv: ignore ENOBUFS in the KVP daemon

2014-11-19 Thread Dexuan Cui
-Original Message- From: Vitaly Kuznetsov Sent: Wednesday, November 19, 2014 18:50 PM To: Dexuan Cui Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; jasow...@redhat.com; Haiyang Zhang Subject

RE: [PATCH] tools: hv: ignore ENOBUFS in the KVP daemon

2014-11-19 Thread Dexuan Cui
-Original Message- From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] Sent: Wednesday, November 19, 2014 20:41 PM To: Dexuan Cui Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; jasow

RE: [PATCH] tools: hv: ignore ENOBUFS in the KVP daemon

2014-11-19 Thread Dexuan Cui
-Original Message- From: Vitaly Kuznetsov -- Vitaly OK, I can add this new check: (I'll send out the v2 tomorrow in case people have new comments) Thanks! --- a/tools/hv/hv_kvp_daemon.c +++ b/tools/hv/hv_kvp_daemon.c @@ -1770,8 +1770,15 @@ kvp_done:

[PATCH v2] tools: hv: ignore ENOBUFS and ENOMEM in the KVP daemon

2014-11-19 Thread Dexuan Cui
it will succeed next time. We can also ignore the errors on sending. Cc: Vitaly Kuznetsov vkuzn...@redhat.com Cc: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Dexuan Cui de...@microsoft.com --- v2: I also ignore the errors on sending, as Vitaly suggested. tools/hv/hv_kvp_daemon.c | 14

RE: [PATCH] hv: hv_fcopy: drop the obsolete message on transfer failure

2014-11-19 Thread Dexuan Cui
-Original Message- From: KY Srinivasan Sent: Thursday, November 20, 2014 6:59 AM diff --git a/drivers/hv/hv_fcopy.c b/drivers/hv/hv_fcopy.c index 23b2ce2..177122a 100644 --- a/drivers/hv/hv_fcopy.c +++ b/drivers/hv/hv_fcopy.c @@ -86,6 +86,15 @@ static void

RE: [PATCH] hv: hv_fcopy: drop the obsolete message on transfer failure

2014-11-20 Thread Dexuan Cui
-Original Message- From: KY Srinivasan Sent: Friday, November 21, 2014 1:58 AM To: Dexuan Cui; gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev-devel@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; jasow...@redhat.com Cc: Haiyang Zhang Subject: RE

[PATCH] hv: hv_balloon: avoid memory leak on alloc_error of 2MB memory block

2014-11-23 Thread Dexuan Cui
If num_ballooned is not 0, we shouldn't neglect the already-allocated 2MB memory block(s). Cc: K. Y. Srinivasan k...@microsoft.com Cc: sta...@vger.kernel.org Signed-off-by: Dexuan Cui de...@microsoft.com --- drivers/hv/hv_balloon.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff

RE: [PATCH] hv: hv_balloon: avoid memory leak on alloc_error of 2MB memory block

2014-11-23 Thread Dexuan Cui
-Original Message- From: Jason Wang [mailto:jasow...@redhat.com] Sent: Monday, November 24, 2014 13:18 PM To: Dexuan Cui; gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev-devel@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; KY Srinivasan Cc: Haiyang

RE: [PATCH] hv: hv_balloon: avoid memory leak on alloc_error of 2MB memory block

2014-11-23 Thread Dexuan Cui
-Original Message- From: Jason Wang [mailto:jasow...@redhat.com] Sent: Monday, November 24, 2014 15:28 PM To: Dexuan Cui; gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev-devel@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; KY Srinivasan Cc: Haiyang

RE: [PATCH] hv: hv_balloon: avoid memory leak on alloc_error of 2MB memory block

2014-11-24 Thread Dexuan Cui
-Original Message- From: Jason Wang [mailto:jasow...@redhat.com] Sent: Monday, November 24, 2014 16:48 PM To: Dexuan Cui; gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev-devel@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; KY Srinivasan Cc: Haiyang

[PATCH v2] hv: hv_balloon: avoid memory leak on alloc_error of 2MB memory block

2014-11-24 Thread Dexuan Cui
If num_ballooned is not 0, we shouldn't neglect the already-partially-allocated 2MB memory block(s). Cc: Jason Wang jasow...@redhat.com Cc: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Dexuan Cui de...@microsoft.com --- v2: I fixed the logic error in v1, pointed by Jason Wang

RE: [PATCH] hv: hv_fcopy: drop the obsolete message on transfer failure

2014-11-26 Thread Dexuan Cui
-Original Message- From: Greg KH [mailto:gre...@linuxfoundation.org] Sent: Thursday, November 27, 2014 7:54 AM To: Dexuan Cui Cc: linux-ker...@vger.kernel.org; driverdev-devel@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; jasow...@redhat.com; KY Srinivasan; Haiyang

[PATCH v2] hv: hv_fcopy: drop the obsolete message on transfer failure

2014-11-26 Thread Dexuan Cui
: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Dexuan Cui de...@microsoft.com --- v2: I removed the FCP prefix as Greg asked. I also updated the output message a little: FCP: failed to acquire the semaphore -- can not acquire the semaphore: it is benign drivers/hv/hv_fcopy.c

RE: [PATCH v2] hv: hv_fcopy: drop the obsolete message on transfer failure

2014-11-27 Thread Dexuan Cui
-Original Message- From: Jason Wang [mailto:jasow...@redhat.com] Sent: Thursday, November 27, 2014 15:15 PM To: Dexuan Cui Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; KY Srinivasan; vkuzn

RE: [PATCH v2] hv: hv_fcopy: drop the obsolete message on transfer failure

2014-11-27 Thread Dexuan Cui
-Original Message- From: Jason Wang [mailto:jasow...@redhat.com] Sent: Thursday, November 27, 2014 17:01 PM To: Dexuan Cui Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; KY Srinivasan; vkuzn

[PATCH v3] hv: hv_fcopy: drop the obsolete message on transfer failure

2014-11-27 Thread Dexuan Cui
Kuznetsov vkuzn...@redhat.com Cc: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Dexuan Cui de...@microsoft.com --- v2: I removed the FCP prefix as Greg asked. I also updated the output message a little: FCP: failed to acquire the semaphore -- can not acquire the semaphore: it is benign

RE: [PATCH v3] hv: hv_fcopy: drop the obsolete message on transfer failure

2014-11-28 Thread Dexuan Cui
-Original Message- From: Jason Wang [mailto:jasow...@redhat.com] Sent: Friday, November 28, 2014 14:47 PM To: Dexuan Cui Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; KY Srinivasan; vkuzn

RE: [PATCH v3] hv: hv_fcopy: drop the obsolete message on transfer failure

2014-12-01 Thread Dexuan Cui
-Original Message- From: Jason Wang [mailto:jasow...@redhat.com] Sent: Monday, December 1, 2014 18:18 PM To: Dexuan Cui Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; KY Srinivasan; vkuzn

RE: [PATCH v2] Drivers: hv: vmbus: prevent cpu offlining on newer hypervisors

2014-12-01 Thread Dexuan Cui
-Original Message- From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] Sent: Monday, December 1, 2014 18:53 PM To: KY Srinivasan; Haiyang Zhang Cc: de...@linuxdriverproject.org; Greg Kroah-Hartman; linux- ker...@vger.kernel.org; Dexuan Cui Subject: [PATCH v2] Drivers: hv: vmbus

RE: [PATCH v3] hv: hv_fcopy: drop the obsolete message on transfer failure

2014-12-01 Thread Dexuan Cui
-Original Message- From: KY Srinivasan Sent: Monday, December 1, 2014 23:55 PM To: Dexuan Cui; Jason Wang Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; vkuzn...@redhat.com; Haiyang Zhang

[PATCH] tools: hv: kvp_daemon: make IPv6-only-injection work

2014-12-09 Thread Dexuan Cui
shouldn't ignore the IPv6 address and we should pass BOOTPROTO=none to the distro-specific script hv_set_ipconfig. Tested in Ubuntu 14.10 and RHEL7. Cc: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Dexuan Cui de...@microsoft.com --- tools/hv/hv_kvp_daemon.c | 47

RE: [PATCH 0/5] Tools: hv: fix compiler warnings and do minor cleanup

2014-12-09 Thread Dexuan Cui
-Original Message- From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] Sent: Tuesday, December 9, 2014 23:48 PM To: KY Srinivasan Cc: Haiyang Zhang; de...@linuxdriverproject.org; linux- ker...@vger.kernel.org; Dexuan Cui Subject: [PATCH 0/5] Tools: hv: fix compiler warnings and do

RE: [PATCH] tools: hv: kvp_daemon: make IPv6-only-injection work

2014-12-09 Thread Dexuan Cui
-Original Message- From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] Sent: Tuesday, December 9, 2014 21:06 PM To: Dexuan Cui Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; jasow

RE: [PATCH] tools: hv: kvp_daemon: make IPv6-only-injection work

2014-12-09 Thread Dexuan Cui
Thanks, -- Dexuan -Original Message- From: Dexuan Cui Sent: Wednesday, December 10, 2014 15:34 PM To: 'Vitaly Kuznetsov' Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; jasow...@redhat.com

RE: [PATCH] tools: hv: kvp_daemon: make IPv6-only-injection work

2014-12-10 Thread Dexuan Cui
-Original Message- From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] Sent: Wednesday, December 10, 2014 17:08 PM To: Dexuan Cui Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; jasow

[PATCH v2] tools: hv: kvp_daemon: make IPv6-only-injection work

2014-12-10 Thread Dexuan Cui
In the case the host only injects an IPv6 address, the dhcp_enabled flag is true (it's only for IPv4 according to Hyper-V host team), but we still need to proceed to parse the IPv6 information. Cc: Vitaly Kuznetsov vkuzn...@redhat.com Cc: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Dexuan

RE: [PATCH] video: hyperv: hyperv_fb: refresh the VM screen by force on VM panic

2014-06-24 Thread Dexuan Cui
On Tue, Jun 24, 2014 at 08:29:17AM +0800, Dexuan Cui wrote: Currently the VSC has no chance to notify the VSP of the dirty rectangle on VM panic because the notification work is done in a workqueue, and in panic() the kernel typically ends up in an infinite loop, and a typical kernel config

[PATCH] video: hyperv: hyperv_fb: refresh the VM screen by force on VM panic

2014-06-25 Thread Dexuan Cui
to a synchronous mode, meaning the VSC flushes any future framebuffer change to the VSP immediately. MS-TFS: 157532 Signed-off-by: Dexuan Cui de...@microsoft.com Reviewed-by: Haiyang Zhang haiya...@microsoft.com --- drivers/video/fbdev/hyperv_fb.c | 58 ++--- 1

RE: [PATCH] video: hyperv: hyperv_fb: refresh the VM screen by force on VM panic

2014-06-26 Thread Dexuan Cui
MS-TFS: 157532 What is this line for? Hi Greg, This line is for our internal bug repository. We have an automated system to correlate bugs with fixes so that our test team knows when a bug fix has been accepted upstream and they need to write a new test case for it. The

[PATCH v2] video: hyperv: hyperv_fb: refresh the VM screen by force on VM panic

2014-06-26 Thread Dexuan Cui
to a synchronous mode, meaning the VSC flushes any future framebuffer change to the VSP immediately. v2: removed the MS-TFS line in the commit message Signed-off-by: Dexuan Cui de...@microsoft.com Reviewed-by: Haiyang Zhang haiya...@microsoft.com --- drivers/video/fbdev/hyperv_fb.c | 58

RE: [PATCH v2] video: hyperv: hyperv_fb: refresh the VM screen by force on VM panic

2014-07-08 Thread Dexuan Cui
-Original Message- From: driverdev-devel-boun...@linuxdriverproject.org [mailto:driverdev- devel-boun...@linuxdriverproject.org] On Behalf Of Dexuan Cui Sent: Friday, June 27, 2014 5:35 AM To: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- de

RE: [PATCH v2] video: hyperv: hyperv_fb: refresh the VM screen by force on VM panic

2014-07-08 Thread Dexuan Cui
-Original Message- From: Dan Carpenter [mailto:dan.carpen...@oracle.com] Sent: Tuesday, July 8, 2014 17:27 PM To: Dexuan Cui Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- de...@linuxdriverproject.org; plagn...@jcrosoft.com; tomi.valkei

[PATCH v3] video: hyperv: hyperv_fb: refresh the VM screen by force on VM panic

2014-07-08 Thread Dexuan Cui
to a synchronous mode, meaning the VSC flushes any future framebuffer change to the VSP immediately. v2: removed the MS-TFS line in the commit message v3: remove some 'unlikely' markings Signed-off-by: Dexuan Cui de...@microsoft.com Reviewed-by: Haiyang Zhang haiya...@microsoft.com --- drivers

[PATCH] Drivers: hv: hv_fcopy: fix a race condition for SMP guest

2014-07-16 Thread Dexuan Cui
...@microsoft.com Cc: Haiyang Zhang haiya...@microsoft.com Cc: sta...@vger.kernel.org Signed-off-by: Dexuan Cui de...@microsoft.com --- drivers/hv/hv_fcopy.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/hv/hv_fcopy.c b/drivers/hv/hv_fcopy.c index eaaa3d8..23b2ce2 100644

RE: [PATCH v3] video: hyperv: hyperv_fb: refresh the VM screen by force on VM panic

2014-07-25 Thread Dexuan Cui
-Original Message- From: gre...@linuxfoundation.org [mailto:gre...@linuxfoundation.org] Sent: Thursday, July 24, 2014 11:53 AM So, hi Greg and all, If you think the patch itself is OK, may I know if it's OK for the patch to go into Greg's char-misc.git tree, as some other

RE: [PATCH v3] video: hyperv: hyperv_fb: refresh the VM screen by force on VM panic

2014-08-01 Thread Dexuan Cui
-Original Message- From: Tomi Valkeinen [mailto:tomi.valkei...@ti.com] Sent: Thursday, July 31, 2014 21:38 PM I think in hvfb_on_panic() we should be able to get the hvfb_info pointer by hvfb_info = container_of(nb, struct hv_fb_panic_nb, nb). If you like that or have a better

[PATCH v4] video: hyperv: hyperv_fb: refresh the VM screen by force on VM panic

2014-08-01 Thread Dexuan Cui
-by: Dexuan Cui de...@microsoft.com --- drivers/video/fbdev/hyperv_fb.c | 62 +++-- 1 file changed, 59 insertions(+), 3 deletions(-) diff --git a/drivers/video/fbdev/hyperv_fb.c b/drivers/video/fbdev/hyperv_fb.c index e23392e..569e756 100644 --- a/drivers/video

[PATCH] HID: hyperv: register as a wakeup source

2014-08-01 Thread Dexuan Cui
With this patch, we can move the mouse to wake up the VM after the VM executes echo freeze /sys/power/state. This addresses part of https://bugzilla.redhat.com/show_bug.cgi?id=1086100 Cc: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Dexuan Cui de...@microsoft.com --- drivers/hid/hid

[PATCH] Input: hyperv-keyboard: register as a wakeup source

2014-08-01 Thread Dexuan Cui
With this patch, we can press a key to wake up the VM after the VM executes echo freeze /sys/power/state. This addresses part of https://bugzilla.redhat.com/show_bug.cgi?id=1086100 Cc: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Dexuan Cui de...@microsoft.com --- drivers/input/serio

RE: [PATCH] Input: hyperv-keyboard: register as a wakeup source

2014-08-06 Thread Dexuan Cui
-Original Message- From: Dmitry Torokhov [mailto:dmitry.torok...@gmail.com] Sent: Thursday, August 7, 2014 4:34 AM To: Dexuan Cui On Fri, Aug 01, 2014 at 07:28:24AM -0700, Dexuan Cui wrote: With this patch, we can press a key to wake up the VM after the VM executes echo freeze

RE: [PATCH] Input: hyperv-keyboard: register as a wakeup source

2014-08-06 Thread Dexuan Cui
-Original Message- From: linux-kernel-ow...@vger.kernel.org [mailto:linux-kernel- ow...@vger.kernel.org] On Behalf Of Dmitry Torokhov Hi Dmitry, Sorry, I didn't realize it's not public... :-) I'll remove the line and re-send the patch. No need to resend, I already applied

RE: [PATCH] Hyperv: Trigger DHCP renew after host hibernation

2014-08-07 Thread Dexuan Cui
; Haiyang Zhang; KY Srinivasan; Thomas Shao; Dexuan Cui Subject: Re: [PATCH] Hyperv: Trigger DHCP renew after host hibernation On Mon, Jul 21, 2014 at 11:32 PM, David Miller da...@davemloft.net wrote: From: Olaf Hering o...@aepfle.de Date: Mon, 21 Jul 2014 11:18:51 +0200 On Mon, Jul 21

RE: [PATCH] Hyperv: Trigger DHCP renew after host hibernation

2014-08-10 Thread Dexuan Cui
-Original Message- From: Greg KH [mailto:gre...@linuxfoundation.org] IMO the most feasible and need-the-least-change solution may be: the hyperv network VSC driver passes the event RNDIS_STATUS_NETWORK_CHANGE to the udev daemon? No, don't do that, again, act like

RE: [PATCH] Hyperv: Trigger DHCP renew after host hibernation

2014-08-10 Thread Dexuan Cui
-Original Message- From: Dexuan Cui Sent: Monday, August 11, 2014 11:24 AM Now I understand it's not good to pass the event to the udev daemon, and it's not good to use a SLEEP(10s) in the kernel space(even if it's in a work task here). Please let me know if it's the correct

[PATCH] Input: serio: make HYPERV_KEYBOARD depend on SERIO_I8042=y

2014-08-11 Thread Dexuan Cui
Claudio for the initial reporting, investigation and suggesting the fix. Signed-off-by: Dexuan Cui de...@microsoft.com Reported-by: Claudio Latini claudio.lat...@live.com Cc: K. Y. Srinivasan k...@microsoft.com --- drivers/input/serio/Kconfig | 2 +- 1 file changed, 1 insertion(+), 1 deletion

RE: [PATCH] Input: serio: make HYPERV_KEYBOARD depend on SERIO_I8042=y

2014-08-11 Thread Dexuan Cui
-Original Message- From: Greg KH [mailto:gre...@linuxfoundation.org] Decoupling the dependency between hyperv_keyboard and i8042 needs non-trivial efforts and is hence a long term goal. For now, let's make the dependency explicit so people can beware of this. You didn't make

RE: [PATCH] Input: serio: make HYPERV_KEYBOARD depend on SERIO_I8042=y

2014-08-12 Thread Dexuan Cui
-Original Message- From: linux-kernel-ow...@vger.kernel.org [mailto:linux-kernel- ow...@vger.kernel.org] On Behalf Of Greg KH What exactly needs to be done to fix this correctly that is going to take too much work at the moment? To decouple the dependency between the

RE: [PATCH] Hyperv: Trigger DHCP renew after host hibernation

2014-08-12 Thread Dexuan Cui
-Original Message- From: Florian Fainelli [mailto:f.faine...@gmail.com] Sent: Monday, August 11, 2014 11:52 AM I suppose you meant the below logic: if (refresh) { rtnl_lock(); netif_carrier_off(net); netif_carrier_on(net); rtnl_unlock();

RE: [PATCH] Hyperv: Trigger DHCP renew after host hibernation

2014-08-12 Thread Dexuan Cui
-Original Message- From: Bill Fink In the case of ifplugd, it has parameters -u | --delay-up= which defaults to 0 seconds, and -d | --delay-down= which defaults to 5 seconds. Maybe for hyperv you could specify --delay-down=0. I don't know if other daemons such as systemd have

RE: [PATCH] Hyperv: Trigger DHCP renew after host hibernation

2014-08-12 Thread Dexuan Cui
From: Tom Gundersen Unluckily this logic doesn't work because the user-space daemons like ifplugd, usually don't renew the DHCP immediately as long as they receive a link-down message: they usually wait for some seconds and if they find the link becomes up soon, they won't trigger renew

RE: [PATCH] Input: serio: make HYPERV_KEYBOARD depend on SERIO_I8042=y

2014-08-12 Thread Dexuan Cui
-Original Message- From: Dmitry Torokhov Sent: Wednesday, August 13, 2014 1:55 AM To decouple the dependency between the hyperv-keyboard and i8042 modules, I suppose we probably have to re-implement hyperv- keyboard by using input_allocate_device(), input_register_device(),

RE: [PATCH] Input: serio: make HYPERV_KEYBOARD depend on SERIO_I8042=y

2014-08-12 Thread Dexuan Cui
-Original Message- From: KY Srinivasan The Arch users can simply select to build i8042 into the kernel as a workaround. The proper solution is to allow loading libps2 module even if i8042 did not find its device. I wish I could simply drop this i8042_lock_chip and stuff, but

RE: [PATCH] Input: serio: make HYPERV_KEYBOARD depend on SERIO_I8042=y

2014-08-14 Thread Dexuan Cui
-Original Message- From: Dmitry Torokhov How about this: in libps2.c let's add and export a function pointer i8042_lock_chip_if_port_owner: it is used to replace the current if (i8042_check_port_owner(ps2dev-serio)) i8042_lock_chip(); The function pointer has

[PATCH] Input: hyperv-keyboard - implement Type Clipboard Text

2014-08-15 Thread Dexuan Cui
sure the CapsLock state of the VM's (virtual) keyboard is OFF because this patch assumes it -- we'll try to fix this later, probably by tracking the state of virtual CapsLock, because it looks the keyboard module doesn't supply an API for us to query the state of the keyboard. Signed-off-by: Dexuan

RE: [PATCH] Input: hyperv-keyboard - implement Type Clipboard Text

2014-08-17 Thread Dexuan Cui
-Original Message- From: Dmitry Torokhov Sent: Saturday, August 16, 2014 0:58 AM To: Dexuan Cui For each char in the string, the host sends 2 events (key down/up with the char's UNICODE value) to the guest. The patch finds each char's scan codes of key down/up, and injects

RE: [PATCH] Input: hyperv-keyboard - implement Type Clipboard Text

2014-08-19 Thread Dexuan Cui
-Original Message- From: Dmitry Torokhov Sent: Tuesday, August 19, 2014 2:52 AM To: Dexuan Cui On Mon, Aug 18, 2014 at 03:54:51AM +, Dexuan Cui wrote: -Original Message- From: Dmitry Torokhov Sent: Saturday, August 16, 2014 0:58 AM To: Dexuan Cui For each

RE: [PANIC, hyperv] BUG: unable to handle kernel paging request at ffff880077800004 (hv_ringbuffer_write)

2014-08-20 Thread Dexuan Cui
-Original Message- From: Sitsofe Wheeler Sent: Wednesday, August 20, 2014 17:27 PM To: Dexuan Cui Cc: KY Srinivasan; Greg Kroah-Hartman; Haiyang Zhang; de...@linuxdriverproject.org; linux-ker...@vger.kernel.org; Jean-Christophe Plagniol-Villard; linux-fb...@vger.kernel.org While

RE: [PANIC, hyperv] BUG: unable to handle kernel paging request at ffff880077800004 (hv_ringbuffer_write)

2014-08-20 Thread Dexuan Cui
-Original Message- From: Sitsofe Wheeler Sent: Wednesday, August 20, 2014 19:40 PM I suppose you're using the latest mainline v3.17-rc1(7d1311b9). That's right - Linux 3.17-rc1 (7d1311b93e58ed55f3a31cc8f94c4b8fe988a2b9). Ok, let me try to reproduce it first. Can you please

RE: [PANIC, hyperv] BUG: unable to handle kernel paging request at ffff880077800004 (hv_ringbuffer_write)

2014-08-25 Thread Dexuan Cui
-Original Message- From: Sitsofe Wheeler Sent: Wednesday, August 20, 2014 17:27 PM While booting a Hyper-V 3.17.0-rc1 guest on a 2012 R2 host a BUG was triggered while registering hyperv_fb which in turn caused a panic. Various kernel debugging options (CONFIG_DEBUG_PAGEALLOC,

RE: [PANIC, hyperv] BUG: unable to handle kernel paging request at ffff880077800004 (hv_ringbuffer_write)

2014-08-26 Thread Dexuan Cui
-Original Message- From: Sitsofe Wheeler Sent: Tuesday, August 26, 2014 1:42 AM [7.645526] hv_vmbus: registering driver hyperv_fb [7.657553] BUG: unable to handle kernel paging request at 88007784 [7.658224] IP: [8159a7ac]

RE: [PANIC, hyperv] BUG: unable to handle kernel paging request at ffff880077800004 (hv_ringbuffer_write)

2014-08-27 Thread Dexuan Cui
-Original Message- From: Sitsofe Wheeler Sent: Wednesday, August 27, 2014 20:16 PM do_hypercall() fails due to HV_STATUS_INVALID_ALIGNMENT, if the specified input or output GPA pointer is not aligned to 8 bytes, or, the specified input or output parameter lists spans pages.

RE: [PANIC, hyperv] BUG: unable to handle kernel paging request at ffff880077800004 (hv_ringbuffer_write)

2014-08-27 Thread Dexuan Cui
-Original Message- From: linux-kernel-ow...@vger.kernel.org [mailto:linux-kernel- ow...@vger.kernel.org] On Behalf Of KY Srinivasan Sent: Thursday, August 28, 2014 7:14 AM From: Sitsofe Wheeler [mailto:sits...@gmail.com] Sent: Wednesday, August 27, 2014 9:19 AM BTW,

RE: [PANIC, hyperv] BUG: unable to handle kernel paging request at ffff880077800004 (hv_ringbuffer_write)

2014-08-28 Thread Dexuan Cui
-Original Message- From: Sitsofe Wheeler Sent: Thursday, August 28, 2014 20:49 PM The only issue seen on boot now is similar to https://lkml.org/lkml/2014/8/19/227 ... I don't see this issue. Do you still see the issue for EVERY boot after you applied KY's

RE: [PANIC, hyperv] BUG: unable to handle kernel paging request at ffff880077800004 (hv_ringbuffer_write)

2014-08-29 Thread Dexuan Cui
-Original Message- From: Dan Carpenter Sent: Friday, August 29, 2014 17:50 PM To: KY Srinivasan Cc: Sitsofe Wheeler; Dexuan Cui; Greg Kroah-Hartman; Haiyang Zhang; linux-ker...@vger.kernel.org; de...@linuxdriverproject.org Subject: Re: [PANIC, hyperv] BUG: unable to handle kernel

RE: [PANIC, hyperv] BUG: unable to handle kernel paging request at ffff880077800004 (hv_ringbuffer_write)

2014-08-29 Thread Dexuan Cui
-Original Message- From: Sitsofe Wheeler Sent: Friday, August 29, 2014 19:20 PM Because this particular return code is overloaded with multiple meanings that can't necessarily be deduced from its name perhaps this documentation patch would be helpful: diff --git

[PATCH] Tools: hv: vssdaemon: ignore the EBUSY on multiple freezing the same partition

2014-09-22 Thread Dexuan Cui
Signed-off-by: Dexuan Cui de...@microsoft.com Reviewed-by: K. Y. Srinivasan k...@microsoft.com --- tools/hv/hv_vss_daemon.c | 21 + 1 file changed, 21 insertions(+) diff --git a/tools/hv/hv_vss_daemon.c b/tools/hv/hv_vss_daemon.c index 6a213b8..19567c5 100644 --- a/tools/hv

[PATCH v2] Tools: hv: vssdaemon: ignore the EBUSY on multiple freezing the same partition

2014-09-22 Thread Dexuan Cui
v2: I added errno = 0; in the ioctl() Signed-off-by: Dexuan Cui de...@microsoft.com Reviewed-by: K. Y. Srinivasan k...@microsoft.com --- tools/hv/hv_vss_daemon.c | 28 1 file changed, 28 insertions(+) diff --git a/tools/hv/hv_vss_daemon.c b/tools/hv/hv_vss_daemon.c

RE: [PATCH] Tools: hv: vssdaemon: ignore the EBUSY on multiple freezing the same partition

2014-09-22 Thread Dexuan Cui
-Original Message- From: linux-kernel-ow...@vger.kernel.org [mailto:linux-kernel- ow...@vger.kernel.org] On Behalf Of Dexuan Cui Sent: Tuesday, September 23, 2014 2:02 AM To: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- de...@linuxdriverproject.org; o

RE: [PATCH v2] Tools: hv: vssdaemon: ignore the EBUSY on multiple freezing the same partition

2014-09-24 Thread Dexuan Cui
-Original Message- From: linux-kernel-ow...@vger.kernel.org [mailto:linux-kernel- ow...@vger.kernel.org] On Behalf Of Greg KH Sent: Wednesday, September 24, 2014 14:32 PM To: Dexuan Cui Cc: linux-ker...@vger.kernel.org; driverdev-devel@linuxdriverproject.org; o...@aepfle.de

[PATCH v3] Tools: hv: vssdaemon: ignore the EBUSY on multiple freezing the same partition

2014-09-25 Thread Dexuan Cui
fails. Signed-off-by: Dexuan Cui de...@microsoft.com Reviewed-by: K. Y. Srinivasan k...@microsoft.com --- v2: Add errno = 0; before the ioctl() (Unnecessary and removed now since we remove syslog() in vss_do_freeze() in v3) v3: Remove the unsafe syslog() in vss_do_freeze(): that could write

RE: [PATCH v2] tools: hv: kvp_daemon: make IPv6-only-injection work

2014-12-16 Thread Dexuan Cui
-Original Message- From: devel [mailto:driverdev-devel-boun...@linuxdriverproject.org] On Behalf Of Dexuan Cui Sent: Wednesday, December 10, 2014 19:33 PM To: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- de...@linuxdriverproject.org; vkuzn...@redhat.com; o

RE: [PATCH v2] tools: hv: kvp_daemon: make IPv6-only-injection work

2015-01-02 Thread Dexuan Cui
-Original Message- From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] Sent: Friday, January 2, 2015 22:51 PM To: Dexuan Cui Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; jasow...@redhat.com

RE: [PATCH 3/4] Drivers: hv: vmbus: protect vmbus_get_outgoing_channel() against channel removal

2015-02-03 Thread Dexuan Cui
-Original Message- From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] Sent: Wednesday, February 4, 2015 1:01 AM To: KY Srinivasan; de...@linuxdriverproject.org Cc: Haiyang Zhang; linux-ker...@vger.kernel.org; Dexuan Cui; Jason Wang Subject: [PATCH 3/4] Drivers: hv: vmbus: protect

RE: [PATCH 2/4] Drivers: hv: vmbus: do not lose rescind offer on failure in vmbus_process_offer()

2015-02-03 Thread Dexuan Cui
-Original Message- From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] Sent: Wednesday, February 4, 2015 1:01 AM To: KY Srinivasan; de...@linuxdriverproject.org Cc: Haiyang Zhang; linux-ker...@vger.kernel.org; Dexuan Cui; Jason Wang Subject: [PATCH 2/4] Drivers: hv: vmbus: do

RE: [PATCH 1/4] Drivers: hv: vmbus: implement get/put usage workflow for vmbus channels

2015-02-04 Thread Dexuan Cui
-Original Message- From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] Sent: Wednesday, February 4, 2015 1:01 AM To: KY Srinivasan; de...@linuxdriverproject.org Cc: Haiyang Zhang; linux-ker...@vger.kernel.org; Dexuan Cui; Jason Wang Subject: [PATCH 1/4] Drivers: hv: vmbus: implement

RE: [PATCH 1/4] Drivers: hv: vmbus: implement get/put usage workflow for vmbus channels

2015-02-04 Thread Dexuan Cui
-Original Message- From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] Sent: Wednesday, February 4, 2015 17:32 PM To: Dexuan Cui Cc: KY Srinivasan; de...@linuxdriverproject.org; Haiyang Zhang; linux- ker...@vger.kernel.org; Jason Wang Subject: Re: [PATCH 1/4] Drivers: hv: vmbus

RE: [PATCH 4/4] hyperv: netvsc: improve protection against rescind offer

2015-02-03 Thread Dexuan Cui
-Original Message- From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] Sent: Wednesday, February 4, 2015 1:01 AM To: KY Srinivasan; de...@linuxdriverproject.org Cc: Haiyang Zhang; linux-ker...@vger.kernel.org; Dexuan Cui; Jason Wang Subject: [PATCH 4/4] hyperv: netvsc: improve

RE: [PATCH v2 1/3] hv: hv_util: move vmbus_open() to a later place

2015-02-02 Thread Dexuan Cui
-Original Message- From: Jason Wang [mailto:jasow...@redhat.com] Sent: Monday, February 2, 2015 17:36 PM To: Dexuan Cui Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; KY Srinivasan; vkuzn

RE: [PATCH v2 2/3] hv: vmbus_post_msg: retry the hypercall on some transient errors

2015-02-02 Thread Dexuan Cui
-Original Message- From: Jason Wang [mailto:jasow...@redhat.com] Sent: Monday, February 2, 2015 17:41 PM To: Dexuan Cui Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; KY Srinivasan; vkuzn

RE: [PATCH 0/4] Drivers: hv: Further protection for the rescind path

2015-02-05 Thread Dexuan Cui
-Original Message- From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] Sent: Thursday, February 5, 2015 18:10 PM To: KY Srinivasan Cc: de...@linuxdriverproject.org; Haiyang Zhang; linux-ker...@vger.kernel.org; Dexuan Cui; Jason Wang Subject: Re: [PATCH 0/4] Drivers: hv: Further

RE: [PATCH 0/4] Drivers: hv: Further protection for the rescind path

2015-02-06 Thread Dexuan Cui
-Original Message- From: KY Srinivasan Sent: Friday, February 6, 2015 6:47 AM To: Dexuan Cui; Vitaly Kuznetsov Cc: de...@linuxdriverproject.org; Haiyang Zhang; linux-ker...@vger.kernel.org; Jason Wang Subject: RE: [PATCH 0/4] Drivers: hv: Further protection for the rescind path

RE: [PATCH v3 3/3] Drivers: hv: vmbus: serialize Offer and Rescind offer

2015-01-28 Thread Dexuan Cui
-Original Message- From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] Sent: Tuesday, January 20, 2015 23:45 PM To: KY Srinivasan; de...@linuxdriverproject.org Cc: Haiyang Zhang; linux-ker...@vger.kernel.org; Dexuan Cui; Jason Wang; Radim Krčmář; Dan Carpenter Subject: [PATCH v3 3/3

RE: [PATCH v3 3/3] Drivers: hv: vmbus: serialize Offer and Rescind offer

2015-01-28 Thread Dexuan Cui
-Original Message- From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] Sent: Wednesday, January 28, 2015 20:09 PM To: Dexuan Cui Cc: KY Srinivasan; de...@linuxdriverproject.org; Haiyang Zhang; linux- ker...@vger.kernel.org; Jason Wang; Radim Krčmář; Dan Carpenter Subject: Re: [PATCH

[PATCH 2/3] hv: vmbus_post_msg: retry the hypercall on HV_STATUS_INVALID_CONNECTION_ID

2015-01-29 Thread Dexuan Cui
I got the hypercall error code on Hyper-V 2008 R2 when keeping running rmmod hv_netvsc; modprobe hv_netvsc; rmmod hv_utils; modprobe hv_utils in a Linux guest. Without the patch, the driver can occasionally fail to load. CC: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Dexuan Cui de

[PATCH 3/3] hv: vmbus_open(): reset the channel state on ENOMEM

2015-01-29 Thread Dexuan Cui
Without this patch, the state is put to CHANNEL_OPENING_STATE, and when the driver is loaded next time, vmbus_open() will fail immediately due to newchannel-state != CHANNEL_OPEN_STATE. CC: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Dexuan Cui de...@microsoft.com --- drivers/hv/channel.c

[PATCH 1/3] hv: hv_util: move vmbus_open() to a later place

2015-01-29 Thread Dexuan Cui
Before the line vmbus_open() returns, srv-util_cb can be already running and the variablies, like util_fw_version, are needed by the srv-util_cb. So we have to make sure the variables are initialized before the vmbus_open(). CC: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Dexuan Cui de

RE: [PATCH 2/3] hv: vmbus_post_msg: retry the hypercall on HV_STATUS_INVALID_CONNECTION_ID

2015-01-31 Thread Dexuan Cui
-Original Message- From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] Sent: Saturday, January 31, 2015 1:29 AM To: Dexuan Cui Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; jasow

RE: [PATCH 3/3] hv: vmbus_open(): reset the channel state on ENOMEM

2015-02-01 Thread Dexuan Cui
-Original Message- From: KY Srinivasan Sent: Monday, February 2, 2015 3:42 AM To: Dexuan Cui; Vitaly Kuznetsov Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; jasow...@redhat.com; Haiyang

[PATCH v2 2/3] hv: vmbus_post_msg: retry the hypercall on some transient errors

2015-02-01 Thread Dexuan Cui
, the driver can occasionally fail to load. Also let's retry HV_STATUS_INSUFFICIENT_MEMORY, though we didn't get it before. Removed 'case -ENOMEM', since the hypervisor doesn't return this. CC: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Dexuan Cui de...@microsoft.com --- v2: updated

[PATCH v2 3/3] hv: vmbus_open(): reset the channel state on ENOMEM

2015-02-01 Thread Dexuan Cui
Without this patch, the state is put to CHANNEL_OPENING_STATE, and when the driver is loaded next time, vmbus_open() will fail immediately due to newchannel-state != CHANNEL_OPEN_STATE. CC: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Dexuan Cui de...@microsoft.com --- v2: this is a RESEND

[PATCH v2 1/3] hv: hv_util: move vmbus_open() to a later place

2015-02-01 Thread Dexuan Cui
vkuzn...@redhat.com Signed-off-by: Dexuan Cui de...@microsoft.com --- v2: This is a RESEND. I just added Vitaly's Reviewed-by. drivers/hv/hv_util.c | 11 ++- 1 file changed, 6 insertions(+), 5 deletions(-) diff --git a/drivers/hv/hv_util.c b/drivers/hv/hv_util.c index 3b9c9ef..c5be773

[PATCH v4] hv: hv_fcopy: drop the obsolete message on transfer failure

2015-01-14 Thread Dexuan Cui
Kuznetsov vkuzn...@redhat.com Cc: K. Y. Srinivasan k...@microsoft.com Signed-off-by: Dexuan Cui de...@microsoft.com --- v2: I removed the FCP prefix as Greg asked. I also updated the output message a little: FCP: failed to acquire the semaphore -- can not acquire the semaphore: it is benign

  1   2   3   4   5   6   >