Thread (4 messages) 4 messages, 2 authors, 2017-11-10

Re: [PATCH v2] locking/lockdep: Revise Documentation/locking/crossrelease.txt

From: Ingo Molnar <mingo@kernel.org>
Date: 2017-11-10 07:30:59
Also in: linux-mm, lkml

* Byungchul Park [off-list ref] wrote:
    Event C depends on event A.
    Event A depends on event B.
    Event B depends on event C.
 
-   NOTE: Precisely speaking, a dependency is one between whether a
-   waiter for an event can be woken up and whether another waiter for
-   another event can be woken up. However from now on, we will describe
-   a dependency as if it's one between an event and another event for
-   simplicity.
Why was this explanation removed?
-Lockdep tries to detect a deadlock by checking dependencies created by
-lock operations, acquire and release. Waiting for a lock corresponds to
-waiting for an event, and releasing a lock corresponds to triggering an
-event in the previous section.
+Lockdep tries to detect a deadlock by checking circular dependencies
+created by lock operations, acquire and release, which are wait and
+event respectively.
What? You changed a readable paragraph into an unreadable one.

Sorry, this text needs to be acked by someone with good English skills, and I 
don't have the time right now to fix it all up. Please send minimal, obvious
typo/grammar fixes only.

Thanks,

	Ingo
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help