On Tue, Jun 21, 2022 at 08:12:23AM +0200, David Jander wrote: > These patches optimize the spi_sync call for the common case that the > worker thread is idle and the queue is empty. It also opens the > possibility to potentially further optimize the async path also, since > it doesn't need to take into account the direct sync path anymore. > > As an example for the performance gain, on an i.MX8MM SoC with a SPI CAN > controller attached (MCP2518FD), the time the interrupt line stays > active (which corresponds roughly with the time it takes to send 3 > relatively short consecutive spi_sync messages) is reduced from 98us to > only 72us by this patch. This seems to be testing fine so far so I'm thinking it's probably a good idea to get it into -next which will hopefully trigger wider testing, unless someone shouts I'll look into that early next week. The only feedback I've seen was Andy's review which is broadly stylistic so can safely be addressed incrementally (like the improvement in patch 4 already does for example), I didn't see any comments there which went to correctness.
Attachment:
signature.asc
Description: PGP signature
