From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from smtp3.osuosl.org (smtp3.osuosl.org [140.211.166.136]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id A3E12C7EE21 for ; Thu, 4 May 2023 10:39:23 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp3.osuosl.org (Postfix) with ESMTP id 32B7060B22; Thu, 4 May 2023 10:39:23 +0000 (UTC) DKIM-Filter: OpenDKIM Filter v2.11.0 smtp3.osuosl.org 32B7060B22 Authentication-Results: smtp3.osuosl.org; dkim=fail reason="signature verification failed" (2048-bit key) header.d=gmail.com header.i=@gmail.com header.a=rsa-sha256 header.s=20221208 header.b=h/WC9bsI X-Virus-Scanned: amavisd-new at osuosl.org Received: from smtp3.osuosl.org ([127.0.0.1]) by localhost (smtp3.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id LC1wcJGLrQ9N; Thu, 4 May 2023 10:39:22 +0000 (UTC) Received: from lists.linuxfoundation.org (lf-lists.osuosl.org [IPv6:2605:bc80:3010:104::8cd3:938]) by smtp3.osuosl.org (Postfix) with ESMTPS id 97E9B60B3B; Thu, 4 May 2023 10:39:21 +0000 (UTC) DKIM-Filter: OpenDKIM Filter v2.11.0 smtp3.osuosl.org 97E9B60B3B Received: from lf-lists.osuosl.org (localhost [127.0.0.1]) by lists.linuxfoundation.org (Postfix) with ESMTP id 677D9C0036; Thu, 4 May 2023 10:39:21 +0000 (UTC) Received: from smtp3.osuosl.org (smtp3.osuosl.org [IPv6:2605:bc80:3010::136]) by lists.linuxfoundation.org (Postfix) with ESMTP id 06BFCC002A for ; Thu, 4 May 2023 10:39:20 +0000 (UTC) Received: from localhost (localhost [127.0.0.1]) by smtp3.osuosl.org (Postfix) with ESMTP id C87C660B3B for ; Thu, 4 May 2023 10:39:19 +0000 (UTC) DKIM-Filter: OpenDKIM Filter v2.11.0 smtp3.osuosl.org C87C660B3B X-Virus-Scanned: amavisd-new at osuosl.org Received: from smtp3.osuosl.org ([127.0.0.1]) by localhost (smtp3.osuosl.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id nnPriJXduT99 for ; Thu, 4 May 2023 10:39:19 +0000 (UTC) X-Greylist: whitelisted by SQLgrey-1.8.0 DKIM-Filter: OpenDKIM Filter v2.11.0 smtp3.osuosl.org BBB2060B22 Received: from mail-lj1-x230.google.com (mail-lj1-x230.google.com [IPv6:2a00:1450:4864:20::230]) by smtp3.osuosl.org (Postfix) with ESMTPS id BBB2060B22 for ; Thu, 4 May 2023 10:39:18 +0000 (UTC) Received: by mail-lj1-x230.google.com with SMTP id 38308e7fff4ca-2ac78bb48eeso3852951fa.1 for ; Thu, 04 May 2023 03:39:18 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20221208; t=1683196756; x=1685788756; h=mime-version:references:in-reply-to:message-id:subject:cc:to:from :date:from:to:cc:subject:date:message-id:reply-to; bh=I8cy10TD23BLiLlfGPG7Cx1kjtU5cY0ZvS7C4oKT/pw=; b=h/WC9bsIDNhXfE4cBGKdONkvL8j3FuZ+LIKlbUE/Fw/TL7LWRGFeA7+1jOBPeeTWjC p2mfRLcjbvnrYAbMyRMn8EfIrEZE5+a8j4/BeBqoFutI3/mHFflZ+EwdzYtnYn6GvWAp bPUoVBVgx9A0T6lJU5SK4AaL617FJQjrmtw4u53vZS4fz15YGCxegKUPxOcroxWYuxtB 6B8WiF8Sz3cFcR4id1NhHDvr88GrA2KcW941R8V+UjU0rnE/5w0aDa23bSUC/FtuPovw frQzLlqgl2WCLudQRozM4TtuWpQ7i+tfLK9Rz9A3Q4JeJd2NXj7oJ6OAuOMTEMmJwxhh qGBA== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20221208; t=1683196756; x=1685788756; h=mime-version:references:in-reply-to:message-id:subject:cc:to:from :date:x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=I8cy10TD23BLiLlfGPG7Cx1kjtU5cY0ZvS7C4oKT/pw=; b=BWwRRuw/KypjKaVbpGZedE7v9nny8t3gBgSmXNgHav30rTxkQ5DaAAK7uGgkk/WU+U wJqcdIfIIKjgt8UxVg6E/r+87PSIDdET3/aa5nhl9JjJQVNj9P9fWZwwAXO9m9syuWK+ aHRKoka3+O/c524PAugnoj/ATDinHQRm2uZZabGkvmnTReDjFI7sfy/o3jcDcLR9SbEv hp5TLjUC3slUKlsnNqEm1aJ2yNkEwJS4jFQ5NzAZbVEIfqHyeTuf1NpmrAsT9Z+j3Oa+ NKkXmjRe+TkqLVHT216lc2f7MGCQ2Ss42qebAavVVrZm1rd1ixXQQO92WB7/CNMiU9Hl pm3Q== X-Gm-Message-State: AC+VfDzxFu9yYQlLVzb1UHB5EtXM4ZPNJUqNA748EHDJbixd3wfFUtdg kuyXfJfVLRhqpebNQR2Z5qg= X-Google-Smtp-Source: ACHHUZ63JYRRIjOWE/UJE3a2S7oNalHCss8wmWBdY5Pf1RyHzSjjoQCnGEjVircjnZeeAkiqJ9Zb0w== X-Received: by 2002:a2e:97c3:0:b0:2a8:d32e:e6bd with SMTP id m3-20020a2e97c3000000b002a8d32ee6bdmr833683ljj.21.1683196756288; Thu, 04 May 2023 03:39:16 -0700 (PDT) Received: from eldfell ([194.136.85.206]) by smtp.gmail.com with ESMTPSA id j15-20020a2e800f000000b00294692d8645sm6564927ljg.17.2023.05.04.03.39.15 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Thu, 04 May 2023 03:39:15 -0700 (PDT) Date: Thu, 4 May 2023 13:39:04 +0300 From: Pekka Paalanen To: Zack Rusin Subject: Re: [PATCH v2 1/8] drm: Disable the cursor plane on atomic contexts with virtualized drivers Message-ID: <20230504133904.4ad3011c@eldfell> In-Reply-To: <17cc969e9f13fab112827e154495eca28c4bd2b6.camel@vmware.com> References: <20220712033246.1148476-1-zack@kde.org> <20220712033246.1148476-2-zack@kde.org> <87lei7xemy.fsf@minerva.mail-host-address-is-not-set> <0dd2fa763aa0e659c8cbae94f283d8101450082a.camel@vmware.com> <87y1m5x3bt.fsf@minerva.mail-host-address-is-not-set> <17cc969e9f13fab112827e154495eca28c4bd2b6.camel@vmware.com> X-Mailer: Claws Mail 4.1.1 (GTK 3.24.24; x86_64-pc-linux-gnu) MIME-Version: 1.0 Cc: "daniel@ffwll.ch" , "hdegoede@redhat.com" , "airlied@linux.ie" , "belmouss@redhat.com" , "javierm@redhat.com" , "stable@vger.kernel.org" , "gurchetansingh@chromium.org" , Martin Krastev , Linux-graphics-maintainer , "dri-devel@lists.freedesktop.org" , "tzimmermann@suse.de" , "spice-devel@lists.freedesktop.org" , "airlied@redhat.com" , "virtualization@lists.linux-foundation.org" , Maaz Mombasawala , "jadahl@gmail.com" X-BeenThere: virtualization@lists.linux-foundation.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: Linux virtualization List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Content-Type: multipart/mixed; boundary="===============8161189739908797716==" Errors-To: virtualization-bounces@lists.linux-foundation.org Sender: "Virtualization" --===============8161189739908797716== Content-Type: multipart/signed; boundary="Sig_/fSX23B7nk8XUDBx03oqMhN_"; protocol="application/pgp-signature"; micalg=pgp-sha256 --Sig_/fSX23B7nk8XUDBx03oqMhN_ Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: quoted-printable On Thu, 4 May 2023 01:50:25 +0000 Zack Rusin wrote: > On Wed, 2023-05-03 at 09:48 +0200, Javier Martinez Canillas wrote: > > Zack Rusin writes: > > =20 > > > On Tue, 2023-05-02 at 11:32 +0200, Javier Martinez Canillas wrote: =20 > > > > AFAICT this is the only remaining thing to be addressed for this se= ries ? =20 > > >=20 > > > No, there was more. tbh I haven't had the time to think about whether= the above > > > makes sense to me, e.g. I'm not sure if having virtualized drivers ex= pose > > > "support > > > universal planes" and adding another plane which is not universal (th= e only > > > "universal" plane on them being the default one) makes more sense tha= n a flag > > > that > > > says "this driver requires a cursor in the cursor plane". There's cer= tainly a > > > huge > > > difference in how userspace would be required to handle it and it's w= ay uglier > > > with > > > two different cursor planes. i.e. there's a lot of ways in which this= could be > > > cleaner in the kernel but they all require significant changes to use= rspace, > > > that go > > > way beyond "attach hotspot info to this plane". I'd like to avoid app= roaches > > > that > > > mean running with atomic kms requires completely separate paths for v= irtualized > > > drivers because no one will ever support and maintain it. > > >=20 > > > It's not a trivial thing because it's fundamentally hard to untangle = the fact > > > the > > > virtualized drivers have been advertising universal plane support wit= hout ever > > > supporting universal planes. Especially because most new userspace in= general > > > checks > > > for "universal planes" to expose atomic kms paths. > > > =20 > >=20 > > After some discussion on the #dri-devel, your approach makes sense and = the > > only contention point is the name of the driver feature flag name. The = one > > you are using (DRIVER_VIRTUAL) seems to be too broad and generic (the f= act > > that vkms won't set and is a virtual driver as well, is a good example). > >=20 > > Maybe something like DRIVER_CURSOR_HOTSPOT or DRIVER_CURSOR_COMMANDEERI= NG > > would be more accurate and self explanatory ? =20 >=20 > Sure, or even more verbose DRIVER_NEEDS_CURSOR_PLANE_HOTSPOT, but it soun= ds like > Pekka doesn't agree with this approach. As I mentioned in my response to = him, I'd be > happy with any approach that gets paravirtualized drivers working with at= omic kms, > but atm I don't have enough time to be creating a new kernel subsystem or= a new set > of uapi's for paravirtualized drivers and then porting mutter/kwin to tho= se. It seems I have not been clear enough, apologies. Once more, in short: Zack, I'm worried about this statement from you (copied from above): > > > I'd like to avoid approaches that mean running with atomic kms > > > requires completely separate paths for virtualized drivers > > > because no one will ever support and maintain it. It feels like you are intentionally limiting your own design options for the fear of "no one will ever support it". I'm worried that over the coming years, that will lead to a hard to use, hard to maintain patchwork of vague or undocumented or just too many little UAPI details. Please, don't limit your designs. There are good reasons why nested KMS drivers behave fundamentally differently to most KMS hardware drivers. Userspace that does not or cannot take that into account is unavoidably crippled. Thanks, pq --Sig_/fSX23B7nk8XUDBx03oqMhN_ Content-Type: application/pgp-signature Content-Description: OpenPGP digital signature -----BEGIN PGP SIGNATURE----- iQIzBAEBCAAdFiEEJQjwWQChkWOYOIONI1/ltBGqqqcFAmRTi0gACgkQI1/ltBGq qqediRAAk0pncI/uXJYfQjhsOa7tB6Ix+d5Y3iIRT1aD3+kt2fGVGlRXhHYamy6f /R8z/yrSfr3md4rpsf73HWdpQJYC33FDFhOnJ5pYaUaIhTw9sqdCNlqdsHOppWVx kWWjKB4obarFgjxLZAxAxJ75IUpj1luPqLCfnvwOpwGgCoB1dv+JLZAjN4mj5Oxj c+nqemTqhZhzX8PpBL4gCiTBkdhF+4gc9F73NHfDb6T8ws7jgXDQ9ePuJY31EKpq +RY5Z0KyH+PBvIOhsYcirez3QBkCc1losZz3FBB6DwiXchI7jnRfpNtRvDrNvlwv rA6DGnjAu8buoPh16Or2EKjESAPq9dJL1tFpYCgFD2Ud8VTLYFuV8h7wHXrOnbvj Jx33L0OL8jlRTwQaUiywNQFUqNF3nK++ogfBU/zzQzzU/RWP6CVJGSiqtbfoyHI2 4kk2fUSt2obvOswJX2XHiWyTHSudgpLzP528HDqFelLRa4B5UAg+QVzBc9j4U/5S NOZjKTIfTX+zQ4R9vhrvMSuEReQ6mcEo63tEaoioF3wPcxeogSY7HdoJE5syAWsS gqKV/b+fVk46q5tDZ9bqW874qQmPAupgrhwWenITrSXF1UULQ64hwyT0CL44MJXw xxnPS9Cqlvk9KsA1u7uUtLDlk3hP8reuUJyzMxDqY3G5yWeKZ5o= =fN1I -----END PGP SIGNATURE----- --Sig_/fSX23B7nk8XUDBx03oqMhN_-- --===============8161189739908797716== Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ Virtualization mailing list Virtualization@lists.linux-foundation.org https://lists.linuxfoundation.org/mailman/listinfo/virtualization --===============8161189739908797716==--