On Mon, Nov 17, 2014 at 11:47:48PM +0100, Beniamino Galvani wrote:
> + list_for_each_entry(xfer, &master->cur_msg->transfers, transfer_list) {
> + xfer->last = list_is_last(&xfer->transfer_list,
> + &master->cur_msg->transfers);
> + }
It's incredibly sad to iterate through the entire list in order to find
the last entry, especially given that it's a doubly linked list and this
is a bit of a hot path. We should look at the previous entry for the
list head instead, or perhaps better yet by doing this as part of
spi_validate() which already itereates over the entire list and is where
we do other similar fixups.
Though looking at this I'm not sure that a flag is the best approach at
all - why not just have the driver call list_is_last() in the transfer
function or ideally provide an inline function that does that so that we
can change the implementation later?
Attachment:
signature.asc
Description: Digital signature
