RE: [PATCH V5 4/9] Drivers: hv: ring_buffer: enhance hv_ringbuffer_read() to support hvsock

2016-01-03 Thread Dexuan Cui
> -Original Message- > From: David Miller [mailto:da...@davemloft.net] > Sent: Saturday, January 2, 2016 12:30 > To: Dexuan Cui > Cc: gre...@linuxfoundation.org; step...@networkplumber.org; > net...@vger.kernel.org; linux-ker...@vger.kernel.org; driverdev- > de...@l

RE: [PATCH V5 4/9] Drivers: hv: ring_buffer: enhance hv_ringbuffer_read() to support hvsock

2016-01-05 Thread Dexuan Cui
> From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] > Sent: Tuesday, January 5, 2016 20:31 > ... > > To get the payload of hvsock, we need raw=0 to skip the level-1 header > > (i.e., struct vmpacket_descriptor desc) and we also need to skip the > > level-2 header (i.e., struct vmpipe_proto_header

RE: [PATCH V5 5/9] Drivers: hv: vmbus: add APIs to send/recv hvsock packets

2016-01-05 Thread Dexuan Cui
> From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] > Sent: Tuesday, January 5, 2016 20:39 > ... > > +/* > > + * vmbus_sendpacket_hvsock - Send the hvsock payload 'buf' of a length > > 'len' > > + */ > > +int vmbus_sendpacket_hvsock(struct vmbus_channel *channel, void *buf, > u32 len) > > ... >

RE: [PATCH V5 7/9] Drivers: hv: vmbus: add a mechanism to pass hvsock events to the hvsock driver

2016-01-05 Thread Dexuan Cui
> From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] > > diff --git a/drivers/hv/channel_mgmt.c b/drivers/hv/channel_mgmt.c > > > > +/* hvsock related definitions */ > > +enum hvsock_event { > > + /* The host application is close()-ing the connection */ > > + HVSOCK_RESCIND_CHANNEL, > > +}; >

RE: [PATCH V5 9/9] hvsock: introduce Hyper-V VM Sockets feature

2016-01-05 Thread Dexuan Cui
> From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] > Sent: Tuesday, January 5, 2016 17:52 > > Just some minor nitpicks below -- I have to admit I didn't test the feature. > > [..skip..] > > > + > > + if (sk->sk_err) { > > + ret = -sk->sk_err; > > + goto out_wait_error; >

[PATCH net-next 1/3] net: add the AF_KCM entries to family name tables

2016-03-21 Thread Dexuan Cui
This is for the recent kcm driver, which introduces AF_KCM(41) in b7ac4eb(kcm: Kernel Connection Multiplexor module). Signed-off-by: Dexuan Cui Cc: Signed-off-by: Tom Herbert --- net/core/sock.c | 9 ++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/net/core/sock.c b/net

[PATCH net-next 2/3] hv_sock: introduce Hyper-V Sockets

2016-03-21 Thread Dexuan Cui
mt_service The patch implements the necessary support in the guest side by introducing a new socket address family AF_HYPERV. Signed-off-by: Dexuan Cui Cc: "K. Y. Srinivasan" Cc: Haiyang Zhang Cc: Vitaly Kuznetsov --- I posted the V6 of the hv_sock patchset in Jan: [PATCH V6 0/8] in

[PATCH net-next 3/3] net: add the AF_HYPERV entries to family name tables

2016-03-21 Thread Dexuan Cui
This is for the hv_sock driver, which introduces AF_HYPERV(42). Signed-off-by: Dexuan Cui Cc: "K. Y. Srinivasan" Cc: Haiyang Zhang --- net/core/sock.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/net/core/sock.c b/net/core/sock.c index 7e73c26..51ff

RE: [PATCH net-next 1/3] net: add the AF_KCM entries to family name tables

2016-03-21 Thread Dexuan Cui
> From: David Miller [mailto:da...@davemloft.net] > Sent: Monday, March 21, 2016 23:28 > To: Dexuan Cui > Cc: gre...@linuxfoundation.org; net...@vger.kernel.org; linux- > ker...@vger.kernel.org; de...@linuxdriverproject.org; o...@aepfle.de; > a...@canonical.com; jasow...@redhat.

KVP daemon/module race on module unloading

2016-03-24 Thread Dexuan Cui
Hi Vitaly, We got the below warning once -- it's pretty hard to repro it. I think the cause may be: before the daemon is terminated, we can successfully "rmmod hv_utils" -- as a result all the code/data segments of the module shouldn't be accessed any more, but in the kernel we still has a dangli

RE: KVP daemon/module race on module unloading

2016-03-24 Thread Dexuan Cui
> From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] > Sent: Friday, March 25, 2016 0:11 > To: Dexuan Cui > Cc: driverdev-devel@linuxdriverproject.org; Chris Valean (Cloudbase Solutions > SRL) ; KY Srinivasan > Subject: Re: KVP daemon/module race on module unloading >

[PATCH v7 net-next 1/1] hv_sock: introduce Hyper-V Sockets

2016-04-07 Thread Dexuan Cui
mt_service The patch implements the necessary support in the guest side by introducing a new socket address family AF_HYPERV. Signed-off-by: Dexuan Cui Cc: "K. Y. Srinivasan" Cc: Haiyang Zhang Cc: Vitaly Kuznetsov --- MAINTAINERS |2 + include/linux/hyperv.h

[PATCH v7 net-next 0/1] introduce Hyper-V VM Sockets(hv_sock)

