[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 16/45] scsi-disk: Track tray locked state
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v2 16/45] scsi-disk: Track tray locked state |
Date: |
Thu, 04 Aug 2011 08:39:56 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/23.2 (gnu/linux) |
Hannes Reinecke <address@hidden> writes:
> On 08/03/2011 03:07 PM, Markus Armbruster wrote:
>> We already track it in BlockDriverState. Just like tray open/close
>> state, we should track it in the device models instead, because it's
>> device state.
>>
>> Signed-off-by: Markus Armbruster<address@hidden>
>> ---
>> hw/scsi-disk.c | 4 +++-
>> 1 files changed, 3 insertions(+), 1 deletions(-)
>>
>> diff --git a/hw/scsi-disk.c b/hw/scsi-disk.c
>> index db72b86..8ca69f2 100644
>> --- a/hw/scsi-disk.c
>> +++ b/hw/scsi-disk.c
>> @@ -73,6 +73,7 @@ struct SCSIDiskState
>> char *serial;
>> SCSISense sense;
>> bool tray_open;
>> + bool tray_locked;
>> };
>>
> Hmm. Shouldn't we use a more generic 'flags' here and have bits for
> the individual tray states?
> Feels like a waste to have individual values here.
On my system, struct SCSIDiskState is 248 bytes before my series (5
bytes of padding at the end), and 248 bytes after (3 bytes padding). We
use one per disk.
I dare say switching to flags won't make a noticable difference in
memory use ;)
The bool members are easy to read and hard to screw up.
Flags can be nicer when you manipulate several of them together.
[Qemu-devel] [PATCH v2 08/45] ide: Reject ATA commands specific to drive kinds, Markus Armbruster, 2011/08/03
[Qemu-devel] [PATCH v2 14/45] block: Drop tray status tracking, no longer used, Markus Armbruster, 2011/08/03
[Qemu-devel] [PATCH v2 02/45] block: Generalize change_cb() to BlockDevOps, Markus Armbruster, 2011/08/03
[Qemu-devel] [PATCH v2 19/45] block: Rename bdrv_set_locked() to bdrv_lock_medium(), Markus Armbruster, 2011/08/03
[Qemu-devel] [PATCH v2 12/45] scsi-disk: Track tray open/close state, Markus Armbruster, 2011/08/03
[Qemu-devel] [PATCH v2 06/45] ide: Fix ATA command READ to set ATAPI signature for CD-ROM, Markus Armbruster, 2011/08/03