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

2014-12-10 Thread Vitaly Kuznetsov
Dexuan Cui de...@microsoft.com writes: 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;

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

2014-12-10 Thread Vitaly Kuznetsov
Dexuan Cui de...@microsoft.com writes: -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]

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;

[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] tools: hv: kvp_daemon: make IPv6-only-injection work

2014-12-10 Thread Vitaly Kuznetsov
Dexuan Cui de...@microsoft.com writes: -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;

[PATCH v2] staging: writeboost: Add dm-writeboost

2014-12-10 Thread Akira Hayakawa
This patch adds dm-writeboost to staging tree. dm-writeboost is a log-structured SSD-caching driver. It caches data in log-structured way on the cache device so that the performance is maximized. The merit of putting this driver in staging tree is to make it possible to get more feedback from

[PATCH] staging: fpga manager: fix platform_no_drv_owner.cocci warnings

2014-12-10 Thread kbuild test robot
drivers/staging/fpga/altera.c:768:3-8: No need to set .owner here. The core will do it. Remove .owner field if calls are used which set it automatically Generated by: scripts/coccinelle/api/platform_no_drv_owner.cocci CC: Alan Tull at...@opensource.altera.com Signed-off-by: Fengguang Wu

[PATCH] staging: fpga manager: fix array_size.cocci warnings

2014-12-10 Thread kbuild test robot
drivers/staging/fpga/altera.c:368:33-34: WARNING: Use ARRAY_SIZE Use ARRAY_SIZE instead of dividing sizeof array with sizeof an element Semantic patch information: This makes an effort to find cases where ARRAY_SIZE can be used such as where there is a division of sizeof the array by the

[PATCH] staging: rts5208: Removed the new typedefs

2014-12-10 Thread Anjana Sasindran
This patch fix the checkpatch.pl warning: WARNING: do not add new typedefs Signed-off-by: Anjana Sasindran anjanasasindran...@gmail.com --- drivers/staging/rts5208/rtsx_sys.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/rts5208/rtsx_sys.h

Re: [PATCH] staging: rts5208: Removed the new typedefs

2014-12-10 Thread Greg KH
On Wed, Dec 10, 2014 at 07:24:16PM +0530, Anjana Sasindran wrote: This patch fix the checkpatch.pl warning: WARNING: do not add new typedefs Signed-off-by: Anjana Sasindran anjanasasindran...@gmail.com --- drivers/staging/rts5208/rtsx_sys.h | 2 +- 1 file changed, 1 insertion(+), 1

Re: [PATCH] staging: lustre: Fix the warning messages about casting without __user macro

2014-12-10 Thread Greg KH
On Wed, Dec 10, 2014 at 07:09:59AM +, Al Viro wrote: On Tue, Dec 09, 2014 at 10:56:12PM -0800, Shalin Mehta wrote: From: Shalin Mehta shalinmeht...@gmail.com This issue is showed up while compiling with sparse. The iov_base in struct iovec struct explicitly declares that the

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

2014-12-10 Thread gre...@linuxfoundation.org
On Wed, Dec 10, 2014 at 10:22:40AM +0100, Vitaly Kuznetsov wrote: Dexuan Cui de...@microsoft.com writes: -Original Message- From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] Sent: Tuesday, December 9, 2014 23:48 PM To: KY Srinivasan Cc: Haiyang Zhang;

[PATCH 1/2] staging: lustre: Fix minor style error in libcfs_string.c

2014-12-10 Thread Matthew Tyler
This patch adds spacing around an '=' operator Signed-off-by: Matthew Tyler matt.ty...@flashics.com --- drivers/staging/lustre/lustre/libcfs/libcfs_string.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/lustre/lustre/libcfs/libcfs_string.c

[PATCH 2/2] staging: lustre: Fix trailing statement error in libcfs_string.c