2016-04-07 Thread Dexuan Cui
cind callback (Thank Vitaly!) - avoided the introduction of new VMBUS driver APIs vmbus_sendpacket_hvsock() and vmbus_recvpacket_hvsock() and used vmbus_sendpacket()/vmbus_recvpacket() in the higher level (i.e., the vmsock driver). Thank Vitaly! Changes since v6 (http://lkml.iu.edu/hypermail/lin

RE: [PATCH v7 net-next 1/1] hv_sock: introduce Hyper-V Sockets

2016-04-07 Thread Dexuan Cui
> From: Joe Perches [mailto:j...@perches.com] > Sent: Thursday, April 7, 2016 19:30 > To: Dexuan Cui ; gre...@linuxfoundation.org; > da...@davemloft.net; net...@vger.kernel.org; linux-ker...@vger.kernel.org; > de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; > j

[PATCH v8 net-next 0/1] introduce Hyper-V VM Sockets(hv_sock)

2016-04-07 Thread Dexuan Cui
rnel/1601.3/01813.html) - only a few minor changes of coding style and comments Changes since v7 - a few minor changes of coding style: thanks, Joe Perches! - added some lines of comments about GUID/UUID before the struct sockaddr_hv. Dexuan Cui (1): hv_sock: introduce Hyper-V Sockets MAINTAINERS

[PATCH v8 net-next 1/1] hv_sock: introduce Hyper-V Sockets

2016-04-07 Thread Dexuan Cui
mt_service The patch implements the necessary support in the guest side by introducing a new socket address family AF_HYPERV. Signed-off-by: Dexuan Cui Cc: "K. Y. Srinivasan" Cc: Haiyang Zhang Cc: Vitaly Kuznetsov --- MAINTAINERS |2 + include/linux/hyperv.h

RE: [PATCH v8 net-next 1/1] hv_sock: introduce Hyper-V Sockets

2016-04-07 Thread Dexuan Cui
> From: Joe Perches [mailto:j...@perches.com] > Sent: Friday, April 8, 2016 9:15 > On Thu, 2016-04-07 at 18:36 -0700, Dexuan Cui wrote: > > diff --git a/include/net/af_hvsock.h b/include/net/af_hvsock.h > [] > > +#define VMBUS_RINGBUFFER_SIZE_HVSOCK_RECV (5 *

RE: [PATCH v8 net-next 1/1] hv_sock: introduce Hyper-V Sockets

2016-04-13 Thread Dexuan Cui
> From: David Miller [mailto:da...@davemloft.net] > Sent: Thursday, April 14, 2016 10:30 > To: Dexuan Cui > Cc: gre...@linuxfoundation.org; net...@vger.kernel.org; linux- > ker...@vger.kernel.org; de...@linuxdriverproject.org; o...@aepfle.de; > a...@canonical.com; jasow.

RE: [PATCH] tools: hv: lsvmbus: add pci pass-through UUID

2016-04-22 Thread Dexuan Cui
> From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] > Sent: Saturday, April 23, 2016 0:21 > To: de...@linuxdriverproject.org > Cc: linux-ker...@vger.kernel.org; KY Srinivasan ; Haiyang > Zhang ; Dexuan Cui > Subject: [PATCH] tools: hv: lsvmbus: add pci pass-through UUID >

RE: [PATCH v2] tools: hv: lsvmbus: add pci pass-through UUID

2016-04-25 Thread Dexuan Cui
> From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] > Sent: Monday, April 25, 2016 19:06 > To: de...@linuxdriverproject.org > Cc: linux-ker...@vger.kernel.org; KY Srinivasan ; Haiyang > Zhang ; Dexuan Cui > Subject: [PATCH v2] tools: hv: lsvmbus: add pci pass-through UUID >

RE: [PATCH v8 net-next 1/1] hv_sock: introduce Hyper-V Sockets

2016-04-26 Thread Dexuan Cui
> From: Cathy Avery [mailto:cav...@redhat.com] > Sent: Wednesday, April 27, 2016 0:19 > To: Dexuan Cui ; gre...@linuxfoundation.org; > da...@davemloft.net; net...@vger.kernel.org; linux-ker...@vger.kernel.org; > de...@linuxdriverproject.org; o...@aepfle.de; Jason Wang > ; KY Sr

[PATCH v9 net-next 0/2] introduce Hyper-V VM Sockets(hv_sock)

2016-05-04 Thread Dexuan Cui
ed the code according to suggestions from Cathy Avery: split big functions into small ones, set .setsockopt and getsockopt to sock_no_setsockopt/sock_no_getsockopt - inline'd some small list helper functions Dexuan Cui (2): hv_sock: introduce Hyper-V Sockets net: add the AF_HYPERV entri

[PATCH v9 net-next 1/2] hv_sock: introduce Hyper-V Sockets

2016-05-04 Thread Dexuan Cui
mt_service The patch implements the necessary support in the guest side by introducing a new socket address family AF_HYPERV. Signed-off-by: Dexuan Cui Cc: "K. Y. Srinivasan" Cc: Haiyang Zhang Cc: Vitaly Kuznetsov Cc: Cathy Avery --- MAINTAINERS |2 + include/l

[PATCH v9 net-next 2/2] net: add the AF_HYPERV entries to family name tables

2016-05-04 Thread Dexuan Cui
This is for the hv_sock driver, which introduces AF_HYPERV(42). Signed-off-by: Dexuan Cui Cc: "K. Y. Srinivasan" Cc: Haiyang Zhang Cc: Vitaly Kuznetsov Cc: Cathy Avery --- net/core/sock.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/net/core/sock.c

RE: [PATCH v9 net-next 1/2] hv_sock: introduce Hyper-V Sockets

2016-05-07 Thread Dexuan Cui
> From: David Miller [mailto:da...@davemloft.net] > Sent: Saturday, May 7, 2016 1:04 > To: Dexuan Cui > Cc: gre...@linuxfoundation.org; net...@vger.kernel.org; linux- > ker...@vger.kernel.org; de...@linuxdriverproject.org; o...@aepfle.de; > a...@canonical.com; jasow.

