Message ID | 20200826014428.828392-1-airlied@gmail.com (mailing list archive) |
---|---|
Headers | show |
Series | ttm tt refactoring. | expand |
A repeating style nit pick that {} can be dropped on single line if and empty line between declaration and code. Apart from that looks mostly good to me :) Patches #1-#4, #6, #10, #12-#15 are Reviewed-by: Christian König <christian.koenig@amd.com> Patch #5, #8, #9, #11 are Acked-by: Christian König <christian.koenig@amd.com> A few nits on patch #7. Regarding patch #16 I'm not sure if the sub-classing into ttm_tt and ttm_dma_tt is really worth it. I mean we can provide allocation helpers for drivers which don't need the DMA address array. But apart from that I'm pretty sure keeping the all fields in ttm_tt is the way to go. Patch #17 has one little style comment. I really need to take an in deep look at patches #18-#23, but the general approach sounds like the right thing to do. Christian. Am 26.08.20 um 03:44 schrieb Dave Airlie: > This is based on top of misc-next with Christians io rework for nouveau. > > https://nam11.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fairlied%2Flinux%2Ftree%2Fttm-tt-cleanup&data=02%7C01%7Cchristian.koenig%40amd.com%7C84fd18e47b124259c45a08d849619a9a%7C3dd8961fe4884e608e11a82d994e183d%7C0%7C0%7C637340030838885795&sdata=WyDLcpJp07bSNxYPWovsnDkUIRDDmK3OlakW%2Fwy9HX4%3D&reserved=0 (is a complete tree) > > The idea is to move towards ttm_tt being just a backing store for system > allocated pages + dma info, and not be anything to do with a global TT. > > First is a couple of minor cleanups, then drops bdev from ttm_tt, > and moves the func table into the bdev driver table instead, makes > drivers deal with the agp differences, then cleans up the mess. > > It tries to cleanup ttm_tt a bit more as well, in the end ttm_tt > is getting to what I'd like it to represent, the next step is tackling > the whole bind/unbind/move stuff I think. > > Dave. > >