Re: [RESEND, PATCH 4/6] memcg: fix broken boolean expression

From: Johannes Weiner
Date: Mon Jan 09 2012 - 09:04:12 EST


On Fri, Jan 06, 2012 at 10:57:50PM +0200, Kirill A. Shutemov wrote:
> From: "Kirill A. Shutemov" <kirill@xxxxxxxxxxxxx>
>
> action != CPU_DEAD || action != CPU_DEAD_FROZEN is always true.
>
> Signed-off-by: Kirill A. Shutemov <kirill@xxxxxxxxxxxxx>
> Cc: <stable@xxxxxxxxxx>

I think you don't need to actually CC stable via email. If you
include that tag, they will pick it up once the patch hits mainline.

The changelog is too terse, doubly so for a patch that should go into
stable. How is the code supposed to work? What are the consequences
of the bug?
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/