On Wed, 23 Apr 2025 at 11:57, Matthew Brost <[email protected]> wrote: > > On Tue, Apr 22, 2025 at 06:48:08PM -0700, Matthew Brost wrote: > > On Wed, Apr 23, 2025 at 09:35:35AM +1000, Dave Airlie wrote: > > > Hey, > > > > > > Been digging a bit into cgroups stuff and I found this force_alloc > > > flags in the TTM operation context that seemed like I should dig > > > deeper, but I can't see this flag being used anywhere, places set it > > > but I can't spot the consumer of it? > > > > > > > A quick grep shows: > > > > mbrost@lstrano-desk:drm$ grep force_alloc */*.c > > ttm/ttm_bo_vm.c: .force_alloc = true > > ttm/ttm_resource.c: .force_alloc = true > > > > We have this wired in xe_bo_evict() too but always set it to false at > > the caller. > > > > Ah, I should read better. Yea, this appears to be unused, only set. > > git format-patch -1 c44dfe4de0539 > > This show when this flag was changed to a bool there was one consumer of > this flag but that user has since been removed. My guess is this had a > use case, that use case changed, and the flag was failed to be removed. > We can probably just remove it since there are no consumers of this.
Oh indeed, I thought this was something going into kernel, but yes it was just leftovers from something exited via vmwgfx. I'll send a cleanup. Dave.