2014-12-10 Thread Matthew Tyler
This patch fixes a trailing statement warning found by checkpatch.pl Signed-off-by: Matthew Tyler matt.ty...@flashics.com --- drivers/staging/lustre/lustre/libcfs/libcfs_string.c | 7 --- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git

[PATCH] staging: vt6656: whitespace checkpatch warning fix.

2014-12-10 Thread Krzysztof Adamski
This fixes following checkpatch.pl error: ERROR: space prohibited before that ',' (ctx:WxW) Signed-off-by: Krzysztof Adamski k...@japko.eu --- drivers/staging/vt6656/rxtx.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/vt6656/rxtx.c

[PATCH 1/1] Drivers: hv: vmbus: Fix a bug in vmbus_establish_gpadl()

2014-12-10 Thread K. Y. Srinivasan
Fix a bug in vmbus_establish_gpadl(). I would like to thank Michael Brown mc...@ipxe.org for seeing this bug. Signed-off-by: K. Y. Srinivasan k...@microsoft.com --- drivers/hv/channel.c |4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/hv/channel.c

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

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

RE: [PATCH 2/5] Tools: hv: remove unused bytes_written from kvp_update_file()

2014-12-10 Thread KY Srinivasan
-Original Message- From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] Sent: Tuesday, December 9, 2014 7:48 AM To: KY Srinivasan Cc: Haiyang Zhang; de...@linuxdriverproject.org; linux- ker...@vger.kernel.org; Dexuan Cui Subject: [PATCH 2/5] Tools: hv: remove unused bytes_written

RE: [PATCH 3/5] Tools: hv: address compiler warnings for hv_kvp_daemon.c

2014-12-10 Thread KY Srinivasan
-Original Message- From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] Sent: Tuesday, December 9, 2014 7:48 AM To: KY Srinivasan Cc: Haiyang Zhang; de...@linuxdriverproject.org; linux- ker...@vger.kernel.org; Dexuan Cui Subject: [PATCH 3/5] Tools: hv: address compiler warnings for

RE: [PATCH 1/5] Tools: hv: add mising fcopyd to the Makefile

2014-12-10 Thread KY Srinivasan
-Original Message- From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] Sent: Tuesday, December 9, 2014 7:48 AM To: KY Srinivasan Cc: Haiyang Zhang; de...@linuxdriverproject.org; linux- ker...@vger.kernel.org; Dexuan Cui Subject: [PATCH 1/5] Tools: hv: add mising fcopyd to the

RE: [PATCH 5/5] Tools: hv: do not add redundant '/' in hv_start_fcopy()

2014-12-10 Thread KY Srinivasan
-Original Message- From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] Sent: Tuesday, December 9, 2014 7:48 AM To: KY Srinivasan Cc: Haiyang Zhang; de...@linuxdriverproject.org; linux- ker...@vger.kernel.org; Dexuan Cui Subject: [PATCH 5/5] Tools: hv: do not add redundant '/' in

RE: [PATCH 4/5] Tools: hv: address compiler warnings for hv_fcopy_daemon.c

2014-12-10 Thread KY Srinivasan
-Original Message- From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] Sent: Tuesday, December 9, 2014 7:48 AM To: KY Srinivasan Cc: Haiyang Zhang; de...@linuxdriverproject.org; linux- ker...@vger.kernel.org; Dexuan Cui Subject: [PATCH 4/5] Tools: hv: address compiler warnings for

Re: [PATCH 1/1] Drivers: hv: vmbus: Fix a bug in vmbus_establish_gpadl()

2014-12-10 Thread Greg KH
On Wed, Dec 10, 2014 at 11:42:43AM -0800, K. Y. Srinivasan wrote: Fix a bug in vmbus_establish_gpadl(). I would like to thank Michael Brown mc...@ipxe.org for seeing this bug. Then use the Reported-by: tag with his name in this patch please. thanks, greg k-h

RE: [PATCH 1/1] Drivers: hv: vmbus: Fix a bug in vmbus_establish_gpadl()

