On 11/06/2019 19:43, Christopher Clark wrote: > On Tue, Jun 11, 2019 at 10:11 AM Nicholas Tsirakis > <niko.tsira...@gmail.com> wrote: >> In its current state, if the destination ring is full, sendv() >> will requeue the message and return the rc of pending_requeue(), >> which will return 0 on success. This prevents the caller from >> distinguishing the difference between a successful write and a >> message that needs to be resent at a later time. >> >> Instead, capture the -EAGAIN value returned from ringbuf_insert() >> and *only* overwrite it if the rc of pending_requeue() is non-zero. >> This allows the caller to make intelligent decisions on -EAGAIN and >> still be alerted if the pending message fails to requeue. >> >> Signed-off-by: Nicholas Tsirakis <tsirak...@ainfosec.com> > Thanks for the correct identification of the problem and the patch. > > Reviewed-by: Christopher Clark <christopher.cla...@baesystems.com>
So I was coming to commit this, but technically according to the maintainers file, ARGO is maintained by <christopher.w.cl...@gmail.com> Looking at the ARGO series as committed, the patches where all From: gmail, SoB: baesystems. Which is the correct alias to use? ~Andrew _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
