From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jan Kiszka Subject: Re: [RFC][PATCH] qemu-kvm: Drop vga dirty logging workarounds Date: Mon, 15 Nov 2010 14:26:36 +0100 Message-ID: <4CE1350C.3080304@siemens.com> References: <4CE10C55.5040104@siemens.com> <4CE132B2.9050507@redhat.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-15 Content-Transfer-Encoding: 7bit Cc: Marcelo Tosatti , kvm To: Avi Kivity Return-path: Received: from thoth.sbs.de ([192.35.17.2]:22144 "EHLO thoth.sbs.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754535Ab0KON0z (ORCPT ); Mon, 15 Nov 2010 08:26:55 -0500 In-Reply-To: <4CE132B2.9050507@redhat.com> Sender: kvm-owner@vger.kernel.org List-ID: Am 15.11.2010 14:16, Avi Kivity wrote: > On 11/15/2010 12:32 PM, Jan Kiszka wrote: >> These diffs to upstream should all date back to the days qemu-kvm >> supported vga dirty logging with restricted/broken kvm kernel modules. >> We no longer do, so there is no need for those workarounds. Even worse >> they can trigger internal bug checks these days: >> >> BUG: kvm_dirty_pages_log_change: invalid parameters 00000000000a8000-00000000000affff > > I'd like to apply this. What kind of testing has this seen? autotest > likely isn't a good enough test. No systematic testing. It's based on the fact that I'm not aware of any VGA issues in upstream when KVM is enabled, on the fact that explicit log enable/disable became obsolete when switching to upstream's logging backend, and the annoying BUG messages only issued under qemu-kvm. There just remains a slight uncertainty about what mark_dirty and s[123] were once doing - or may still be useful for. Jan -- Siemens AG, Corporate Technology, CT T DE IT 1 Corporate Competence Center Embedded Linux