RE: [PATCH v9 net-next 1/2] hv_sock: introduce Hyper-V Sockets

2016-05-07 Thread Dexuan Cui
> From: David Miller [mailto:da...@davemloft.net] > Sent: Sunday, May 8, 2016 1:41 > To: Dexuan Cui > Cc: gre...@linuxfoundation.org; net...@vger.kernel.org; linux- > ker...@vger.kernel.org; de...@linuxdriverproject.org; o...@aepfle.de; > a...@canonical.com; jasow...@redhat.com;

RE: [PATCH v9 net-next 1/2] hv_sock: introduce Hyper-V Sockets

2016-05-09 Thread Dexuan Cui
> From: David Miller [mailto:da...@davemloft.net] > Sent: Monday, May 9, 2016 1:45 > To: Dexuan Cui > Cc: gre...@linuxfoundation.org; net...@vger.kernel.org; linux- > ker...@vger.kernel.org; de...@linuxdriverproject.org; o...@aepfle.de; > a...@canonical.com; jasow...@redhat.com;

[PATCH v10 net-next 0/1] introduce Hyper-V VM Sockets(hv_sock)

2016-05-11 Thread Dexuan Cui
. the buffers are allocated by kmalloc() in __hvsock_create() now. - minimized the sizes of the send/recv buffers and the vmbus ringbuffers. Dexuan Cui (1): hv_sock: introduce Hyper-V Sockets MAINTAINERS |2 + include/linux/hyperv.h | 14 + include/linu

[PATCH v10 net-next 1/1] hv_sock: introduce Hyper-V Sockets

2016-05-11 Thread Dexuan Cui
mt_service The patch implements the necessary support in the guest side by introducing a new socket address family AF_HYPERV. Signed-off-by: Dexuan Cui Cc: "K. Y. Srinivasan" Cc: Haiyang Zhang Cc: Vitaly Kuznetsov Cc: Cathy Avery --- You can also get the patch on this branch: http

[PATCH v11 net-next 1/1] hv_sock: introduce Hyper-V Sockets

2016-05-15 Thread Dexuan Cui
mt_service The patch implements the necessary support in the guest side by introducing a new socket address family AF_HYPERV. Signed-off-by: Dexuan Cui Cc: "K. Y. Srinivasan" Cc: Haiyang Zhang Cc: Vitaly Kuznetsov Cc: Cathy Avery --- You can also get the patch on this branch: http

[PATCH v11 net-next 0/1] introduce Hyper-V VM Sockets(hv_sock)

2016-05-15 Thread Dexuan Cui
s_open() fails, hvsock_close_connection() can do nothing and we count on vmbus_onoffer_rescind() -> vmbus_device_unregister() to clean up the device. 6) some stylistic modificiation. Dexuan Cui (1): hv_sock: introduce Hyper-V Sockets MAINTAINERS |2 + include/linux/hyperv.h

RE: [PATCH v11 net-next 0/1] introduce Hyper-V VM Sockets(hv_sock)

2016-05-16 Thread Dexuan Cui
> From: David Miller [mailto:da...@davemloft.net] > Sent: Monday, May 16, 2016 1:16 > To: Dexuan Cui > Cc: gre...@linuxfoundation.org; net...@vger.kernel.org; linux- > ker...@vger.kernel.org; de...@linuxdriverproject.org; o...@aepfle.de; > a...@canonical.com; jasow.

[PATCH] Drivers: hv: vmbus: fix the race when querying & updating the percpu list

2016-05-16 Thread Dexuan Cui
ported-by: Rolf Neugebauer Signed-off-by: Dexuan Cui --- drivers/hv/channel.c | 3 +++ drivers/hv/channel_mgmt.c | 20 +--- include/linux/hyperv.h| 3 +++ 3 files changed, 15 insertions(+), 11 deletions(-) diff --git a/drivers/hv/channel.c b/drivers/hv/channel.c index 5

RE: [PATCH] Drivers: hv: vmbus: fix the race when querying & updating the percpu list

2016-05-17 Thread Dexuan Cui
> From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] > Sent: Tuesday, May 17, 2016 16:15 > 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; KY Sr

RE: [PATCH] Drivers: hv: vmbus: fix the race when querying & updating the percpu list

2016-05-17 Thread Dexuan Cui
> From: devel [mailto:driverdev-devel-boun...@linuxdriverproject.org] On Behalf > Of Dexuan Cui > > > > Do we need to do tasklet_schedule() to make sure there are no events > > pending? This is probably not a big issue as some other event will > > trigger scheduling b

[PATCH v2] Drivers: hv: vmbus: fix the race when querying & updating the percpu list

2016-05-17 Thread Dexuan Cui
ported-by: Rolf Neugebauer Cc: Vitaly Kuznetsov Signed-off-by: Dexuan Cui --- v2: added tasklet_schedule() after tasklet_enable(). Thanks, Vitaly! drivers/hv/channel.c | 5 + drivers/hv/channel_mgmt.c | 24 +--- include/linux/hyperv.h| 3 +++ 3 files changed

RE: [PATCH v11 net-next 0/1] introduce Hyper-V VM Sockets(hv_sock)

2016-05-18 Thread Dexuan Cui
> From: devel [mailto:driverdev-devel-boun...@linuxdriverproject.org] On Behalf > Of Dexuan Cui > Sent: Tuesday, May 17, 2016 10:46 > To: David Miller > Cc: o...@aepfle.de; gre...@linuxfoundation.org; jasow...@redhat.com; > linux-ker...@vger.kernel.org; j...@perches.com; net.

RE: [PATCH v11 net-next 0/1] introduce Hyper-V VM Sockets(hv_sock)

