Hello Alexey, Fred. Thanks for reviewing!


On Wed, 2021-07-21 at 13:32 +1000, Alexey Kardashevskiy wrote:

quoted

quoted

    spin_lock(&direct_window_list_lock);

Somewhere around here, we have:

out_remove_win:
     remove_ddw(pdn, true, DIRECT64_PROPNAME);

We should replace with:
     remove_ddw(pdn, true, win_name);

True. Good spotting. Or rework remove_dma_window() to take just a
liobn. 
Thanks,

Fred,
Good catch! On rebasing the last changes I did miss this one.

Due to reworking v5 06/11, I ended up with a solution that takes a
liobn instead of a win name (as suggested by Alexey), and this one is
no more :)

quoted

   Fred

quoted

Thanks!