Project

General

Profile

Actions

Bug #7313

closed

bump nvme admin command timeout to 1s

Added by Hans Rosenfeld almost 6 years ago. Updated almost 6 years ago.

Status:
Closed
Priority:
Normal
Category:
driver - device drivers
Start date:
2016-08-18
Due date:
% Done:

100%

Estimated time:
Difficulty:
Bite-size
Tags:
needs-triage
Gerrit CR:

Description

Youzhong Yang reported that some NVMe devices need longer than 100ms to complete some commands such as GET LOGPAGE.

The original 100ms timeout was chosen because it worked, it's not in the spec or anything. So we can just increase it now that we know it's too short. While at it we can make it tunable so no driver recompile is needed to increase it, should the need for that arise.

Actions #1

Updated by Electric Monk almost 6 years ago

  • Status changed from New to Closed
  • % Done changed from 0 to 100

git commit e8ba2a389f6ca6999ca72dabbe2871e894bf6b67

commit  e8ba2a389f6ca6999ca72dabbe2871e894bf6b67
Author: Hans Rosenfeld <hans.rosenfeld@nexenta.com>
Date:   2016-08-25T15:04:45.000Z

    7313 bump nvme admin command timeout to 1s
    Reviewed by: Yuri Pankov <yuri.pankov@nexenta.com>
    Reviewed by: Rick McNeal <rick.mcneal@nexenta.com>
    Reviewed by: Richard Lowe <richlowe@richlowe.net>
    Reviewed by: Garrett D'Amore <garrett@damore.org>
    Reviewed by: Dan McDonald <danmcd@omniti.com>
    Approved by: Robert Mustacchi <rm@joyent.com>

Actions

Also available in: Atom PDF