On 03/12/2025 13:20, Christian König wrote:
On 12/3/25 14:09, Tvrtko Ursulin wrote:
On 03/12/2025 12:57, Christian König wrote:
Sorry, I'm mostly on sick leave at the moment (cold/flu).
No rush in this case, series has been waiting from September so it can wait a
bit longer.
I already crawled out of bed, so I can as well take a look :)
I couldn't find the original patches, neither in my inbox nor using google.
Could you send them out once more?
This is the series:
https://lore.kernel.org/dri-devel/[email protected]/
It still applies but let me know if you would regardless prefer me to re-send.
Patches #1-#3 are Reviewed-by: Christian König <[email protected]>
Patch #4 looks good to me as well, but has a bit to much changes at once for my
current brain status.
Feel free to add my acked-by if you're sure that this is correct, otherwise
please split up into smaller cleanups.
I was pretty confident, having triple checked everything, and so have
pushed it all to drm-misc-next, thank you! (And on the off-chance I
broke something I'll be around to fix it as always.)
Regards,
Tvrtko
P.S. Btw I have some amdgpu cleanup patches which remove almost half a
megabyte of .text, so if during your recovery you find the need to look
at something not very demanding it is at
https://patchwork.freedesktop.org/series/157011/. Just beware Alex had
tried the earlier version of the "compact IB emissions" patches (tail
end of the series) and there was potentially some unexplained regression
in there.
On 12/3/25 12:34, Tvrtko Ursulin wrote:
3rd ping, and apologies, for some reason I thought Thomas you were listed as
maintainer for TTM.
Correcting the To: list to Christian and Huang.
Again, asking for acks to merge this trivial series.
Regards,
Tvrtko
On 21/11/2025 10:02, Tvrtko Ursulin wrote:
A gentle ping. Two really trivial cleanups are ready to merge just need
maintainer acks.
On 05/11/2025 15:33, Tvrtko Ursulin wrote:
+ Christian & Thomas
On 19/09/2025 14:15, Tvrtko Ursulin wrote:
Some small cleanups, nothing intentionally functional. It is mostly about
applying some consistency to when to use a local variable to cache something,
when not, and making the code use the ones already there.
Tvrtko Ursulin (4):
drm/ttm: Make ttm_bo_init_validate safe against ttm_operation_ctx
re-ordering
drm/ttm: Resource cannot be NULL in ttm_resource_intersects
drm/ttm: Tidy ttm_operation_ctx initialization
drm/ttm: Tidy usage of local variables a little bit
I got acks for trivial cleanups patch 1 & 3 from this series. Are you okay if I
push the two to drm-misc-next?
Regards,
Tvrtko
drivers/gpu/drm/ttm/ttm_bo.c | 64 +++++++++++++++---------------
drivers/gpu/drm/ttm/ttm_bo_util.c | 57 ++++++++++++--------------
drivers/gpu/drm/ttm/ttm_bo_vm.c | 12 +++---
drivers/gpu/drm/ttm/ttm_device.c | 5 +--
drivers/gpu/drm/ttm/ttm_pool.c | 26 ++++++------
drivers/gpu/drm/ttm/ttm_resource.c | 14 ++-----
drivers/gpu/drm/ttm/ttm_tt.c | 2 +-
7 files changed, 83 insertions(+), 97 deletions(-)
