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

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] scsi:storvsc enable reading from VPD pages on SPC-2

2014-12-09 Thread Long Li
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 process. Reviewed-by: K. Y. Srinivasan Signed-off-by: Long Li

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

2014-12-09 Thread Al Viro
On Tue, Dec 09, 2014 at 10:56:12PM -0800, Shalin Mehta wrote: > From: Shalin Mehta > > This issue is showed up while compiling with sparse. The iov_base in struct > iovec struct explicitly declares that the assigned value should be user space > pointer with __user macro. Where as here, the __us

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

2014-12-09 Thread Shalin Mehta
From: Shalin Mehta This issue is showed up while compiling with sparse. The iov_base in struct iovec struct explicitly declares that the assigned value should be user space pointer with __user macro. Where as here, the __user macro isn't used while casting. Signed-off-by: Shalin Mehta --- d

[PATCH] staging: vt6655: remove unused function in mac.c

2014-12-09 Thread Fred Chou
From: Fred Chou Removed the unused function MACvGetShortRetryLimit, which also fixed the following sparse warning: drivers/staging/vt6655/mac.c:162:6: warning: symbol 'MACvGetShortRetryLimit' was not declared. Should it be static? Signed-off-by: Fred Chou --- drivers/staging/vt6655/mac.c | 19

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 an

Re: [PATCH 0/20] fix misspelling of current function in string

2014-12-09 Thread Julian Calaby
Hi Julia, On Mon, Dec 8, 2014 at 5:43 PM, Julia Lawall wrote: > On Mon, 8 Dec 2014, Julian Calaby wrote: > >> Hi Julia, >> >> On Mon, Dec 8, 2014 at 6:20 AM, Julia Lawall wrote: >> > These patches replace what appears to be a reference to the name of the >> > current function but is misspelled i

[socfpga-nex:for_next-fpgamgr-without-overlays-v4 8/9] drivers/staging/fpga/fpga-mgr.c:46:5: sparse: symbol 'fpga_mgr_low_level_state' was not declared. Should it be static?

2014-12-09 Thread kbuild test robot
tree: git://git.rocketboards.org/linux-socfpga-next for_next-fpgamgr-without-overlays-v4 head: 590b747bfa6791797c241116453acf50bc4e5dda commit: 356ae534a603de3aa8b1cfc173a777f59273d32e [8/9] staging: fpga manager: framework core reproduce: # apt-get install sparse git checkout 356ae534a60

[PATCH socfpga-nex] staging: fpga manager: fpga_mgr_low_level_state() can be static

2014-12-09 Thread kbuild test robot
drivers/staging/fpga/fpga-mgr.c:46:5: sparse: symbol 'fpga_mgr_low_level_state' was not declared. Should it be static? drivers/staging/fpga/fpga-mgr.c:324:30: sparse: symbol 'fpga_mgr_groups' was not declared. Should it be static? drivers/staging/fpga/fpga-mgr.c:365:25: sparse: symbol 'fpga_mgr_d

Blessed Is The Hand That Giveth.

2014-12-09 Thread PATRICIA WAGNER
Good day and God bless you. I feel quite safe and satisfy dealing with you in this charity project.My name is MRS PATRICIA WAGNER , a merchant in Dubai, in the U.A.E. I have been diagnosed with Esophageal cancer . It has defiled all forms of medical treatment, and right now I have only about a

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

2014-12-09 Thread Vitaly Kuznetsov
This patch addresses two types of compiler warnings: ... warning: unused variable ‘fd’ [-Wunused-variable] and ... warning: format ‘%s’ expects argument of type ‘char *’, but argument 5 has type ‘__u16 *’ [-Wformat=] Signed-off-by: Vitaly Kuznetsov --- tools/hv/hv_fcopy_daemon.c | 4 ++-- 1 fil

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

2014-12-09 Thread Vitaly Kuznetsov
We don't need to add additional '/' to smsg->path_name as snprintf("%s/%s") does the right thing. Without the patch we get doubled '//' in the log message. Signed-off-by: Vitaly Kuznetsov --- tools/hv/hv_fcopy_daemon.c | 6 -- 1 file changed, 6 deletions(-) diff --git a/tools/hv/hv_fcopy_da

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

2014-12-09 Thread Vitaly Kuznetsov
This patch addresses two types of compiler warnings: ... warning: comparison between signed and unsigned integer expressions [-Wsign-compare] and ... warning: pointer targets in passing argument N of ‘kvp_...’ differ in signedness [-Wpointer-sign] Signed-off-by: Vitaly Kuznetsov --- tools/hv/h

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

2014-12-09 Thread Vitaly Kuznetsov
fwrite() does not actually return the number of bytes written and this value is being ignored anyway and ferror() is being called to check for an error. As we assign to this variable and never use it we get the following compile-time warning: hv_kvp_daemon.c:149:9: warning: variable ‘bytes_written’

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