2014-12-10 Thread KY Srinivasan
-Original Message- From: Greg KH [mailto:gre...@linuxfoundation.org] Sent: Wednesday, December 10, 2014 11:00 AM To: KY Srinivasan Cc: linux-ker...@vger.kernel.org; de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; jasow...@redhat.com; mc...@ipxe.org Subject: Re:

[PATCH V2 1/1] Drivers: hv: vmbus: Fix a bug in vmbus_establish_gpadl()

2014-12-10 Thread K. Y. Srinivasan
Fix a bug in vmbus_establish_gpadl(). I would like to thank Michael Brown mc...@ipxe.org for seeing this bug. In this version, I have added the Reported-by tag. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reported-by: Michael Brown mc...@ipxe.org --- drivers/hv/channel.c |4 ++-- 1

Re: [PATCH 3/3] Drivers: hv: hv_balloon: Don't post pressure status from interrupt context

2014-12-10 Thread Sitsofe Wheeler
On Mon, Dec 08, 2014 at 06:04:35AM +, KY Srinivasan wrote: Greg has not committed these patches yet. One of the patches changes the balloon floor. This means that the guest will not be ballooned down below the floor. Is this what you are seeing? In our testing we did not see anything

Re: [PATCH] scsi:storvsc enable reading from VPD pages on SPC-2

2014-12-10 Thread Sitsofe Wheeler
On Wed, Dec 10, 2014 at 12:38:25AM -0800, Long Li wrote: MSFT targets currently claim SPC-2 compliance while they implement post SPC-2 features. With this patch we can correctly handle WRITE_SAME_16 issues. This patch fixes an issue where the flag is setup too late in drive initialization

Re: [PATCH] scsi:storvsc enable reading from VPD pages on SPC-2

