From mboxrd@z Thu Jan  1 00:00:00 1970
Received: from eggs.gnu.org ([2001:4830:134:3::10]:39716)
	by lists.gnu.org with esmtp (Exim 4.71)
	(envelope-from <agraf@suse.de>) id 1Z9sDO-00010P-BD
	for qemu-devel@nongnu.org; Tue, 30 Jun 2015 05:52:31 -0400
Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71)
	(envelope-from <agraf@suse.de>) id 1Z9sDJ-0005Cj-L5
	for qemu-devel@nongnu.org; Tue, 30 Jun 2015 05:52:30 -0400
Message-ID: <559266D8.7070108@suse.de>
Date: Tue, 30 Jun 2015 11:52:24 +0200
From: Alexander Graf <agraf@suse.de>
MIME-Version: 1.0
References: <55925E00.4090308@suse.de>
	<1435657794-6230-1-git-send-email-laurent@vivier.eu>
In-Reply-To: <1435657794-6230-1-git-send-email-laurent@vivier.eu>
Content-Type: text/plain; charset=windows-1252; format=flowed
Content-Transfer-Encoding: 7bit
Subject: Re: [Qemu-devel] [PATCH] linux-user,
	ppc: mftbl can be used by user application
List-Id: <qemu-devel.nongnu.org>
List-Unsubscribe: <https://lists.nongnu.org/mailman/options/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=unsubscribe>
List-Archive: <http://lists.nongnu.org/archive/html/qemu-devel>
List-Post: <mailto:qemu-devel@nongnu.org>
List-Help: <mailto:qemu-devel-request@nongnu.org?subject=help>
List-Subscribe: <https://lists.nongnu.org/mailman/listinfo/qemu-devel>,
	<mailto:qemu-devel-request@nongnu.org?subject=subscribe>
To: Laurent Vivier <laurent@vivier.eu>, Riku Voipio <riku.voipio@iki.fi>
Cc: qemu-ppc@nongnu.org, qemu-devel@nongnu.org

On 06/30/15 11:49, Laurent Vivier wrote:
> In qemu-linux-user, when calling gethostbyname2(),
> it was hanging in .__res_nmkquery.
>
> (gdb) bt
> 0 in .__res_nmkquery () from /lib64/libresolv.so.2
> 1 in .__libc_res_nquery () from /lib64/libresolv.so.2
> 2 in .__libc_res_nsearch () from /lib64/libresolv.so.2
> 3 in ._nss_dns_gethostbyname3_r () from /lib64/libnss_dns.so.2
> 4 in ._nss_dns_gethostbyname2_r () from /lib64/libnss_dns.so.2
> 5 in .gethostbyname2_r () from /lib64/libc.so.6
> 6 in .gethostbyname2 () from /lib64/libc.so.6
>
> .__res_nmkquery() is:
>
> ...
> do { RANDOM_BITS (randombits); } while ((randombits & 0xffff) == 0);
> ...
>
> <.__res_nmkquery+112>:	mftbl   r11
> <.__res_nmkquery+116>:	clrlwi  r10,r11,16
> <.__res_nmkquery+120>:	cmpwi   cr7,r10,0
> <.__res_nmkquery+124>:	beq     cr7,<.__res_nmkquery+112>
>
> but as mftbl (Move From Time Base Lower) is not implemented,
> r11 is always 0, so we have an infinite loop.
>
> This patch fills the Time Base register with cpu_get_real_ticks().
>
> Signed-off-by: Laurent Vivier <laurent@vivier.eu>

Thanks, applied to ppc-next. Riku, if you want to veto, feel free to :).


Alex