Hello Ben, On Wed, 2021-11-24 at 19:43 -0600, Benjamin Marzinski wrote: > On Thu, Nov 18, 2021 at 11:58:24PM +0100, mwilck@xxxxxxxx wrote: > > From: Martin Wilck <mwilck@xxxxxxxx> > > > > send_packet() may busy-loop. By polling for POLLOUT, we can > > avoid that, even if it's very unlikely in practice. > > > > The last time I reviewed this, I mentioned that when we fall through > from CLT_WORK to CLT_SEND, the client hasn't checked for a POLLOUT > revent, so it's possible to block here. I'm not sure that we care. > If not > > Reviewed-by: Benjamin Marzinski <bmarzins@xxxxxxxxxx> Unlike the other patches, I did remove your "Reviewed-by:" on this patch in the v3 series, because I made a non-trivial change in order to fix the issue you mentioned. Now it's the only patch in the v3 series that doesn't have your Reviewed-by: tag. Will you review this patch again? If you're fine with the patch, and you have no other objections, I'll push the v4 of the series (with the whitespace issues fixed) to my "queue2 branch and start preparing a submission for Christophe. Regards Martin -- dm-devel mailing list dm-devel@xxxxxxxxxx https://listman.redhat.com/mailman/listinfo/dm-devel