2014-12-09 Thread Vitaly Kuznetsov
fcopyd in missing in the Makefile, add it there. Signed-off-by: Vitaly Kuznetsov --- tools/hv/Makefile | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/tools/hv/Makefile b/tools/hv/Makefile index bd22f78..99ffe61 100644 --- a/tools/hv/Makefile +++ b/tools/hv/Makefile @@ -5

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

2014-12-09 Thread Vitaly Kuznetsov
When someone does 'make' in tools/hv/ issues appear: - hv_fcopy_daemon is not being built; - lots of compiler warnings. This is just a cleanup. Compile-tested by myself on top of linux-next/master. Piggyback this series and send "[PATCH 5/5] Tools: hv: do not add redundant '/' in hv_start_fcopy(

[PATCH 1/1] [PATCH] Staging:vt6655: sparse warning fix

2014-12-09 Thread Adithya K
From: Adithya Krishnamurthy Fixed sparse warning(symbol was not declared. Should it be static) fix. Signed-off-by: Adithya Krishnamurthy --- --- drivers/staging/vt6655/device_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/vt6655/device_main.c b/driv

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

2014-12-09 Thread Vitaly Kuznetsov
Dexuan Cui writes: > Currently IPv6-only-injection doesn't work because the daemon doesn't parse > any IPv6 information at all once it finds the dhcp_enabled flag is true. > > But according to the Hyper-v host team, the flag is only for IPv4. > In the case the host only injects 1 IPv6 address, th

BUSINESS ENQUIRY.

2014-12-09 Thread Taylor
BUSINESS ENQUIRY. Hello Sir, My company based in United Kingdom is in need of your product I will appreciate if you can get back to me with your grading qualities and prices , i will appreciate if you send me your catalog too. Sincerely, Mr Simon Taylor. Address:Brierl

Re: [PATCH 1/1] [PATCH] Staging:vt6655: sparse warning fix

2014-12-09 Thread Sudip Mukherjee
On Mon, Dec 08, 2014 at 11:33:25PM +0530, Adithya K wrote: > From: Adithya Krishnamurthy > > Fixed sparse warning fix. you should mention what type of warning you have fixed. sudip > > Signed-off-by: Adithya Krishnamurthy > --- > --- > drivers/staging/vt6655/device_main.c | 2 +- > 1 file ch

[PATCH] staging: vt6656: Use ether_addr_copy() on vnt_private members.

2014-12-09 Thread Krzysztof Adamski
This patch fixes checkpatch.pl warning: WARNING: Prefer ether_addr_copy() over memcpy() if the Ethernet addresses are __aligned(2) current_net_addr and permanent_net_addr members of vnt_private alignment is changed to at last 16 bits so that ether_addr_copy can be safely used on them. buf->data i

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

2014-12-09 Thread Michal Hocko
On Tue 09-12-14 19:25:50, Yasuaki Ishimatsu wrote: > (2014/12/09 18:08), Michal Hocko wrote: [...] > >Doesn't udev retry the operation if it gets EBUSY or EAGAIN? > > It depend on implementation of udev.rules. So we can retry online/offline > operation in udev.rules. [...] # Memory hotadd request

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

2014-12-09 Thread Dexuan Cui
Currently IPv6-only-injection doesn't work because the daemon doesn't parse any IPv6 information at all once it finds the dhcp_enabled flag is true. But according to the Hyper-v host team, the flag is only for IPv4. In the case the host only injects 1 IPv6 address, the dhcp flag is true, but we sh

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

2014-12-09 Thread Yasuaki Ishimatsu
(2014/12/09 18:08), Michal Hocko wrote: On Tue 09-12-14 10:23:51, Yasuaki Ishimatsu wrote: (2014/12/09 0:04), Michal Hocko wrote: On Fri 05-12-14 16:41:38, K. Y. Srinivasan wrote: Andy Whitcroft initially saw this deadlock. We have seen this as well. Here is the original description of the pr

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

2014-12-09 Thread Michal Hocko
On Tue 09-12-14 10:23:51, Yasuaki Ishimatsu wrote: > (2014/12/09 0:04), Michal Hocko wrote: > >On Fri 05-12-14 16:41:38, K. Y. Srinivasan wrote: > >>Andy Whitcroft initially saw this deadlock. We > >>have seen this as well. Here is the original description of the > >>problem (and a potential solut

[PATCH 1/1] [PATCH] Staging:vt6655: sparse warning fix

2014-12-09 Thread Adithya K
From: Adithya Krishnamurthy Fixed sparse warning fix. Signed-off-by: Adithya Krishnamurthy --- --- drivers/staging/vt6655/device_main.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/vt6655/device_main.c b/drivers/staging/vt6655/device_main.c index 83e4162