From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:38379) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1W38Zx-00059w-0k for qemu-devel@nongnu.org; Tue, 14 Jan 2014 13:19:13 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1W38Zs-0007fO-4O for qemu-devel@nongnu.org; Tue, 14 Jan 2014 13:19:08 -0500 From: Juan Quintela In-Reply-To: <52D57673.7020100@weilnetz.de> (Stefan Weil's message of "Tue, 14 Jan 2014 18:40:03 +0100") References: <1389679228-15870-1-git-send-email-sw@weilnetz.de> <52D57343.8000205@msgid.tls.msk.ru> <52D57673.7020100@weilnetz.de> Date: Tue, 14 Jan 2014 19:18:50 +0100 Message-ID: <87a9ey1l6d.fsf@elfo.mitica> MIME-Version: 1.0 Content-Type: text/plain Subject: Re: [Qemu-devel] [Qemu-trivial] [PATCH] exec: Exclude non portable function for MinGW Reply-To: quintela@redhat.com List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Stefan Weil Cc: qemu-trivial@nongnu.org, Michael Tokarev , qemu-devel@nongnu.org Stefan Weil wrote: > Am 14.01.2014 18:26, schrieb Michael Tokarev: >> 14.01.2014 10:00, Stefan Weil wrote: >>> cpu_physical_memory_set_dirty_lebitmap calls getpageaddr and ffsl which are >>> unavailable for MinGW. As the function is unused for MinGW, it can simply >>> be excluded from compilation. >> I applied it to -trivial. But maybe it's better to just move whole >> thing to kvm-all.c where >> it is actually used? >> >> Thanks, >> >> /mjt > > That's a good suggestion. Let it as your change. Functions on ram_addr.h should be opaque, and nothing else should access the bitmap. > > Juan, a comment in include/exec/ram_addr.h says that those functions > will be removed soon. Would you suggest moving them to kvm-all.c now, or > would you prefer the conditional compilation for MinGW which I > introduced with my patch? The "soon" was introduced when Memory API was included, and we are still waiting on TCG. Later, Juan.