From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:55279) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a3u09-0005kJ-QS for qemu-devel@nongnu.org; Tue, 01 Dec 2015 18:06:26 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1a3u05-0007w0-MU for qemu-devel@nongnu.org; Tue, 01 Dec 2015 18:06:25 -0500 Received: from mail-qg0-x232.google.com ([2607:f8b0:400d:c04::232]:33933) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1a3u05-0007vw-HX for qemu-devel@nongnu.org; Tue, 01 Dec 2015 18:06:21 -0500 Received: by qgeb1 with SMTP id b1so19315337qge.1 for ; Tue, 01 Dec 2015 15:06:21 -0800 (PST) Sender: Richard Henderson References: <1448986767-28016-1-git-send-email-rth@twiddle.net> <20151201163451.GC9577@aurel32.net> <20151201164054.GD9577@aurel32.net> From: Richard Henderson Message-ID: <565E27E9.5050601@twiddle.net> Date: Tue, 1 Dec 2015 15:06:17 -0800 MIME-Version: 1.0 In-Reply-To: <20151201164054.GD9577@aurel32.net> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH for-2.5] tcg: Increase the highwater reservation List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Aurelien Jarno , Peter Maydell Cc: QEMU Developers On 12/01/2015 08:40 AM, Aurelien Jarno wrote: > On 2015-12-01 17:34, Aurelien Jarno wrote: >> On 2015-12-01 16:28, Peter Maydell wrote: >>> On 1 December 2015 at 16:19, Richard Henderson wrote: >>>> If there are a lot of guest memory ops in the TB, the amount of >>>> code generated by tcg_out_tb_finalize could be well more than 1k. >>>> In the short term, increase the reservation larger than any TB >>>> seen in practice. >>>> >>>> Reported-by: Aurelien Jarno >>>> Signed-off-by: Richard Henderson >>>> --- >>>> >>>> Reported and discussed with Aurelien on IRC yesterday. This seems >>>> to be the easiest fix for the upcoming release. I will fix this >>>> properly (by modifying every backend's finalize routines) for 2.6. >>> >>> What would be the result of our hitting this bug? I ask because >>> there's a report on qemu-discuss about a qemu-i386-on-ARM-host >>> bug: http://lists.nongnu.org/archive/html/qemu-discuss/2015-11/msg00042.html >>> and the debug log (http://www.mediafire.com/download/ge611be9vbebbw7/qemu.log) >>> suggests we're segfaulting in translation on the TB shortly >>> after we (successfully) translate a TB whose final 'out' size >>> is 1100 and which has 64 guest writes in it. So I'm wondering >>> if that's actually the same bug this is fixing... >> >> I don't think this is the same bug. The problem happens because the slow >> path of the softmmu load/store access is written at the end of the TB. >> In user mode, there is no slow path, so nothing is written at the end. Oh quite right. Duh. > That said the problem reported is likely fixed by this commit that went > just after it has been reported: It does seem likely, but I don't see how we can know that the out size is 1100 in that situation. The disassembler dump doesn't happen until after we've done all of the writes that would have resulted in a highwater overflow segv. I don't see how this can be the same bug. r~