On Fri, Jun 28, 2013 at 07:27:21PM +0200, Markus Armbruster wrote: > Eric Blake <ebl...@redhat.com> writes: > > > On 06/26/2013 08:15 AM, Markus Armbruster wrote: > >> Luiz Capitulino <lcapitul...@redhat.com> writes: > >> > >>> On Wed, 26 Jun 2013 14:00:30 +0200 > >>> Markus Armbruster <arm...@redhat.com> wrote: > >>> > >>>> Meh, I got confused and reviewed an out-of-date version. Hope it's not > >>>> entirely noise. > >>> > >>> I don't think it is. But this series got applied to Michael's tree, so > >>> if you want your comments addressed before applying to master (I think > >>> we do) then it's better to state it clearly. > >> > >> Michael, please give Amos a chance to reply to my review. > > > > Looks like the pull request is already live but had issues; meanwhile, I > > also found an issue when reviewing the pull request: > > https://lists.gnu.org/archive/html/qemu-devel/2013-06/msg05127.html > > Missed it, thanks!
In mst's v2 pull, it still contains my patch, it's not merged to master by Anthony. I think mst should re-send a v3 without my patch as said in changelog. I will fix the problem in your reply, thanks. -- Amos.