Re: [PATCH v2 1/3] block: Add bdrv_co_get_lba_status

2020-06-29 Thread Stefan Hajnoczi
On Thu, Jun 25, 2020 at 01:07:14PM +, Lin Ma wrote: > On 2020-06-25 20:59, Lin Ma wrote: > > Please initialize is_deallocated to false at the beginning of the > > function to avoid accidental uninitialized variable accesses in the > > caller. > > It has already been initialized to 0 by 'data =

Re: [PATCH v2 1/3] block: Add bdrv_co_get_lba_status

2020-06-25 Thread Lin Ma
On 2020-06-25 20:59, Lin Ma wrote: > From: Stefan Hajnoczi > Sent: Monday, June 22, 2020 6:25 PM > ... >> diff --git a/block/io.c b/block/io.c >> index df8f2a98d4..2064016b19 100644 >> --- a/block/io.c >> +++ b/block/io.c >> @@ -2208,6 +2208,49 @@ int coroutine_fn bdrv_co_pwrite_zeroes(BdrvChild >

Re: [PATCH v2 1/3] block: Add bdrv_co_get_lba_status

2020-06-22 Thread Stefan Hajnoczi
On Wed, Jun 17, 2020 at 06:30:16PM +0800, Lin Ma wrote: > The get lba status wrapper based on the bdrv_block_status. The following > patches will add GET LBA STATUS 16 support for scsi emulation layer. > > Signed-off-by: Lin Ma > --- > block/io.c| 43 +

[PATCH v2 1/3] block: Add bdrv_co_get_lba_status

2020-06-17 Thread Lin Ma
The get lba status wrapper based on the bdrv_block_status. The following patches will add GET LBA STATUS 16 support for scsi emulation layer. Signed-off-by: Lin Ma --- block/io.c| 43 +++ include/block/block_int.h | 5 + 2 files changed, 4

[PATCH v2 1/3] block: Add bdrv_co_get_lba_status

2020-06-17 Thread Lin Ma
The get lba status wrapper based on the bdrv_block_status. The following patches will add GET LBA STATUS 16 support for scsi emulation layer. Signed-off-by: Lin Ma --- block/io.c| 43 +++ include/block/block_int.h | 5 + 2 files changed, 4