From mboxrd@z Thu Jan 1 00:00:00 1970 Received: by 2002:a17:907:2723:b0:a5a:9152:cbdc with SMTP id d3csp770426ejl; Wed, 15 May 2024 09:22:15 -0700 (PDT) X-Forwarded-Encrypted: i=2; AJvYcCVncYvSfdnndXdztXriPunImdFnfP3ZIctl6nsNC5w6M8/2advvh9pKwkhANGz0SKMYmQ8oqI9jgAF48i+Dshh4aOg0NIa0 X-Google-Smtp-Source: AGHT+IH6+iveLH8bKV+IoJsTJ+GiDbQTxnDqcTZjko8cATSoycR6hPcQjlwK06yT0u1yvQMp67m7 X-Received: by 2002:a25:e20e:0:b0:dc6:ddfc:1736 with SMTP id 3f1490d57ef6-dee4f32a723mr14562683276.17.1715790135138; Wed, 15 May 2024 09:22:15 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1715790135; cv=none; d=google.com; s=arc-20160816; b=dmZ0BLDVJA4byiui0JWm1ujBZwc3dzIAbfBx85kfYyExYYtln3urai2P4g2sMVlb8I dcLmrZXTsZ6DIsDGUuT/dPxlN4VjyiJ2FhJEUZ3x9Op5Bbdv6zEvv/2m7ZRFL1LDSCLW x9voJglcf4Fkg6uijRvD2GEMzJXHQ+JtjvCAe5Y8EY5b5Honr7CA+gbzb4LgEV+vBhox leo7jkFccvAcz+tA4fPTn8PrFMmUdGD5tNLrMqQI/MmNymQ1nKBuC8nePxfcKryHXex+ eOgbNmc6zEds9G+ATFfOIiYkaQQO2TaNyI/XOyMv/OZLGRK9VqvpDKa72s7IQBxcvQfc WF3g== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=sender:errors-to:reply-to:list-subscribe:list-help:list-post :list-archive:list-unsubscribe:list-id:precedence:user-agent :in-reply-to:content-transfer-encoding:content-disposition :mime-version:references:message-id:subject:cc:to:from:date :dkim-signature; bh=nmkTWaisy4R2Mu7NUhC46zzhrGGene3P5HaV6fxWV04=; fh=/uKE6QiKKM35cOFAAq5elCEuHK69ydv6F6oZ+yfkDHQ=; b=Flb+OqvRKDG3J2qf5s2LSz1XidO+Je23Y+laNvIqCWthtOxgMb5qjUByzazmx9qLvi 9pnNK6kojJWCp9VNo89iYvKE+xDoUCiOjmB81QppTmrDAARRFZiACCYoIFYGeW3u8YjZ L41Y6xl0ja879OxiwHER7e5qX95FO+1Zpe+Nl0X943w4q9cG26w5FvUcy2VyJ2Yu6SQz ZLJM783MT2qfta5yP7w1kAhTpPxjwlu2h2LTP3hT2opugHHrC9YCz9p20KuEeN/ThhoX AC28oXB53V3TOYlF7Vu1cA3pknRGQnN4AN4SNGmmL9dyg5sZDt3kowYWMKpxQgLojlzD GsRw==; dara=google.com ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b="Iow/CFwi"; spf=pass (google.com: domain of qemu-devel-bounces+alex.bennee=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+alex.bennee=linaro.org@nongnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=redhat.com Return-Path: Received: from lists.gnu.org (lists.gnu.org. [209.51.188.17]) by mx.google.com with ESMTPS id d75a77b69052e-43df566f3aesi140510771cf.381.2024.05.15.09.22.14 for (version=TLS1_2 cipher=ECDHE-ECDSA-CHACHA20-POLY1305 bits=256/256); Wed, 15 May 2024 09:22:15 -0700 (PDT) Received-SPF: pass (google.com: domain of qemu-devel-bounces+alex.bennee=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) client-ip=209.51.188.17; Authentication-Results: mx.google.com; dkim=pass header.i=@redhat.com header.s=mimecast20190719 header.b="Iow/CFwi"; spf=pass (google.com: domain of qemu-devel-bounces+alex.bennee=linaro.org@nongnu.org designates 209.51.188.17 as permitted sender) smtp.mailfrom="qemu-devel-bounces+alex.bennee=linaro.org@nongnu.org"; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=redhat.com Received: from localhost ([::1] helo=lists1p.gnu.org) by lists.gnu.org with esmtp (Exim 4.90_1) (envelope-from ) id 1s7HOL-0004CL-5q; Wed, 15 May 2024 12:22:09 -0400 Received: from eggs.gnu.org ([2001:470:142:3::10]) by lists.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1s7HOJ-0004Bv-O1 for qemu-devel@nongnu.org; Wed, 15 May 2024 12:22:07 -0400 Received: from us-smtp-delivery-124.mimecast.com ([170.10.129.124]) by eggs.gnu.org with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.90_1) (envelope-from ) id 1s7HOG-0007jd-De for qemu-devel@nongnu.org; Wed, 15 May 2024 12:22:07 -0400 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=redhat.com; s=mimecast20190719; t=1715790123; h=from:from:reply-to:reply-to:subject:subject:date:date: message-id:message-id:to:to:cc:cc:mime-version:mime-version: content-type:content-type: content-transfer-encoding:content-transfer-encoding: in-reply-to:in-reply-to:references:references; bh=nmkTWaisy4R2Mu7NUhC46zzhrGGene3P5HaV6fxWV04=; b=Iow/CFwiw26ZwFYGM7AUYv+EzfTmDHtu2sRU3CUNZ7PppQ+U6BufPu/hGh8ZsyQDz7OBDG RNdVfV02eYmG8p80vv32ZjiCUuwg38RPTOa/d8f3UG8/C/ExRWMiLPAbxAL8BhALilQ9cu WdZ6ZLl4SqSjCVUWv5HwvjimX5WQDYY= Received: from mimecast-mx02.redhat.com (mx-ext.redhat.com [66.187.233.73]) by relay.mimecast.com with ESMTP with STARTTLS (version=TLSv1.3, cipher=TLS_AES_256_GCM_SHA384) id us-mta-303-2Khz-HrLOcK4JNNGKh6JCg-1; Wed, 15 May 2024 12:22:00 -0400 X-MC-Unique: 2Khz-HrLOcK4JNNGKh6JCg-1 Received: from smtp.corp.redhat.com (int-mx10.intmail.prod.int.rdu2.redhat.com [10.11.54.10]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256) (No client certificate requested) by mimecast-mx02.redhat.com (Postfix) with ESMTPS id 574FD3803908; Wed, 15 May 2024 16:21:59 +0000 (UTC) Received: from redhat.com (unknown [10.42.28.55]) by smtp.corp.redhat.com (Postfix) with ESMTPS id 0A0D3491032; Wed, 15 May 2024 16:21:55 +0000 (UTC) Date: Wed, 15 May 2024 17:21:53 +0100 From: Daniel =?utf-8?B?UC4gQmVycmFuZ8Op?= To: Peter Xu Cc: marcandre.lureau@redhat.com, qemu-devel@nongnu.org, Eduardo Habkost , "Michael S. Tsirkin" , Marcel Apfelbaum , Fiona Ebner , Paolo Bonzini , Richard Henderson , qemu-arm@nongnu.org, Peter Maydell , Fabiano Rosas , Gerd Hoffmann , Yanan Wang , Philippe =?utf-8?Q?Mathieu-Daud=C3=A9?= , Thomas Huth Subject: Re: [PATCH v3 0/5] Fix "virtio-gpu: fix scanout migration post-load" Message-ID: References: <20240515141557.1277999-1-marcandre.lureau@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: User-Agent: Mutt/2.2.12 (2023-09-09) X-Scanned-By: MIMEDefang 3.4.1 on 10.11.54.10 Received-SPF: pass client-ip=170.10.129.124; envelope-from=berrange@redhat.com; helo=us-smtp-delivery-124.mimecast.com X-Spam_score_int: -29 X-Spam_score: -3.0 X-Spam_bar: --- X-Spam_report: (-3.0 / 5.0 requ) BAYES_00=-1.9, DKIMWL_WL_HIGH=-0.935, DKIM_SIGNED=0.1, DKIM_VALID=-0.1, DKIM_VALID_AU=-0.1, DKIM_VALID_EF=-0.1, RCVD_IN_DNSWL_NONE=-0.0001, RCVD_IN_MSPIKE_H4=0.001, RCVD_IN_MSPIKE_WL=0.001, SPF_HELO_NONE=0.001, SPF_PASS=-0.001 autolearn=unavailable autolearn_force=no X-Spam_action: no action X-BeenThere: qemu-devel@nongnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: Daniel =?utf-8?B?UC4gQmVycmFuZ8Op?= Errors-To: qemu-devel-bounces+alex.bennee=linaro.org@nongnu.org Sender: qemu-devel-bounces+alex.bennee=linaro.org@nongnu.org X-TUID: U3vSZX6YKrui On Wed, May 15, 2024 at 10:07:31AM -0600, Peter Xu wrote: > On Wed, May 15, 2024 at 06:15:51PM +0400, marcandre.lureau@redhat.com wrote: > > From: Marc-André Lureau > > > > Hi, > > > > The aforementioned patch breaks virtio-gpu device migrations for versions > > pre-9.0/9.0, both forwards and backwards. Versioning of `VMS_STRUCT` is more > > complex than it may initially appear, as evidenced in the problematic commit > > dfcf74fa68c ("virtio-gpu: fix scanout migration post-load"). > > > > v2: > > - use a manual version field test (instead of the more complex struct variant) > > > > v3: > > - introduce machine_check_version() > > - drop the VMSD version, and use machine version field test > > Thanks for trying this out already. > > Last time I mentioned this may for the long term because I remember Dan and > Thomas were trying to work on some machine deprecation work, and maybe such > things may collapse with that work (and perhaps easier with that work > landed, too?). Just to copy them both here so we know where we are now, as > I didn't follow that discussion. IOW, patch 3/4 may need separate review > from outside migration.. You'll be refering to my series here: https://lists.nongnu.org/archive/html/qemu-devel/2024-05/msg00084.html Note that series very delibrately did *not* expose the version numbers as accessible fields to code. The version number info is only accessible within the machine type macros, and once the macros are expanded, the version digits remains hidden within the opaque machine type name strings, and/or method names. With regards, Daniel -- |: https://berrange.com -o- https://www.flickr.com/photos/dberrange :| |: https://libvirt.org -o- https://fstop138.berrange.com :| |: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|