Michal Kazior <michal.kazior@xxxxxxxxx> writes: > This prevents leaving incomplete scatter-gather > transfer on CE rings which can lead firmware to > crash. > > Reported-By: Avery Pennarun <apenwarr@xxxxxxxxx> > Signed-off-by: Michal Kazior <michal.kazior@xxxxxxxxx> The title is a bit misleading as usually the commit log with the word revert means that the commit is reverting another git commit. Maybe something like this is better: ath10k: drop incomplete scatter-gather pci tx transfers > + if (WARN_ON(src_ring->write_index == src_ring->sw_index)) > + return; > + > + if (WARN_ON(src_ring->write_index == > + ath10k_ce_src_ring_write_index_get(ar, ctrl_addr))) > + return; WARN_ON() on data path is dangerous. WARN_ON_ONCE() or ath10k_warn() is better. > +err: > + for (; i > 0; i--) Isn't this just a fancy way to say 'while (i-- > 0)'? -- Kalle Valo -- To unsubscribe from this list: send the line "unsubscribe linux-wireless" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html