diff options
author | Matthew Wilcox <matthew.r.wilcox@intel.com> | 2012-01-10 16:35:08 -0500 |
---|---|---|
committer | Matthew Wilcox <matthew.r.wilcox@intel.com> | 2012-01-10 16:29:23 -0500 |
commit | 4eeb9215a0d5c9494ca8b20158cc8ee82618840c (patch) | |
tree | e78031c97d64d7545454f729f14e4885fa5f5f5c | |
parent | 1c2ad9faaf662b4a525348775deca3ac8e6c35a0 (diff) |
NVMe: Set queue flags correctly
QUEUE_FLAG_* are flags (other than QUEUE_FLAG_DEFAULT), so they cannot
be ORed together. Set the queue flags using queue_flag_set_unlocked().
Reported-by: Donald Wood <donald.e.wood@intel.com>
Signed-off-by: Matthew Wilcox <matthew.r.wilcox@intel.com>
-rw-r--r-- | drivers/block/nvme.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/drivers/block/nvme.c b/drivers/block/nvme.c index 3cf82c27a544..b583603fae5b 100644 --- a/drivers/block/nvme.c +++ b/drivers/block/nvme.c | |||
@@ -1315,8 +1315,10 @@ static struct nvme_ns *nvme_alloc_ns(struct nvme_dev *dev, int nsid, | |||
1315 | ns->queue = blk_alloc_queue(GFP_KERNEL); | 1315 | ns->queue = blk_alloc_queue(GFP_KERNEL); |
1316 | if (!ns->queue) | 1316 | if (!ns->queue) |
1317 | goto out_free_ns; | 1317 | goto out_free_ns; |
1318 | ns->queue->queue_flags = QUEUE_FLAG_DEFAULT | QUEUE_FLAG_NOMERGES | | 1318 | ns->queue->queue_flags = QUEUE_FLAG_DEFAULT; |
1319 | QUEUE_FLAG_NONROT | QUEUE_FLAG_DISCARD; | 1319 | queue_flag_set_unlocked(QUEUE_FLAG_NOMERGES, ns->queue); |
1320 | queue_flag_set_unlocked(QUEUE_FLAG_NONROT, ns->queue); | ||
1321 | /* queue_flag_set_unlocked(QUEUE_FLAG_DISCARD, ns->queue); */ | ||
1320 | blk_queue_make_request(ns->queue, nvme_make_request); | 1322 | blk_queue_make_request(ns->queue, nvme_make_request); |
1321 | ns->dev = dev; | 1323 | ns->dev = dev; |
1322 | ns->queue->queuedata = ns; | 1324 | ns->queue->queuedata = ns; |