From mboxrd@z Thu Jan  1 00:00:00 1970
From: "Keith Packard" <keithp@keithp.com>
Subject: Re: [PATCH 1/3] drm/lease: debug output for lease
 creation
Date: Fri, 02 Nov 2018 08:02:12 -0700
Message-ID: <87bm77tte3.fsf@keithp.com>
References: <20181102132543.16486-1-daniel.vetter@ffwll.ch>
Mime-Version: 1.0
Content-Type: multipart/mixed; boundary="===============0396346239=="
Return-path: <intel-gfx-bounces@lists.freedesktop.org>
In-Reply-To: <20181102132543.16486-1-daniel.vetter@ffwll.ch>
List-Unsubscribe: <https://lists.freedesktop.org/mailman/options/intel-gfx>,
 <mailto:intel-gfx-request@lists.freedesktop.org?subject=unsubscribe>
List-Archive: <https://lists.freedesktop.org/archives/intel-gfx>
List-Post: <mailto:intel-gfx@lists.freedesktop.org>
List-Help: <mailto:intel-gfx-request@lists.freedesktop.org?subject=help>
List-Subscribe: <https://lists.freedesktop.org/mailman/listinfo/intel-gfx>,
 <mailto:intel-gfx-request@lists.freedesktop.org?subject=subscribe>
Errors-To: intel-gfx-bounces@lists.freedesktop.org
Sender: "Intel-gfx" <intel-gfx-bounces@lists.freedesktop.org>
To: DRI Development <dri-devel@lists.freedesktop.org>
Cc: Daniel Vetter <daniel.vetter@ffwll.ch>, Intel Graphics Development <intel-gfx@lists.freedesktop.org>
List-Id: dri-devel@lists.freedesktop.org

--===============0396346239==
Content-Type: multipart/signed; boundary="=-=-=";
	micalg=pgp-sha256; protocol="application/pgp-signature"

--=-=-=
Content-Type: text/plain
Content-Transfer-Encoding: quoted-printable

Daniel Vetter <daniel.vetter@ffwll.ch> writes:

> I spent a bit of time scratching heads and figuring out why the igts
> don't work. Probably useful to keep this work.

Acked-by: Keith Packard <keithp@keithp.com>

>  	/* Do not allow sub-leases */
> -	if (lessor->lessor)
> +	if (lessor->lessor) {
> +		DRM_DEBUG_LEASE("recursive leasing not allowed\n");
>  		return -EINVAL;
> +	}

(not review about this patch, just noticing that the relevant code is here:=
-)

We may want to revisit this restriction as it looks like multi-seat may
end up using leases from a manager to multiple X servers and letting
those X servers support leases would be a good thing.

=2D-=20
=2Dkeith

--=-=-=
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAEBCAAdFiEEw4O3eCVWE9/bQJ2R2yIaaQAAABEFAlvcZvQACgkQ2yIaaQAA
ABGXrBAAoiWkItoXzNPJG2UJNCVBoytCdqdNAP2cZlZXEHo1LQhOA2ZjmRkZ3lvi
P0QR11KkeGRNBAWMFKOdUNQ1PFTmUheRvmVjMZtG/+3yy9+c+C3YvTzVano/+eSG
AJPA5rYYdy85c4zl5B7cF1lnE5D39oVmoEg9+x1FAFS34EgpwvlEp0IO9PT5XAA6
tJ4umACg3ke0Z9JAlMg39f6NuSHUe4YLujvwBsoPb7dwFCLj+dvVRJq6mDZ7Wetn
v0k+dJjAGzTjDkc+h848J0cSlcDFZHC0l6mD7KjNSF89I14FR6DyEYJTc3zcnX3L
vb/U7mfTeJ9lywvSFDS1DZDyEAEVk6GdADRvB2scxcbT8yVLelDZOlRT3XESc6+Y
PrSi5VndJKP83v96okdRwOyTfw4yyxnm5TbHnJaNK8EhuwEGoJhqnJOOaqmGTJBg
QxEZLfVdHMLIGoa1kvY3mhNjZR7yWUESfJBX07dfmKt6Kqgc/U4AZNt7lDTWmN4Q
u+Lz+VmaNQNvJ8VN4vZJg2X8G9CgjUJKZG1MgHYSvEwXQFTUHsZfGo+hd8d62SDo
VwCcFB4D43w3+MKZiND3icjpHm/1OJY4hDDjw2bM4nxxHAFUs37QRNZZrqase5Gh
aLIt8ndeCSx4Uy1/onLxSnB/yC/0K5kldwl0XGRs2Lwwcnvdc6Y=
=+tvR
-----END PGP SIGNATURE-----
--=-=-=--

--===============0396346239==
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: base64
Content-Disposition: inline

X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4
IG1haWxpbmcgbGlzdApJbnRlbC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz
dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vaW50ZWwtZ2Z4Cg==

--===============0396346239==--