On 24 February 2015 at 19:14, Laszlo Ersek <ler...@redhat.com> wrote: > On 02/24/15 20:01, Ard Biesheuvel wrote: >> On 24 February 2015 at 18:41, Laszlo Ersek <ler...@redhat.com> wrote: >>> On 02/24/15 19:37, Ard Biesheuvel wrote: >>>> On 24 February 2015 at 18:34, Laszlo Ersek <ler...@redhat.com> wrote: >>>>> On 02/24/15 19:02, Ard Biesheuvel wrote: >>>>> >>>>>> Changes since v4: >>>>>> - rename InterlockedCompareExchange16 () patch as suggested by Jordan, >>>>>> and added >>>>>> his ack >>>>>> - fix a bug spotted by Anthony in the TestAndClearBit () implementation >>>>>> - added more acks and R-b's >>>>> >>>>> - Are there any patches missing review tags? >>>> >>>> There is just one patch, #23, which had a bug so I was hoping Anthony >>>> would add his R-b to the now fixed version. >>>> The InterlockedCompareExchange16() has Jordan's ack, but no reply yet >>>> from Michael Kinney. >>>> >>>>> - Who should push this once everything is reviewed? >>>>> >>>> >>>> Good question. Are you volunteering? :-) >>> >>> Yes. Kick me when you have a public branch with the final review tags added. >>> >> >> Grr. Patch #13 is authored jointly by Olivier and myself, and touches >> only ARM specific code under MdePkg/ >> I guess that needs someone else's ack as well? >> >> Anyways, the branch is here >> https://git.linaro.org/people/ard.biesheuvel/uefi-next.git/shortlog/refs/heads/linaro-topic-xen >> >> with acks/r-b's on all patches except #13 > > If Olivier is okay with patch #13, then I don't think that should be a > worry. However we should await Michael's feedback for patch #14. Please > write him an email directly, with a distinct subject. >
He should be, he wrote the initial version. The reason he did not ack or r-b it is because he is already in a S-o-b tag Re #14: I sent Michael an email yesterday, to which he responded he was going to review it the same day, which he apparently didn't :-( Anyway, I will wait another day or so for him to respond _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel
