RE: [PATCH 49/49] Staging: hv: vmbus: Increase the timeout value in vmbus_request_offers()
From: KY Srinivasan <kys@microsoft.com>
Date: 2011-06-07 21:39:58
Also in:
lkml
-----Original Message----- From: Greg KH [mailto:gregkh@suse.de] Sent: Tuesday, June 07, 2011 5:28 PM To: KY Srinivasan Cc: Greg KH; linux-kernel@vger.kernel.org; devel@linuxdriverproject.org; virtualization@lists.osdl.org; Haiyang Zhang; Abhishek Kane (Mindtree Consulting PVT LTD) Subject: Re: [PATCH 49/49] Staging: hv: vmbus: Increase the timeout value in vmbus_request_offers() On Tue, Jun 07, 2011 at 09:20:16PM +0000, KY Srinivasan wrote:quoted
quoted
-----Original Message----- From: Greg KH [mailto:greg@kroah.com] Sent: Tuesday, June 07, 2011 4:45 PM To: KY Srinivasan Cc: gregkh@suse.de; linux-kernel@vger.kernel.org; devel@linuxdriverproject.org; virtualization@lists.osdl.org; Haiyang Zhang; Abhishek Kane (Mindtree Consulting PVT LTD) Subject: Re: [PATCH 49/49] Staging: hv: vmbus: Increase the timeout value in vmbus_request_offers() On Mon, Jun 06, 2011 at 03:50:13PM -0700, K. Y. Srinivasan wrote:quoted
Increase the timeout value in vmbus_request_offers().Why? What does this solve/fix/prevent?On some very loaded systems, when we tested the 1 second timeout that we had prior to this patch was insufficient. So I bumped it to 5 seconds.Ok, then this should be something that goes into 3.0 and older kernels, right? If so, please say so, with this type of description, when you submit it. Remember, don't say only what you did in the changelog comment, but why you did it if it's not obvious (and it wasn't here.) Care to resend this one with this information?
Will do. Thanks, K. Y