From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S966996AbcHBPL2 (ORCPT ); Tue, 2 Aug 2016 11:11:28 -0400 Received: from home.keithp.com ([63.227.221.253]:60290 "EHLO elaine.keithp.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S965480AbcHBPLG (ORCPT ); Tue, 2 Aug 2016 11:11:06 -0400 From: Keith Packard To: Daniel Vetter Cc: linux-kernel@vger.kernel.org, Daniel Vetter , intel-gfx@lists.freedesktop.org, dri-devel@lists.freedesktop.org Subject: Re: [PATCH] drm/i915: cleanup_plane_fb: also drop reference to current state wait_req In-Reply-To: <20160802133841.GN6232@phenom.ffwll.local> References: <1469951691-5938-1-git-send-email-keithp@keithp.com> <20160802133841.GN6232@phenom.ffwll.local> User-Agent: Notmuch/0.22 (http://notmuchmail.org) Emacs/24.5.1 (x86_64-pc-linux-gnu) Date: Tue, 02 Aug 2016 08:11:02 -0700 Message-ID: <86lh0f435l.fsf@hiro.keithp.com> MIME-Version: 1.0 Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha256; protocol="application/pgp-signature" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --=-=-= Content-Type: text/plain Content-Transfer-Encoding: quoted-printable Daniel Vetter writes: > Hm, I think we should just clean up wiat_req in ->atomic_destroy_state > instead of littering cleanup code all over. But this gets the job done, so > applied. Thanks. It's required for the DRM patch I posted that makes moving the cursor not block on rendering. I'm hoping Dave will get that merged for 4.8; it's really annoying to have the server moving the cursor in a separate thread and then have the motion blocked in the kernel for a few seconds worth of rendering. =2D-=20 =2Dkeith --=-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIVAwUBV6C4BtsiGmkAAAARAQilYg//cLqVbPteBH/FuqGtw1rvoEBOzKCV8ptV uoeNEz2w5GkxUERmT1HsdvecfUclEZ1S7+I7suYDOMxX/4EStJP5L0jn42U5G6Ds gHh1UQisaf+dri3LMk6zrvRgjKPWeBOh4cRY/1Jhx4g7nVXONVLvrOuhll1lCuah QuGCk4wTt4Eyf1TV4+QpkGWV1rwQ+PV48Rda+gobzSsx6tzQFNfaXoGRt9b0kDza k/t6h33zUYONeHNtvEp3V2GVNJK99J1+ZM19RduUeGc4vX3MlvEe+e2vqyPCHftj gHphDS0larKdaqhh/aTml3FcbEw7L5MT8eDf2Qrz2rWnVRyXfdyzq2LS8RhfihtZ Ks035b5pFFSfVfNsYC+oHDaaKk0DKXMOoCQ3C/5EpOSj8/tP3bQh7Rz57WjFdo/Q ZSWTK/1SfMJJoN/pJdhhc7jRDHUKrO8tl44843oS2mdp5L0sgzATvMkzJgqWw7ur V/+n4vUycshUWsEChAdVsenuJ4XOGej7Tm4udC0vBrlwFuCV4zSzG7erz0u8xALI rqKTWOFPAQZU0wn/+rva0aUYp8r6u7ty+XZiSGoN0oDAoZtO6qz90uKV8MmuuIh0 2MaQ45TPxJ1emluAFelyYhOOU9ZE72UGqc/TYItSm2dut0FKPseDDD7oJJ2/enxQ a+wEgHip9B4= =qsJU -----END PGP SIGNATURE----- --=-=-=-- From mboxrd@z Thu Jan 1 00:00:00 1970 From: Keith Packard Subject: Re: [PATCH] drm/i915: cleanup_plane_fb: also drop reference to current state wait_req Date: Tue, 02 Aug 2016 08:11:02 -0700 Message-ID: <86lh0f435l.fsf@hiro.keithp.com> References: <1469951691-5938-1-git-send-email-keithp@keithp.com> <20160802133841.GN6232@phenom.ffwll.local> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0489037836==" Return-path: In-Reply-To: <20160802133841.GN6232@phenom.ffwll.local> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Daniel Vetter Cc: Daniel Vetter , intel-gfx@lists.freedesktop.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org --===============0489037836== Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha256; protocol="application/pgp-signature" --=-=-= Content-Type: text/plain Content-Transfer-Encoding: quoted-printable Daniel Vetter writes: > Hm, I think we should just clean up wiat_req in ->atomic_destroy_state > instead of littering cleanup code all over. But this gets the job done, so > applied. Thanks. It's required for the DRM patch I posted that makes moving the cursor not block on rendering. I'm hoping Dave will get that merged for 4.8; it's really annoying to have the server moving the cursor in a separate thread and then have the motion blocked in the kernel for a few seconds worth of rendering. =2D-=20 =2Dkeith --=-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIVAwUBV6C4BtsiGmkAAAARAQilYg//cLqVbPteBH/FuqGtw1rvoEBOzKCV8ptV uoeNEz2w5GkxUERmT1HsdvecfUclEZ1S7+I7suYDOMxX/4EStJP5L0jn42U5G6Ds gHh1UQisaf+dri3LMk6zrvRgjKPWeBOh4cRY/1Jhx4g7nVXONVLvrOuhll1lCuah QuGCk4wTt4Eyf1TV4+QpkGWV1rwQ+PV48Rda+gobzSsx6tzQFNfaXoGRt9b0kDza k/t6h33zUYONeHNtvEp3V2GVNJK99J1+ZM19RduUeGc4vX3MlvEe+e2vqyPCHftj gHphDS0larKdaqhh/aTml3FcbEw7L5MT8eDf2Qrz2rWnVRyXfdyzq2LS8RhfihtZ Ks035b5pFFSfVfNsYC+oHDaaKk0DKXMOoCQ3C/5EpOSj8/tP3bQh7Rz57WjFdo/Q ZSWTK/1SfMJJoN/pJdhhc7jRDHUKrO8tl44843oS2mdp5L0sgzATvMkzJgqWw7ur V/+n4vUycshUWsEChAdVsenuJ4XOGej7Tm4udC0vBrlwFuCV4zSzG7erz0u8xALI rqKTWOFPAQZU0wn/+rva0aUYp8r6u7ty+XZiSGoN0oDAoZtO6qz90uKV8MmuuIh0 2MaQ45TPxJ1emluAFelyYhOOU9ZE72UGqc/TYItSm2dut0FKPseDDD7oJJ2/enxQ a+wEgHip9B4= =qsJU -----END PGP SIGNATURE----- --=-=-=-- --===============0489037836== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4 IG1haWxpbmcgbGlzdApJbnRlbC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vaW50ZWwtZ2Z4Cg== --===============0489037836==--