|
From: | Paolo Bonzini |
Subject: | Re: [Qemu-devel] qemu - SCSI disk "Device Model", "Serial Number", and "Firmware Version"? |
Date: | Tue, 07 Jun 2011 09:35:34 +0200 |
User-agent: | Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.9.2.17) Gecko/20110428 Fedora/3.1.10-1.fc14 Lightning/1.0b3pre Mnenhy/0.8.3 Thunderbird/3.1.10 |
On 06/07/2011 09:04 AM, Dave Seddon wrote:
Here's an example of the nasty/QEMU output of smartmontools: ------------------------------------------------------- # smartctl -d sat --all /dev/sg0 smartctl version 5.38 [i686-spcdn-linux-gnu] Copyright (C) 2002-8 Bruce Allen Home page is http://smartmontools.sourceforge.net/ === START OF INFORMATION SECTION === Device Model: [No Information Found] Serial Number: [No Information Found] Firmware Version: �" Device is: Not in smartctl database [for details use: -P showall] ATA Version is: 1 ATA Standard is: Exact ATA specification draft version not indicated Local Time is: Tue Jun 7 16:57:08 2011 UTC SMART is only available in ATA Version 3 Revision 3 or greater. We will try to proceed in spite of this. SMART support is: Ambiguous - ATA IDENTIFY DEVICE words 82-83 don't show if SMART supported. Checking for SMART support by trying SMART ENABLE command. SMART ENABLE appeared to work! Continuing. SMART support is: Ambiguous - ATA IDENTIFY DEVICE words 85-87 don't show if SMART is enabled. A mandatory SMART command failed: exiting. To continue, add one or more '-T permissive' options -------------------------------------------------------
Here we should perhaps try to improve the ATA emulation.
Looking in the source, I can see that the "QEMU HARDDISK" for example is statically defined. Would this be difficult to make an option for the '-drive '? ------------------------------------------------------- address@hidden hw]# grep -R 'QEMU' scsi-disk.c QEMUIOVector qiov; QEMUBH *bh; memcpy(&outbuf[16], "QEMU CD-ROM ", 16); memcpy(&outbuf[16], "QEMU HARDDISK ", 16); memcpy(&outbuf[8], "QEMU ", 8); s->version = qemu_strdup(QEMU_VERSION); -------------------------------------------------------
It should not be hard. It probably would be better to do it for if=ide too, though.
[Prev in Thread] | Current Thread | [Next in Thread] |