NVMe: Remove outdated comments
authorMatthew Wilcox <matthew.r.wilcox@intel.com>
Wed, 16 Mar 2011 20:29:24 +0000 (16:29 -0400)
committerMatthew Wilcox <matthew.r.wilcox@intel.com>
Fri, 4 Nov 2011 19:53:00 +0000 (15:53 -0400)
The head can never overrun the tail since we won't allocate enough command
IDs to let that happen.  The status codes are in sync with the spec.

Signed-off-by: Matthew Wilcox <matthew.r.wilcox@intel.com>
drivers/block/nvme.c
include/linux/nvme.h

index 740a9c1b81aa7744f1fd57a14263716c40804411..d4f95eb51dc13c6843c021c8bda2cd9217c3ff2e 100644 (file)
@@ -245,7 +245,6 @@ static int nvme_submit_cmd(struct nvme_queue *nvmeq, struct nvme_command *cmd)
 {
        unsigned long flags;
        u16 tail;
-       /* XXX: Need to check tail isn't going to overrun head */
        spin_lock_irqsave(&nvmeq->q_lock, flags);
        tail = nvmeq->sq_tail;
        memcpy(&nvmeq->sq_cmds[tail], cmd, sizeof(*cmd));
index c46a9b7988fb3bec0d44dbcbb159744da1d53fd4..6b5a8d19daf5a68127faa0bb7b3b7025df918685 100644 (file)
@@ -323,7 +323,6 @@ struct nvme_command {
        };
 };
 
-/* XXX: Sync with spec */
 enum {
        NVME_SC_SUCCESS                 = 0x0,
        NVME_SC_INVALID_OPCODE          = 0x1,