mirror of https://github.com/xqemu/xqemu.git
iostatus: change is_read to a bool
Do this while we are touching this part of the code, before introducing more uses of "int is_read". Signed-off-by: Paolo Bonzini <pbonzini@redhat.com> Reviewed-by: Eric Blake <eblake@redhat.com> Signed-off-by: Kevin Wolf <kwolf@redhat.com>
This commit is contained in:
parent
92aa5c6d77
commit
1ceee0d5cc
4
block.c
4
block.c
|
@ -1387,7 +1387,7 @@ void bdrv_set_dev_ops(BlockDriverState *bs, const BlockDevOps *ops,
|
|||
}
|
||||
|
||||
void bdrv_emit_qmp_error_event(const BlockDriverState *bdrv,
|
||||
BlockErrorAction action, int is_read)
|
||||
BlockErrorAction action, bool is_read)
|
||||
{
|
||||
QObject *data;
|
||||
const char *action_str;
|
||||
|
@ -2481,7 +2481,7 @@ void bdrv_set_on_error(BlockDriverState *bs, BlockdevOnError on_read_error,
|
|||
bs->on_write_error = on_write_error;
|
||||
}
|
||||
|
||||
BlockdevOnError bdrv_get_on_error(BlockDriverState *bs, int is_read)
|
||||
BlockdevOnError bdrv_get_on_error(BlockDriverState *bs, bool is_read)
|
||||
{
|
||||
return is_read ? bs->on_read_error : bs->on_write_error;
|
||||
}
|
||||
|
|
4
block.h
4
block.h
|
@ -109,7 +109,7 @@ void bdrv_iostatus_disable(BlockDriverState *bs);
|
|||
bool bdrv_iostatus_is_enabled(const BlockDriverState *bs);
|
||||
void bdrv_iostatus_set_err(BlockDriverState *bs, int error);
|
||||
void bdrv_emit_qmp_error_event(const BlockDriverState *bdrv,
|
||||
BlockErrorAction action, int is_read);
|
||||
BlockErrorAction action, bool is_read);
|
||||
void bdrv_info_print(Monitor *mon, const QObject *data);
|
||||
void bdrv_info(Monitor *mon, QObject **ret_data);
|
||||
void bdrv_stats_print(Monitor *mon, const QObject *data);
|
||||
|
@ -281,7 +281,7 @@ int bdrv_is_allocated(BlockDriverState *bs, int64_t sector_num, int nb_sectors,
|
|||
|
||||
void bdrv_set_on_error(BlockDriverState *bs, BlockdevOnError on_read_error,
|
||||
BlockdevOnError on_write_error);
|
||||
BlockdevOnError bdrv_get_on_error(BlockDriverState *bs, int is_read);
|
||||
BlockdevOnError bdrv_get_on_error(BlockDriverState *bs, bool is_read);
|
||||
int bdrv_is_read_only(BlockDriverState *bs);
|
||||
int bdrv_is_sg(BlockDriverState *bs);
|
||||
int bdrv_enable_write_cache(BlockDriverState *bs);
|
||||
|
|
|
@ -238,7 +238,7 @@ static void drive_put_ref_bh_schedule(DriveInfo *dinfo)
|
|||
qemu_bh_schedule(s->bh);
|
||||
}
|
||||
|
||||
static int parse_block_error_action(const char *buf, int is_read)
|
||||
static int parse_block_error_action(const char *buf, bool is_read)
|
||||
{
|
||||
if (!strcmp(buf, "ignore")) {
|
||||
return BLOCKDEV_ON_ERROR_IGNORE;
|
||||
|
|
|
@ -556,7 +556,7 @@ void ide_dma_error(IDEState *s)
|
|||
|
||||
static int ide_handle_rw_error(IDEState *s, int error, int op)
|
||||
{
|
||||
int is_read = (op & BM_STATUS_RETRY_READ);
|
||||
bool is_read = (op & BM_STATUS_RETRY_READ) != 0;
|
||||
BlockdevOnError action = bdrv_get_on_error(s->bs, is_read);
|
||||
|
||||
if (action == BLOCKDEV_ON_ERROR_IGNORE) {
|
||||
|
|
|
@ -188,7 +188,7 @@ static void bmdma_restart_bh(void *opaque)
|
|||
{
|
||||
BMDMAState *bm = opaque;
|
||||
IDEBus *bus = bm->bus;
|
||||
int is_read;
|
||||
bool is_read;
|
||||
int error_status;
|
||||
|
||||
qemu_bh_delete(bm->bh);
|
||||
|
@ -198,7 +198,7 @@ static void bmdma_restart_bh(void *opaque)
|
|||
return;
|
||||
}
|
||||
|
||||
is_read = !!(bus->error_status & BM_STATUS_RETRY_READ);
|
||||
is_read = (bus->error_status & BM_STATUS_RETRY_READ) != 0;
|
||||
|
||||
/* The error status must be cleared before resubmitting the request: The
|
||||
* request may fail again, and this case can only be distinguished if the
|
||||
|
|
|
@ -386,7 +386,7 @@ static void scsi_read_data(SCSIRequest *req)
|
|||
*/
|
||||
static int scsi_handle_rw_error(SCSIDiskReq *r, int error)
|
||||
{
|
||||
int is_read = (r->req.cmd.xfer == SCSI_XFER_FROM_DEV);
|
||||
bool is_read = (r->req.cmd.xfer == SCSI_XFER_FROM_DEV);
|
||||
SCSIDiskState *s = DO_UPCAST(SCSIDiskState, qdev, r->req.dev);
|
||||
BlockdevOnError action = bdrv_get_on_error(s->qdev.conf.bs, is_read);
|
||||
|
||||
|
|
|
@ -64,7 +64,7 @@ static void virtio_blk_req_complete(VirtIOBlockReq *req, int status)
|
|||
}
|
||||
|
||||
static int virtio_blk_handle_rw_error(VirtIOBlockReq *req, int error,
|
||||
int is_read)
|
||||
bool is_read)
|
||||
{
|
||||
BlockdevOnError action = bdrv_get_on_error(req->dev->bs, is_read);
|
||||
VirtIOBlock *s = req->dev;
|
||||
|
@ -98,7 +98,7 @@ static void virtio_blk_rw_complete(void *opaque, int ret)
|
|||
trace_virtio_blk_rw_complete(req, ret);
|
||||
|
||||
if (ret) {
|
||||
int is_read = !(ldl_p(&req->out->type) & VIRTIO_BLK_T_OUT);
|
||||
bool is_read = !(ldl_p(&req->out->type) & VIRTIO_BLK_T_OUT);
|
||||
if (virtio_blk_handle_rw_error(req, -ret, is_read))
|
||||
return;
|
||||
}
|
||||
|
|
Loading…
Reference in New Issue