On 11/06/2019 20:22, Christopher Clark wrote: > On Tue, Jun 11, 2019 at 12:16 PM Andrew Cooper > <andrew.coop...@citrix.com> wrote: >> 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? > For this purpose: > Reviewed-by: Christopher Clark <christopher.w.cl...@gmail.com>
Fixed up and pushed. Thanks. ~Andrew _______________________________________________ Xen-devel mailing list Xen-devel@lists.xenproject.org https://lists.xenproject.org/mailman/listinfo/xen-devel
