From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753982AbcKASOj (ORCPT ); Tue, 1 Nov 2016 14:14:39 -0400 Received: from p3plsmtps2ded01.prod.phx3.secureserver.net ([208.109.80.58]:60996 "EHLO p3plsmtps2ded01.prod.phx3.secureserver.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753859AbcKASOh (ORCPT ); Tue, 1 Nov 2016 14:14:37 -0400 x-originating-ip: 72.167.245.219 From: kys@exchange.microsoft.com To: gregkh@linuxfoundation.org, linux-kernel@vger.kernel.org, devel@linuxdriverproject.org, olaf@aepfle.de, apw@canonical.com, vkuznets@redhat.com, jasowang@redhat.com, leann.ogasawara@canonical.com, sthemmin@microsoft.com Cc: Alex Ng , Alex Ng , "K. Y. Srinivasan" Subject: [PATCH V2 07/14] Drivers: hv: vss: Operation timeouts should match host expectation Date: Tue, 1 Nov 2016 13:07:27 -0700 Message-Id: <1478030854-5970-7-git-send-email-kys@exchange.microsoft.com> X-Mailer: git-send-email 1.7.4.1 In-Reply-To: <1478030854-5970-1-git-send-email-kys@exchange.microsoft.com> References: <1478030807-5915-1-git-send-email-kys@exchange.microsoft.com> <1478030854-5970-1-git-send-email-kys@exchange.microsoft.com> Reply-To: kys@microsoft.com X-CMAE-Envelope: MS4wfHLJXns+hTFGQcTDWgmPX7OdanOfo+E2n53c9tWoSgEHJOAst6gyrj4i+vq5i7XaU5kNdxair9eDFQIL5+QhZzotsOiC/ByrV1Ttu60bLSKhryP91uop d52CvwkBSqk7WxCHDlYV9CoxwEfituR0vBTDkoSOz4Fr6/cZYd399MG4LDjCkAUVVHtzheYhqyOWYVONU6IgpVJOf6GQlGTdv7fj8wu0KKZBfMly676CIU44 q1kClT8xCG/wXjY9NbPtbuwgNgBN5Xrm3ImDAc8wxkRKg/Fh3xYez0V6chJxbcD9Y1/CDEkXWzcNEg7GTHyYWOw+PPglcCeH4ZCfSbYQQaVAAWFZZWggU7wu t+r4H1HKH2JbjlffQeFH5184XT95WzFEjUDLu80Cv67CYxEWj+yunWlqEVe7+YwQvSTLpa+0m7nE1rK63lasXAinrfmpDHIeStlEFTHFKJ2ifjkPk7AdHOvA 3rPOZb/mw44h7+ZtuswyjE9Sb4f+2ZrTTD+9Ot+cp3p9MIvy47hirxBIQdi38wdpcnkR/QOnfXCjV94LAuIGViG0C+/vKA0W5xlThg== Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Alex Ng Increase the timeout of backup operations. When system is under I/O load, it needs more time to freeze. These timeout values should also match the host timeout values more closely. Signed-off-by: Alex Ng Signed-off-by: K. Y. Srinivasan --- drivers/hv/hv_snapshot.c | 8 ++++++-- 1 files changed, 6 insertions(+), 2 deletions(-) diff --git a/drivers/hv/hv_snapshot.c b/drivers/hv/hv_snapshot.c index 5c95ba1..eee238c 100644 --- a/drivers/hv/hv_snapshot.c +++ b/drivers/hv/hv_snapshot.c @@ -31,7 +31,10 @@ #define VSS_MINOR 0 #define VSS_VERSION (VSS_MAJOR << 16 | VSS_MINOR) -#define VSS_USERSPACE_TIMEOUT (msecs_to_jiffies(10 * 1000)) +/* + * Timeout values are based on expecations from host + */ +#define VSS_FREEZE_TIMEOUT (15 * 60) /* * Global state maintained for transaction that is being processed. For a class @@ -186,7 +189,8 @@ static void vss_send_op(void) vss_transaction.state = HVUTIL_USERSPACE_REQ; - schedule_delayed_work(&vss_timeout_work, VSS_USERSPACE_TIMEOUT); + schedule_delayed_work(&vss_timeout_work, op == VSS_OP_FREEZE ? + VSS_FREEZE_TIMEOUT * HZ : HV_UTIL_TIMEOUT * HZ); rc = hvutil_transport_send(hvt, vss_msg, sizeof(*vss_msg), NULL); if (rc) { -- 1.7.4.1