4 ms·

Thanks, I've corrected the links, please check them again to see they are pre-regression versions [1] > Not true, it is architecture independent (commit also

Thanks, I've corrected the links, please check them again to see they are pre-regression versions [1] > Not true, it is architecture independent (commit also mentions Android) I saw it (and arm64 comment in this thread) but I didn't include them because I don't think it would be a production-serious issue there. Thanks for clarification. > Please check your facts next time. Thanks to your post I've corrected links, but no need for such an aggressive tone, huh? Maybe you should try to be more polite in your next refutal tries? [1] https://github.com/torvalds/linux/commit/b0c29f79ecea0b6fbcefc999e70f2843ae8306db https://github.com/torvalds/linux/commit/b0c29f79ecea0b6fbce...

buster 11y ago
Please note that i was just looking for the missing default case. It might still be the case that the bug is not in precise.. The patch also replaces atomic_inc() with futex_get_mm(): - atomic_inc(&key->private.mm->mm_count); + futex_get_mm(key); /* implies MB (B) */ And your links are using atomic_inc().. What that means with regards to this bug? I don't know.
- kasabali 11y ago
  Fix commit and OP claims bug was introduced in [1], which has been never backported to 3.2.x, so I assume lack of default case was not a problem before that commit. > And your links are using atomic_inc().. What that means with regards to this bug? I don't know. It means they are simply old versions and they were never affected by the bug, if we believe the commit message of the fix commit and OP. [1] https://github.com/torvalds/linux/commit/b0c29f79ecea0b6fbcefc999e70f2843ae8306db https://github.com/torvalds/linux/commit/b0c29f79ecea0b6fbce...
  - buster 11y ago
    Or not recognized.. But you are right, i suppose that the atomic_inc() doesn't need a memory barrier because it's atomic and the new function may need one, guessing.
deleted 11y ago
[deleted]

buster 11y ago

Please note that i was just looking for the missing default case. It might still be the case that the bug is not in precise.. The patch also replaces atomic_inc() with futex_get_mm(): - atomic_inc(&key->private.mm->mm_count); + futex_get_mm(key); /* implies MB (B) */ And your links are using atomic_inc().. What that means with regards to this bug? I don't know.

kasabali 11y ago
Fix commit and OP claims bug was introduced in [1], which has been never backported to 3.2.x, so I assume lack of default case was not a problem before that commit. > And your links are using atomic_inc().. What that means with regards to this bug? I don't know. It means they are simply old versions and they were never affected by the bug, if we believe the commit message of the fix commit and OP. [1] https://github.com/torvalds/linux/commit/b0c29f79ecea0b6fbcefc999e70f2843ae8306db https://github.com/torvalds/linux/commit/b0c29f79ecea0b6fbce...
- buster 11y ago
  Or not recognized.. But you are right, i suppose that the atomic_inc() doesn't need a memory barrier because it's atomic and the new function may need one, guessing.

deleted 11y ago

[deleted]