lkml.org 
[lkml]   [2016]   [Nov]   [1]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH V2 07/14] Drivers: hv: vss: Operation timeouts should match host expectation
    Date
    From: Alex Ng <alexng@messages.microsoft.com>

    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 <alexng@microsoft.com>
    Signed-off-by: K. Y. Srinivasan <kys@microsoft.com>
    ---
    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
    \
     
     \ /
      Last update: 2016-11-01 19:15    [W:2.520 / U:0.492 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site