From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:50989) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZoNEN-0003Zq-9M for qemu-devel@nongnu.org; Mon, 19 Oct 2015 23:04:56 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZoNEI-0005WX-Ak for qemu-devel@nongnu.org; Mon, 19 Oct 2015 23:04:55 -0400 Received: from mx1.redhat.com ([209.132.183.28]:51382) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZoNEI-0005WI-54 for qemu-devel@nongnu.org; Mon, 19 Oct 2015 23:04:50 -0400 References: <5620F082.5040007@redhat.com> <56216E1A.30609@weilnetz.de> From: Jason Wang Message-ID: <5625AF4B.9060409@redhat.com> Date: Tue, 20 Oct 2015 11:04:43 +0800 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PATCH] eepro100: prevent an infinite loop over same command block List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Qinghao Tang Cc: P J P , Peter Maydell , Paolo Bonzini , QEMU Developers , Stefan Weil On 10/17/2015 07:35 PM, Peter Maydell wrote: > On 16 October 2015 at 22:37, Stefan Weil wrote: >> Maybe real hardware will run an endless loop? >> Or the "endless" loop is terminated because the driver >> changes the link while the loop is running? >> >> The goal of eepro100.c should be emulation of the >> real hardware, even of a potential design weakness. > I agree in general, but we need to be sure that if we're > letting the guest put a device into an infinite loop this > doesn't lock up the whole VM (ie preventing the user > from using the qemu monitor or otherwise rebooting it). > > thanks > -- PMM > Yes, so the reproducer needs to be tested on real hardware. Qinghao: Any chance to test the reproducer on real e100? If not, I will try to find one to test. Thanks