On Fri, Mar 20, 2015 at 12:53 AM, KY Srinivasan k...@microsoft.com
wrote:
-Original Message-
From: Jason Wang [mailto:jasow...@redhat.com]
Sent: Tuesday, March 17, 2015 8:09 PM
To: KY Srinivasan
Cc: da...@davemloft.net; net...@vger.kernel.org; linux-
ker...@vger.kernel.org
On Wed, Mar 18, 2015 at 12:02 AM, K. Y. Srinivasan k...@microsoft.com
wrote:
Optimize notifying the host by deferring notification until there
are no more packets to be sent. This will help in batching the
requests
on the host.
Signed-off-by: K. Y. Srinivasan k...@microsoft.com
---
On Thu, Mar 12, 2015 at 3:04 AM, K. Y. Srinivasan k...@microsoft.com
wrote:
Optimize notifying the host by deferring notification until there
are no more packets to be sent. This will help in batching the
requests
on the host.
Signed-off-by: K. Y. Srinivasan k...@microsoft.com
---
packets
may have been placed on the ring buffer.
I would like to thank Jason Wang jasow...@redhat.com for pointing
out this issue.
Signed-off-by: K. Y. Srinivasan k...@microsoft.com
---
drivers/hv/channel.c | 32
1 files changed, 32 insertions(+), 0 deletions
ret;
}
+EXPORT_SYMBOL_GPL(vmbus_sendpacket_pagebuffer_ctl);
/*
* vmbus_sendpacket_pagebuffer - Send a range of single-page buffer
--
1.7.4.1
Acked-by: Jason Wang jasow...@redhat.com
___
devel mailing list
de...@linuxdriverproject.org
http
On Wed, Mar 11, 2015 at 2:50 AM, K. Y. Srinivasan k...@microsoft.com
wrote:
Optimize notifying the host by deferring notification until there
are no more packets to be sent. This will help in batching the
requests
on the host.
Signed-off-by: K. Y. Srinivasan k...@microsoft.com
---
!
This happens as we don't acquire ha_region_mutex and hot_add_req() expects us
to as it does unconditional mutex_unlock(). Acquire the lock on the error
path.
Signed-off-by: Vitaly Kuznetsov vkuzn...@redhat.com
Acked-by: Jason Wang jasow...@redhat.com
---
This patch is dependent on the previously
On Wed, Feb 4, 2015 at 1:00 AM, Vitaly Kuznetsov vkuzn...@redhat.com
wrote:
free_channel() function frees the channel unconditionally so we need
to make
sure nobody has any link to it. This is not trivial and there are
several
examples of races we have:
1) In vmbus_onoffer_rescind() we
On Tue, Feb 3, 2015 at 11:46 PM, Haiyang Zhang haiya...@microsoft.com
wrote:
-Original Message-
From: Jason Wang [mailto:jasow...@redhat.com]
Sent: Monday, February 2, 2015 1:49 AM
btw, I find during netvsc_start_xmit(), ret was change to
-ENOSPC
when
queue_sends[q_idx
(open_info);
+ newchannel-state = CHANNEL_OPEN_STATE;
return err;
}
EXPORT_SYMBOL_GPL(vmbus_open);
--
1.9.1
Reviewed-by: Jason Wang jasow...@redhat.com
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org
if
some kinds of hypercalls are sent too frequently.
Better to clarify this kind of throttling. Looks like it only affects
HVCALL_POST_MESSAGE. Other looks good.
Reviewed-by: Jason Wang jasow...@redhat.com
Without the patch, the driver can occasionally fail to load.
Also let's retry
On Mon, Feb 2, 2015 at 12:35 PM, Dexuan Cui de...@microsoft.com wrote:
Before the line vmbus_open() returns, srv-util_cb can be already
running
and the variables, like util_fw_version, are needed by the
srv-util_cb.
A questions is why we do this for util only? Can callbacks of other
On Tue, Feb 3, 2015 at 11:30 AM, KY Srinivasan k...@microsoft.com
wrote:
-Original Message-
From: Jason Wang [mailto:jasow...@redhat.com]
Sent: Monday, February 2, 2015 7:09 PM
To: Dexuan Cui
Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org;
driverdev-
de
On Mon, Feb 2, 2015 at 6:09 PM, Dexuan Cui de...@microsoft.com wrote:
-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
On Tue, Feb 3, 2015 at 11:40 AM, KY Srinivasan k...@microsoft.com
wrote:
-Original Message-
From: Jason Wang [mailto:jasow...@redhat.com]
Sent: Monday, February 2, 2015 7:38 PM
To: KY Srinivasan
Cc: Dexuan Cui; gre...@linuxfoundation.org;
linux-ker...@vger.kernel.org
;
error:
--
1.9.1
Reviewed-by: Jason Wang jasow...@redhat.com
___
devel mailing list
de...@linuxdriverproject.org
http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
On Fri, Jan 30, 2015 at 11:05 PM, Haiyang Zhang
haiya...@microsoft.com wrote:
-Original Message-
From: Jason Wang [mailto:jasow...@redhat.com]
Sent: Friday, January 30, 2015 5:25 AM
+ if (ret != 0) {
+ if (section_index != NETVSC_INVALID_INDEX
On Fri, Jan 30, 2015 at 4:34 AM, Haiyang Zhang haiya...@microsoft.com
wrote:
The existing code frees the skb in EAGAIN case, in which the skb will
be
retried from upper layer and used again.
Also, the existing code doesn't free send buffer slot in error case,
because
there is no completion
; Dexuan Cui; Jason
Wang;
Radim Krčmář; Dan Carpenter
Subject: [PATCH v3 3/3] Drivers: hv: vmbus: serialize Offer and
Rescind offer
Commit 4b2f9abea52a (staging: hv: convert channel_mgmt.c to not
call
osd_schedule_callback)' was written under an assumption that we
never
receive
Rescind
...@linuxdriverproject.org; Haiyang Zhang;
linux-
ker...@vger.kernel.org; Jason Wang; Radim Krčmář; Dan Carpenter
Subject: Re: [PATCH v3 3/3] Drivers: hv: vmbus: serialize Offer
and Rescind
offer
Dexuan Cui de...@microsoft.com writes:
-Original Message-
From: Vitaly Kuznetsov
...@vger.kernel.org; Jason Wang; Radim Krčmář; Dan Carpenter
Subject: Re: [PATCH v3 3/3] Drivers: hv: vmbus: serialize Offer and
Rescind
offer
Dexuan Cui de...@microsoft.com writes:
-Original Message-
From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com]
Sent: Tuesday, January 20
On Thu, Jan 29, 2015 at 7:02 PM, Dexuan Cui de...@microsoft.com wrote:
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.
'lock' and
add the
description.
Signed-off-by: Vitaly Kuznetsov vkuzn...@redhat.com
---
drivers/hv/channel.c | 6 +++---
drivers/hv/channel_mgmt.c | 10 +-
include/linux/hyperv.h| 7 ++-
3 files changed, 14 insertions(+), 9 deletions(-)
Acked-by: Jason Wang jasow
.
Reported-by: Jason Wang jasow...@redhat.com
Signed-off-by: Vitaly Kuznetsov vkuzn...@redhat.com
---
drivers/hv/channel_mgmt.c | 14 +-
1 file changed, 9 insertions(+), 5 deletions(-)
Acked-by: Jason Wang jasow...@redhat.com
diff --git a/drivers/hv/channel_mgmt.c b/drivers/hv
rescind offer during or before vmbus_process_offer()
is done
we set rescind flag to true and we check it at the end of
vmbus_process_offer()
so such offer will not get lost.
Suggested-by: Radim Krčmář rkrc...@redhat.com
Signed-off-by: Vitaly Kuznetsov vkuzn...@redhat.com
---
Acked-by: Jason
On Wed, Jan 14, 2015 at 9:43 AM, Dexuan Cui de...@microsoft.com wrote:
-Original Message-
From: Vitaly Kuznetsov [mailto:vkuzn...@redhat.com]
Sent: Tuesday, January 13, 2015 21:52 PM
To: Dexuan Cui; KY Srinivasan
Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org;
insertions(+), 0 deletions(-)
Acked-by: Jason Wang jasow...@redhat.com
diff --git a/arch/x86/kernel/cpu/mshyperv.c
b/arch/x86/kernel/cpu/mshyperv.c
index a450373..939155f 100644
--- a/arch/x86/kernel/cpu/mshyperv.c
+++ b/arch/x86/kernel/cpu/mshyperv.c
@@ -107,6 +107,7 @@ static struct clocksource
)
+ hv_process_timer_expiration(msg, cpu);
+ else
+ tasklet_schedule(msg_dpc);
+ }
}
/*
--
1.7.4.1
Reviewed-by: Jason Wang jasow...@redhat.com
___
devel mailing list
de...@linuxdriverproject.org
http
On Fri, Nov 28, 2014 at 7:54 PM, Dexuan Cui de...@microsoft.com wrote:
-Original Message-
From: Jason Wang [mailto:jasow...@redhat.com]
Sent: Friday, November 28, 2014 18:13 PM
To: Dexuan Cui
Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org;
driverdev-
de
On Mon, Dec 1, 2014 at 5:47 PM, Dexuan Cui de...@microsoft.com wrote:
-Original Message-
From: Jason Wang [mailto:jasow...@redhat.com]
Sent: Monday, December 1, 2014 16:23 PM
To: Dexuan Cui
Cc: gre...@linuxfoundation.org; linux-ker...@vger.kernel.org;
driverdev-
de
On Tue, Dec 2, 2014 at 1:33 PM, Dexuan Cui de...@microsoft.com wrote:
-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
On Thu, Nov 27, 2014 at 4:50 PM, Dexuan Cui de...@microsoft.com wrote:
-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
will be used immediately.
Cc: Jason Wang jasow...@redhat.com
Cc: Vitaly 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
- Original Message -
In the case the user-space daemon crashes, hangs or is killed, we
need to down the semaphore, otherwise, after the daemon starts next
time, the obsolete data in fcopy_transaction.message or
fcopy_transaction.fcopy_msg will be used immediately.
Reviewed-by:
On 11/24/2014 03:54 PM, Dexuan Cui wrote:
-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
On 11/25/2014 12:32 PM, Dexuan Cui wrote:
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
On 11/24/2014 02:08 PM, Dexuan Cui wrote:
-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
{
__u64 offset;
__u32 size;
__u8data[DATA_FRAGMENT];
-};
+} __attribute__((packed));
/*
* An implementation of HyperV key value pair (KVP) functionality for Linux.
Acked-by: Jason Wang jasow...@redhat.com
___
devel mailing
haiya...@microsoft.com
Signed-off-by: Jason Wang jasow...@redhat.com
---
drivers/hv/channel.c |6 +-
1 files changed, 1 insertions(+), 5 deletions(-)
diff --git a/drivers/hv/channel.c b/drivers/hv/channel.c
index 284cf66..531a593 100644
--- a/drivers/hv/channel.c
+++ b/drivers/hv/channel.c
On 03/05/2014 12:57 AM, Haiyang Zhang wrote:
-Original Message-
From: Jason Wang [mailto:jasow...@redhat.com]
Sent: Monday, March 3, 2014 10:10 PM
To: Haiyang Zhang; da...@davemloft.net; net...@vger.kernel.org
Cc: KY Srinivasan; o...@aepfle.de; linux-ker...@vger.kernel.org; driverdev
On 03/04/2014 07:54 AM, Haiyang Zhang wrote:
It moves the state setting for query into rndis_filter_receive_response().
All callbacks including query-complete and status-callback are synchronized
by channel-inbound_lock. This prevents pentential race between them.
This still looks racy to me.
lpj instead.
Cc: K. Y. Srinivasan k...@microsoft.com
Cc: Haiyang Zhang haiya...@microsoft.com
Cc: sta...@vger.kernel.org
Acked-by: K. Y. Srinivasan k...@microsoft.com
Signed-off-by: Jason Wang jasow...@redhat.com
---
arch/x86/kernel/cpu/mshyperv.c | 6 ++
1 file changed, 6 insertions(+)
diff
On 02/13/2014 11:04 PM, Haiyang Zhang wrote:
-Original Message-
From: Jason Wang [mailto:jasow...@redhat.com]
Sent: Wednesday, February 12, 2014 10:52 PM
To: Haiyang Zhang; da...@davemloft.net; net...@vger.kernel.org
Cc: KY Srinivasan; o...@aepfle.de; linux-ker...@vger.kernel.org
On 02/13/2014 08:54 AM, Haiyang Zhang wrote:
Without this patch, the cat /sys/class/net/ethN/operstate shows
unknown, and ethtool ethN shows Link detected: yes, when VM
boots up with or without vNIC connected.
This patch fixed the problem.
Signed-off-by: Haiyang Zhang haiya...@microsoft.com
On 01/26/2014 12:42 PM, Jason Wang wrote:
On 01/25/2014 05:20 AM, H. Peter Anvin wrote:
On 01/23/2014 10:02 PM, Jason Wang wrote:
This patch bypass the timer_irq_works() check for hyperv guest since:
- It was guaranteed to work.
- timer_irq_works() may fail sometime due to the lpj
On 02/11/2014 02:15 AM, Haiyang Zhang wrote:
Without this patch, the cat /sys/class/net/ethN/operstate shows
unknown, and ethtool ethN shows Link detected: yes, when VM
boots up with or without vNIC connected.
This patch fixed the problem.
Signed-off-by: Haiyang Zhang haiya...@microsoft.com
On 01/27/2014 04:35 PM, David Miller wrote:
From: Jason Wang jasow...@redhat.com
Date: Mon, 27 Jan 2014 15:30:54 +0800
Call netif_carrier_on() after register_device(). Otherwise it won't work
since
the device was still in NETREG_UNINITIALIZED state.
Fixes
On 01/27/2014 06:22 PM, Ben Hutchings wrote:
On Mon, 2014-01-27 at 17:40 +0800, Jason Wang wrote:
On 01/27/2014 04:35 PM, David Miller wrote:
From: Jason Wang jasow...@redhat.com
Date: Mon, 27 Jan 2014 15:30:54 +0800
Call netif_carrier_on() after register_device(). Otherwise it won't work
On 01/27/2014 06:30 PM, Ben Hutchings wrote:
On Mon, 2014-01-27 at 18:28 +0800, Jason Wang wrote:
On 01/27/2014 06:22 PM, Ben Hutchings wrote:
On Mon, 2014-01-27 at 17:40 +0800, Jason Wang wrote:
On 01/27/2014 04:35 PM, David Miller wrote:
From: Jason Wang jasow...@redhat.com
Date: Mon, 27
...@microsoft.com
Reported-by: Di Nie d...@redhat.com
Tested-by: Di Nie d...@redhat.com
Signed-off-by: Jason Wang jasow...@redhat.com
---
drivers/net/hyperv/netvsc_drv.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/net/hyperv/netvsc_drv.c b/drivers/net/hyperv
On 01/25/2014 05:20 AM, H. Peter Anvin wrote:
On 01/23/2014 10:02 PM, Jason Wang wrote:
This patch bypass the timer_irq_works() check for hyperv guest since:
- It was guaranteed to work.
- timer_irq_works() may fail sometime due to the lpj calibration were
inaccurate
in a hyperv guest
lpj instead.
Cc: K. Y. Srinivasan k...@microsoft.com
Cc: Haiyang Zhang haiya...@microsoft.com
Signed-off-by: Jason Wang jasow...@redhat.com
---
arch/x86/kernel/cpu/mshyperv.c | 6 ++
1 file changed, 6 insertions(+)
diff --git a/arch/x86/kernel/cpu/mshyperv.c b/arch/x86/kernel/cpu/mshyperv.c
by NETDEV_NOTIFY_PEERS was
synchronized with the netif_tx_disable() called by netvsc_change_mtu().
Reported-by: Yaju Cao ya...@redhat.com
Tested-by: Yaju Cao ya...@redhat.com
Cc: K. Y. Srinivasan k...@microsoft.com
Cc: Haiyang Zhang haiya...@microsoft.com
Signed-off-by: Jason Wang jasow...@redhat.com
---
The patch
On 07/25/2013 04:54 PM, Jason Wang wrote:
We try to handle the hypervisor compatibility mode by detecting hypervisor
through a specific order. This is not robust, since hypervisors may implement
each others features.
This patch tries to handle this situation by always choosing the last one
Cc: Frederic Weisbecker fweis...@gmail.com
Cc: linux-ker...@vger.kernel.org
Cc: de...@linuxdriverproject.org
Cc: k...@vger.kernel.org
Cc: xen-de...@lists.xensource.com
Cc: virtualizat...@lists.linux-foundation.org
Signed-off-by: Jason Wang jasow...@redhat.com
---
arch/x86/include/asm/hypervisor.h
55 matches
Mail list logo