Re: [PATCH 2/6] drm/amdgpu: Separate eviction from VM status.

2023-10-31 Thread kernel test robot
-implicit-sync-PRT-maps/20231031-224530 base: git://anongit.freedesktop.org/drm/drm-misc drm-misc-next patch link: https://lore.kernel.org/r/20231031134059.171277-3-ishitatsuyuki%40gmail.com patch subject: [PATCH 2/6] drm/amdgpu: Separate eviction from VM status. config: arc-randconfig-001

Re: [PATCH 2/6] drm/amdgpu: Separate eviction from VM status.

2023-10-31 Thread Christian König
Am 31.10.23 um 15:39 schrieb Tatsuyuki Ishi: On Oct 31, 2023, at 22:55, Christian König wrote: Am 31.10.23 um 14:40 schrieb Tatsuyuki Ishi: In short, eviction never really belonged to the vm_status state machine. I strongly disagree to that. Even when evicted, the BO could belong to

Re: [PATCH 2/6] drm/amdgpu: Separate eviction from VM status.

2023-10-31 Thread Tatsuyuki Ishi
> On Oct 31, 2023, at 22:55, Christian König wrote: > > Am 31.10.23 um 14:40 schrieb Tatsuyuki Ishi: >> In short, eviction never really belonged to the vm_status state machine. > > I strongly disagree to that. > >> Even when evicted, the BO could belong to either the moved or done state. >>

Re: [PATCH 2/6] drm/amdgpu: Separate eviction from VM status.

2023-10-31 Thread Christian König
Am 31.10.23 um 14:40 schrieb Tatsuyuki Ishi: In short, eviction never really belonged to the vm_status state machine. I strongly disagree to that. Even when evicted, the BO could belong to either the moved or done state. The "evicted" state needed to handle both cases, causing greater

[PATCH 2/6] drm/amdgpu: Separate eviction from VM status.

2023-10-31 Thread Tatsuyuki Ishi
In short, eviction never really belonged to the vm_status state machine. Even when evicted, the BO could belong to either the moved or done state. The "evicted" state needed to handle both cases, causing greater confusion. Additionally, there were inconsistencies in the definition of an evicted