From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrea Arcangeli Subject: Re: [PATCH 2/5] i915: flush gem obj freeing workqueues to add accuracy to the i915 shrinker Date: Fri, 7 Apr 2017 15:06:00 +0200 Message-ID: <20170407130600.GA5035@redhat.com> References: <87pogtplxr.fsf@intel.com> <20170406232347.988-1-aarcange@redhat.com> <20170406232347.988-3-aarcange@redhat.com> <20170407100211.GG10496@nuc-i3427.alporthouse.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: Content-Disposition: inline In-Reply-To: <20170407100211.GG10496@nuc-i3427.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 , Martin Kepplinger , Thorsten Leemhuis , daniel.vetter@intel.com, Dave Airlie , intel-gfx@lists.freedesktop.org, linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org List-Id: dri-devel@lists.freedesktop.org T24gRnJpLCBBcHIgMDcsIDIwMTcgYXQgMTE6MDI6MTFBTSArMDEwMCwgQ2hyaXMgV2lsc29uIHdy b3RlOgo+IE9uIEZyaSwgQXByIDA3LCAyMDE3IGF0IDAxOjIzOjQ0QU0gKzAyMDAsIEFuZHJlYSBB cmNhbmdlbGkgd3JvdGU6Cj4gPiBXYWl0aW5nIGEgUkNVIGdyYWNlIHBlcmlvZCBvbmx5IGd1YXJh bnRlZXMgdGhlIHdvcmsgZ2V0cyBxdWV1ZWQsIGJ1dAo+ID4gdW50aWwgYWZ0ZXIgdGhlIHF1ZXVl ZCB3b3JrcXVldWUgcmV0dXJucywgdGhlcmUncyBubyBndWFyYW50ZWUgdGhlCj4gPiBtZW1vcnkg d2FzIGFjdHVhbGx5IGZyZWVkLiBTbyBmbHVzaCB0aGUgd29yayB0byBwcm92aWRlIGJldHRlcgo+ ID4gZ3VhcmFudGVlcyB0byB0aGUgcmVjbGFpbSBjb2RlIGluIGFkZGl0aW9uIG9mIHdhaXRpbmcg YSBSQ1UgZ3JhY2UKPiA+IHBlcmlvZCB0byBwYXNzLgo+IAo+IFdlIGFyZSBub3QgYWxsb3dlZCB0 byBjYWxsIGZsdXNoX3dvcmsoKSBmcm9tIHRoZSBzaHJpbmtlciwgdGhlIHdvcmtxdWV1ZQo+IGRv ZXNuJ3QgaGF2ZSBhbmQgY2FuJ3QgaGF2ZSB0aGUgcmlnaHQgcmVjbGFpbSBmbGFncy4KCkkgZmln dXJlZCB0aGUgZmx1c2hfd29yayBoYWQgdG8gYmUgY29uZGl0aW9uYWwgdG8gInVubG9jayIgYmVp bmcgdHJ1ZQp0b28gaW4gdGhlIGk5MTUgc2hyaW5rZXIgKG5vdCBvbmx5IHN5bmNocm9uaXplX3Jj dV9leHBlZGl0ZWQoKSksIGFuZCBJCmFscmVhZHkgZml4ZWQgdGhhdCBiaXQsIGJ1dCBJIGRpZG4n dCB0aGluayBpdCB3b3VsZCBiZSBhIHByb2JsZW0gdG8Kd2FpdCBmb3IgdGhlIHdvcmtxdWV1ZSBh cyBsb25nIGFzIHJlY2xhaW0gZGlkbid0IHJlY3Vyc2Ugb24gdGhlCnN0cnVjdF9tdXRleCAoaXQg aXMgYSBwcm9ibGVtIGlmIHVubG9jayBpcyBmYWxzZSBvZiBjb3Vyc2UgYXMgd2Ugd291bGQKYmUg YmFjayB0byBzcXVhcmUgb25lKS4gSSBkaWRuJ3QgZ2V0IGZ1cnRoZXIgaGFuZ3MgYW5kIEkgYXNz dW1lIEkndmUKYmVlbiBydW5uaW5nIGEgY291cGxlIG9mIHN5bmNocm9uaXplX3JjdV9leHBlZGl0 ZWQoKSBhbmQgZmx1c2hfd29yayAoSQpzaG91bGQgYWRkIGR5bmFtaWMgdHJhY2luZyB0byBiZSBz dXJlKS4KCkFsc28gbm90ZSwgSSBkaWRuJ3QgZ2V0IGFueSBsb2NrZGVwIHdhcm5pbmcgd2hlbiBJ IHJlcHJvZHVjZWQgdGhlCndvcmtxdWV1ZSBoYW5nIGluIDQuMTEtcmM1IHNvIGF0IGxlYXN0IGFz IGZhciBhcyBsb2NrZGVwIGlzIGNvbmNlcm5lZAp0aGVyZSdzIG5vIHByb2JsZW0gdG8gY2FsbCBz eW5jaHJvbml6ZV9yY3VfZXhwZWRpdGVkIGFuZCBpdCBjb3VsZG4ndApub3RpY2Ugd2Ugd2VyZSBo b2xkaW5nIHRoZSBzdHJ1Y3RfbXV0ZXggd2hpbGUgd2FpdGluZyBmb3IgdGhlIG5ldwp3b3JrcXVl dWUgdG8gcnVuLgoKQWxzbyBub3RlIHJlY3Vyc2luZyBvbiB0aGUgbG9jayAodW5sb2NrIGZhbHNl IGNhc2UpIGlzIHNvbWV0aGluZwpub3RoaW5nIGVsc2UgZG9lcywgSSdtIG5vdCBzdXJlIGlmIGl0 J3Mgd29ydGggdGhlIHJpc2sgYW5kIGlmIHlvdQpzaG91bGRuJ3QganVzdCBjYWxsIG11dGV4X3Ry eWxvY2sgaW4gdGhlIHNocmlua2VyIGluc3RlYWQgb2YKbXV0ZXhfdHJ5bG9ja19yZWN1cnNpdmUu IE9uZSB0aGluZyB3YXMgdG8gcmVjdXJzZSBvbiB0aGUgbG9jawppbnRlcm5hbGx5IGluIHRoZSBz YW1lIGNvbnRleHQsIGJ1dCByZWN1cnNpbmcgdGhyb3VnaCB0aGUgd2hvbGUKcmVjbGFpbSBpcyBt b3JlIGR1YmlvdXMgYXMgc2FmZS4KCllvdSBjb3VsZCBzdGFydCBkcm9wcGluZyBvYmplY3RzIGFu ZCB3aXBpbmcgdm1hcyBhbmQgc3R1ZmYgaW4gdGhlCm1pZGRsZSBvZiBzb21lIGttYWxsb2MvYWxs b2NfcGFnZXMgdGhhdCBkb2Vzbid0IGV4cGVjdCBpdCBhbmQgdGhlbgpjcmFzaCBmb3Igb3RoZXIg cmVhc29ucy4gU28gdGhpcyByZWNsYWltIHJlY3Vyc2lvbiBtb2RlbCBvZiB0aGUKc2hpbmtlciBp cyBxdWl0ZSB1bmlxdWUgYW5kIHF1aXRlIGNoYWxsZW5naW5nIHRvIHByb29mIGFzIHNhZmUgaWYg eW91CmtlZXAgdXNpbmcgbXV0ZXhfdHJ5bG9ja19yZWN1cnNpdmUgaW4gaTkxNV9nZW1fc2hyaW5r ZXJfc2Nhbi4KCkxvY2sgcmVjdXJzaW9uIGluIGFsbCBvdGhlciBwbGFjZXMgY291bGQgYmUgZHJv cHBlZCB3aXRob3V0IHJ1bnRpbWUKZG93bnNpZGVzLCB0aGUgb25seSBwbGFjZSBtdXRleF90cnls b2NrX3JlY3Vyc2l2ZSBtYWtlcyBhIGRlc2lnbgpkaWZmZXJlbmNlIGFuZCBtYWtlcyBzZW5zZSB0 byBiZSB1c2VkIGlzIGluIGk5MTVfZ2VtX3Nocmlua2VyX3NjYW4sCnRoZSByZXN0IGFyZSBpbXBs ZW1lbnRhdGlvbiBpc3N1ZXMgbm90IGZ1bmRhbWVudGFsIHNocmlua2VyIGRlc2lnbiBhbmQKaXQn ZCBiZSBuaWNlIGlmIHRob3NlIG90aGVyIG11dGV4X3RyeWxvY2tfcmVjdXJzaXZlIHdvdWxkIGFs bCBiZQpyZW1vdmVkIGFuZCB0aGUgb25seSBvbmUgdGhhdCBpcyBsZWZ0IGlzIGluIGk5MTVfZ2Vt X3Nocmlua2VyX3NjYW4gYW5kCm5vd2hlcmUgZWxzZSAob3IgdG8gZHJvcCBpdCBhbHNvIGZyb20g aTkxNV9nZW1fc2hyaW5rZXJfc2NhbikuCgptdXRleF90cnlsb2NrX3JlY3Vyc2l2ZSgpIHNob3Vs ZCBhbHNvIGJlIHBhdGNoZWQgdG8gdXNlClJFQURfT05DRShfX211dGV4X293bmVyKGxvY2spKSBi ZWNhdXNlIGN1cnJlbnRseSBpdCBicmVha3MgQy4KCkluIHRoZSB3aG9sZSBrZXJuZWwgaTkxNSBh bmQgbXNtIGRybSBhcmUgdGhlIG9ubHkgdHdvIHVzZXJzIG9mIHN1Y2gKZnVuY3Rpb24gaW4gZmFj dC4KCkFub3RoZXIgdGhpbmcgaXMgd2hhdCB2YWx1ZSByZXR1cm4gZnJvbSBpOTE1X2dlbV9zaHJp bmtlcl9zY2FuIHdoZW4KdW5sb2NrIGlzIGZhbHNlLCBhbmQgd2UgY2FuJ3QgcG9zc2libHkgd2Fp dCBmb3IgdGhlIG1lbW9yeSB0byBiZSBmcmVlZApsZXQgYWxvbmUgZm9yIGEgcmN1IGdyYWNlIHBl cmlvZC4gRm9yIHZhcmlvdXMgcmVhc29ucyBJIHRoaW5rIGl0J3MKc2FmZXIgdG8gcmV0dXJuIHRo ZSBjdXJyZW50ICJmcmVlIiBldmVuIGlmIHdlIGNvdWxkIGFsc28gcmV0dXJuICIwIiBpbgpzdWNo IGNhc2UuIFRoZXJlIGFyZSBkaWZmZXJlbnQgdHJhZGVvZmZzLCByZXR1cm5pbmcgImZyZWUiIGlz IGxlc3MKbGlrZWx5IHRvIHRyaWdnZXIgYW4gZWFybHkgT09NIGFzIHRoZSBWTSB0aGlua3MgaXQn cyBzdGlsbCBtYWtpbmcKcHJvZ3Jlc3MgYW5kIGluIGZhY3QgaXQgd2lsbCBnZXQgbW9yZSBmcmVl IG1lbW9yeSBzaG9ydGx5LCB3aGlsZQpyZXR1cm5pbmcgU0hSSU5LX1NUT1Agd291bGQgYWxzbyBi ZSBhbiBvcHRpb24gYW5kIGl0IHdvdWxkIGluc2lzdCBtb3JlCm9uIHRoZSBvdGhlciBzbGFicyBz byBpdCB3b3VsZCBiZSBtb3JlIHJlbGlhYmxlIGF0IGZyZWVpbmcgbWVtb3J5CnRpbWVseSwgYnV0 IGl0IHdvdWxkIGJlIG1vcmUgYXQgcmlzayBvZiBlYXJseSBPT00uIEkgdGhpbmsgcmV0dXJuaW5n CiJmcmVlIiBpcyB0aGUgYmV0dGVyIHRyYWRlb2ZmIG9mIHRoZSB0d28sIGJ1dCBJIHN1Z2dlc3Qg dG8gYWRkIGEKY29tbWVudCBhcyBpdCdzIG5vdCBleGFjdGx5IG9idmlvdXMgd2hhdCBpcyBiZXR0 ZXIuCgpUaGFua3MsCkFuZHJlYQpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f X19fX19fX19fXwpJbnRlbC1nZnggbWFpbGluZyBsaXN0CkludGVsLWdmeEBsaXN0cy5mcmVlZGVz a3RvcC5vcmcKaHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9p bnRlbC1nZngK