David Howells <[EMAIL PROTECTED]> wrote:
> Peter Zijlstra <[EMAIL PROTECTED]> wrote:
>
> > - if (unlikely(vma->vm_flags & VM_SHARED)) {
> > + if (unlikely(vma->vm_flags & (VM_SHARED|VM_WRITE) ==
> > + VM_SHARED|VM_WRITE) {
>
> NAK!
>
> "==" is higher priority than "|". What you meant was:
>
> - if (unlikely(vma->vm_flags & VM_SHARED)) {
> + if (unlikely(vma->vm_flags & (VM_SHARED|VM_WRITE) ==
> + (VM_SHARED|VM_WRITE)) {
Or, rather:
- if (unlikely(vma->vm_flags & VM_SHARED)) {
+ if (unlikely(vma->vm_flags & (VM_SHARED|VM_WRITE) ==
+ (VM_SHARED|VM_WRITE))) {
It has insufficient closing brackets otherwise.
David
_______________________________________________
Ocfs2-devel mailing list
[email protected]
http://oss.oracle.com/mailman/listinfo/ocfs2-devel
