Re: [PATCH v3] drm/amdgpu: Add debugfs entry for printing VM info

2020-10-13 Thread Christian König
[SNIP] Thinking a bit more about it, when you also look at bo->preferred_domains and compare that with the current domain you can also figure out if a BO is swapped out or not. Christian. Thanks Christian for quick review/help and details. Summarizing your inputs and some queries: 1. Adding B

RE: [PATCH v3] drm/amdgpu: Add debugfs entry for printing VM info

2020-10-13 Thread Chauhan, Madhav
October 12, 2020 3:14 PM >> To: Patel, Mihir ; amd-gfx@lists.freedesktop.org >> Cc: Chauhan, Madhav ; Deucher, Alexander >> ; Surampalli, Kishore >> ; Kamliya, Prakash >> >> Subject: Re: [PATCH v3] drm/amdgpu: Add debugfs entry for printing VM >> info >

RE: [PATCH v3] drm/amdgpu: Add debugfs entry for printing VM info

2020-10-12 Thread Patel, Mihir
: [PATCH v3] drm/amdgpu: Add debugfs entry for printing VM info Am 12.10.20 um 12:50 schrieb Christian König: > Am 12.10.20 um 12:00 schrieb Patel, Mihir: >> [AMD Official Use Only - Internal Distribution Only] >> >> >> >> -Original Message- >> From: Koen

Re: [PATCH v3] drm/amdgpu: Add debugfs entry for printing VM info

2020-10-12 Thread Christian König
: Chauhan, Madhav ; Deucher, Alexander ; Surampalli, Kishore ; Kamliya, Prakash Subject: Re: [PATCH v3] drm/amdgpu: Add debugfs entry for printing VM info [SNIP] Hi Christian, One question regarding getting total allocations by app and also swapped size for the app. Shouldn’t we print mapped entries

Re: [PATCH v3] drm/amdgpu: Add debugfs entry for printing VM info

2020-10-12 Thread Christian König
, Kishore ; Kamliya, Prakash Subject: Re: [PATCH v3] drm/amdgpu: Add debugfs entry for printing VM info [SNIP] Hi Christian, One question regarding getting total allocations by app and also swapped size for the app. Shouldn’t we print mapped entries as well to get total allocation by the process

RE: [PATCH v3] drm/amdgpu: Add debugfs entry for printing VM info

2020-10-12 Thread Patel, Mihir
: [PATCH v3] drm/amdgpu: Add debugfs entry for printing VM info [SNIP] > Hi Christian, > One question regarding getting total allocations by app and also swapped size > for the app. > Shouldn’t we print mapped entries as well to get total allocation by the > process ? > Also which li

Re: [PATCH v3] drm/amdgpu: Add debugfs entry for printing VM info

2020-10-12 Thread Christian König
[SNIP] Hi Christian, One question regarding getting total allocations by app and also swapped size for the app. Shouldn’t we print mapped entries as well to get total allocation by the process ? Also which list would represent swapped memory? None, we don't really have that information anywhe

RE: [PATCH v3] drm/amdgpu: Add debugfs entry for printing VM info

2020-10-12 Thread Patel, Mihir
: [PATCH v3] drm/amdgpu: Add debugfs entry for printing VM info Am 12.10.20 um 11:01 schrieb Mihir Patel: > From: Mihir Bhogilal Patel > > Create new debugfs entry to print memory info using VM buffer objects. > > Pending: > - Consolidated memory utilization info like total, swap

Re: [PATCH v3] drm/amdgpu: Add debugfs entry for printing VM info

2020-10-12 Thread Christian König
Am 12.10.20 um 11:01 schrieb Mihir Patel: From: Mihir Bhogilal Patel Create new debugfs entry to print memory info using VM buffer objects. Pending: - Consolidated memory utilization info like total, swap etc. V2: Added Common function for printing BO info. Dump more VM lists for evicted

[PATCH v3] drm/amdgpu: Add debugfs entry for printing VM info

2020-10-12 Thread Mihir Patel
From: Mihir Bhogilal Patel Create new debugfs entry to print memory info using VM buffer objects. Pending: - Consolidated memory utilization info like total, swap etc. V2: Added Common function for printing BO info. Dump more VM lists for evicted, moved, relocated, invalidated. Removed