From mboxrd@z Thu Jan 1 00:00:00 1970 From: "Kuehling, Felix" Subject: Re: [PATCH 27/27] drm/amdgpu: Fix GTT size calculation Date: Tue, 30 Apr 2019 17:25:09 +0000 Message-ID: <8ba952ab-4836-4ca3-cd80-99f7367a7979@amd.com> References: <20190428074331.30107-1-Felix.Kuehling@amd.com> <20190428074331.30107-28-Felix.Kuehling@amd.com> <86fa9fc3-7a8f-9855-ae1d-5c7ccf2b5260@gmail.com> <1b1ec993-1c4b-8661-9b3f-ac0ad8ae64c7@amd.com> <134a4999-776f-44c6-99a2-42e8b9366a73@gmail.com> <9f882acd-c48f-3bbd-2d90-659c2edead39@amd.com> Mime-Version: 1.0 Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: base64 Return-path: In-Reply-To: Content-Language: en-US Content-ID: <73672055CE8C194B8177067904EF8BD1-asWib9pRmPqcE4WynfumptQqCkab/8FMAL8bYrjMMd8@public.gmane.org> List-Id: Discussion list for AMD gfx List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: amd-gfx-bounces-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org Sender: "amd-gfx" To: "Koenig, Christian" , "amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org" Cc: "Russell, Kent" T24gMjAxOS0wNC0zMCAxOjAzIHAubS4sIEtvZW5pZywgQ2hyaXN0aWFuIHdyb3RlOg0KPiBBbSAz MC4wNC4xOSB1bSAxNzozNiBzY2hyaWViIEt1ZWhsaW5nLCBGZWxpeDoNCj4+IE9uIDIwMTktMDQt MzAgNTozMiBhLm0uLCBDaHJpc3RpYW4gS8O2bmlnIHdyb3RlOg0KPj4+IFtDQVVUSU9OOiBFeHRl cm5hbCBFbWFpbF0NCj4+Pg0KPj4+IEFtIDMwLjA0LjE5IHVtIDAxOjE2IHNjaHJpZWIgS3VlaGxp bmcsIEZlbGl4Og0KPj4+PiBPbiAyMDE5LTA0LTI5IDg6MzQgYS5tLiwgQ2hyaXN0aWFuIEvDtm5p ZyB3cm90ZToNCj4+Pj4+IEFtIDI4LjA0LjE5IHVtIDA5OjQ0IHNjaHJpZWIgS3VlaGxpbmcsIEZl bGl4Og0KPj4+Pj4+IEZyb206IEtlbnQgUnVzc2VsbCA8a2VudC5ydXNzZWxsQGFtZC5jb20+DQo+ Pj4+Pj4NCj4+Pj4+PiBHVFQgc2l6ZSBpcyBjdXJyZW50bHkgbGltaXRlZCB0byB0aGUgbWluaW11 bSBvZiBWUkFNIHNpemUgb3IgMy80IG9mDQo+Pj4+Pj4gc3lzdGVtIG1lbW9yeS4gVGhpcyBzZXZl cmVseSBsaW1pdHMgdGhlIHF1YW5pdGl0eSBvZiBzeXN0ZW0gbWVtb3J5DQo+Pj4+Pj4gdGhhdCBj YW4gYmUgdXNlZCBieSBST0NtIGFwcGxpY2F0aW9uLg0KPj4+Pj4+DQo+Pj4+Pj4gSW5jcmVhc2Ug R1RUIHNpemUgdG8gdGhlIG1heGltdW0gb2YgVlJBTSBzaXplIG9yIHN5c3RlbSBtZW1vcnkgc2l6 ZS4NCj4+Pj4+IFdlbGwsIE5BSy4NCj4+Pj4+DQo+Pj4+PiBUaGlzIGxpbWl0IHdhcyBkb25lIG9u IHB1cnBvc2UgYmVjYXVzZSB3ZSBvdGhlcndpc2UgdGhlDQo+Pj4+PiBtYXgtdGV4dHVyZS1zaXpl IHdvdWxkIGJlIGNyYXNoaW5nIHRoZSBzeXN0ZW0gYmVjYXVzZSB0aGUgT09NIGtpbGxlcg0KPj4+ Pj4gd291bGQgYmUgY2F1c2luZyBhIHN5c3RlbSBwYW5pYy4NCj4+Pj4+DQo+Pj4+PiBVc2luZyBt b3JlIHRoYW4gNzUlIG9mIHN5c3RlbSBtZW1vcnkgYnkgdGhlIEdQVSBhdCB0aGUgc2FtZSB0aW1l IG1ha2VzDQo+Pj4+PiB0aGUgc3lzdGVtIHVuc3RhYmxlIGFuZCBzbyB3ZSBjYW4ndCBhbGxvdyB0 aGF0IGJ5IGRlZmF1bHQuDQo+Pj4+IExpa2Ugd2UgZGlzY3Vzc2VkLCB0aGUgY3VycmVudCBpbXBs ZW1lbnRhdGlvbiBpcyB0b28gbGltaXRpbmcuIE9uIGEgRmlqaQ0KPj4+PiBzeXN0ZW0gd2l0aCA0 R0IgVlJBTSBhbmQgMzJHQiBzeXN0ZW0gbWVtb3J5LCBpdCBsaW1pdHMgc3lzdGVtIG1lbW9yeQ0K Pj4+PiBhbGxvY2F0aW9ucyB0byA0R0IuIEkgdGhpbmsgdGhpcyB3b3JrYXJvdW5kIHdhcyBmaXhl ZCBvbmNlIGJlZm9yZSBhbmQNCj4+Pj4gcmV2ZXJ0ZWQgYmVjYXVzZSBpdCBicm9rZSBhIENaIHN5 c3RlbSB3aXRoIDFHQiBzeXN0ZW0gbWVtb3J5LiBTbyBJDQo+Pj4+IHN1c3BlY3QgdGhhdCB0aGlz IGlzIGFuIGlzc3VlIGFmZmVjdGluZyBzbWFsbCBtZW1vcnkgc3lzdGVtcyB3aGVyZSBtYXliZQ0K Pj4+PiB0aGUgMS8yIHN5c3RlbSBtZW1vcnkgbGltaXQgaW4gVFRNIGlzbid0IHN1ZmZpY2llbnQg dG8gcHJvdGVjdCBmcm9tIE9PTQ0KPj4+PiBwYW5pY3MuDQo+Pj4gV2VsbCBpdCBub3Qgb25seSBi cm9rZSBvbiBhIDFHQiBDWiBzeXN0ZW0sIHRoaXMgd2FzIGp1c3Qgd2hlcmUgQW5kcmV5DQo+Pj4g cmVwcm9kdWNlZCBpdC4gV2UgZ290IHJlcG9ydHMgZnJvbSBhbGwga2luZCBvZiBzeXN0ZW1zLg0K Pj4gSSdkIGxpa2UgdG8gc2VlIHRob3NlIHJlcG9ydHMuIFRoaXMgcGF0Y2ggaGFzIGJlZW4gaW5j bHVkZWQgaW4gTGludXggUHJvDQo+PiByZWxlYXNlcyBzaW5jZSAxOC4yMC4gSSdtIG5vdCBhd2Fy ZSB0aGF0IGFueW9uZSBjb21wbGFpbmVkIGFib3V0IGl0Lg0KPiBXZWxsIHRvIGJlIGhvbmVzdCBv dXIgUHJvIGRyaXZlciBpcyBhY3R1YWxseSBub3QgdXNlZCB0aGF0IHdpZGVseSBhbmQNCj4gb25s eSB1c2VkIG9uIHJhdGhlciBob21vZ2VuZW91cyBzeXN0ZW1zLg0KPg0KPiBXaGljaCBpcyBub3Qg cmVhbGx5IHN1cnByaXNpbmcgc2luY2Ugd2Ugb25seSBhZHZpc2UgdG8gdXNlIGl0IG9uDQo+IHBy b2Zlc3Npb25hbCB1c2UgY2FzZXMuDQo+DQo+Pj4+IFRoZSBPT00ga2lsbGVyIHByb2JsZW0gaXMg YSBtb3JlIGdlbmVyYWwgcHJvYmxlbSB0aGF0IHBvdGVudGlhbGx5DQo+Pj4+IGFmZmVjdHMgb3Ro ZXIgZHJpdmVycyB0b28uIEtlZXBpbmcgdGhpcyBHVFQgbGltaXQgYnJva2VuIGluIEFNREdQVSBp cyBhbg0KPj4+PiBpbmFkZXF1YXRlIHdvcmthcm91bmQgYXQgYmVzdC4gSSdkIGxpa2UgdG8gbG9v ayBmb3IgYSBiZXR0ZXIgc29sdXRpb24sDQo+Pj4+IHByb2JhYmx5IHNvbWUgYWRqdXN0bWVudCBv ZiB0aGUgVFRNIHN5c3RlbSBtZW1vcnkgbGltaXRzIG9uIHN5c3RlbXMgd2l0aA0KPj4+PiBzbWFs bCBtZW1vcnksIHRvIGF2b2lkIE9PTSBwYW5pY3Mgb24gc3VjaCBzeXN0ZW1zLg0KPj4+IFRoZSBj b3JlIHByb2JsZW0gaGVyZSBpcyB0aGF0IHRoZSBPT00ga2lsbGVyIGV4cGxpY2l0bHkgZG9lc24n dCB3YW50IHRvDQo+Pj4gYmxvY2sgZm9yIHNoYWRlcnMgdG8gZmluaXNoIHdoYXRldmVyIGl0IGlz IGRvaW5nLg0KPj4+DQo+Pj4gU28gY3VycmVudGx5IGFzIHNvb24gYXMgdGhlIGhhcmR3YXJlIGlz IHVzaW5nIHNvbWUgbWVtb3J5IGl0IGNhbid0IGJlDQo+Pj4gcmVjbGFpbWVkIGltbWVkaWF0ZWx5 Lg0KPj4+DQo+Pj4gVGhlIG9yaWdpbmFsIGxpbWl0IGluIFRUTSB3YXMgMi8zIG9mIHN5c3RlbSBt ZW1vcnkgYW5kIHRoYXQgd29ya2VkDQo+Pj4gcmVhbGx5IHJlbGlhYmxlIGFuZCB3ZSByYW4gaW50 byBwcm9ibGVtcyBvbmx5IGFmdGVyIHJhaXNpbmcgaXQgdG8gMy80Lg0KPj4gVGhlIFRUTSBzeXN0 ZW0gbWVtb3J5IGxpbWl0IGlzIHN0aWxsIDMvOCBzb2Z0IGFuZCAxLzIgaGFyZCwgMy80IGZvcg0K Pj4gZW1lcmdlbmNpZXMuIFNlZSB0dG1fbWVtX2luaXRfa2VybmVsX3pvbmUuIEFGQUlDVCwgdGhl IGVtZXJnZW5jeSBsaW1pdA0KPj4gaXMgb25seSBhdmFpbGFibGUgdG8gcm9vdC4NCj4gQWghIEkg dGhpbmsgSSBrbm93IHdoeSB0aG9zZSBsaW1pdHMgZG9lc24ndCBraWNrIGluIGhlcmUhDQo+DQo+ IFdoZW4gR1RUIHNwYWNlIGlzIHVzZWQgYnkgZXZpY3Rpb25zIGZyb20gVlJBTSB0aGVuIHdlIHdp bGwgdXNlIHRoZQ0KPiBlbWVyZ2VuY3kgbGltaXQgYXMgd2VsbC4NCj4NCj4+IFRoaXMgR1RUIGxp bWl0IGtpY2tzIGluIGJlZm9yZSBJIGdldCBhbnl3aGVyZSBjbG9zZSB0byB0aGUgVFRNIGxpbWl0 Lg0KPj4gVGhhdCdzIHdoeSBJIHRoaW5rIGl0IGlzIGJvdGggYnJva2VuIGFuZCByZWR1bmRhbnQu DQo+IFRoYXQgd2FzIGFsc28gdGhlIGFyZ3VtZW50IHdoZW4gd2UgcmVtb3ZlZCBpdCB0aGUgbGFz dCB0aW1lLCBidXQgaXQgZ290DQo+IGltbWVkaWF0ZWx5IHJldmVydGVkLg0KPg0KPj4+IFRvIHN1 bSBpdCB1cCB0aGUgcmVxdWlyZW1lbnQgb2YgdXNpbmcgYWxtb3N0IGFsbCBzeXN0ZW0gbWVtb3J5 IGJ5IGEgR1BVDQo+Pj4gaXMgc2ltcGx5IG5vdCBwb3NzaWJsZSB1cHN0cmVhbSBhbmQgZXZlbiBp biBhbnkgcHJvZHVjdGlvbiBzeXN0ZW0gcmF0aGVyDQo+Pj4gcXVlc3Rpb25hYmxlLg0KPj4gSXQg c2hvdWxkIGJlIGRvYWJsZSB3aXRoIHVzZXJwdHIsIHdoaWNoIG5vdyB1c2VzIHVucGlubmVkIHBh Z2VzIHRocm91Z2gNCj4+IEhNTS4gQ3VycmVudGx5IHRoZSBHVFQgbGltaXQgYWZmZWN0cyB0aGUg bGFyZ2VzdCBwb3NzaWJsZSB1c2VycHRyDQo+PiBhbGxvY2F0aW9uLCB0aG91Z2ggbm90IHRoZSB0 b3RhbCBzdW0gb2YgYWxsIHVzZXJwdHIgYWxsb2NhdGlvbnMuIE1heWJlDQo+PiBtYWtpbmcgdXNl cnB0ciBjb21wbGV0ZWx5IGluZGVwZW5kZW50IG9mIEdUVCBzaXplIHdvdWxkIGJlIGFuIGVhc2ll cg0KPj4gcHJvYmxlbSB0byB0YWNrbGUuIFRoZW4gSSBjYW4gbGVhdmUgdGhlIEdUVCBsaW1pdCBh bG9uZS4NCj4gV2VsbCB0aGlzIHdheSB3ZSB3b3VsZCBvbmx5IGF2b2lkIHRoZSBzeW1wdG9tcywg YnV0IG5vdCB0aGUgcmVhbCBwcm9ibGVtLg0KDQpJdCBhbGxvY2F0ZXMgcGFnZXMgaW4gdXNlciBt b2RlIHJhdGhlciB0aGFuIGtlcm5lbCBtb2RlLiBUaGF0IG1lYW5zLCBPT00gDQpzaXR1YXRpb25z IHRha2UgYSBjb21wbGV0ZWx5IGRpZmZlcmVudCBjb2RlIHBhdGguIEJlZm9yZSBydW5uaW5nIG91 dCBvZiANCm1lbW9yeSBjb21wbGV0ZWx5LCB0cmlnZ2VyaW5nIHRoZSBPT00ga2lsbGVyLCB0aGUg a2VybmVsIHdvdWxkIHN0YXJ0IA0Kc3dhcHBpbmcgcGFnZXMsIHdoaWNoIHdvdWxkIHRyaWdnZXIg dGhlIE1NVSBub3RpZmllciB0byBzdG9wIHRoZSB1c2VyIA0KbW9kZSBxdWV1ZXMgb3IgaW52YWxp ZGF0ZSBHUFUgcGFnZSB0YWJsZSBlbnRyaWVzLCBhbmQgYWxsb3cgdGhlIHBhZ2VzIHRvIA0KYmUg c3dhcHBlZCBvdXQuDQoNCg0KPg0KPj4+IFRoZSBvbmx5IHJlYWwgc29sdXRpb24gSSBjYW4gc2Vl IGlzIHRvIGJlIGFibGUgdG8gcmVsaWFibGUga2lsbCBzaGFkZXJzDQo+Pj4gaW4gYW4gT09NIHNp dHVhdGlvbi4NCj4+IFdlbGwsIHdlIGNhbiBpbiBmYWN0IHByZWVtcHQgb3VyIGNvbXB1dGUgc2hh ZGVycyB3aXRoIGxvdyBsYXRlbmN5Lg0KPj4gS2lsbGluZyBhIEtGRCBwcm9jZXNzIHdpbGwgZG8g ZXhhY3RseSB0aGF0Lg0KPiBJJ3ZlIHRha2VuIGEgbG9vayBhdCB0aGF0IHRoaW5nIGFzIHdlbGwg YW5kIHRvIGJlIGhvbmVzdCBpdCBpcyBub3QgZXZlbg0KPiByZW1vdGVseSBzdWZmaWNpZW50Lg0K Pg0KPiBXZSBuZWVkIHNvbWV0aGluZyB3aGljaCBzdG9wcyB0aGUgaGFyZHdhcmUgKmltbWVkaWF0 ZWx5KiBmcm9tIGFjY2Vzc2luZw0KPiBzeXN0ZW0gbWVtb3J5LCBhbmQgbm90IHdhaXQgZm9yIHRo ZSBTUSB0byBraWxsIGFsbCB3YXZlcywgZmx1c2ggY2FjaGVzDQo+IGV0Yy4uLg0KDQpJdCdzIGFw cGFyZW50bHkgc3VmZmljaWVudCB0byB1c2UgaW4gb3VyIE1NVSBub3RpZmllci4gVGhlcmUgaXMg YWxzbyBhIA0Kd2F5IHRvIGRpc2FibGUgdGhlIGdyYWNlIHBlcmlvZCB0aGF0IGFsbG93cyBzaG9y dCB3YXZlcyB0byBjb21wbGV0ZSANCmJlZm9yZSBiZWluZyBwcmVlbXB0ZWQsIHRob3VnaCB3ZSdy ZSBub3QgdXNpbmcgdGhhdCBhdCB0aGUgbW9tZW50Lg0KDQoNCj4NCj4gT25lIHBvc3NpYmlsaXR5 IEknbSBwbGF5aW5nIGFyb3VuZCB3aXRoIGZvciBhIHdoaWxlIGlzIHRvIHJlcGxhY2UgdGhlDQo+ IHJvb3QgUEQgZm9yIHRoZSBWTUlEcyBpbiBxdWVzdGlvbiBvbiB0aGUgZmx5LiBFLmcuIHdlIGp1 c3QgbGV0IGl0IHBvaW50DQo+IHRvIHNvbWUgZHVtbXkgd2hpY2ggcmVkaXJlY3RzIGV2ZXJ5dGhp bmcgaW50byBuaXJ2YW5hLg0KDQpFdmVuIHRoYXQncyBub3Qgc3VmZmljaWVudC4gWW91J2xsIGFs c28gbmVlZCB0byBmcmVlIHRoZSBwYWdlcyANCmltbWVkaWF0ZWx5LiBGb3IgS0ZEIHByb2Nlc3Nl cywgY2xlYW5pbmcgdXAgb2YgbWVtb3J5IGlzIGRvbmUgaW4gYSANCndvcmtlciB0aHJlYWQgdGhh dCBnZXRzIGtpY2tlZCBvZmYgYnkgYSByZWxlYXNlIE1NVSBub3RpZmllciB3aGVuIHRoZSANCnBy b2Nlc3MnIG1tX3N0cnVjdCBpcyB0YWtlbiBkb3duLg0KDQpUaGVuIHRoZXJlIGlzIHN0aWxsIFRU TSdzIGRlbGF5ZWQgZnJlZWluZyBvZiBCT3MgdGhhdCB3YWl0cyBmb3IgZmVuY2VzLiANClNvIHlv dSdkIG5lZWQgdG8gc2lnbmFsIGFsbCB0aGUgQk8gZmVuY2VzIHRvIGFsbG93IHRoZW0gdG8gYmUg cmVsZWFzZWQuDQoNClRCSCwgSSBkb24ndCB1bmRlcnN0YW5kIHdoeSB3YWl0aW5nIGlzIG5vdCBh biBvcHRpb24sIGlmIHRoZSBhbHRlcm5hdGl2ZSANCmlzIGEga2VybmVsIHBhbmljLiBJZiB5b3Vy IE9PTSBraWxsZXIga2lja3MgaW4sIHlvdXIgc3lzdGVtIGlzIGJhc2ljYWxseSANCmRlYWQuIFdh aXRpbmcgZm9yIGEgZnJhY3Rpb24gb2YgYSBzZWNvbmQgdG8gbGV0IGEgR1BVIGZpbmlzaCBpdHMg bWVtb3J5IA0KYWNjZXNzIHNob3VsZCBiZSBhIHNtYWxsIHByaWNlIHRvIHBheSBpbiB0aGF0IHNp dHVhdGlvbi4NCg0KUmVnYXJkcywNCiDCoCBGZWxpeA0KDQoNCj4NCj4gQnV0IGltcGxlbWVudGlu ZyB0aGlzIGlzIGVhc2llciBzYWlkIHRoYW4gZG9uZS4uLg0KPg0KPiBSZWdhcmRzLA0KPiBDaHJp c3RpYW4uDQo+DQpfX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19f XwphbWQtZ2Z4IG1haWxpbmcgbGlzdAphbWQtZ2Z4QGxpc3RzLmZyZWVkZXNrdG9wLm9yZwpodHRw czovL2xpc3RzLmZyZWVkZXNrdG9wLm9yZy9tYWlsbWFuL2xpc3RpbmZvL2FtZC1nZng=