From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:44669) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XAJiw-0006L7-CX for qemu-devel@nongnu.org; Thu, 24 Jul 2014 10:10:30 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1XAJir-0000P3-EU for qemu-devel@nongnu.org; Thu, 24 Jul 2014 10:10:22 -0400 Received: from mx1.redhat.com ([209.132.183.28]:11475) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1XAJir-0000OC-2w for qemu-devel@nongnu.org; Thu, 24 Jul 2014 10:10:17 -0400 Message-ID: <1406211011.13444.20.camel@nilsson.home.kraxel.org> From: Gerd Hoffmann Date: Thu, 24 Jul 2014 16:10:11 +0200 In-Reply-To: <53D0FA0F.2030501@suse.de> References: <1406190486-16026-1-git-send-email-kraxel@redhat.com> <53D0FA0F.2030501@suse.de> Content-Type: text/plain; charset="UTF-8" Mime-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: Re: [Qemu-devel] [PULL for-2.1 0/2] vnc: fix two vnc update issues. List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Andreas =?ISO-8859-1?Q?F=E4rber?= Cc: Peter Maydell , Stephan Kulow , QEMU Developers , Bruce Rogers Hi, > > So are these *really* release critical bugs, if they've been > > only found in code review? We're really close to release now > > and so my preference is not to include changes unless they're > > really necessary... > > These are fixing openQA breakage (os-autoinst), In more detail: Stephan's patch fixes a rather serve violation of the vnc protocol. If vnc clients ask for a complete framebuffer update they may not get it. The complete display is tagged dirty, but nothing is sent out until something changes on the screen, thereby triggering processing of all dirty display regions. Which may be never. The other patch fixes a simliar situation. Screen updates might become stuck in case vnc delays processing due to output buffers being filled. Again other screen updates will trigger dirty processing and un-stuck the updates. That kind of bug you usually don't notice as normal user. You'll wiggle the mouse and the mouse pointer update will make vnc flush things. Or, if in text mode, the friendly blinking cursor causes regular vnc screen update activity. Thats why it didn't trip up normal users. Scrips are much more likely to hit it as they don't do random mouse activity. But even openQA doesn't hit it on every run, but only now and then. Also asking for a complete framebuffer update isn't something desktop vnc clients usually do (after initial connect). openQA does it though. cheers, Gerd