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

From: Kirill A. Shutemov
Date: Mon Jan 16 2012 - 06:54:03 EST


On Mon, Jan 09, 2012 at 03:04:04PM +0100, Johannes Weiner wrote:
> 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.

I don't think it's a problem for stable@.

>
> 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?

Is it enough?

---