From: "Denis V. Lunev" <den@openvz.org>
To: Eric Blake <eblake@redhat.com>,
Michael Roth <mdroth@linux.vnet.ibm.com>,
qemu-devel@nongnu.org
Cc: peter.maydell@linaro.org, Olga Krishtal <okrishtal@virtuozzo.com>
Subject: Re: [Qemu-devel] [PATCH 09/10] qga: added bus type and disk location path
Date: Mon, 20 Jul 2015 18:19:01 +0300 [thread overview]
Message-ID: <55AD1165.2000209@openvz.org> (raw)
In-Reply-To: <55AD0E7C.6000301@redhat.com>
[-- Attachment #1: Type: text/plain, Size: 1759 bytes --]
On 07/20/2015 06:06 PM, Eric Blake wrote:
> On 07/06/2015 10:40 PM, Michael Roth wrote:
>> From: Olga Krishtal <okrishtal@virtuozzo.com>
>>
>> According to Microsoft disk location path can be obtained via
>> IOCTL_SCSI_GET_ADDRESS. Unfortunately this ioctl can not be used for all
>> devices. There are certain bus types which could be obtained with this
>> API. Please, refer to the following link for more details
>> https://technet.microsoft.com/en-us/library/ee851589(v=ws.10).aspx
>>
>> Bus type could be obtained using IOCTL_STORAGE_QUERY_PROPERTY. Enum
>> STORAGE_BUS_TYPE describes all buses supported by OS.
>>
>> Windows defines more bus types than Linux. Thus some values have been added
>> to GuestDiskBusType.
>>
>> Signed-off-by: Olga Krishtal <okrishtal@virtuozzo.com>
>> Signed-off-by: Denis V. Lunev <den@openvz.org>
>> CC: Eric Blake <eblake@redhat.com>
>> CC: Michael Roth <mdroth@linux.vnet.ibm.com>
>> * fixed warning in CreateFile due to use of NULL instead of 0
>> Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>
>> ---
>> +++ b/qga/qapi-schema.json
>> @@ -703,12 +703,24 @@
>> # @uml: UML disks
>> # @sata: SATA disks
>> # @sd: SD cards
>> +# @unknown: Unknown bus type
>> +# @ieee1394: Win IEEE 1394 bus type
>> +# @ssa: Win SSA bus type
>> +# @fibre: Win fiber channel bus type
>> +# @raid: Win RAID bus type
>> +# @iscsi: Win iScsi bus type
>> +# @sas: Win serial-attaches SCSI bus type
>> +# @mmc: Win multimedia card (MMC) bus type
>> +# @virtual: Win virtual bus type
>> +# @file-backed virtual: Win file-backed bus type
>> #
>> # Since: 2.2
> It would be nice to have a followup patch (since it is doc-only, it
> could still make 2.4) that mentions that all these new enum members were
> added in 2.4.
>
sure!
[-- Attachment #2: Type: text/html, Size: 2788 bytes --]
next prev parent reply other threads:[~2015-07-20 15:19 UTC|newest]
Thread overview: 31+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-07-07 4:40 [Qemu-devel] [PULL 00/10] qemu-ga patches for 2.4.0 Michael Roth
2015-07-07 4:40 ` [Qemu-devel] [PATCH 01/10] qga/commands-posix: Fix bug in guest-fstrim Michael Roth
2015-07-07 4:40 ` [Qemu-devel] [PATCH 02/10] qga/qmp_guest_fstrim: Return per path fstrim result Michael Roth
2015-07-07 4:40 ` [Qemu-devel] [PATCH 03/10] Revert "guest agent: remove g_strcmp0 usage" Michael Roth
2015-07-07 4:40 ` [Qemu-devel] [PATCH 04/10] qga: add win32 library iphlpapi Michael Roth
2015-07-07 4:40 ` [Qemu-devel] [PATCH 05/10] qga: win32 qmp_guest_network_get_interfaces implementation Michael Roth
2015-07-07 4:40 ` [Qemu-devel] [PATCH 06/10] qga: fail early for invalid time Michael Roth
2015-07-07 4:40 ` [Qemu-devel] [PATCH 07/10] qga: added empty qmp_quest_get_fsinfo functionality Michael Roth
2015-07-07 4:40 ` [Qemu-devel] [PATCH 08/10] qga: added mountpoint and filesystem type for single volume Michael Roth
2015-07-07 4:40 ` [Qemu-devel] [PATCH 09/10] qga: added bus type and disk location path Michael Roth
2015-07-20 15:06 ` Eric Blake
2015-07-20 15:19 ` Denis V. Lunev [this message]
2015-07-07 4:40 ` [Qemu-devel] [PATCH 10/10] qga: added GuestPCIAddress information Michael Roth
2015-07-07 10:32 ` [Qemu-devel] [PULL 00/10] qemu-ga patches for 2.4.0 Peter Maydell
2015-07-07 13:06 ` Olga Krishtal
2015-07-07 13:15 ` Peter Maydell
2015-07-07 13:18 ` Peter Maydell
2015-07-07 13:25 ` Olga Krishtal
2015-07-07 13:30 ` Peter Maydell
2015-07-07 17:13 ` [Qemu-devel] [PATCH] qga: fix build for older versions of mingw Olga Krishtal
2015-07-07 18:06 ` Denis V. Lunev
2015-07-07 19:16 ` Peter Maydell
2015-07-07 20:02 ` Michael Roth
2015-07-07 20:04 ` Denis V. Lunev
2015-07-07 21:23 ` Michael Roth
2015-07-07 21:28 ` Peter Maydell
-- strict thread matches above, loose matches on Subject: below --
2015-06-30 10:25 [Qemu-devel] [PATCH v6 0/10] QGA: disk and volume info for Windows & guest exec Denis V. Lunev
2015-06-30 10:25 ` [Qemu-devel] [PATCH 09/10] qga: added bus type and disk location path Denis V. Lunev
2015-06-19 16:57 [Qemu-devel] [PATCH v5 0/10] QGA: disk and volume info for Windows & guest exec Denis V. Lunev
2015-06-19 16:57 ` [Qemu-devel] [PATCH 09/10] qga: added bus type and disk location path Denis V. Lunev
2015-06-19 17:10 ` Eric Blake
2015-06-19 18:02 ` Denis V. Lunev
2015-06-19 16:51 [Qemu-devel] [PATCH v4 0/10] QGA: disk and volume info for Windows & guest exec Denis V. Lunev
2015-06-19 16:51 ` [Qemu-devel] [PATCH 09/10] qga: added bus type and disk location path Denis V. Lunev
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=55AD1165.2000209@openvz.org \
--to=den@openvz.org \
--cc=eblake@redhat.com \
--cc=mdroth@linux.vnet.ibm.com \
--cc=okrishtal@virtuozzo.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.