Re: [PATCH] Drivers: hv: vmbus: fix error return code in vmbus_connect()

2013-09-12 Thread Dan Carpenter
On Thu, Sep 12, 2013 at 12:36:46AM +, KY Srinivasan wrote: > When you try a given version, vmbus_negotiate_version() returns an error code > if the host does not support that version. Hence, in this code you will not > be able to > run on anything prior to ws2012. I have already sent in a patc

RE: [PATCH] Drivers: hv: vmbus: fix error return code in vmbus_connect()

2013-09-11 Thread KY Srinivasan
4:20 AM > >> To: KY Srinivasan; Haiyang Zhang > >> Cc: yongjun_...@trendmicro.com.cn; de...@linuxdriverproject.org; linux- > >> ker...@vger.kernel.org > >> Subject: [PATCH] Drivers: hv: vmbus: fix error return code in > >> vmbus_connect() > >> > &

Re: [PATCH] Drivers: hv: vmbus: fix error return code in vmbus_connect()

2013-09-11 Thread Wei Yongjun
xdriverproject.org; linux- >> ker...@vger.kernel.org >> Subject: [PATCH] Drivers: hv: vmbus: fix error return code in vmbus_connect() >> >> From: Wei Yongjun >> >> Fix to return -EINVAL in the version check error handling >> case instead of 0, as done elsewhere i

RE: [PATCH] Drivers: hv: vmbus: fix error return code in vmbus_connect()

2013-09-11 Thread KY Srinivasan
H] Drivers: hv: vmbus: fix error return code in vmbus_connect() > > From: Wei Yongjun > > Fix to return -EINVAL in the version check error handling > case instead of 0, as done elsewhere in this function. The return will not be zero in this case. If you look at the function vmbus_n

[PATCH] Drivers: hv: vmbus: fix error return code in vmbus_connect()

2013-09-11 Thread Wei Yongjun
From: Wei Yongjun Fix to return -EINVAL in the version check error handling case instead of 0, as done elsewhere in this function. Signed-off-by: Wei Yongjun --- drivers/hv/connection.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/drivers/hv/connection.c b/drivers/hv/