(apologies for not quoting, I wasn't subscribed before now)
some quick thoughts:
- Can we split these patches in to two series, one for each topic. They
don't seem specifically related.
- to simplify 64K page support, could we just set minimum allocation
size to 64K and round up for allocation requests?
Placement then becomes much simpler, no need to align the va to 2MB,
just fit it in wherever it fits and always use 64K PTEs in GTT
This would simplify the code a lot and would benefit performance due up
to 16x fewer page walks.
If we did this, we would not have to consider 2MB boundaries at all, we
could drop all the colour handling etc.
The only down side might be some waste of allocation if there are lots
of very small buffers.
However, I think most gfx related use cases would not be badly affected
by this (even a cursor plane is 64k, usually).
Are there any usecases that you are aware of that would be impacted
badly by this idea? (maybe some compute workload?)
- flat ccs modifiers: there seems to be some confusion over whether
there should be a separate modifier for this.
As it dictates a new layout it seems like it should be a new modifier.
Was there any internal discussions about this that you could elaborate
on here?
