From mboxrd@z Thu Jan 1 00:00:00 1970 From: Kenneth Graunke Subject: Re: [PATCH] i965: Always try to create a logical context Date: Mon, 20 Nov 2017 10:43:26 -0800 Message-ID: <6924474.czhiFrq7ZV@kirito> References: <20171120092942.3225-1-chris@chris-wilson.co.uk> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0548467991==" Return-path: Received: from smtp97.ord1c.emailsrvr.com (smtp97.ord1c.emailsrvr.com [108.166.43.97]) by gabe.freedesktop.org (Postfix) with ESMTPS id A15FC8920E for ; Mon, 20 Nov 2017 18:51:43 +0000 (UTC) Received: from smtp13.relay.ord1c.emailsrvr.com (localhost [127.0.0.1]) by smtp13.relay.ord1c.emailsrvr.com (SMTP Server) with ESMTP id 3DEBFA056E for ; Mon, 20 Nov 2017 13:43:32 -0500 (EST) Received: from smtp105.ord1d.emailsrvr.com (relay.ord1c.rsapps.net [172.28.255.120]) by smtp13.relay.ord1c.emailsrvr.com (SMTP Server) with ESMTPS id 395C8A038B for ; Mon, 20 Nov 2017 13:43:32 -0500 (EST) In-Reply-To: <20171120092942.3225-1-chris@chris-wilson.co.uk> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" To: Chris Wilson Cc: Jason Ekstrand , intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org --===============0548467991== Content-Type: multipart/signed; boundary="nextPart3295005.QJQx3aQ8ry"; micalg="pgp-sha256"; protocol="application/pgp-signature" --nextPart3295005.QJQx3aQ8ry Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="us-ascii" On Monday, November 20, 2017 1:29:42 AM PST Chris Wilson wrote: > Always enable use of HW logical contexts to preserve GPU state between > batches when the kernel supports such constructs, continuing to enforce > the required support for gen6+. > > At runtime, this effectively removes the BRW_NEW_CONTEXT flag (and the > upload of invariant state) from the start of every batch for any kernel > supporting contexts. So long as the older atoms are correctly listening > to the right flag (NEW_CONTEXT rather than NEW_BATCH) this should > eliminate a few redundant state uploads for the older platforms. > > Cc: Jason Ekstrand > Cc: Kenneth Graunke Nice, I saw Ironlake RC6 go by and was wondering if contexts were up next :) It makes sense to use them if the kernel supports them. Reviewed-by: Kenneth Graunke --nextPart3295005.QJQx3aQ8ry Content-Type: application/pgp-signature; name="signature.asc" Content-Description: This is a digitally signed message part. Content-Transfer-Encoding: 7Bit -----BEGIN PGP SIGNATURE----- iQIzBAABCAAdFiEE6OtbNAgc4e6ibv4ZW1vaBx1JzDgFAloTIk4ACgkQW1vaBx1J zDhTWw//fB0GJ4q94pcfyWT7ENzNfaQsDUhMWEgERQBN3GVOqB2PIAjFa5/C4ZfR eQjAN9M8Ca5zsOCctCi5q2+OX7WyZU9Ud5mvU22eSjApf5SJfIIKwNV6BWk2TrF1 jhYiZ2E1/PKjscdFlWc1emsOTKbWuostVsp+NHPLXvEQKIEMZReQGMju6n1w8m/N wUHc8uOJYkdUeB3mYG6FNxERs0NAqikNQ0dwLJb3zu+m0ePj8cg7DpTFODe95V7f 2fWYIHwllhguexU1rD5B+aGLwdpo5/h1KXjItSp0gMbUgXIk5JER20O+VvU5IAe7 /cgQOZjhzJkRw5f60AmROk/q4xC1iWyZryWaiXB2CVoxLfn4xX8ohLspkHOlG/b0 cJlWxJZAY7xCICsZMKhCecKaEWgUd/ZlUo2UAnkHE7526nejoiYGxZJ5sE7/V2xA MxHRNq0eRLX1dl8ebEHe3FolyW5R8Z1DFcq3vMlzYdmRZf4UfSZNdxVnZUPY807G hIu9oPbvdivpdH9jTG14Yxaasg+g08dVaNnWfJ7liQdM83xLGrjj+K7GO9iDSrLz CJ/Gle5DLRLXiSkVTSW9dHzNIigUUFnC1PBFWfMVC48lZDdqbWQIAGHXfhUVfmTa CNZIMfwDxg/Gvua8GUa1hPB0CVQrN09VP0fFpF8gBPBBbJkWY1Q= =nBHe -----END PGP SIGNATURE----- --nextPart3295005.QJQx3aQ8ry-- --===============0548467991== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4 IG1haWxpbmcgbGlzdApJbnRlbC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vaW50ZWwtZ2Z4Cg== --===============0548467991==--