From: Rusty Russell<rusty@xxxxxxxxxxxxxxx>
Date: Thu, 16 Jan 2014 10:25:26 +1030
Rusty Russell<rusty@xxxxxxxxxxxxxxx> writes:Jason please repost this with Rusty's ACK, thanks.Jason Wang<jasowang@xxxxxxxxxx> writes:Oops, this doesn't require any core virtio changes, so it's for DaveM:It looks like there's no need for those two fields:Nice. These fields date from when the vq struct was opaque.
- Unless there's a failure for the first refill try, rq->max should be always
equal to the vring size.
- rq->num is only used to determine the condition that we need to do the refill,
we could check vq->num_free instead.
- rq->num was required to be increased or decreased explicitly after each
get/put which results a bad API.
So this patch removes them both to make the code simpler.
Applied,
Rusty.
Acked-by: Rusty Russell<rusty@xxxxxxxxxxxxxxx>