2016-05-18 Thread Dexuan Cui
> From: David Miller [mailto:da...@davemloft.net] > Sent: Thursday, May 19, 2016 12:13 > To: Dexuan Cui > Cc: KY Srinivasan ; o...@aepfle.de; > gre...@linuxfoundation.org; jasow...@redhat.com; linux- > ker...@vger.kernel.org; j...@perches.com; net...@vger.kernel.org; > a

RE: [PATCH v2] Drivers: hv: vmbus: fix the race when querying & updating the percpu list

2016-05-20 Thread Dexuan Cui
> From: devel [mailto:driverdev-devel-boun...@linuxdriverproject.org] On > Behalf Of Dexuan Cui > Sent: Wednesday, May 18, 2016 11:44 > To: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org; driverdev- > de...@linuxdriverproject.org; o...@aepfle.de; a...@canonical.com; > j

[PATCH v3] Drivers: hv: vmbus: fix the race when querying & updating the percpu list

2016-05-21 Thread Dexuan Cui
the per-cpu and the global lists. Reported-by: Rolf Neugebauer Cc: Vitaly Kuznetsov Signed-off-by: Dexuan Cui --- v2: added tasklet_schedule() after tasklet_enable(). Thanks, Vitaly! v3: I shouldn't have moved percpu_channel_deq() from hv_process_channel_removal() to vmbus_close_int

RE: [PATCH v3] Drivers: hv: vmbus: fix the race when querying & updating the percpu list

2016-05-31 Thread Dexuan Cui
> From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com] > Sent: Wednesday, June 1, 2016 0:27 > 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; KY Sr

RE: [PATCH v3] Drivers: hv: vmbus: fix the race when querying & updating the percpu list

2016-06-01 Thread Dexuan Cui
> >> [5.472143] BUG: unable to handle kernel paging request at > >> 00079fff5288 > >> [5.477107] IP: [] vmbus_onoffer+0x311/0x570 > >> [hv_vmbus] > >> ... > >> Vitaly > > > > I can't reproduce the panic somehow, but I did find a bug in > vmbus_process_offer(): > > > > "hv_event_taskle

[PATCH v4] Drivers: hv: vmbus: fix the race when querying & updating the percpu list

2016-06-01 Thread Dexuan Cui
the per-cpu and the global lists. Reported-by: Rolf Neugebauer Signed-off-by: Dexuan Cui --- v2: added tasklet_schedule() after tasklet_enable(). Thanks, Vitaly! v3: I shouldn't have moved percpu_channel_deq() from hv_process_channel_removal() to vmbus_close_internal(): the channel couldn&#

[PATCH V6 0/8] introduce Hyper-V VM Socket(hv_sock)

2016-01-26 Thread Dexuan Cui
ike.notify_recv_init .notify_recv_pre_block .notify_recv_pre_dequeue .notify_recv_post_dequeue .notify_send_init .notify_send_pre_block .notify_send_pre_enqueue .notify_send_post_enqueue etc. So I think we'd better introduce a new address fami

[PATCH V6 1/8] Drivers: hv: vmbus: add a helper function to set a channel's pending send size

2016-01-26 Thread Dexuan Cui
This will be used by the coming net/hvsock driver. Signed-off-by: Dexuan Cui --- include/linux/hyperv.h | 6 ++ 1 file changed, 6 insertions(+) diff --git a/include/linux/hyperv.h b/include/linux/hyperv.h index 753dbad..e4867a7 100644 --- a/include/linux/hyperv.h +++ b/include/linux

[PATCH V6 2/8] Drivers: hv: vmbus: define the new offer type for Hyper-V socket (hvsock)

2016-01-26 Thread Dexuan Cui
A helper function is also added. Signed-off-by: Dexuan Cui --- include/linux/hyperv.h | 7 +++ 1 file changed, 7 insertions(+) diff --git a/include/linux/hyperv.h b/include/linux/hyperv.h index e4867a7..c0eddd7 100644 --- a/include/linux/hyperv.h +++ b/include/linux/hyperv.h @@ -235,6

[PATCH V6 3/8] Drivers: hv: vmbus: vmbus_sendpacket_ctl: hvsock: avoid unnecessary signaling

2016-01-26 Thread Dexuan Cui
When the hvsock channel's outbound ringbuffer is full (i.e., hv_ringbuffer_write() returns -EAGAIN), we should avoid the unnecessary signaling the host. Signed-off-by: Dexuan Cui --- drivers/hv/channel.c | 6 +- 1 file changed, 5 insertions(+), 1 deletion(-) diff --git a/drive

[PATCH V6 5/8] Drivers: hv: vmbus: add a hvsock flag in struct hv_driver

2016-01-26 Thread Dexuan Cui
Only the coming hv_sock driver has a "true" value for this flag. We treat the hvsock offers/channels as special VMBus devices. Since the hv_sock driver handles all the hvsock offers/channels, we need to tweak vmbus_match() for hv_sock driver, so we introduce this flag. Signed-off-by:

[PATCH V6 4/8] Drivers: hv: vmbus: define a new VMBus message type for hvsock

2016-01-26 Thread Dexuan Cui
A function to send the type of message is also added. The coming net/hvsock driver will use this function to proactively request the host to offer a VMBus channel for a new hvsock connection. Signed-off-by: Dexuan Cui --- drivers/hv/channel.c | 15 +++ drivers/hv

[PATCH V6 7/8] Drivers: hv: vmbus: add an API vmbus_hvsock_device_unregister()

2016-01-26 Thread Dexuan Cui
The hvsock driver needs this API to release all the resources related to the channel. Signed-off-by: Dexuan Cui --- drivers/hv/channel_mgmt.c | 33 - drivers/hv/connection.c | 4 ++-- include/linux/hyperv.h| 2 ++ 3 files changed, 32 insertions(+), 7

