[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v8 01/15] block: Add "file" output parameter to
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH v8 01/15] block: Add "file" output parameter to block status query functions |
Date: |
Tue, 26 Jan 2016 11:36:23 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Tue, 01/26 11:27, Fam Zheng wrote:
> On Mon, 01/25 14:04, Kevin Wolf wrote:
> > Am 25.01.2016 um 03:44 hat Fam Zheng geschrieben:
> > > diff --git a/block/vvfat.c b/block/vvfat.c
> > > index 2ea5a4a..b8d29e1 100644
> > > --- a/block/vvfat.c
> > > +++ b/block/vvfat.c
> > > @@ -2884,7 +2884,7 @@ static coroutine_fn int
> > > vvfat_co_write(BlockDriverState *bs, int64_t sector_num,
> > > }
> > >
> > > static int64_t coroutine_fn vvfat_co_get_block_status(BlockDriverState
> > > *bs,
> > > - int64_t sector_num, int nb_sectors, int* n)
> > > + int64_t sector_num, int nb_sectors, int *n, BlockDriverState **file)
> > > {
> > > BDRVVVFATState* s = bs->opaque;
> > > *n = s->sector_count - sector_num;
> >
> > This still returns NULL at the end of the series. Shouldn't it return bs
> > like other protocol drivers do?
>
> Yes, we need another patch for vvfat.
No, I now remember why vvfat didn't need a patch: it never sets the
BDRV_BLOCK_OFFSET_VALID bit.
Fam
- [Qemu-devel] [PATCH v8 00/15] qemu-img map: Allow driver to return file of the allocated block, Fam Zheng, 2016/01/24
- [Qemu-devel] [PATCH v8 03/15] qcow2: Assign bs->file->bs to file in qcow2_co_get_block_status, Fam Zheng, 2016/01/24
- [Qemu-devel] [PATCH v8 04/15] raw: Assign bs to file in raw_co_get_block_status, Fam Zheng, 2016/01/24
- [Qemu-devel] [PATCH v8 05/15] iscsi: Assign bs to file in iscsi_co_get_block_status, Fam Zheng, 2016/01/24
- [Qemu-devel] [PATCH v8 06/15] parallels: Assign bs->file->bs to file in parallels_co_get_block_status, Fam Zheng, 2016/01/24
- [Qemu-devel] [PATCH v8 07/15] qed: Assign bs->file->bs to file in bdrv_qed_co_get_block_status, Fam Zheng, 2016/01/24
- [Qemu-devel] [PATCH v8 08/15] sheepdog: Assign bs to file in sd_co_get_block_status, Fam Zheng, 2016/01/24
- [Qemu-devel] [PATCH v8 09/15] vdi: Assign bs->file->bs to file in vdi_co_get_block_status, Fam Zheng, 2016/01/24
- [Qemu-devel] [PATCH v8 10/15] vpc: Assign bs->file->bs to file in vpc_co_get_block_status, Fam Zheng, 2016/01/24