qemu-stable
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] qapi: better document NVMe blockdev @device parameter


From: Eric Blake
Subject: Re: [PATCH] qapi: better document NVMe blockdev @device parameter
Date: Fri, 6 Dec 2019 08:51:15 -0600
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.2.2

On 12/6/19 8:38 AM, Daniel P. Berrangé wrote:
Mention that this is a PCI device address & give the format it is
expected it. Also mention that it must be first unbound from any

maybe s/expected it/expecting/

host kernel driver.

Signed-off-by: Daniel P. Berrangé <address@hidden>
---
  qapi/block-core.json | 6 +++++-
  1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/qapi/block-core.json b/qapi/block-core.json
index 0cf68fea14..fcb52ec24f 100644
--- a/qapi/block-core.json
+++ b/qapi/block-core.json
@@ -2963,9 +2963,13 @@
  #
  # Driver specific block device options for the NVMe backend.
  #
-# @device:    controller address of the NVMe device.
+# @device:    PCI controller address of the NVMe device in
+#             format hhhh:bb:ss.f (host:bus:slot.function)
  # @namespace: namespace number of the device, starting from 1.
  #
+# Note that the PCI @device must have been unbound from any host
+# kernel driver before instructing QEMU to add the blockdev.
+#

Reviewed-by: Eric Blake <address@hidden>

Doesn't affect code, but similarly no reason to hurry this into 4.2. 5.0 and qemu-stable (cc'd) is good enough.

--
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3226
Virtualization:  qemu.org | libvirt.org




reply via email to

[Prev in Thread] Current Thread [Next in Thread]