Kevin Wolf
2012-May-15 10:32 UTC
Re: [Qemu-devel] [PATCH, v2] qemu/xendisk: properly update stats in ioreq_release()
Am 15.05.2012 12:16, schrieb Stefano Stabellini:> On Tue, 15 May 2012, Kevin Wolf wrote: >> Am 14.05.2012 16:57, schrieb Stefano Stabellini: >>> On Mon, 14 May 2012, Jan Beulich wrote: >>>> While for the "normal" case (called from blk_send_response_all()) >>>> decrementing requests_finished is correct, doing so in the parse error >>>> case is wrong; requests_inflight needs to be decremented instead. >>>> >>>> Change in v2: Adjust coding style. >>>> >>>> Signed-off-by: Jan Beulich <jbeulich@suse.com> >>> >>> Acked-by: Stefano Stabellini <stefano.stabellini@eu.citrix.com> >> >> Aren''t you going to send a pull request yourself, Stefano? > > Yes, I am: I am collecting a series of Xen patches for 1.1 and I am > aiming at sending a single pull request by the end of the week. > > http://marc.info/?l=qemu-devel&m=133701567429388&w=2Ok, great. I was just wondering because I always understood an Acked-by as a message to the maintainer who picks the patch up, so I wasn''t sure if you expected me to do it and the patch would fall through the cracks. Kevin