Hi Michael, >> It seems there is no need to keep the inuse field signed and end up with >> compiler warnings for sign-compare. >> >> CC libvhost-user.o >> libvhost-user.c: In function ‘vu_queue_pop’: >> libvhost-user.c:2763:19: error: comparison of integer expressions of >> different signedness: ‘int’ and ‘unsigned int’ [-Werror=sign-compare] >> 2763 | if (vq->inuse >= vq->vring.num) { >> | ^~ >> libvhost-user.c: In function ‘vu_queue_rewind’: >> libvhost-user.c:2808:13: error: comparison of integer expressions of >> different signedness: ‘unsigned int’ and ‘int’ [-Werror=sign-compare] >> 2808 | if (num > vq->inuse) { >> | ^ >> >> Instead of casting the comparision to unsigned int, just make the inuse >> field unsigned int in the fist place. >> >> Signed-off-by: Marcel Holtmann <mar...@holtmann.org> > > > Is this a part of a patchset? No threading visible and I'd rather not > guess.
I am going to re-send both series as v2. Regards Marcel