From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:40269) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZAMdE-0001bR-GX for qemu-devel@nongnu.org; Wed, 01 Jul 2015 14:21:13 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZAMdA-0007Cn-Cn for qemu-devel@nongnu.org; Wed, 01 Jul 2015 14:21:12 -0400 Received: from mout.kundenserver.de ([212.227.17.24]:55044) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZAMdA-0007B1-2I for qemu-devel@nongnu.org; Wed, 01 Jul 2015 14:21:08 -0400 References: <1435681199-14392-1-git-send-email-laurent@vivier.eu> <5592CE38.2010509@vivier.eu> <55932D09.5030102@vivier.eu> <20150701131536.GD11361@aurel32.net> From: Laurent Vivier Message-ID: <55942F8F.9070106@vivier.eu> Date: Wed, 1 Jul 2015 20:21:03 +0200 MIME-Version: 1.0 In-Reply-To: <20150701131536.GD11361@aurel32.net> Content-Type: text/plain; charset=iso-8859-15 Content-Transfer-Encoding: 8bit Subject: Re: [Qemu-devel] [PATCH] linux-user: Avoid compilation error with --disable-guest-base List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Aurelien Jarno Cc: Peter Maydell , Riku Voipio , QEMU Developers Le 01/07/2015 15:15, Aurelien Jarno a écrit : > On 2015-07-01 01:58, Laurent Vivier wrote: >> >> >> Le 30/06/2015 19:20, Peter Maydell a écrit : >>> On 30 June 2015 at 18:13, Laurent Vivier wrote: >>>> >>>> >>>> Le 30/06/2015 18:45, Peter Maydell a écrit : >>>>> On 30 June 2015 at 17:19, Laurent Vivier wrote: >>>>>> When guest base is disabled, RESERVED_VA is 0, and >>>>>> (__guest < RESERVED_VA) is always false as __guest is unsigned. >>>>>> >>>>>> With -Werror=type-limits, this triggers an error: >>>>>> >>>>>> include/exec/cpu_ldst.h:60:31: error: comparison of unsigned expression < 0 is always false [-Werror=type-limits] >>>>>> (!RESERVED_VA || (__guest < RESERVED_VA)); \ >>>>>> >>>>>> This patch removes this comparison when guest base is disabled. >>>>> >>>>> Is there a useful reason to compile with --disable-guest-base >>>>> (ie why we should retain the !CONFIG_USE_GUEST_BASE code >>>>> in QEMU at all) ? It was originally optional because we >>>>> didn't support it in all our TCG hosts, but we fixed that >>>>> back in 2012... >>>> >>>> TCG generates less code, so performance is better (well, it is what I >>>> guess). >>>> >>>> I've compiled a kernel with and without guest base in a chrooted >>>> linux-user-qemu. >>>> Without guest base it is ~1 minute less for a 13 minutes build. >>>> >>>> I can do more tests if you want. >>> >>> Hmm. That's a fair chunk of speedup. On the downside: >>> * you only get this if you're willing to build QEMU from >>> source with funny options >>> * it won't work for all guest/host combinations (sometimes >>> the guest really wants to be able to map at low addresses >>> the host won't permit) >>> * it's an extra configuration to maintain which we're >>> clearly not testing at all upstream >>> >>> I'd still favour removing it completely, personally... >> >> In fact, I have made more measurements, it saves only ~10 seconds on a >> 13 minutes build. >> >> my test is: "make -j 4 vmlinux" >> (target: m68k, host: x86_64, 4 cores x 2 threads) > > Note that on x86_64, guest base is implemented by using the gs segment > register. That explains why the impact should be relatively low, as your > test shows. I did a similar test on a PowerPC host, It is 2 seconds MORE on an 1m27s build WITH --disable-guest-base. So, definitively, I think the option can be dropped. Laurent