[PATCH V6 6/8] Drivers: hv: vmbus: add a per-channel rescind callback

2016-01-26 Thread Dexuan Cui
This will be used by the coming hv_sock driver. Signed-off-by: Dexuan Cui --- drivers/hv/channel_mgmt.c | 11 +++ include/linux/hyperv.h| 9 + 2 files changed, 20 insertions(+) diff --git a/drivers/hv/channel_mgmt.c b/drivers/hv/channel_mgmt.c index 4611b50..b782944 100644

[PATCH V6 8/8] hvsock: introduce Hyper-V Socket feature

2016-01-26 Thread Dexuan Cui
ectly by the traditional BSD-style socket APIs. Hyper-V Socket is only available on new Windows hosts. The patch implements the necessary support in the guest side by introducing a new socket address family AF_HYPERV. Signed-off-by: Dexuan Cui Cc: Vitaly Kuznetsov --- MAINTA

RE: [PATCH V6 8/8] hvsock: introduce Hyper-V Socket feature

2016-01-26 Thread Dexuan Cui
> From: Olaf Hering [mailto:o...@aepfle.de] > Sent: Tuesday, January 26, 2016 18:24 > To: Dexuan Cui > Cc: gre...@linuxfoundation.org; da...@davemloft.net; > step...@networkplumber.org; net...@vger.kernel.org; linux- > ker...@vger.kernel.org; driverdev-devel@linuxdriv

RE: [PATCH V2 06/13] Drivers: hv: vmbus: define the new offer type for Hyper-V socket (hvsock)

2016-02-09 Thread Dexuan Cui
> From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Monday, February 8, 2016 13:34 > To: KY Srinivasan > Cc: linux-ker...@vger.kernel.org; de...@linuxdriverproject.org; > o...@aepfle.de; > a...@canonical.com; vkuzn...@redhat.com; jasow...@redhat.com; Dexuan > Cui

RE: [PATCH V6 0/8] introduce Hyper-V VM Socket(hv_sock)

2016-02-13 Thread Dexuan Cui
> From: devel [mailto:driverdev-devel-boun...@linuxdriverproject.org] On Behalf > Of Dexuan Cui > Sent: Tuesday, January 26, 2016 17:40 > ... > Dexuan Cui (8): > Drivers: hv: vmbus: add a helper function to set a channel's pending > send size > Drivers: hv: vmb

[PATCH] x86/mm: fix slow_virt_to_phys() for X86_PAE again

2016-02-25 Thread Dexuan Cui
d long" by mistake -- it should be "phys_addr_t". As a result, Hyper-V network driver in 32-PAE Linux guest can't work again. Fixes: "commmit 34437e67a672: x86/mm: Fix slow_virt_to_phys() to handle large PAT bit" Signed-off-by: Dexuan Cui Cc: Toshi Kani Cc: Andrew Morton

[PATCH 0/7] introduce Hyper-V VM Sockets(hvsock)

2015-07-06 Thread Dexuan Cui
Looking forward to your comments! Dexuan Cui (7): Drivers: hv: vmbus: define the new offer type for Hyper-V socket (hvsock) Drivers: hv: vmbus: define a new VMBus message type for hvsock Drivers: hv: vmbus: add APIs to send/recv hvsock packet and get the r/w-ability Drivers: hv: vm

[PATCH 1/7] Drivers: hv: vmbus: define the new offer type for Hyper-V socket (hvsock)

2015-07-06 Thread Dexuan Cui
A helper function is also added. Signed-off-by: Dexuan Cui --- include/linux/hyperv.h | 7 +++ 1 file changed, 7 insertions(+) diff --git a/include/linux/hyperv.h b/include/linux/hyperv.h index 30d3a1f..aa21814 100644 --- a/include/linux/hyperv.h +++ b/include/linux/hyperv.h @@ -236,6

[PATCH 2/7] Drivers: hv: vmbus: define a new VMBus message type for hvsock

2015-07-06 Thread Dexuan Cui
A function to send the type of message is also added. The coming net/hvsock driver will use this function to proactively request the host to offer a VMBus channel for a new hvsock connection. Signed-off-by: Dexuan Cui --- drivers/hv/channel.c | 15 +++ drivers/hv

[PATCH 3/7] Drivers: hv: vmbus: add APIs to send/recv hvsock packet and get the r/w-ability

2015-07-06 Thread Dexuan Cui
This will be used by the coming net/hvsock driver. Signed-off-by: Dexuan Cui --- drivers/hv/channel.c | 131 ++ drivers/hv/hyperv_vmbus.h | 4 ++ drivers/hv/ring_buffer.c | 14 + include/linux/hyperv.h| 33 4 files

[PATCH 4/7] Drivers: hv: vmbus: add APIs to register callbacks to process hvsock connection

2015-07-06 Thread Dexuan Cui
With the 2 APIs supplied by the VMBus driver, the coming net/hvsock driver can register 2 callbacks and can know when a new hvsock connection is offered by the host, and when a hvsock connection is being closed by the host. Signed-off-by: Dexuan Cui --- drivers/hv/Makefile | 4

[PATCH 5/7] Drivers: hv: vmbus: add a helper function to set a channel's pending send size

2015-07-06 Thread Dexuan Cui
This will be used by the coming net/hvsock driver. Signed-off-by: Dexuan Cui --- include/linux/hyperv.h | 6 ++ 1 file changed, 6 insertions(+) diff --git a/include/linux/hyperv.h b/include/linux/hyperv.h index 307910b3..c52f4cb 100644 --- a/include/linux/hyperv.h +++ b/include/linux

[PATCH 6/7] hvsock: introduce Hyper-V VM Sockets feature

