[Qemu-devel] Re: [PATCH 2/2] Add serial number support for virtio_blk, V4

[john cooper <john.cooper@redhat.com>]

Jens Axboe wrote:
> On Mon, Jun 01 2009, Rusty Russell wrote:
>> On Fri, 29 May 2009 01:45:27 pm john cooper wrote:
>>> virtio_blk-serial-4.patch
>> Hate to ask dumb questions, but is there a scsi equivalent of this?  It'd be 
>> nice if we could avoid being ATA-specific in the long run...
> 
> SCSI has mode pages, where ATA pretty much stuffs everything into the
> identify data.
> 
>> Also, why u16?
> 
> The identify page is word based, so u16 makes sense.

It is actually an artifact left over from the previous
version.  In that case the driver was only pulling
the serial number over, itself constructing a mocked-up
identify page, and needed to be aware of the internal
structure.

Currently qemu fabricates the identify page which the
driver treats as opaque data passed onto the user.  So
the u16 * structure has no meaning.

Sorry to be late in responding here.  A patch to address
this wrinkle and other feedback to follow shortly.

-john


>>> +/* return ATA identify data
>>> + */
>>> +static int virtblk_identify(struct gendisk *disk, void *argp)
>>> +{
>>> +	struct virtio_blk *vblk = disk->private_data;
>>> +	u16 *id;
>>> +	int err = -ENOMEM;
>>> +
>>> +	id = kmalloc(VIRTIO_BLK_ID_BYTES, GFP_KERNEL);
>>> +	if (!id)
>>> +		goto out;
>>> +
>>> +	err = virtio_config_buf(vblk->vdev, VIRTIO_BLK_F_IDENTIFY,
>>> +		offsetof(struct virtio_blk_config, identify), id,
>>> +		VIRTIO_BLK_ID_BYTES);
> 


-- 
john.cooper@redhat.com