On Tue, Oct 03, 2017 at 09:57:02AM -0700, Linus Torvalds wrote: > On Tue, Oct 3, 2017 at 9:54 AM, Linus Torvalds > <torva...@linux-foundation.org> wrote: > > > > Can we consider just reverting the crossrelease thing? > > > > The apparent stack corruption really worries me [...] > > Side note: I also think the thing is just broken. > > Any actual cross-releaser should be way more annotated than just "set > cross to 1" in the lockdep map. > > The place where the release is done should simply be special. > > Because we should *not* encourage the whole "acquire by one context, > release by another" as being something normal and "just set the flag > to let lockdep know".
Could you explain it more? Please let me apply what you point out. Now, I don't understand your intention. > So that commit is apparently buggy, but I think it might be more > fundamentally the wrong model too. It would be appriciated if you let me know what is buggy wrt crossrelease and the model, then I will do my best to fix it. But I believe the model crossrelease uses is what lockdep should have adopted before. Anyway, I might be wrong. It would be appriciated if you tell me why you think so. Thank you.
