• Boqun Feng's avatar
    lockdep: Make __bfs() visit every dependency until a match · d563bc6e
    Boqun Feng authored
    Currently, __bfs() will do a breadth-first search in the dependency
    graph and visit each lock class in the graph exactly once, so for
    example, in the following graph:
    
    	A ---------> B
    	|            ^
    	|            |
    	+----------> C
    
    a __bfs() call starts at A, will visit B through dependency A -> B and
    visit C through dependency A -> C and that's it, IOW, __bfs() will not
    visit dependency C -> B.
    
    This is OK for now, as we only have strong dependencies in the
    dependency graph, so whenever there is a traverse path from A to B in
    __bfs(), it means A has strong dependencies to B (IOW, B depends on A
    strongly). So no need to visit all dependencies in the graph.
    
    However, as we are going to add recursive-read lock into the dependency
    graph, as a result, not all the paths mean strong dependencies, in the
    same example above, dependency A -> B may be a weak dependency and
    traverse A -> C -> B may be a strong dependency path. And with the old
    way of __bfs() (i.e. visiting every lock class exactly once), we will
    miss the strong dependency path, which will result into failing to find
    a deadlock. To cure this for the future, we need to find a way for
    __bfs() to visit each dependency, rather than each class, exactly once
    in the search until we find a match.
    
    The solution is simple:
    
    We used to mark lock_class::lockdep_dependency_gen_id to indicate a
    class has been visited in __bfs(), now we change the semantics a little
    bit: we now mark lock_class::lockdep_dependency_gen_id to indicate _all
    the dependencies_ in its lock_{after,before} have been visited in the
    __bfs() (note we only take one direction in a __bfs() search). In this
    way, every dependency is guaranteed to be visited until we find a match.
    
    Note: the checks in mark_lock_accessed() and lock_accessed() are
    removed, because after this modification, we may call these two
    functions on @source_entry of __bfs(), which may not be the entry in
    "list_entries"
    Signed-off-by: default avatarBoqun Feng <boqun.feng@gmail.com>
    Signed-off-by: default avatarPeter Zijlstra (Intel) <peterz@infradead.org>
    Link: https://lkml.kernel.org/r/20200807074238.1632519-5-boqun.feng@gmail.com
    d563bc6e
lockdep.c 147 KB