Re: + fs-reiser4-add-parenths-around-x-y.patch added to -mm tree

View: New views
1 Messages — Rating Filter:   Alert me  

Parent Message unknown Re: + fs-reiser4-add-parenths-around-x-y.patch added to -mm tree

by Edward Shishkin-2 :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

I don't understand this brain damaged deducing
GN flags from COPI flags, but it seems that it
doesn't have any effect and we can simply remove
both those if-branches.

Anyway, thanks!

Edward.

akpm@... wrote:

> The patch titled
>      fs: reiser4, add parenths around !X & Y
> has been added to the -mm tree.  Its filename is
>      fs-reiser4-add-parenths-around-x-y.patch
>
> Before you just go and hit "reply", please:
>    a) Consider who else should be cc'ed
>    b) Prefer to cc a suitable mailing list as well
>    c) Ideally: find the original patch on the mailing list and do a
>       reply-to-all to that, adding suitable additional cc's
>
> *** Remember to use Documentation/SubmitChecklist when testing your code ***
>
> See http://userweb.kernel.org/~akpm/stuff/added-to-mm.txt to find
> out what to do about this
>
> The current -mm tree may be found at http://userweb.kernel.org/~akpm/mmotm/
>
> ------------------------------------------------------
> Subject: fs: reiser4, add parenths around !X & Y
> From: Jiri Slaby <jirislaby@...>
>
> Change !X & Y to !(X & Y) to avoid compiler confusion and fix a bug.
>
> Signed-off-by: Jiri Slaby <jirislaby@...>
> Cc: Edward Shishkin <edward.shishkin@...>
> Signed-off-by: Andrew Morton <akpm@...>
> ---
>
>  fs/reiser4/carry_ops.c |    4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff -puN fs/reiser4/carry_ops.c~fs-reiser4-add-parenths-around-x-y fs/reiser4/carry_ops.c
> --- a/fs/reiser4/carry_ops.c~fs-reiser4-add-parenths-around-x-y
> +++ a/fs/reiser4/carry_ops.c
> @@ -79,7 +79,7 @@ static carry_node *find_left_neighbor(ca
>   left->free = 1;
>  
>   flags = GN_TRY_LOCK;
> - if (!op->u.insert.flags & COPI_LOAD_LEFT)
> + if (!(op->u.insert.flags & COPI_LOAD_LEFT))
>   flags |= GN_NO_ALLOC;
>  
>   /* then, feeling lucky, peek left neighbor in the cache. */
> @@ -203,7 +203,7 @@ static carry_node *find_right_neighbor(c
>   read_unlock_tree(tree);
>  
>   flags = GN_CAN_USE_UPPER_LEVELS;
> - if (!op->u.insert.flags & COPI_LOAD_RIGHT)
> + if (!(op->u.insert.flags & COPI_LOAD_RIGHT))
>   flags = GN_NO_ALLOC;
>  
>   /* then, try to lock right neighbor */
> _
>
> Patches currently in -mm which might be from jirislaby@... are
>
> linux-next.patch
> toshiba_acpi-return-on-a-fail-path.patch
> mtd-sst25l-fix-lock-imbalance.patch
> isdn-hisax-fix-lock-imbalance.patch
> icom-converting-space-to-tabs.patch
> cyclades-read-buffer-overflow.patch
> serial167-fix-read-buffer-overflow.patch
> usb-make-usb_buffer_map_sg-consistent-with-doc.patch
> char-riscom8-fix-shutdown-declaration.patch
> kcore-use-registerd-physmem-information-define-node__pfn-for-non-numa-builds.patch
> kcore-use-registerd-physmem-information-fix2.patch
> video-console-use-div_round_up.patch
> memory-controller-soft-limit-organize-cgroups-v9-fix.patch
> cyclades-allow-overriding-isa-defaults-also-when-the-driver-is-built-in.patch
> drivers-char-rio-rioctrlc-off-by-one-error-in-rioctrlc.patch
> reiser4.patch
> fs-reiser4-add-parenths-around-x-y.patch
>
>
>  

--
To unsubscribe from this list: send the line "unsubscribe reiserfs-devel" in
the body of a message to majordomo@...
More majordomo info at  http://vger.kernel.org/majordomo-info.html