From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:38927) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1gN0eV-0005SM-MM for qemu-devel@nongnu.org; Wed, 14 Nov 2018 14:16:40 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1gN0eU-0007YE-Vd for qemu-devel@nongnu.org; Wed, 14 Nov 2018 14:16:39 -0500 Received: from mail-wr1-x435.google.com ([2a00:1450:4864:20::435]:37657) by eggs.gnu.org with esmtps (TLS1.0:RSA_AES_128_CBC_SHA1:16) (Exim 4.71) (envelope-from ) id 1gN0eU-0007Xj-PO for qemu-devel@nongnu.org; Wed, 14 Nov 2018 14:16:38 -0500 Received: by mail-wr1-x435.google.com with SMTP id j10so4330893wru.4 for ; Wed, 14 Nov 2018 11:16:38 -0800 (PST) MIME-Version: 1.0 References: <20181101021226.6353-1-zhangckid@gmail.com> <20181114104720.GB3955@xz-x1> <74cca5d1-d2b4-6b44-32fe-332136b8b171@redhat.com> In-Reply-To: From: Zhang Chen Date: Thu, 15 Nov 2018 03:16:25 +0800 Message-ID: Content-Type: text/plain; charset="UTF-8" Subject: Re: [Qemu-devel] [PATCH V2] migration/colo.c: Fix compilation issue when disable replication List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Peter Maydell Cc: Thomas Huth , peterx@redhat.com, qemu-devel@nongnu.org, "zhang.zhanghailiang@huawei.com" , "Dr . David Alan Gilbert" , f4bug@amsat.org On Wed, Nov 14, 2018 at 7:17 PM Peter Maydell wrote: > On 14 November 2018 at 11:06, Thomas Huth wrote: > > On 2018-11-14 11:47, Peter Xu wrote: > >> On Thu, Nov 01, 2018 at 10:12:26AM +0800, Zhang Chen wrote: > >>> This compilation issue will occur when user use --disable-replication > >>> to config Qemu. > >>> > >>> Reported-by: Thomas Huth > >>> Signed-off-by: Zhang Chen > >> > >> Hi, > >> > >> How's the status of this patch? Are we gonna merge it for 3.1? > >> > >> I just posted a similar one without knowing this (until Dave pointed > >> it out). IMHO it can be a good candidate for 3.1. > > > > Maybe Peter Maydell could apply this directly to the repo as a built fix? > > I'd rather it just went through the migration tree, really... > it isn't actually breaking any of our travis jobs or other CI. > > Hi Dave, What do you think about peter's comments? Thanks Zhang Chen > thanks > -- PMM >