2014-12-10 Thread Martin K. Petersen
Long == Long Li lon...@microsoft.com writes: Long MSFT targets currently claim SPC-2 compliance while they implement Long post SPC-2 features. With this patch we can correctly handle Long WRITE_SAME_16 issues. Handle the issues or handle WRITE SAME(10/16)? + {Msft, Virtual Disk, 1.0,

RE: [PATCH] scsi:storvsc enable reading from VPD pages on SPC-2

2014-12-10 Thread Long Li
Long == Long Li lon...@microsoft.com writes: Long MSFT targets currently claim SPC-2 compliance while they implement Long post SPC-2 features. With this patch we can correctly handle Long WRITE_SAME_16 issues. Handle the issues or handle WRITE SAME(10/16)? With this patch, the SCSI

RE: [PATCH] scsi:storvsc enable reading from VPD pages on SPC-2

2014-12-10 Thread Long Li
-Original Message- From: Sitsofe Wheeler [mailto:sits...@gmail.com] Sent: Wednesday, December 10, 2014 12:58 PM To: Long Li Cc: KY Srinivasan; Haiyang Zhang; jbottom...@parallels.com; linux- s...@vger.kernel.org; de...@linuxdriverproject.org; linux- ker...@vger.kernel.org Subject:

[PATCH v4 1/2] staging: media: lirc: lirc_zilog.c: fix quoted strings split across lines

2014-12-10 Thread Luis de Bethencourt
checkpatch makes an exception to the 80-colum rule for quotes strings, and Documentation/CodingStyle recommends not splitting quotes strings across lines because it breaks the ability to grep for the string. Fixing these. WARNING: quoted string split across lines Signed-off-by: Luis de

[PATCH v4 2/2] staging: media: lirc: lirc_zilog.c: keep consistency in dev functions

2014-12-10 Thread Luis de Bethencourt
The previous patch switched some dev functions to move the string to a second line. Doing this for all similar functions because it makes the driver easier to read if all similar lines use the same criteria. Signed-off-by: Luis de Bethencourt l...@debethencourt.com ---

RE: [PATCH] scsi:storvsc enable reading from VPD pages on SPC-2

2014-12-10 Thread KY Srinivasan
-Original Message- From: Long Li Sent: Wednesday, December 10, 2014 2:28 PM To: Martin K. Petersen Cc: KY Srinivasan; Haiyang Zhang; jbottom...@parallels.com; linux- s...@vger.kernel.org; de...@linuxdriverproject.org; linux- ker...@vger.kernel.org Subject: RE: [PATCH]

RE: [PATCH 3/3] Drivers: hv: hv_balloon: Don't post pressure status from interrupt context

2014-12-10 Thread KY Srinivasan
-Original Message- From: Sitsofe Wheeler [mailto:sits...@gmail.com] Sent: Wednesday, December 10, 2014 12:50 PM To: KY Srinivasan Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; jasow...@redhat.com

Re: [PATCH v4 1/2] staging: media: lirc: lirc_zilog.c: fix quoted strings split across lines

2014-12-10 Thread Luis de Bethencourt
On Wed, Dec 10, 2014 at 03:39:09PM -0800, Joe Perches wrote: On Wed, 2014-12-10 at 22:33 +, Luis de Bethencourt wrote: checkpatch makes an exception to the 80-colum rule for quotes strings, and Documentation/CodingStyle recommends not splitting quotes strings across lines because it

Re: [PATCH v4 1/2] staging: media: lirc: lirc_zilog.c: fix quoted strings split across lines

2014-12-10 Thread Joe Perches
On Wed, 2014-12-10 at 23:57 +, Luis de Bethencourt wrote: On Wed, Dec 10, 2014 at 03:39:09PM -0800, Joe Perches wrote: On Wed, 2014-12-10 at 22:33 +, Luis de Bethencourt wrote: diff --git a/drivers/staging/media/lirc/lirc_zilog.c b/drivers/staging/media/lirc/lirc_zilog.c [] +

[PATCH V3 1/1] Drivers: hv: vmbus: Fix a bug in vmbus_establish_gpadl()

2014-12-10 Thread K. Y. Srinivasan
Correctly compute the local (gpadl) handle. I would like to thank Michael Brown mc...@ipxe.org for seeing this bug. Signed-off-by: K. Y. Srinivasan k...@microsoft.com Reported-by: Michael Brown mc...@ipxe.org --- Changes in V2: Added the Reported-by tag. Changes in V3: Cleaned up the commit log.

Re: [PATCH v4 1/2] staging: media: lirc: lirc_zilog.c: fix quoted strings split across lines

2014-12-10 Thread Luis de Bethencourt
On Wed, Dec 10, 2014 at 04:01:04PM -0800, Joe Perches wrote: On Wed, 2014-12-10 at 23:57 +, Luis de Bethencourt wrote: On Wed, Dec 10, 2014 at 03:39:09PM -0800, Joe Perches wrote: On Wed, 2014-12-10 at 22:33 +, Luis de Bethencourt wrote: diff --git

RE: [PATCH 2/2] Drivers: hv: balloon: Fix the deadlock issue in the memory hot-add code

2014-12-10 Thread KY Srinivasan
-Original Message- From: Michal Hocko [mailto:mho...@suse.cz] Sent: Tuesday, December 9, 2014 2:56 AM To: Yasuaki Ishimatsu Cc: KY Srinivasan; gre...@linuxfoundation.org; linux- ker...@vger.kernel.org; de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com;

[PATCH v5 1/3] staging: media: lirc: lirc_zilog.c: fix quoted strings split across lines

2014-12-10 Thread Luis de Bethencourt
checkpatch makes an exception to the 80-colum rule for quotes strings, and Documentation/CodingStyle recommends not splitting quotes strings across lines because it breaks the ability to grep for the string. Fixing these. WARNING: quoted string split across lines Signed-off-by: Luis de

[PATCH v5 2/3] staging: media: lirc: lirc_zilog.c: keep consistency in dev functions

2014-12-10 Thread Luis de Bethencourt
The previous patch switched some dev functions to move the string to a second line. Doing this for all similar functions because it makes the driver easier to read if all similar lines use the same criteria. Signed-off-by: Luis de Bethencourt l...@debethencourt.com ---

[PATCH v5 3/3] staging: media: lirc: lirc_zilog.c: missing newline in dev_err()

2014-12-10 Thread Luis de Bethencourt
Missing newline character at the end of string passed to dev_err() Signed-off-by: Luis de Bethencourt l...@debethencourt.com --- drivers/staging/media/lirc/lirc_zilog.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/media/lirc/lirc_zilog.c

Fwd: improve support for usb board camera

2014-12-10 Thread Kristopher Borer
Hello, I am looking to improve support for a USB board camera. It currently works fairly well, but we occasionally cause kernel panics while using OpenCV on Ubuntu 14.04. Here is the device: e-consystems.com/5mp-usb-cameraboard.asp We do not need an NDA. We can provide hardware, test

[PATCH socfpga-nex] staging: fpga manager: socfpga_fpga_ops can be static

2014-12-10 Thread kbuild test robot
drivers/staging/fpga/socfpga.c:655:25: sparse: symbol 'socfpga_fpga_ops' was not declared. Should it be static? Signed-off-by: Fengguang Wu fengguang...@intel.com --- socfpga.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/fpga/socfpga.c

[socfpga-nex:for_next-fpgamgr-without-overlays-v4.3 8/9] drivers/staging/fpga/socfpga.c:655:25: sparse: symbol 'socfpga_fpga_ops' was not declared. Should it be static?

2014-12-10 Thread kbuild test robot
tree: git://git.rocketboards.org/linux-socfpga-next for_next-fpgamgr-without-overlays-v4.3 head: 7920447fc75922e9a72db640cddf890d75a30e0d commit: 99aa2d008ce7a2a925368bfddd53ae83d307fb38 [8/9] staging: fpga manager: add driver for socfpga fpga manager reproduce: # apt-get install sparse

Re: Fwd: improve support for usb board camera

2014-12-10 Thread Greg KH
On Wed, Dec 10, 2014 at 07:35:48PM -0500, Kristopher Borer wrote: Hello, I am looking to improve support for a USB board camera. It currently works fairly well, but we occasionally cause kernel panics while using OpenCV on Ubuntu 14.04. What kernel version is that, and what is the oops

Re: [PATCH v4 1/2] staging: media: lirc: lirc_zilog.c: fix quoted strings split across lines

2014-12-10 Thread Joe Perches
On Wed, 2014-12-10 at 22:33 +, Luis de Bethencourt wrote: checkpatch makes an exception to the 80-colum rule for quotes strings, and Documentation/CodingStyle recommends not splitting quotes strings across lines because it breaks the ability to grep for the string. Fixing these. [] diff

Re: [PATCH V3 1/1] Drivers: hv: vmbus: Fix a bug in vmbus_establish_gpadl()

2014-12-10 Thread Jeremiah Mahler
K. Y. Srinivasan, On Wed, Dec 10, 2014 at 05:13:00PM -0800, K. Y. Srinivasan wrote: Correctly compute the local (gpadl) handle. This description is still too sparse for me. How was it computed before and why was this incorrect? Pretend like you are trying to explain your patch to someone who

Re: [PATCH v18 12/12] drm: bridge/dw_hdmi: add rockchip rk3288 support

2014-12-10 Thread Daniel Kurtz
Hi Andy, This driver adds HDMI to rockchip/drm. The fact that rockchip's hdmi uses dw_hdmi is an implementation detail. I do not think that the names used for rk3288-hdmi should include dw in them. See inline for what I mean... On Thu, Dec 4, 2014 at 10:34 PM, Andy Yan andy@rock-chips.com