[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 2/2] ISCSI: Force scsi-generic for MMC with blan
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 2/2] ISCSI: Force scsi-generic for MMC with blank disks |
Date: |
Sun, 19 Aug 2012 00:20:44 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:14.0) Gecko/20120717 Thunderbird/14.0 |
Il 19/08/2012 00:02, ronnie sahlberg ha scritto:
> On Sun, Aug 19, 2012 at 7:58 AM, Paolo Bonzini <address@hidden> wrote:
>> Il 17/08/2012 04:36, Ronnie Sahlberg ha scritto:
>>> There is no bdrv_* API for the commands for burning a blank MMC disk
>>> so when iSCSI LUNs are specified and the LUN is a MMC device with
>>> 0 available blocks. This is a blank disk so force scsi generic.
>>>
>>> This allows the guest to talk directly to the target to burn data on
>>> the disk.
>>>
>>> Signed-off-by: Ronnie Sahlberg <address@hidden>
>>
>> What happens without the patch? It's ok that scsi-{hd,cd} does not
>> work, but do scsi-{block,generic} work without the patch?
>>
>
> Neither of them work, basically because in
> block.c:find_image_format()
>
> if bs->sg is not set in
>
> if (bs->sg || !bdrv_is_inserted(bs)) {
>
> then we continue to
>
> ret = bdrv_pread(bs, 0, buf, sizeof(buf));
>
> which fails with an error.
> So this patch is basically to prevent find_image_format() from trying
> to read from a blank disk.
>
> Maybe there is a better way to do this?
Ah, BTW does using -drive ...,format=raw work, or does it hiccup later
again on something else?
Paolo
- Re: [Qemu-devel] [PATCH 1/2] ISCSI: Set number of blocks to 0 for blank CDROM devices, (continued)
- [Qemu-devel] [PATCH 2/2] ISCSI: Force scsi-generic for MMC with blank disks, Ronnie Sahlberg, 2012/08/16
- Re: [Qemu-devel] [PATCH 2/2] ISCSI: Force scsi-generic for MMC with blank disks, Blue Swirl, 2012/08/18
- Re: [Qemu-devel] [PATCH 2/2] ISCSI: Force scsi-generic for MMC with blank disks, Paolo Bonzini, 2012/08/18
- Re: [Qemu-devel] [PATCH 2/2] ISCSI: Force scsi-generic for MMC with blank disks, ronnie sahlberg, 2012/08/18
- Re: [Qemu-devel] [PATCH 2/2] ISCSI: Force scsi-generic for MMC with blank disks, ronnie sahlberg, 2012/08/18
- Re: [Qemu-devel] [PATCH 2/2] ISCSI: Force scsi-generic for MMC with blank disks, Paolo Bonzini, 2012/08/18
- Re: [Qemu-devel] [PATCH 2/2] ISCSI: Force scsi-generic for MMC with blank disks, ronnie sahlberg, 2012/08/18
- Re: [Qemu-devel] [PATCH 2/2] ISCSI: Force scsi-generic for MMC with blank disks, Paolo Bonzini, 2012/08/18
- Re: [Qemu-devel] [PATCH 2/2] ISCSI: Force scsi-generic for MMC with blank disks,
Paolo Bonzini <=
- Re: [Qemu-devel] [PATCH 2/2] ISCSI: Force scsi-generic for MMC with blank disks, ronnie sahlberg, 2012/08/18
- Re: [Qemu-devel] [PATCH 2/2] ISCSI: Force scsi-generic for MMC with blank disks, Paolo Bonzini, 2012/08/19