From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:52482) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZoNKA-00068c-Mm for qemu-devel@nongnu.org; Mon, 19 Oct 2015 23:10:55 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZoNK7-00078V-Fs for qemu-devel@nongnu.org; Mon, 19 Oct 2015 23:10:54 -0400 Received: from mail-pa0-x236.google.com ([2607:f8b0:400e:c03::236]:35571) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZoNK7-00078L-9y for qemu-devel@nongnu.org; Mon, 19 Oct 2015 23:10:51 -0400 Received: by pasz6 with SMTP id z6so6201684pas.2 for ; Mon, 19 Oct 2015 20:10:50 -0700 (PDT) Content-Type: text/plain; charset=gb2312 Mime-Version: 1.0 (1.0) From: max In-Reply-To: <5625AF4B.9060409@redhat.com> Date: Tue, 20 Oct 2015 11:10:44 +0800 Content-Transfer-Encoding: quoted-printable Message-Id: References: <5620F082.5040007@redhat.com> <56216E1A.30609@weilnetz.de> <5625AF4B.9060409@redhat.com> 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: Jason Wang Cc: P J P , Peter Maydell , Paolo Bonzini , QEMU Developers , Stefan Weil I will try to test the PoC on real e100. But this work may need some more time. =B7=A2=D7=D4=CE=D2=B5=C4 iPhone > =D4=DA 2015=C4=EA10=D4=C220=C8=D5=A3=AC=C9=CF=CE=E711:04=A3=ACJason Wang <= jasowang@redhat.com> =D0=B4=B5=C0=A3=BA >=20 >=20 >=20 >> 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? >>>=20 >>> 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). >>=20 >> thanks >> -- PMM >=20 > Yes, so the reproducer needs to be tested on real hardware. >=20 > Qinghao: >=20 > Any chance to test the reproducer on real e100? If not, I will try to > find one to test. >=20 > Thanks