2015-07-06 Thread Dexuan Cui
other directly by the traditional BSD-style socket APIs. Hyper-V VM Sockets is only available on Windows 10 host and later. The patch implements the necessary support in the guest side by introducing a new socket address family AF_HYPERV. Signed-off-by: Dexuan Cui --- MAINTAINERS |

[PATCH 7/7] Drivers: hv: vmbus: disable local interrupt when hvsock's callback is running

2015-07-06 Thread Dexuan Cui
_cb() to A, trying to set channel->onchannel_callbackto NULL; on A, if the IPI handler happens between "if (channel->onchannel_callback != NULL)" and invoking channel->onchannel_callback, we'll invoke a function pointer of NULL. This is why the patch is necessary. Signed-off-b

[PATCH 0/7] introduce Hyper-V VM Sockets(hvsock)

2015-07-06 Thread Dexuan Cui
Looking forward to your comments! Dexuan Cui (7): Drivers: hv: vmbus: define the new offer type for Hyper-V socket (hvsock) Drivers: hv: vmbus: define a new VMBus message type for hvsock Drivers: hv: vmbus: add APIs to send/recv hvsock packet and get the r/w-ability Drivers: hv: vm

[PATCH] Tools: hv: kvp: fix a build warning -Wformat-security

2015-07-06 Thread Dexuan Cui
It is to fix: hv_kvp_daemon.c:705:2: warning: format not a string literal and no format arguments [-Wformat-security] Signed-off-by: Dexuan Cui --- tools/hv/hv_kvp_daemon.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/hv/hv_kvp_daemon.c b/tools/hv/hv_kvp_daemon.c

RE: [PATCH 6/7] hvsock: introduce Hyper-V VM Sockets feature

2015-07-07 Thread Dexuan Cui
> -Original Message- > From: Paul Bolle > Sent: Tuesday, July 7, 2015 17:38 > To: Dexuan Cui > Subject: Re: [PATCH 6/7] hvsock: introduce Hyper-V VM Sockets feature > > Just two nits. > > On ma, 2015-07-06 at 07:47 -0700, Dexuan Cui wrote: > > --- /dev/n

RE: [PATCH 6/7] hvsock: introduce Hyper-V VM Sockets feature

2015-07-07 Thread Dexuan Cui
> -Original Message- > From: Olaf Hering [mailto:o...@aepfle.de] > Sent: Tuesday, July 7, 2015 18:10 > To: Dexuan Cui; Paul Bolle > Cc: gre...@linuxfoundation.org; da...@davemloft.net; > net...@vger.kernel.org; linux-ker...@vger.kernel.org; driverdev- > de...@linu

RE: [PATCH 6/7] hvsock: introduce Hyper-V VM Sockets feature

