On Thu, Sep 4, 2008 at 2:19 PM, Andrew Morton <akpm@xxxxxxxxxxxxxxxxxxxx> wrote: > On Thu, 4 Sep 2008 01:43:51 +0400 > Ilya Yanok <yanok@xxxxxxxxxxx> wrote: > >> From: Yuri Tikhonov <yur@xxxxxxxxxxx> >> >> Should clear the next pointer of the TX if we are sure that the >> next TX (say NXT) will be submitted to the channel too. Overwise, >> we break the chain of descriptors, because we lose the information >> about the next descriptor to run. So next time, when invoke >> async_tx_run_dependencies() with TX, it's TX->next will be NULL, and >> NXT will be never submitted. >> >> Signed-off-by: Yuri Tikhonov <yur@xxxxxxxxxxx> > > This patch should include your signed-off-by: as well. Because you > were on the delivery path, as described in > Documentation/SubmittingPatches, section 12. > Yes, Ilya once I have your signed-off-by I will push this to Linus and -stable. As far as impact for in-tree drivers: iop13xx by is immune, iop3xx can hit this, and it looks like the orion5x implementation is immune since copy and xor are available on the same channel. -- Dan -- To unsubscribe from this list: send the line "unsubscribe linux-raid" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html