From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-13.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 9DCF0C47082 for ; Tue, 8 Jun 2021 07:40:08 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id 6373D60FE8 for ; Tue, 8 Jun 2021 07:40:08 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 6373D60FE8 Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linux.intel.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=dri-devel-bounces@lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id 6BCC56EB03; Tue, 8 Jun 2021 07:40:07 +0000 (UTC) Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by gabe.freedesktop.org (Postfix) with ESMTPS id 03F3E6EB03; Tue, 8 Jun 2021 07:40:05 +0000 (UTC) IronPort-SDR: Kxc1Ou8zh4VOgBueZmKXmCkYtzEeo8cMFIgaa/0nsldF83NH8CsPO3/DVxgrkpk/UypNmODLvc wZU9lWUA7iKA== X-IronPort-AV: E=McAfee;i="6200,9189,10008"; a="204605927" X-IronPort-AV: E=Sophos;i="5.83,257,1616482800"; d="scan'208";a="204605927" Received: from fmsmga007.fm.intel.com ([10.253.24.52]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 08 Jun 2021 00:40:02 -0700 IronPort-SDR: GgfC07aVKe5jpiWdewHbG0l49J8VZfusRbVuC7rhtHqhL2HWwnlRkSqmt6wjC35bO22ci3rUQo JHfRHAfzqUyg== X-IronPort-AV: E=Sophos;i="5.83,257,1616482800"; d="scan'208";a="413296834" Received: from delmer-mobl.ger.corp.intel.com (HELO [10.249.254.231]) ([10.249.254.231]) by fmsmga007-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 08 Jun 2021 00:40:01 -0700 Message-ID: Subject: Re: [PATCH 5/6] drm/i915/ttm: switch over to ttm_buddy_man From: Thomas =?ISO-8859-1?Q?Hellstr=F6m?= To: Matthew Auld , intel-gfx@lists.freedesktop.org Date: Tue, 08 Jun 2021 09:39:58 +0200 In-Reply-To: <20210607182210.99036-6-matthew.auld@intel.com> References: <20210607182210.99036-1-matthew.auld@intel.com> <20210607182210.99036-6-matthew.auld@intel.com> Content-Type: text/plain; charset="UTF-8" User-Agent: Evolution 3.40.1 (3.40.1-1.fc34) MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: dri-devel@lists.freedesktop.org Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" On Mon, 2021-06-07 at 19:22 +0100, Matthew Auld wrote: > Move back to the buddy allocator for managing device local memory, > and > restore the lost mock selftests. Keep around the range manager > related > bits, since we likely need this for managing stolen at some point. > For > stolen we also don't need to reserve anything so no need to support a > generic reserve interface. > > Signed-off-by: Matthew Auld > Cc: Thomas Hellström > --- >  drivers/gpu/drm/i915/gem/i915_gem_ttm.c       |  26 +-- >  drivers/gpu/drm/i915/intel_memory_region.c    |  55 +----- >  drivers/gpu/drm/i915/intel_memory_region.h    |  17 -- >  drivers/gpu/drm/i915/intel_region_ttm.c       | 100 +++-------- >  .../drm/i915/selftests/intel_memory_region.c  | 170 ++++++++++++---- > -- > ... >   >  static int i915_ttm_move(struct ttm_buffer_object *bo, bool evict, > @@ -661,20 +661,8 @@ int __i915_gem_ttm_object_init(struct > intel_memory_region *mem, >         static struct lock_class_key lock_class; >         struct drm_i915_private *i915 = mem->i915; >         enum ttm_bo_type bo_type; > -       size_t alignment = 0; >         int ret; >   > -       /* Adjust alignment to GPU- and CPU huge page sizes. */ > - > -       if (mem->is_range_manager) { > -               if (size >= SZ_1G) > -                       alignment = SZ_1G >> PAGE_SHIFT; > -               else if (size >= SZ_2M) > -                       alignment = SZ_2M >> PAGE_SHIFT; > -               else if (size >= SZ_64K) > -                       alignment = SZ_64K >> PAGE_SHIFT; > -       } > - >         drm_gem_private_object_init(&i915->drm, &obj->base, size); >         i915_gem_object_init(obj, &i915_gem_ttm_obj_ops, &lock_class, > flags); >         i915_gem_object_init_memory_region(obj, mem); > @@ -696,7 +684,7 @@ int __i915_gem_ttm_object_init(struct > intel_memory_region *mem, >          */ >         obj->base.vma_node.driver_private = i915_gem_to_ttm(obj); >         ret = ttm_bo_init(&i915->bdev, i915_gem_to_ttm(obj), size, > -                         bo_type, &i915_sys_placement, alignment, > +                         bo_type, &i915_sys_placement, PAGE_SIZE, Actually just realized that the alignment is specified in PAGE_SIZE units, so above should be s/PAGE_SIZE/1/. Might need to check that the buddy TTM interface gets this right as well. From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-13.8 required=3.0 tests=BAYES_00, HEADER_FROM_DIFFERENT_DOMAINS,INCLUDES_CR_TRAILER,INCLUDES_PATCH, MAILING_LIST_MULTI,SPF_HELO_NONE,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 0B20DC47082 for ; Tue, 8 Jun 2021 07:40:13 +0000 (UTC) Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id C73046127A for ; Tue, 8 Jun 2021 07:40:12 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org C73046127A Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=linux.intel.com Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=intel-gfx-bounces@lists.freedesktop.org Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id DC9E16EB04; Tue, 8 Jun 2021 07:40:07 +0000 (UTC) Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by gabe.freedesktop.org (Postfix) with ESMTPS id 03F3E6EB03; Tue, 8 Jun 2021 07:40:05 +0000 (UTC) IronPort-SDR: Kxc1Ou8zh4VOgBueZmKXmCkYtzEeo8cMFIgaa/0nsldF83NH8CsPO3/DVxgrkpk/UypNmODLvc wZU9lWUA7iKA== X-IronPort-AV: E=McAfee;i="6200,9189,10008"; a="204605927" X-IronPort-AV: E=Sophos;i="5.83,257,1616482800"; d="scan'208";a="204605927" Received: from fmsmga007.fm.intel.com ([10.253.24.52]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 08 Jun 2021 00:40:02 -0700 IronPort-SDR: GgfC07aVKe5jpiWdewHbG0l49J8VZfusRbVuC7rhtHqhL2HWwnlRkSqmt6wjC35bO22ci3rUQo JHfRHAfzqUyg== X-IronPort-AV: E=Sophos;i="5.83,257,1616482800"; d="scan'208";a="413296834" Received: from delmer-mobl.ger.corp.intel.com (HELO [10.249.254.231]) ([10.249.254.231]) by fmsmga007-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 08 Jun 2021 00:40:01 -0700 Message-ID: From: Thomas =?ISO-8859-1?Q?Hellstr=F6m?= To: Matthew Auld , intel-gfx@lists.freedesktop.org Date: Tue, 08 Jun 2021 09:39:58 +0200 In-Reply-To: <20210607182210.99036-6-matthew.auld@intel.com> References: <20210607182210.99036-1-matthew.auld@intel.com> <20210607182210.99036-6-matthew.auld@intel.com> User-Agent: Evolution 3.40.1 (3.40.1-1.fc34) MIME-Version: 1.0 Subject: Re: [Intel-gfx] [PATCH 5/6] drm/i915/ttm: switch over to ttm_buddy_man X-BeenThere: intel-gfx@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Intel graphics driver community testing & development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: dri-devel@lists.freedesktop.org Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Errors-To: intel-gfx-bounces@lists.freedesktop.org Sender: "Intel-gfx" T24gTW9uLCAyMDIxLTA2LTA3IGF0IDE5OjIyICswMTAwLCBNYXR0aGV3IEF1bGQgd3JvdGU6Cj4g TW92ZSBiYWNrIHRvIHRoZSBidWRkeSBhbGxvY2F0b3IgZm9yIG1hbmFnaW5nIGRldmljZSBsb2Nh bCBtZW1vcnksCj4gYW5kCj4gcmVzdG9yZSB0aGUgbG9zdCBtb2NrIHNlbGZ0ZXN0cy4gS2VlcCBh cm91bmQgdGhlIHJhbmdlIG1hbmFnZXIKPiByZWxhdGVkCj4gYml0cywgc2luY2Ugd2UgbGlrZWx5 IG5lZWQgdGhpcyBmb3IgbWFuYWdpbmcgc3RvbGVuIGF0IHNvbWUgcG9pbnQuCj4gRm9yCj4gc3Rv bGVuIHdlIGFsc28gZG9uJ3QgbmVlZCB0byByZXNlcnZlIGFueXRoaW5nIHNvIG5vIG5lZWQgdG8g c3VwcG9ydCBhCj4gZ2VuZXJpYyByZXNlcnZlIGludGVyZmFjZS4KPiAKPiBTaWduZWQtb2ZmLWJ5 OiBNYXR0aGV3IEF1bGQgPG1hdHRoZXcuYXVsZEBpbnRlbC5jb20+Cj4gQ2M6IFRob21hcyBIZWxs c3Ryw7ZtIDx0aG9tYXMuaGVsbHN0cm9tQGxpbnV4LmludGVsLmNvbT4KPiAtLS0KPiDCoGRyaXZl cnMvZ3B1L2RybS9pOTE1L2dlbS9pOTE1X2dlbV90dG0uY8KgwqDCoMKgwqDCoCB8wqAgMjYgKy0t Cj4gwqBkcml2ZXJzL2dwdS9kcm0vaTkxNS9pbnRlbF9tZW1vcnlfcmVnaW9uLmPCoMKgwqAgfMKg IDU1ICstLS0tLQo+IMKgZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfbWVtb3J5X3JlZ2lvbi5o wqDCoMKgIHzCoCAxNyAtLQo+IMKgZHJpdmVycy9ncHUvZHJtL2k5MTUvaW50ZWxfcmVnaW9uX3R0 bS5jwqDCoMKgwqDCoMKgIHwgMTAwICsrKy0tLS0tLS0tCj4gwqAuLi4vZHJtL2k5MTUvc2VsZnRl c3RzL2ludGVsX21lbW9yeV9yZWdpb24uY8KgIHwgMTcwICsrKysrKysrKysrKy0tLS0KPiAtLQo+ IAoKLi4uCgo+IMKgCj4gwqBzdGF0aWMgaW50IGk5MTVfdHRtX21vdmUoc3RydWN0IHR0bV9idWZm ZXJfb2JqZWN0ICpibywgYm9vbCBldmljdCwKPiBAQCAtNjYxLDIwICs2NjEsOCBAQCBpbnQgX19p OTE1X2dlbV90dG1fb2JqZWN0X2luaXQoc3RydWN0Cj4gaW50ZWxfbWVtb3J5X3JlZ2lvbiAqbWVt LAo+IMKgwqDCoMKgwqDCoMKgwqBzdGF0aWMgc3RydWN0IGxvY2tfY2xhc3Nfa2V5IGxvY2tfY2xh c3M7Cj4gwqDCoMKgwqDCoMKgwqDCoHN0cnVjdCBkcm1faTkxNV9wcml2YXRlICppOTE1ID0gbWVt LT5pOTE1Owo+IMKgwqDCoMKgwqDCoMKgwqBlbnVtIHR0bV9ib190eXBlIGJvX3R5cGU7Cj4gLcKg wqDCoMKgwqDCoMKgc2l6ZV90IGFsaWdubWVudCA9IDA7Cj4gwqDCoMKgwqDCoMKgwqDCoGludCBy ZXQ7Cj4gwqAKPiAtwqDCoMKgwqDCoMKgwqAvKiBBZGp1c3QgYWxpZ25tZW50IHRvIEdQVS0gYW5k IENQVSBodWdlIHBhZ2Ugc2l6ZXMuICovCj4gLQo+IC3CoMKgwqDCoMKgwqDCoGlmIChtZW0tPmlz X3JhbmdlX21hbmFnZXIpIHsKPiAtwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgaWYgKHNp emUgPj0gU1pfMUcpCj4gLcKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDC oMKgwqBhbGlnbm1lbnQgPSBTWl8xRyA+PiBQQUdFX1NISUZUOwo+IC3CoMKgwqDCoMKgwqDCoMKg wqDCoMKgwqDCoMKgwqBlbHNlIGlmIChzaXplID49IFNaXzJNKQo+IC3CoMKgwqDCoMKgwqDCoMKg wqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgYWxpZ25tZW50ID0gU1pfMk0gPj4gUEFHRV9T SElGVDsKPiAtwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgZWxzZSBpZiAoc2l6ZSA+PSBT Wl82NEspCj4gLcKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqBh bGlnbm1lbnQgPSBTWl82NEsgPj4gUEFHRV9TSElGVDsKPiAtwqDCoMKgwqDCoMKgwqB9Cj4gLQo+ IMKgwqDCoMKgwqDCoMKgwqBkcm1fZ2VtX3ByaXZhdGVfb2JqZWN0X2luaXQoJmk5MTUtPmRybSwg Jm9iai0+YmFzZSwgc2l6ZSk7Cj4gwqDCoMKgwqDCoMKgwqDCoGk5MTVfZ2VtX29iamVjdF9pbml0 KG9iaiwgJmk5MTVfZ2VtX3R0bV9vYmpfb3BzLCAmbG9ja19jbGFzcywKPiBmbGFncyk7Cj4gwqDC oMKgwqDCoMKgwqDCoGk5MTVfZ2VtX29iamVjdF9pbml0X21lbW9yeV9yZWdpb24ob2JqLCBtZW0p Owo+IEBAIC02OTYsNyArNjg0LDcgQEAgaW50IF9faTkxNV9nZW1fdHRtX29iamVjdF9pbml0KHN0 cnVjdAo+IGludGVsX21lbW9yeV9yZWdpb24gKm1lbSwKPiDCoMKgwqDCoMKgwqDCoMKgICovCj4g wqDCoMKgwqDCoMKgwqDCoG9iai0+YmFzZS52bWFfbm9kZS5kcml2ZXJfcHJpdmF0ZSA9IGk5MTVf Z2VtX3RvX3R0bShvYmopOwo+IMKgwqDCoMKgwqDCoMKgwqByZXQgPSB0dG1fYm9faW5pdCgmaTkx NS0+YmRldiwgaTkxNV9nZW1fdG9fdHRtKG9iaiksIHNpemUsCj4gLcKgwqDCoMKgwqDCoMKgwqDC oMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoCBib190eXBlLCAmaTkxNV9zeXNfcGxhY2Vt ZW50LCBhbGlnbm1lbnQsCj4gK8KgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKg wqDCoMKgwqDCoCBib190eXBlLCAmaTkxNV9zeXNfcGxhY2VtZW50LCBQQUdFX1NJWkUsCgpBY3R1 YWxseSBqdXN0IHJlYWxpemVkIHRoYXQgdGhlIGFsaWdubWVudCBpcyBzcGVjaWZpZWQgaW4gUEFH RV9TSVpFCnVuaXRzLCBzbyBhYm92ZSBzaG91bGQgYmUgcy9QQUdFX1NJWkUvMS8uIE1pZ2h0IG5l ZWQgdG8gY2hlY2sgdGhhdCB0aGUKYnVkZHkgVFRNIGludGVyZmFjZSBnZXRzIHRoaXMgcmlnaHQg YXMgd2VsbC4KCgpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f XwpJbnRlbC1nZnggbWFpbGluZyBsaXN0CkludGVsLWdmeEBsaXN0cy5mcmVlZGVza3RvcC5vcmcK aHR0cHM6Ly9saXN0cy5mcmVlZGVza3RvcC5vcmcvbWFpbG1hbi9saXN0aW5mby9pbnRlbC1nZngK