2015-07-07 Thread Dexuan Cui
> -Original Message- > From: Stephen Hemminger > Sent: Wednesday, July 8, 2015 2:31 > Subject: Re: [PATCH 6/7] hvsock: introduce Hyper-V VM Sockets feature > > On Mon, 6 Jul 2015 07:47:29 -0700 > Dexuan Cui wrote: > > > Hyper-V VM sockets (hvsock

RE: [PATCH 6/7] hvsock: introduce Hyper-V VM Sockets feature

2015-07-07 Thread Dexuan Cui
> -Original Message- > From: Olaf Hering > Sent: Tuesday, July 7, 2015 18:59 > Subject: Re: [PATCH 6/7] hvsock: introduce Hyper-V VM Sockets feature > > On Tue, Jul 07, Dexuan Cui wrote: > > > OK, removing the line seems better than 'default n'

[V2 0/7] introduce Hyper-V VM Sockets(hvsock)

2015-07-14 Thread Dexuan Cui
fy_recv_init .notify_recv_pre_block .notify_recv_pre_dequeue .notify_recv_post_dequeue .notify_send_init .notify_send_pre_block .notify_send_pre_enqueue .notify_send_post_enqueue etc. So I think we'd better introduce a new address family: AF_HYPE

[V2 2/7] Drivers: hv: vmbus: define a new VMBus message type for hvsock

2015-07-14 Thread Dexuan Cui
A function to send the type of message is also added. The coming net/hvsock driver will use this function to proactively request the host to offer a VMBus channel for a new hvsock connection. Signed-off-by: Dexuan Cui --- drivers/hv/channel.c | 15 +++ drivers/hv

[V2 1/7] Drivers: hv: vmbus: define the new offer type for Hyper-V socket (hvsock)

2015-07-14 Thread Dexuan Cui
A helper function is also added. Signed-off-by: Dexuan Cui --- include/linux/hyperv.h | 7 +++ 1 file changed, 7 insertions(+) diff --git a/include/linux/hyperv.h b/include/linux/hyperv.h index 30d3a1f..aa21814 100644 --- a/include/linux/hyperv.h +++ b/include/linux/hyperv.h @@ -236,6

[V2 3/7] Drivers: hv: vmbus: add APIs to send/recv hvsock packet and get the r/w-ability

2015-07-14 Thread Dexuan Cui
This will be used by the coming net/hvsock driver. Signed-off-by: Dexuan Cui --- drivers/hv/channel.c | 131 ++ drivers/hv/hyperv_vmbus.h | 4 ++ drivers/hv/ring_buffer.c | 14 + include/linux/hyperv.h| 33 4 files

[V2 5/7] Drivers: hv: vmbus: add a helper function to set a channel's pending send size

2015-07-14 Thread Dexuan Cui
This will be used by the coming net/hvsock driver. Signed-off-by: Dexuan Cui --- include/linux/hyperv.h | 6 ++ 1 file changed, 6 insertions(+) diff --git a/include/linux/hyperv.h b/include/linux/hyperv.h index 307910b3..c52f4cb 100644 --- a/include/linux/hyperv.h +++ b/include/linux

[V2 4/7] Drivers: hv: vmbus: add APIs to register callbacks to process hvsock connection

2015-07-14 Thread Dexuan Cui
With the 2 APIs supplied by the VMBus driver, the coming net/hvsock driver can register 2 callbacks and can know when a new hvsock connection is offered by the host, and when a hvsock connection is being closed by the host. Signed-off-by: Dexuan Cui --- drivers/hv/Makefile | 4

[V2 6/7] hvsock: introduce Hyper-V VM Sockets feature

2015-07-14 Thread Dexuan Cui
other directly by the traditional BSD-style socket APIs. Hyper-V VM Sockets is only available on Windows 10 host and later. The patch implements the necessary support in the guest side by introducing a new socket address family AF_HYPERV. Signed-off-by: Dexuan Cui --- Changes since v1: -

[V2 7/7] Drivers: hv: vmbus: disable local interrupt when hvsock's callback is running

2015-07-14 Thread Dexuan Cui
_cb() to A, trying to set channel->onchannel_callbackto NULL; on A, if the IPI handler happens between "if (channel->onchannel_callback != NULL)" and invoking channel->onchannel_callback, we'll invoke a function pointer of NULL. This is why the patch is necessary. Signed-off-b

RE: [PATCH] Drivers: hv: vmbus: prevent new subchannel creation on device shutdown

2015-07-14 Thread Dexuan Cui
> -Original Message- > From: Vitaly Kuznetsov > Sent: Monday, July 13, 2015 20:19 > Subject: [PATCH] Drivers: hv: vmbus: prevent new subchannel creation on device > shutdown > > When a new subchannel offer from host comes during device shutdown (e.g. > when a netvsc/storvsc module is unloa

RE: [V2 1/7] Drivers: hv: vmbus: define the new offer type for Hyper-V socket (hvsock)

2015-07-16 Thread Dexuan Cui
> From: David Miller > Sent: Thursday, July 16, 2015 12:13 > > From: Dexuan Cui > Date: Tue, 14 Jul 2015 02:58:03 -0700 > > > A helper function is also added. > > > > diff --git a/include/linux/hyperv.h b/include/linux/hyperv.h > > @@ -236,6 +236,7 @@

RE: [V2 3/7] Drivers: hv: vmbus: add APIs to send/recv hvsock packet and get the r/w-ability

2015-07-16 Thread Dexuan Cui
> -Original Message- > From: David Miller > Sent: Thursday, July 16, 2015 12:16 > > From: Dexuan Cui > Date: Tue, 14 Jul 2015 02:58:56 -0700 > > > +int vmbus_sendpacket_hvsock(struct vmbus_channel *channel, void *buf, > u32 len) > > +{ >

RE: [V2 6/7] hvsock: introduce Hyper-V VM Sockets feature

2015-07-16 Thread Dexuan Cui
> From: David Miller > Sent: Thursday, July 16, 2015 12:19 > > From: Dexuan Cui > Date: Tue, 14 Jul 2015 03:00:48 -0700 > > > + pr_debug("hvsock_sk_destruct: called\n"); > > Debug logging just to state that a function is called is not appropria

RE: [PATCH 0/7] introduce Hyper-V VM Sockets(hvsock)

2015-07-16 Thread Dexuan Cui
> -Original Message- > From: Stefan Hajnoczi > Sent: Thursday, July 16, 2015 23:59 > On Mon, Jul 06, 2015 at 07:39:35AM -0700, Dexuan Cui wrote: > > Hyper-V VM Sockets (hvsock) is a byte-stream based communication > mechanism > > between Windowsd 10 (or later)

RE: [PATCH net-next 1/1] hv_netvsc: Wait for sub-channels to be processed during probe

2015-07-17 Thread Dexuan Cui
> From: K. Y. Srinivasan > Sent: Friday, July 17, 2015 3:17 > Subject: [PATCH net-next 1/1] hv_netvsc: Wait for sub-channels to be processed > during probe > diff --git a/drivers/net/hyperv/hyperv_net.h b/drivers/net/hyperv/hyperv_net.h > ... > @@ -1116,6 +1127,9 @@ int rndis_filter_device_add(str

RE: [PATCH net-next 1/1] hv_netvsc: Wait for sub-channels to be processed during probe

2015-07-17 Thread Dexuan Cui
> From: K. Y. Srinivasan > Sent: Friday, July 17, 2015 3:17 > Subject: [PATCH net-next 1/1] hv_netvsc: Wait for sub-channels to be processed > during probe > > The current code returns from probe without waiting for the proper handling > of subchannels that may be requested. If the netvsc driver w

RE: [V2 6/7] hvsock: introduce Hyper-V VM Sockets feature

2015-07-19 Thread Dexuan Cui
> -Original Message- > From: Vitaly Kuznetsov > Sent: Friday, July 17, 2015 23:04 > To: Dexuan Cui > Dexuan Cui writes: > > >> From: David Miller > >> Sent: Thursday, July 16, 2015 12:19 > >> > >> From: Dexuan Cui > >> Date

RE: [PATCH net-next 1/1] hv_netvsc: Wait for sub-channels to be processed during probe

2015-07-20 Thread Dexuan Cui
> From: KY Srinivasan > Sent: Friday, July 17, 2015 23:33 > > From: Dexuan Cui > > Sent: Friday, July 17, 2015 3:01 AM > > > From: K. Y. Srinivasan > > > Sent: Friday, July 17, 2015 3:17 > > > Subject: [PATCH net-next 1/1] hv_netvsc: Wait for sub-ch

RE: [PATCH 3/5] Drivers: hv_vmbus: Fix signal to host condition

2015-07-20 Thread Dexuan Cui
> -Original Message- > From: deve On Behalf of K. Y. Srinivasan > Sent: Monday, July 20, 2015 11:37 > > From: Christopher Oo > > Fixes a bug where previously hv_ringbuffer_read would pass in the old > number of bytes available to read instead of the expected old read index > when calculat

RE: [PATCH 5/5] Drivers: hv: vmbus: Implement a clocksource based on the TSC page

2015-07-20 Thread Dexuan Cui
> From: devel [mailto:driverdev-devel-bounces...linuxdriverproject.org] On > Behalf > Of K. Y. Srinivasan > Sent: Monday, July 20, 2015 11:37 > > The current Hyper-V clock source is based on the per-partition reference > counter > and this counter is being accessed via s synthetic MSR - > HV_X64

[PATCH V3 0/7] introduce Hyper-V VM Sockets(hvsock)

2015-07-21 Thread Dexuan Cui
cv_pre_block .notify_recv_pre_dequeue .notify_recv_post_dequeue .notify_send_init .notify_send_pre_block .notify_send_pre_enqueue .notify_send_post_enqueue etc. So I think we'd better introduce a new address family: AF_HYPERV. Please review the patchs

[PATCH V3 3/7] Drivers: hv: vmbus: add APIs to send/recv hvsock packet and get the r/w-ability

2015-07-21 Thread Dexuan Cui
This will be used by the coming net/hvsock driver. Signed-off-by: Dexuan Cui --- drivers/hv/channel.c | 133 ++ drivers/hv/hyperv_vmbus.h | 4 ++ drivers/hv/ring_buffer.c | 14 + include/linux/hyperv.h| 32 +++ 4 files

[PATCH V3 5/7] Drivers: hv: vmbus: add a helper function to set a channel's pending send size

2015-07-21 Thread Dexuan Cui
This will be used by the coming net/hvsock driver. Signed-off-by: Dexuan Cui --- include/linux/hyperv.h | 6 ++ 1 file changed, 6 insertions(+) diff --git a/include/linux/hyperv.h b/include/linux/hyperv.h index fda9790..47c5c1a 100644 --- a/include/linux/hyperv.h +++ b/include/linux

[PATCH V3 4/7] Drivers: hv: vmbus: add APIs to register callbacks to process hvsock connection

2015-07-21 Thread Dexuan Cui
With the 2 APIs supplied by the VMBus driver, the coming net/hvsock driver can register 2 callbacks and can know when a new hvsock connection is offered by the host, and when a hvsock connection is being closed by the host. Signed-off-by: Dexuan Cui --- drivers/hv/Makefile | 4

[PATCH V3 2/7] Drivers: hv: vmbus: define a new VMBus message type for hvsock

2015-07-21 Thread Dexuan Cui
A function to send the type of message is also added. The coming net/hvsock driver will use this function to proactively request the host to offer a VMBus channel for a new hvsock connection. Signed-off-by: Dexuan Cui --- drivers/hv/channel.c | 15 +++ drivers/hv

[PATCH V3 1/7] Drivers: hv: vmbus: define the new offer type for Hyper-V socket (hvsock)

2015-07-21 Thread Dexuan Cui
A helper function is also added. Signed-off-by: Dexuan Cui --- include/linux/hyperv.h | 7 +++ 1 file changed, 7 insertions(+) diff --git a/include/linux/hyperv.h b/include/linux/hyperv.h index 30d3a1f..2ca3ac1 100644 --- a/include/linux/hyperv.h +++ b/include/linux/hyperv.h @@ -236,6

[PATCH V3 7/7] Drivers: hv: vmbus: disable local interrupt when hvsock's callback is running

2015-07-21 Thread Dexuan Cui
_cb() to A, trying to set channel->onchannel_callbackto NULL; on A, if the IPI handler happens between "if (channel->onchannel_callback != NULL)" and invoking channel->onchannel_callback, we'll invoke a function pointer of NULL. This is why the patch is necessary. Signed-off-b

[PATCH V3 6/7] hvsock: introduce Hyper-V VM Sockets feature

2015-07-21 Thread Dexuan Cui
other directly by the traditional BSD-style socket APIs. Hyper-V VM Sockets is only available on Windows 10 host and later. The patch implements the necessary support in the guest side by introducing a new socket address family AF_HYPERV. Signed-off-by: Dexuan Cui --- Changes since v1: -

RE: [PATCH V3 3/7] Drivers: hv: vmbus: add APIs to send/recv hvsock packet and get the r/w-ability

2015-07-22 Thread Dexuan Cui
> From: Vitaly Kuznetsov > Sent: Tuesday, July 21, 2015 22:07 > > diff --git a/drivers/hv/channel.c b/drivers/hv/channel.c >> ... > > +int vmbus_recvpacket_hvsock(struct vmbus_channel *channel, void *buffer, > > + u32 bufferlen, u32 *buffer_actual_len) > > +{ > > + struct vm

RE: [PATCH V3 3/7] Drivers: hv: vmbus: add APIs to send/recv hvsock packet and get the r/w-ability

2015-07-22 Thread Dexuan Cui
> From: Dan Carpenter > Sent: Wednesday, July 22, 2015 18:36 > To: Dexuan Cui > On Wed, Jul 22, 2015 at 10:09:10AM +0000, Dexuan Cui wrote: > > > I'd suggest you do something like > > > > > > if (ret == -EAGIAIN) > > > retur

RE: [PATCH V3 3/7] Drivers: hv: vmbus: add APIs to send/recv hvsock packet and get the r/w-ability

2015-07-23 Thread Dexuan Cui
> From: Dan Carpenter > Sent: Thursday, July 23, 2015 18:25 > On Thu, Jul 23, 2015 at 01:10:57PM +0300, Dan Carpenter wrote: > > In this specific case, writing it as "if (ret != 0)" caused the bug. If > > we had written it as "if (ret) return ret;" then there are no zeroes so > > wouldn't have bee

<    1   2   3   4   5   6   >