Project

General

Profile

Bug #11228

nvme may queue more submissions than allowed

Added by Paul Winder 15 days ago. Updated 6 days ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
-
Start date:
Due date:
% Done:

100%

Estimated time:
Difficulty:
Bite-size
Tags:

Description

The NVMe specification 1.2b in section "4.1.2 Full Queue" says "The number of entries in a queue when full is one less than the queue size".

The driver uses a semaphore to prevent this, it is currently initialised to the size of the queue. It should be one less.


Related issues

Related to illumos gate - Feature #11202: Allow the number of NVMe submission and completion queues to be differentClosed

Actions

History

#1

Updated by Gergő Mihály Doma 14 days ago

  • Related to Feature #11202: Allow the number of NVMe submission and completion queues to be different added
#2

Updated by Electric Monk 6 days ago

  • Status changed from In Progress to Closed
  • % Done changed from 80 to 100

git commit 0999c1123c1ab769df080ccc5f1626d50663e7a8

commit  0999c1123c1ab769df080ccc5f1626d50663e7a8
Author: Paul Winder <Paul.Winder@wdc.com>
Date:   2019-06-20T14:02:46.000Z

    11202 Allow the number of NVMe submission and completion queues to be different
    11228 nvme may queue more submissions than allowed
    11229 nvme_get_logpage() can allocate a too small buffer to receive logpage data
    11230 Panic in nvme_fill_prp() because of miscalculation of the number of PRPs per page
    11231 nvme in polled mode ignores the command call back
    Reviewed by: Robert Mustacchi <rm@joyent.com>
    Reviewed by: Hans Rosenfeld <rosenfeld@grumpf.hope-2000.org>
    Reviewed by: Gergő Mihály Doma <domag02@gmail.com>
    Reviewed by: Youzhong Yang <youzhong@gmail.com>
    Approved by: Dan McDonald <danmcd@joyent.com>

Also available in: Atom PDF