diff options
author | Markus Armbruster <armbru@redhat.com> | 2011-01-28 11:21:40 +0100 |
---|---|---|
committer | Kevin Wolf <kwolf@redhat.com> | 2011-01-31 10:42:42 +0100 |
commit | ca7c8e98be596aa951648065b1a70aedb96d5da8 (patch) | |
tree | 607105473a590a78369b14863e80dabcea6b9533 /blockdev.h | |
parent | 4eaa76258108624b8da147a69e76183b65643aa2 (diff) | |
download | qemu-ca7c8e98be596aa951648065b1a70aedb96d5da8.tar.gz qemu-ca7c8e98be596aa951648065b1a70aedb96d5da8.tar.bz2 qemu-ca7c8e98be596aa951648065b1a70aedb96d5da8.zip |
blockdev: Fix regression in -drive if=scsi,index=N
Before commit 622b520f, index=12 meant bus=1,unit=5.
Since the commit, it means bus=0,unit=12. The drive is created, but
not the guest device. That's because the controllers we use with
if=scsi drives (lsi53c895a and esp) support only 7 units, and
scsi_bus_legacy_handle_cmdline() ignores drives with unit numbers
exceeding that limit.
Changing the mapping of index to bus, unit is a regression. Breaking
-drive invocations that used to work just makes it worse.
Revert the part of commit 622b520f that causes this, and clean up
some.
Note that the fix only affects if=scsi. You can still put more than 7
units on a SCSI bus with -device & friends.
Signed-off-by: Markus Armbruster <armbru@redhat.com>
Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Diffstat (limited to 'blockdev.h')
-rw-r--r-- | blockdev.h | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/blockdev.h b/blockdev.h index 2cbbb87dc7..cf8fc0112d 100644 --- a/blockdev.h +++ b/blockdev.h @@ -37,9 +37,6 @@ struct DriveInfo { QTAILQ_ENTRY(DriveInfo) next; }; -#define MAX_IDE_DEVS 2 -#define MAX_SCSI_DEVS 255 - DriveInfo *drive_get(BlockInterfaceType type, int bus, int unit); int drive_get_max_bus(BlockInterfaceType type); DriveInfo *drive_get_next(BlockInterfaceType type); |