From mboxrd@z Thu Jan 1 00:00:00 1970 From: Zhenyu Wang Subject: Re: [PATCH 1/4] drm/i915: Remove i915_gem_context_create_gvt() Date: Mon, 12 Aug 2019 15:57:56 +0800 Message-ID: <20190812075756.GD14790@zhen-hp.sh.intel.com> References: <20190809154146.16397-1-chris@chris-wilson.co.uk> <874l2q8gij.fsf@gaia.fi.intel.com> <156536737864.32306.8520357726207994926@skylake-alporthouse-com> Reply-To: Zhenyu Wang Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============0340921777==" Return-path: Received: from mga18.intel.com (mga18.intel.com [134.134.136.126]) by gabe.freedesktop.org (Postfix) with ESMTPS id 330CD88503 for ; Mon, 12 Aug 2019 08:03:49 +0000 (UTC) In-Reply-To: <156536737864.32306.8520357726207994926@skylake-alporthouse-com> 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: intel-gfx@lists.freedesktop.org List-Id: intel-gfx@lists.freedesktop.org --===============0340921777== Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="XvKFcGCOAo53UbWW" Content-Disposition: inline --XvKFcGCOAo53UbWW Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On 2019.08.09 17:16:18 +0100, Chris Wilson wrote: > Quoting Mika Kuoppala (2019-08-09 17:12:20) > > Chris Wilson writes: > >=20 > > > As we are phasing out using the GEM context for internal clients that > > > need to manipulate logical context state directly, remove the > > > constructor for the GVT context. We are not using it for anything oth= er > > > than default setup and allocation of an i915_ppgtt. > > > > > > Signed-off-by: Chris Wilson > >=20 > > Not familiar with gvt but seems to create similar enough > > context.=20 > >=20 > > Higher priority, not closed and using more strict > > lock. >=20 > I thought they would appreciate the higher priority ;) >=20 > gvt currently doesn't have anyway for it to determine the relative > priority of its guest vs the host, so if we set it too low we impact the > qos of the guest, too high and its affects the qos of the host. Keeping > it at max user though still gives the kernel some leeway to usurp it. > And who knows one day we may get a scheduler. yep, appreciate that. ;) Acked-by: Zhenyu Wang --=20 Open Source Technology Center, Intel ltd. $gpg --keyserver wwwkeys.pgp.net --recv-keys 4D781827 --XvKFcGCOAo53UbWW Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iF0EARECAB0WIQTXuabgHDW6LPt9CICxBBozTXgYJwUCXVEcAwAKCRCxBBozTXgY JxTCAJ9lwqOfVtvzettshXd4QrLzWALoZQCfd8DDPOfqD0GOTSH2vegBbnqSBYs= =qUnv -----END PGP SIGNATURE----- --XvKFcGCOAo53UbWW-- --===============0340921777== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KSW50ZWwtZ2Z4 IG1haWxpbmcgbGlzdApJbnRlbC1nZnhAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vaW50ZWwtZ2Z4 --===============0340921777==--