diff options
| author | jimharris <jimharris@FreeBSD.org> | 2014-03-17 22:36:04 +0000 |
|---|---|---|
| committer | jimharris <jimharris@FreeBSD.org> | 2014-03-17 22:36:04 +0000 |
| commit | 797abe98034de6a3173ea662f06ebf1250a6d7b8 (patch) | |
| tree | 0b68830e86e3d7bedd621f45421d20a860052c9b | |
| parent | f97bf60ca069e62590e34a5575fed367eafe6526 (diff) | |
| download | FreeBSD-src-797abe98034de6a3173ea662f06ebf1250a6d7b8.zip FreeBSD-src-797abe98034de6a3173ea662f06ebf1250a6d7b8.tar.gz | |
nvme: Remove the software progress marker SET_FEATURE command during
controller initialization.
The spec says OS drivers should send this command after controller
initialization completes successfully, but other NVMe OS drivers are
not sending this command. This change will therefore reduce differences
between the FreeBSD and other OS drivers.
Sponsored by: Intel
MFC after: 3 days
| -rw-r--r-- | sys/dev/nvme/nvme_ctrlr.c | 10 |
1 files changed, 0 insertions, 10 deletions
diff --git a/sys/dev/nvme/nvme_ctrlr.c b/sys/dev/nvme/nvme_ctrlr.c index 8acd645..2afd527 100644 --- a/sys/dev/nvme/nvme_ctrlr.c +++ b/sys/dev/nvme/nvme_ctrlr.c @@ -842,16 +842,6 @@ nvme_ctrlr_start(void *ctrlr_arg) for (i = 0; i < ctrlr->num_io_queues; i++) nvme_io_qpair_enable(&ctrlr->ioq[i]); - - /* - * Clear software progress marker to 0, to indicate to pre-boot - * software that OS driver load was successful. - * - * Chatham does not support this feature. - */ - if (pci_get_devid(ctrlr->dev) != CHATHAM_PCI_ID) - nvme_ctrlr_cmd_set_feature(ctrlr, - NVME_FEAT_SOFTWARE_PROGRESS_MARKER, 0, NULL, 0, NULL, NULL); } void |
