Bart Van Assche
2020-Sep-22 02:45 UTC
[PATCH 2/8] vhost: add helper to check if a vq has been setup
On 2020-09-21 11:23, Mike Christie wrote:> +bool vhost_vq_is_setup(struct vhost_virtqueue *vq) > +{ > + if (vq->avail && vq->desc && vq->used && vhost_vq_access_ok(vq)) > + return true; > + else > + return false; > +}Has it been considered changing the body of this function into "return vq->avail && vq->desc && vq->used && vhost_vq_access_ok(vq)"? I'm concerned otherwise one or another build bot will suggest to make that change. Thanks, Bart.
Possibly Parallel Threads
- [PATCH 2/8] vhost: add helper to check if a vq has been setup
- [PATCH 2/8] vhost: add helper to check if a vq has been setup
- [PATCH v6 02/11] vhost, vhost-net: add helper to check if vq has work
- [PATCH 3/8] vhost scsi: alloc cmds per vq instead of session
- [PATCH 4/8] vhost scsi: fix cmd completion race
Wisdom of the Ancients
