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 ------------------------------------------------------------------------------ Dive into the World of Parallel Programming The Go Parallel Website, sponsored by Intel and developed in partnership with Slashdot Media, is your hub for all things parallel software development, from weekly thought leadership blogs to news, videos, case studies, tutorials and more. Take a look and join the conversation now. http://goparallel.sourceforge.net/ _______________________________________________ edk2-devel mailing list edk2-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/edk2-devel
