On Sun, Oct 30, 2022 at 10:03:57PM +0800, Jinlong Chen wrote: > Hi, Christoph! > > Only elevator_find_get is calling __elevator_find after applying the > series. Maybe we can just remove __elevator_find and move the list > iterating logic into elevator_find_get? We could. But then we'd need another local variable to track what was found, so I'm not sure it is a win. In general having a pure list lookup in a helper while all the locking and refcounting in a wrapper around it tends to be a quite nice pattern.
