3 ms·
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_in
by 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 agoFix 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 agoOr 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.