From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([208.118.235.92]:35862) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SrpFY-00009h-M6 for qemu-devel@nongnu.org; Thu, 19 Jul 2012 07:50:36 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1SrpFS-0000PW-UX for qemu-devel@nongnu.org; Thu, 19 Jul 2012 07:50:32 -0400 Received: from mx1.redhat.com ([209.132.183.28]:48028) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1SrpFS-0000P3-Kz for qemu-devel@nongnu.org; Thu, 19 Jul 2012 07:50:26 -0400 Message-ID: <5007F473.8070301@redhat.com> Date: Thu, 19 Jul 2012 14:50:11 +0300 From: Avi Kivity MIME-Version: 1.0 References: <1342531805-29894-1-git-send-email-anthony.perard@citrix.com> <1342531805-29894-4-git-send-email-anthony.perard@citrix.com> <50056AA1.9010004@redhat.com> <50056FAB.8030404@citrix.com> <50057A5E.1070602@redhat.com> <5007F25E.4020502@citrix.com> In-Reply-To: <5007F25E.4020502@citrix.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH 3/4] exec, memory: Call to xen_modified_memory. List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Anthony PERARD Cc: Luiz Capitulino , Anthony Liguori , Xen Devel , QEMU-devel , Stefano Stabellini On 07/19/2012 02:41 PM, Anthony PERARD wrote: > On 17/07/12 19:36, Stefano Stabellini wrote: >> On Tue, 17 Jul 2012, Avi Kivity wrote: >>> How about pushing the call into cpu_physical_memory_set_dirty_flags()? >>> Would that reduce the number of call sites? >> >> Pushing the calls to cpu_physical_memory_set_dirty_flags and >> cpu_physical_memory_set_dirty_range would make the code much nicer. >> However being these functions in exec-obsolete.h, are they at risk of >> removal? > > I thought about it, but when I saw that set_dirty were called only when > it was not already set as dirty where the call seams to be necessary. > > I just try to call xen_modified_mem only within > cpu_phy_mem_set_dirty_flags but it does not work, even when I tried to > clear the dirtybits. But I maybe don't do the right thing yet to clear > the dirty bits You can wrap the if (not dirty) make_it_dirty() sequence in a helper, and insert your hypercall in the helper, unconditionally. -- error compiling committee.c: too many arguments to function