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

2023-10-31 Thread kernel test robot
Hi Tatsuyuki, kernel test robot noticed the following build warnings: [auto build test WARNING on drm-misc/drm-misc-next] [also build test WARNING on drm/drm-next drm-exynos/exynos-drm-next drm-intel/for-linux-next drm-intel/for-linux-next-fixes drm-tip/drm-tip linus/master v6.6 next-20231031]

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 eithe

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 confusi