On Thu, 2018-07-26 at 16:42 +-0530, Varun Prakash wrote: +AD4- cxgbit driver is not working with this patch as transport+AF8-free+AF8-pages() +AD4- is called before calling conn-+AD4-conn+AF8-transport-+AD4-iscsit+AF8-release+AF8-cmd(), cxgbit +AD4- driver calls dma+AF8-unmap+AF8-sg() in -+AD4-iscsit+AF8-release+AF8-cmd(), transport+AF8-free+AF8-pages() +AD4- frees the scatterlist pages so dma unmap will not work. +AD4- +AD4- cxgbit driver needs a callback before target frees the pages +AD4- so that it can dma unmap. +AD4- +AD4- We can remove -+AD4-iscsit+AF8-release+AF8-cmd() from +AF8AXw-iscsit+AF8-free+AF8-cmd() and call it +AD4- directly from iscsit+AF8-free+AF8-cmd(), iscsit+AF8-aborted+AF8-task(). Hello Varun, Thank you for your report. Since no other patches in this series depend on this patch, I will send a revert. Bart. -- To unsubscribe from this list: send the line "unsubscribe target-devel" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
