Linus Torvalds writes: (Summary) wrote:
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.
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".
to let lockdep know".
So that commit is apparently buggy, but I think it might be more fundamentally the wrong model too.
fundamentally the wrong model too.
Linus
Linus
Linus
[...]
The apparent stack corruption really worries me [...] Side note: I also think the thing is just broken.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.
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".
to let lockdep know".
So that commit is apparently buggy, but I think it might be more fundamentally the wrong model too.
fundamentally the wrong model too.
Linus
Linus
Linus